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

Flaky tests pass 1 #616

Merged

bretambrose merged 3 commits into main from FlakyTestPass1

Jan 6, 2025

Contributor

bretambrose commented Dec 19, 2024 •

edited

Loading

Update shared subscription test to remove invalid assertions on a property that does not necessarily hold (all subscriptions receive messages)
Make MQTT will tests more reliable by adding a delay between subscribe and disconnect, and looping the connection attempt in the 311 test which does not have the ability to send a DISCONNECT_WITH_WILL message.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Bret Ambrose added 3 commits

December 19, 2024 11:26


          Don't assert about shared publish routing distribution

f9d25b4


          Update will tests with pause and retry connect in the 311 case

a1c406b


          Autopep

9b45578

sfod approved these changes

View reviewed changes

bretambrose merged commit 87746bb into main

64 checks passed

bretambrose deleted the FlakyTestPass1 branch

January 6, 2025 21:35

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet