ctubbsii commented on code in PR #5192:
URL: https://github.com/apache/accumulo/pull/5192#discussion_r1901368154
##########
server/base/src/test/java/org/apache/accumulo/server/conf/SystemConfigurationTest.java:
##########
@@ -78,7 +81,7 @@ public void initMocks() {
VersionedProperties sysProps =
new VersionedProperties(1, Instant.now(), Map.of(GC_PORT.getKey(),
"1234",
TSERV_SCAN_MAX_OPENFILES.getKey(), "19",
TABLE_BLOOM_ENABLED.getKey(), "true"));
- expect(propStore.get(eq(sysPropKey))).andReturn(sysProps).times(2);
+ expect(propStore.get(eq(sysPropKey))).andReturn(sysProps).once();
Review Comment:
It was always only ever called once. But the test didn't verify the mocks,
so it didn't fail before. I fixed that by adding a method to verify the mock
objects, and corrected the number of expected calls to the correct number of
expected calls.
--
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...@accumulo.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
