ibessonov commented on code in PR #2261:
URL: https://github.com/apache/ignite-3/pull/2261#discussion_r1244920862


##########
modules/metastorage/src/integrationTest/java/org/apache/ignite/internal/metastorage/impl/ItMetaStorageManagerImplTest.java:
##########
@@ -284,4 +286,24 @@ void testMetaStorageStopBeforeRaftServiceStarted() throws 
Exception {
 
         assertThat(metaStorageManager.metaStorageServiceFuture(), 
willThrowFast(CancellationException.class));
     }
+
+    @Test
+    void testUpdateRevisionListener() {
+        // I'm using the future because mock+verify doesn't work.

Review Comment:
   It didn't work because the code was wrong, I guess. Verifying with timeout 
resolved the issue. I already changed the test anyway...



-- 
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: notifications-unsubscr...@ignite.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org

Reply via email to