search

coherence-community / oracle-bedrock

Oracle Bedrock
Other
55 stars 31 forks source link

Eventually.assertThat(T value, ...) should not retry on non-deferred values #321

Closed brianoliver closed 8 years ago

brianoliver commented 8 years ago

Consider the case where by an application does the following;

Eventually.assertThat(1, is(2));

There's no reason that this method should attempt to retry this test for the Timeout. Instead it should try once and then give up, potentially with a warning/advice that the assert was not retried as it was a simple value. If the intent was to retry, a Deferred should be used of an invoking(...) call.

brianoliver commented 8 years ago

This issue was imported from JIRA ORACLETOOLS-321

brianoliver commented 8 years ago

Reported by @brianoliver