accumulo-commits mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From ctubb...@apache.org
Subject [accumulo] 02/02: Fix checkstyle failures
Date Fri, 14 Sep 2018 03:33:06 GMT
This is an automated email from the ASF dual-hosted git repository.

ctubbsii pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/accumulo.git

commit 9d951016d85a1ca5a223f3bd6f3248c37db48040
Author: Christopher Tubbs <ctubbsii@apache.org>
AuthorDate: Thu Sep 13 22:43:21 2018 -0400

    Fix checkstyle failures
    
    Use static import for Assert.* methods
---
 core/src/test/java/org/apache/accumulo/core/data/LoadPlanTest.java | 4 ++--
 .../main/java/org/apache/accumulo/test/functional/BulkLoadIT.java  | 7 +++----
 2 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/core/src/test/java/org/apache/accumulo/core/data/LoadPlanTest.java b/core/src/test/java/org/apache/accumulo/core/data/LoadPlanTest.java
index fd619d1..0851568 100644
--- a/core/src/test/java/org/apache/accumulo/core/data/LoadPlanTest.java
+++ b/core/src/test/java/org/apache/accumulo/core/data/LoadPlanTest.java
@@ -18,6 +18,7 @@ package org.apache.accumulo.core.data;
 
 import static java.nio.charset.StandardCharsets.UTF_8;
 import static java.util.stream.Collectors.toSet;
+import static org.junit.Assert.assertEquals;
 
 import java.util.HashSet;
 import java.util.Set;
@@ -25,7 +26,6 @@ import java.util.Set;
 import org.apache.accumulo.core.data.LoadPlan.Destination;
 import org.apache.accumulo.core.data.LoadPlan.RangeType;
 import org.apache.hadoop.io.Text;
-import org.junit.Assert;
 import org.junit.Test;
 
 public class LoadPlanTest {
@@ -90,7 +90,7 @@ public class LoadPlanTest {
     Set<String> actual = loadPlan.getDestinations().stream().map(LoadPlanTest::toString)
         .collect(toSet());
 
-    Assert.assertEquals(expected, actual);
+    assertEquals(expected, actual);
 
   }
 
diff --git a/test/src/main/java/org/apache/accumulo/test/functional/BulkLoadIT.java b/test/src/main/java/org/apache/accumulo/test/functional/BulkLoadIT.java
index e5ccfcf..a115385 100644
--- a/test/src/main/java/org/apache/accumulo/test/functional/BulkLoadIT.java
+++ b/test/src/main/java/org/apache/accumulo/test/functional/BulkLoadIT.java
@@ -64,7 +64,6 @@ import org.apache.hadoop.fs.Path;
 import org.apache.hadoop.fs.RawLocalFileSystem;
 import org.apache.hadoop.fs.permission.FsPermission;
 import org.apache.hadoop.io.Text;
-import org.junit.Assert;
 import org.junit.Before;
 import org.junit.Test;
 
@@ -294,7 +293,7 @@ public class BulkLoadIT extends AccumuloClusterHarness {
         .loadFileTo("f3.rf", RangeType.TABLE, null, row(666)).build();
     try {
       c.tableOperations().addFilesTo(tableName).from(dir).plan(loadPlan).load();
-      Assert.fail();
+      fail();
     } catch (IllegalArgumentException e) {
       // ignore
     }
@@ -303,7 +302,7 @@ public class BulkLoadIT extends AccumuloClusterHarness {
     loadPlan = LoadPlan.builder().loadFileTo("f1.rf", RangeType.TABLE, null, row(333)).build();
     try {
       c.tableOperations().addFilesTo(tableName).from(dir).plan(loadPlan).load();
-      Assert.fail();
+      fail();
     } catch (IllegalArgumentException e) {
       // ignore
     }
@@ -313,7 +312,7 @@ public class BulkLoadIT extends AccumuloClusterHarness {
         .loadFileTo("f2.rf", RangeType.TABLE, null, row(555)).build();
     try {
       c.tableOperations().addFilesTo(tableName).from(dir).plan(loadPlan).load();
-      Assert.fail();
+      fail();
     } catch (AccumuloException e) {
       // ignore
     }


Mime
View raw message