Skip to content
This repository has been archived by the owner on Apr 15, 2024. It is now read-only.

test: wait also for second dht in test to pass before doing the checks #307

Merged
merged 1 commit into from
Apr 26, 2023

Conversation

rach-id
Copy link
Member

@rach-id rach-id commented Apr 22, 2023

Overview

This should fix a test that fails from time to time because the test isn't waiting for the second DHT routing table to populate:
https://github.com/celestiaorg/orchestrator-relayer/actions/runs/4773521264/jobs/8486641970

Checklist

  • New and updated code has appropriate documentation
  • New and updated code has new and/or updated testing
  • Required CI checks are passing
  • Visual proof for any user facing features like CLI or documentation updates
  • Linked issues closed with keywords

@rach-id rach-id self-assigned this Apr 22, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #307 (ef79d55) into main (bdfe030) will increase coverage by 0.20%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main     #307      +/-   ##
==========================================
+ Coverage   43.39%   43.59%   +0.20%     
==========================================
  Files          26       26              
  Lines        1991     1991              
==========================================
+ Hits          864      868       +4     
+ Misses       1005     1002       -3     
+ Partials      122      121       -1     

see 1 file with indirect coverage changes

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@rach-id rach-id changed the title test: wait also for second dht in test to pass before doing the cheks test: wait also for second dht in test to pass before doing the checks Apr 25, 2023
@rach-id rach-id merged commit 69e9f3c into celestiaorg:main Apr 26, 2023
20 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants