Add functional tests and integration tests for Waiters

[Quanzzzz]

Add integration tests for DynamoDB Waiters;
Add functional tests for RestJsonWithWaiters Waiters;

License

• I confirm that this pull request can be released under the Apache 2 license

[dagnir]

Seems like we need to test other scenarios as well such as

• What happens when we reach max number of attempts
• What happens when we get an unexpected exception
• What happens to the client and scheduled executor when the the waiter is closed?
• etc.

cc: @zoewangg for more suggestions

[Quanzzzz]

The max number of attempts case is added in sync Waiters. Unexpected exception case has been covered by the tests for generic Waiters. For the third case, if the client and scheduled executor are configured while creating the Waiters then they won't be closed while the Waiters are closed, and tests related to this case are also added.

[zoewangg]

@dagnir Good call! I added the test cases with unexpected exception and caught a bug in waiterRuntime 😅

[Quanzzzz]

😅Guess that's my fault. I misunderstood the unmatched exception case in the BaseWaiterTest, thanks for making sure about it again.

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!

0 Bugs
0 Vulnerabilities (and 0 Security Hotspots to review)
1 Code Smell

100.0% Coverage
0.0% Duplication

The version of Java (1.8.0_252) you have used to run this analysis is deprecated and we will stop accepting it from October 2020. Please update to at least Java 11.
Read more here