[
https://issues.apache.org/jira/browse/ACCUMULO-2418?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13914757#comment-13914757
]
Mike Drob commented on ACCUMULO-2418:
-------------------------------------
minor nit: you dropped the fail message from {{NoMutationRecoveryIT.java}} when
it could have been part of the assert.
{noformat}
- for (@SuppressWarnings("unused") Entry<Key, Value> refs : getLogRefs(conn,
MetadataTable.NAME)) {
- fail("should not have any refs");
- }
+ assertEquals(0, FunctionalTestUtils.count(getLogRefs(conn,
MetadataTable.NAME)));
{noformat}
> replace empty loops in tests with simple count method
> -----------------------------------------------------
>
> Key: ACCUMULO-2418
> URL: https://issues.apache.org/jira/browse/ACCUMULO-2418
> Project: Accumulo
> Issue Type: Bug
> Components: test
> Reporter: Eric Newton
> Assignee: Eric Newton
> Priority: Trivial
> Fix For: 1.6.0
>
>
> While fixing ACCUMULO-2405, I created a simple method to eliminate a loop.
> There are many tests that could use this method.
--
This message was sent by Atlassian JIRA
(v6.1.5#6160)
