sk0x50 commented on a change in pull request #121:
URL: https://github.com/apache/ignite-3/pull/121#discussion_r638641278
##########
File path:
modules/metastorage/src/test/java/org/apache/ignite/internal/metastorage/WatchAggregatorTest.java
##########
@@ -131,6 +147,167 @@ public void testCancelByFalseFromListener() {
}
+ /**
+ *
+ */
+ @Test
+ public void testOneCriterionInference() {
+ var watchAggregator = new WatchAggregator();
+
+ watchAggregator.add(new ByteArray("key"), null);
+
+ var keyCriterion = watchAggregator.watch(0, null).get().keyCriterion();
+ assertEquals(new KeyCriterion.ExactCriterion(new ByteArray("key")),
keyCriterion);
Review comment:
Please try to add a human-readable message that provides details.
"Test failed: expected 1, actual 2" looks very concise to me.
--
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.
For queries about this service, please contact Infrastructure at:
[email protected]
