When I started working as a Java Developer, me and my teammate got a first task to repair all broken tests (great task for new starters!) in some old project. Replacing some old configuration and upgrading a few libraries helped making the tests status green but there was another problem. End-to-end test suite took a few hours to pass making it impossible to get the feedback for a new change quickly. Sometimes build run for a few hours and hanged so E2E tetst became excluded from the feedback loop at the end.
Most of the tests from that suite did the same thing:
- Invoke a few commands on the mainframe through HTTP interface
- Sleep for 30 seconds (probably just to be extra safe) for each command and check if the result was persisted to database by another system.
It turned out, that first try to reduce the sleep time to 15 seconds by my teammate, didn’t make any tests fail, so that was a nice first try. But what was the optimal wait time for each test?
After some time, we tried another approach. If the flow has a asynchronous nature then why not testing it asynchronously.
I introduced this simple class (or at least class which does the same):
Method await checks some assertion in a loop until either it won’t throw AssertionError or the timeout is passed. This assertion is usually some state we are waiting for that should be ready in the nearest future. If it is not ready after the timeout, the last message from the AssertionError will be thrown wrapped as a RuntimeException. This greatly reduces test time because if the result is ready earlier, we don’t have to spend unnecessary time in the sleep method.
To represent the assertion result we can use classes (minimal implementation):
To represent the assertion result we can use classes (minimal implementation):
We also need to prepare our AssertionResult suppliers - wrappers which fetch the data and invoke assertions on the result. Usually it is possible to define generic supplier if the way we fetch the data is similar. For example we can write this builder method:
We are ready to replace Thread.sleep() with call to Await in our tests:
Conclusion
Don’t overuse Thread.sleep() in your test code. According to this Google research it is one of the main reasons of flaky tests. Asynchronous approach may complicate your tests but the time savings can be tremendous. If you are looking for some existing implementations there is a nice class in Spock which does the same.
Add a comment