Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(state): retry fetching identity #653

Merged
merged 3 commits into from
Oct 1, 2024
Merged

Conversation

Flemmli97
Copy link
Contributor

What this PR does 📖

  • Retries fetching identity a couple times before giving up

@github-actions github-actions bot added the Missing dev review Two dev reviews are required on PR label Oct 1, 2024
Copy link

github-actions bot commented Oct 1, 2024

Download the app installers for this pull request:

Copy link

github-actions bot commented Oct 1, 2024

Automated Test Results

passed  120 passed
flaky  2 flaky
skipped  4 skipped

Details

stats  126 tests across 16 suites
duration  19 minutes, 56 seconds
commit  a39d8c9

Flaky tests

Automated Tests on Chrome Desktop › 03-friends-two-instances.spec.ts › Two instances tests - Friends and Chats › B7, B57, B58 - Favorites tests
Automated Tests on Chrome Desktop › 09-settings-customizations.spec.ts › Settings Customization Tests › K18 - Default Profile Picture Style dropdown should show expected default profile picture styles

Skipped tests

Automated Tests on Chrome Desktop › 03-friends-two-instances.spec.ts › Two instances tests - Friends and Chats › B8 to B14 - Quick Profile Local and Remote - Updating note
Automated Tests on Chrome Desktop › 03-friends-two-instances.spec.ts › Two instances tests - Friends and Chats › B8 to B14 - Quick Profile Local - Updating username, status, banner and profile picture
Automated Tests on Chrome Desktop › 03-friends-two-instances.spec.ts › Two instances tests - Friends and Chats › B8 to B14 - Quick Profile Remote - Updating username, status, banner and profile picture
Automated Tests on Chrome Desktop › 20-settings-developer.spec.ts › Settings Developer Tests › T4 - Clicking Clear State should clear users state

Copy link

github-actions bot commented Oct 1, 2024

Playwright test execution is complete! You can find the test results report here

@lgmarchi lgmarchi added ⌨️ Ready for Test PR is ready for test and removed Missing dev review Two dev reviews are required on PR labels Oct 1, 2024
@phillsatellite phillsatellite added QA Approved PR has been tested by QA Team and removed ⌨️ Ready for Test PR is ready for test labels Oct 1, 2024
@dariusc93
Copy link
Contributor

Do keep in mind that in the background it will attempt to obtain the identity of another peer and would emit an event once it is discovered.

@Flemmli97
Copy link
Contributor Author

Do keep in mind that in the background it will attempt to obtain the identity of another peer and would emit an event once it is discovered.

wdym about that? isnt get_identity just a lookup on multipass?

@github-actions github-actions bot added the Failed Automated Test Failed Automated Tests CI label Oct 1, 2024
@phillsatellite phillsatellite removed the Failed Automated Test Failed Automated Tests CI label Oct 1, 2024
@dariusc93
Copy link
Contributor

wdym about that? isnt get_identity just a lookup on multipass?

It is, but internally, depending on what is used, it will dispatch a task to attempt to request it too if it havent been previously discovered and an event is emitted later if it is discovered.

@stavares843 stavares843 merged commit 0339655 into dev Oct 1, 2024
8 checks passed
@stavares843 stavares843 deleted the retry-fetch-identity branch October 1, 2024 18:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA Approved PR has been tested by QA Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants