Remove warnings from apache#1495

Remove unused variable warnings
ctubbsii · Feb 20, 2020 · 2b0b971 · 2b0b971
1 parent d02a662
commit 2b0b971
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 5 deletions.
diff --git a/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java b/test/src/main/java/org/apache/accumulo/test/functional/CleanUpIT.java
@@ -39,6 +39,8 @@
 import org.slf4j.Logger;
 import org.slf4j.LoggerFactory;
 
+import com.google.common.collect.Iterables;
+
 /**
  * Ensures that all threads spawned for ZooKeeper and Thrift connectivity are reaped after calling
  * CleanUp.shutdown().
@@ -130,10 +132,7 @@ public void run() throws Exception {
       }
 
       try {
-        count = 0;
-        for (Entry<Key,Value> keyValueEntry : scanner) {
-          count++;
-        }
+        count = Iterables.size(scanner);
         fail("scanner did not fail");
       } catch (Exception e) {
 

diff --git a/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java b/test/src/main/java/org/apache/accumulo/test/functional/PermissionsIT.java
@@ -20,6 +20,7 @@
 
 import static org.junit.Assert.assertEquals;
 import static org.junit.Assert.assertFalse;
+import static org.junit.Assert.assertNotNull;
 import static org.junit.Assert.assertTrue;
 
 import java.io.IOException;
@@ -722,7 +723,7 @@ private void testGrantedTablePermission(AccumuloClient test_user_client, TablePe
       case READ:
         try (Scanner scanner = test_user_client.createScanner(tableName, Authorizations.EMPTY)) {
           for (Entry<Key,Value> keyValueEntry : scanner) {
-            // empty
+            assertNotNull(keyValueEntry);
           }
         }
         break;