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.

Sign up for GitHub

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

Jump to bottom

[Tests] Fix flaky tests 2024/05/09 #2083

Merged
merged 3 commits into from
May 10, 2024
Merged

[Tests] Fix flaky tests 2024/05/09 #2083

merged 3 commits into from
May 10, 2024

Commits on May 9, 2024

  1. wait for operation repo to start completely 

    Ran 100 times to ensure this test is now stable.
    @jkasten2
    jkasten2 committed May 9, 2024
    Configuration menu
    Copy the full SHA
    346b4ce View commit details
    Browse the repository at this point in the history

  2. flaky test due to not waiting for OperationRepo 

    This test had two problems:
1. We were not waiting for OperationRepo.loadSavedOperations to finish
2. After we added waiting, it didn't work we didn't wrap
RecoverFromDroppedLoginBug in spyk(). This was causing all fields
and properties being cleared from it!
    @jkasten2
    jkasten2 committed May 9, 2024
    Configuration menu
    Copy the full SHA
    685e471 View commit details
    Browse the repository at this point in the history

Commits on May 10, 2024

  1. fix flaky Retry-After test 

    Test sometimes failed on "response2 shouldBe null"
    @jkasten2
    jkasten2 committed May 10, 2024
    Configuration menu
    Copy the full SHA
    939a164 View commit details
    Browse the repository at this point in the history