hiero-ledger / hiero-ledger/hiero-consensus-node
Remove Thread.sleep from tests
Open
HapiThursday
- Dominant language
- Java
- Stars
- 406
- Forks
- 226
- Avg merge
- 3d 4h
- Merged PRs (30d)
- 210
Description
We probably shouldn't be doing a Thread.sleep anywhere in the tests. Doing a Thread.sleep is going to increase the amount of time it takes to run tests. We should instead be acting based on events in the system or changes in the state.
Contributor guide
Research direction
The issue does not identify specific files or tests. Search the Java test suite for Thread.sleep, then determine which event or state change each test should observe instead; done means the sleeps are removed and the affected tests still pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- java
- Domain
- testing
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 30/100