Remove arbitrary sleep from multiDM integrationTest.

amino-os / Amino.Run

Amino Distributed OS - Runtime Manager

Apache License 2.0

29 stars 12 forks source link

Remove arbitrary sleep from multiDM integrationTest. #797

Closed quinton-hoole closed 5 years ago

quinton-hoole commented 5 years ago

In summary:

Rather than wait an arbitrary 5 seconds for ConsensusRSM leader to be elected, explicitly catch LeaderException, and retry until timeout, but only if AtLeastOnceRPC is not in the DM chain.

quinton-hoole commented 5 years ago

Addressed all review comments, confirmed that all tests and examples succeed:

$ ./gradlew check examples:run
BUILD SUCCESSFUL in 7m 16s
99 actionable tasks: 99 executed