Skip to content

tests: Remove flaky persistent peers test from wsNetwork#5772

Merged
gmalouf merged 1 commit intoalgorand:masterfrom
gmalouf:remove-persistent-peers-flaky-test
Oct 3, 2023
Merged

tests: Remove flaky persistent peers test from wsNetwork#5772
gmalouf merged 1 commit intoalgorand:masterfrom
gmalouf:remove-persistent-peers-flaky-test

Conversation

@gmalouf
Copy link
Copy Markdown
Contributor

@gmalouf gmalouf commented Oct 3, 2023

Remove flaky persistent peers test from wsNetwork (TestUpdatePhonebookAddresses covers this).

@gmalouf gmalouf added Enhancement test Improves testing of existing code labels Oct 3, 2023
@gmalouf gmalouf requested review from jasonpaulos and winder October 3, 2023 19:23
@ohill ohill changed the title tests: Remove flaky persistent peers test from wsNetwork (TestUpdatePhoneboo… tests: Remove flaky persistent peers test from wsNetwork Oct 3, 2023
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 3, 2023

Codecov Report

Merging #5772 (47f6d44) into master (79fb8fe) will decrease coverage by 0.05%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #5772      +/-   ##
==========================================
- Coverage   55.50%   55.45%   -0.05%     
==========================================
  Files         473      473              
  Lines       66684    66684              
==========================================
- Hits        37010    36979      -31     
- Misses      27156    27186      +30     
- Partials     2518     2519       +1     

see 9 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@gmalouf gmalouf merged commit a23449d into algorand:master Oct 3, 2023
@gmalouf gmalouf deleted the remove-persistent-peers-flaky-test branch October 3, 2023 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Enhancement test Improves testing of existing code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants