[CAMEL-19523] camel-iec60870: replace Thread.sleep in tests

• Type: Task
• Status: Resolved
• Priority: Minor
• Resolution: Fixed
• Affects Version/s: 4.0.0
• Fix Version/s: 4.7.0
• Component/s: camel-iec60870, tests
• Labels: easy help-wanted
• Estimated Complexity: Novice
• Assignee: Aurélien Pupier
• Reporter: Otavio Rodolfo Piske
• Created: 28/Jun/23 08:23
• Updated: 14/Jun/24 11:35
• Resolved: 14/Jun/24 11:35

Description

We have many tests which use Thread.sleep for synchronization. This is bug prone and can introduce flakiness when running on environments with different capacities.

Ideally we should replace these with:

• Awaitility
• Java's native syncronization mechanism (Latches, Phasers, Locks, etc)
• Nothing (i.e.; in some cases the sleep can simply be removed)

 

 

Issue Links

• is a clone of: CAMEL-19514 camel-aws: replace Thread.sleep in tests
• links to: GitHub Pull Request #14530