alex-plekhanov commented on code in PR #12763:
URL: https://github.com/apache/ignite/pull/12763#discussion_r2822739831
##########
modules/core/src/test/java/org/apache/ignite/internal/processors/cache/GridCacheAbstractFailoverSelfTest.java:
##########
@@ -374,11 +379,7 @@ private void remove(Ignite ignite, IgniteCache<String,
Integer> cache, final int
* @throws Exception If failed.
*/
private void check(final IgniteCache<String, Integer> cache, final int
expSize) throws Exception {
- GridTestUtils.waitForCondition(new GridAbsPredicate() {
- @Override public boolean apply() {
- return cache.size() >= expSize;
- }
- }, 5000);
+ assertTrue(GridTestUtils.waitForCondition(() -> cache.size() >=
expSize, 5000));
Review Comment:
Maybe leave here waitForCondition without assertTrue? There are checks for
size with correct message below. Or, at least move this message to this
assertTrue and remove checks below.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
