Fix checkstyle warnings.

Author: javadev

checkstyle.xml

@@ -85,11 +85,11 @@
         <module name="UnusedImports"/>
         <module name="RedundantImport"/>
         <module name="LineLength">
-            <property name="severity" value="warning"/>
+            <property name="severity" value="error"/>
             <property name="max" value="120"/>
         </module>
         <module name="MethodLength">
-            <property name="severity" value="warning"/>
+            <property name="severity" value="error"/>
             <property name="max" value="120"/>
         </module>
         <module name="AnonInnerLength">

src/main/java/com/github/underscore/U.java

@@ -1873,7 +1873,8 @@ public static <T> List<List<T>> chunkFill(final Iterable<T> iterable, final int
         return chunkFill(iterable, size, size, fillValue);
     }
 
-    public static <T> List<List<T>> chunkFill(final Iterable<T> iterable, final int size, final int step, final T fillValue) {
+    public static <T> List<List<T>> chunkFill(final Iterable<T> iterable, final int size,
+        final int step, final T fillValue) {
         if (step <= 0 || size < 0) {
             return newArrayList();
         }

src/test/java/com/github/underscore/ArraysTest.java

@@ -300,7 +300,8 @@ public void chunk() {
         assertEquals("[]", new U<Integer>(U.range(7)).chunk(3, 0).toString());
         assertEquals("[[a, b], [c, d]]", U.chain(asList("a", "b", "c", "d")).chunk(2).value().toString());
         assertEquals("[]", U.chain(asList("a", "b", "c", "d")).chunk(0).value().toString());
-        assertEquals("[[a, b], [b, c], [c, d], [d]]", U.chain(asList("a", "b", "c", "d")).chunk(2, 1).value().toString());
+        assertEquals("[[a, b], [b, c], [c, d], [d]]", U.chain(asList("a", "b", "c", "d"))
+            .chunk(2, 1).value().toString());
         assertEquals("[]", U.chain(asList("a", "b", "c", "d")).chunk(4, 0).value().toString());
     }
 
@@ -315,13 +316,16 @@ public void chunkFill() {
         assertEquals("[]", U.chunkFill(U.range(7), -2, 3, 500).toString());
         assertEquals("[]", U.chunkFill(U.range(7), 2, 0, 500).toString());
         assertEquals("[]", U.chunkFill(U.range(7), 2, -2, 500).toString());
-        assertEquals("[[a, b, c], [d, fill, fill]]", new U<String>(asList("a", "b", "c", "d")).chunkFill(3, "fill").toString());
+        assertEquals("[[a, b, c], [d, fill, fill]]", new U<String>(asList("a", "b", "c", "d"))
+            .chunkFill(3, "fill").toString());
         assertEquals("[]", new U<String>(asList("a", "b", "c", "d")).chunkFill(0, "fill").toString());
-        assertEquals("[[0, 1, 2], [2, 3, 4], [4, 5, 500]]", new U<Integer>(U.range(6)).chunkFill(3, 2, 500).toString());
+        assertEquals("[[0, 1, 2], [2, 3, 4], [4, 5, 500]]", new U<Integer>(U.range(6))
+            .chunkFill(3, 2, 500).toString());
         assertEquals("[]", new U<Integer>(U.range(7)).chunkFill(3, 0, 500).toString());
         assertEquals("[[a, b], [c, d]]", U.chain(asList("a", "b", "c", "d")).chunkFill(2, "fill").value().toString());
         assertEquals("[]", U.chain(asList("a", "b", "c", "d")).chunkFill(0, "fill").value().toString());
-        assertEquals("[[a, b], [b, c], [c, d], [d, fill]]", U.chain(asList("a", "b", "c", "d")).chunkFill(2, 1, "fill").value().toString());
+        assertEquals("[[a, b], [b, c], [c, d], [d, fill]]", U.chain(asList("a", "b", "c", "d"))
+            .chunkFill(2, 1, "fill").value().toString());
         assertEquals("[]", U.chain(asList("a", "b", "c", "d")).chunkFill(4, 0, "fill").value().toString());
     }
 
@@ -383,13 +387,15 @@ public void interpose() {
         assertEquals("[]", U.interpose(U.newArrayList(), null).toString());
         assertEquals("[0, 1, 2, 3]", U.interpose(U.newArrayList(U.range(4)), null).toString());
         assertEquals("[0]", U.interpose(U.range(1), 500).toString());
-        assertEquals("[a, interpose, b, interpose, c]", new U<String>(asList("a", "b", "c")).interpose("interpose").toString());
+        assertEquals("[a, interpose, b, interpose, c]", new U<String>(asList("a", "b", "c"))
+            .interpose("interpose").toString());
         assertEquals("[a]", new U<String>(asList("a")).interpose("interpose").toString());
         assertEquals("[a, b]", new U<String>(asList("a, b")).interpose(null).toString());
         assertEquals("[a]", U.chain(asList("a")).interpose("interpose").toString());
         assertEquals("[]", U.chain(U.newArrayList()).interpose("interpose").toString());
         assertEquals("[a, b, c]", U.chain(asList("a", "b", "c")).interpose(null).toString());
-        assertEquals("[?, interpose, !, interpose, -]", U.chain(asList("?", "!", "-")).interpose("interpose").toString());
+        assertEquals("[?, interpose, !, interpose, -]", U.chain(asList("?", "!", "-"))
+            .interpose("interpose").toString());
     }
 
 /*
@@ -417,12 +423,14 @@ public void interposeByList() {
         assertEquals("[a, b, c]", new U<String>(asList("a", "b", "c")).interposeByList(null).toString());
         assertEquals("[a]", new U<String>(asList("a")).interposeByList(asList("zzz")).toString());
         assertEquals("[a, b, c]", new U<String>(asList("a", "b", "c")).interposeByList(list1).toString());
-        assertEquals("[a, aaa, b, bbb, c]", new U<String>(asList("a", "b", "c")).interposeByList(asList("aaa", "bbb", "ccc")).toString());
+        assertEquals("[a, aaa, b, bbb, c]", new U<String>(asList("a", "b", "c"))
+            .interposeByList(asList("aaa", "bbb", "ccc")).toString());
         assertEquals("[a]", U.chain(asList("a")).interposeByList(asList("aaa", "bbb", "ccc")).toString());
         assertEquals("[aaa, bbb, ccc]", U.chain(asList("aaa", "bbb", "ccc")).interposeByList(null).toString());
         list2.clear();
         assertEquals("[]", U.chain(list2).interposeByList(U.range(6)).toString());
-        assertEquals("[?, aaa, !, bbb, -]", U.chain(asList("?", "!", "-")).interposeByList(asList("aaa", "bbb", "ccc")).toString());
+        assertEquals("[?, aaa, !, bbb, -]", U.chain(asList("?", "!", "-"))
+            .interposeByList(asList("aaa", "bbb", "ccc")).toString());
     }
 
 /*

src/test/java/com/github/underscore/UnderscoreTest.java

@@ -211,7 +211,8 @@ public void splitAt() {
         assertEquals("[[0, 1], [2, 3, 4]]", new U<Integer>(U.range(5)).splitAt(2).toString());
         assertEquals("[[0, 1], [2, 3, 4]]", U.chain(U.range(5)).splitAt(2).value().toString());
         assertEquals("[[a, b], [c, d, e]]", U.splitAt(asList('a', 'b', 'c', 'd', 'e'), 2).toString());
-        assertEquals("[[ant, bird], [camel, dog, elephant]]", U.splitAt(asList("ant", "bird", "camel", "dog", "elephant"), 2).toString());
+        assertEquals("[[ant, bird], [camel, dog, elephant]]", U.splitAt(
+            asList("ant", "bird", "camel", "dog", "elephant"), 2).toString());
         assertEquals("[[0.1, 0.2], [0.3, 0.4, 0.5]]", U.splitAt(asList(0.1, 0.2, 0.3, 0.4, 0.5), 2).toString());
         final Integer[] array = {0, 1, 2, 3, 4};
         assertEquals("[[0, 1], [2, 3, 4]]", U.splitAt(array, 2).toString());
@@ -234,7 +235,8 @@ public void takeSkipping() {
         assertEquals("[0, 2, 4]", new U<Integer>(U.range(5)).takeSkipping(2).toString());
         assertEquals("[0, 2, 4]", U.chain(U.range(5)).takeSkipping(2).value().toString());
         assertEquals("[a, c, e]", U.takeSkipping(asList('a', 'b', 'c', 'd', 'e'), 2).toString());
-        assertEquals("[ant, camel, elephant]", U.takeSkipping(asList("ant", "bird", "camel", "dog", "elephant"), 2).toString());
+        assertEquals("[ant, camel, elephant]", U.takeSkipping(
+            asList("ant", "bird", "camel", "dog", "elephant"), 2).toString());
         assertEquals("[0.1, 0.3, 0.5]", U.takeSkipping(asList(0.1, 0.2, 0.3, 0.4, 0.5), 2).toString());
         final Integer[] array = {0, 1, 2, 3, 4, 5, 6, 7, 8, 9};
         assertEquals("[0, 3, 6, 9]", U.takeSkipping(array, 3).toString());

src/test/java/com/github/underscore/lodash/LodashTest.java

@@ -755,6 +755,11 @@ public void accept(String str) {
         U.chain(new Integer[] {0}).distinct();
         U.chain(new Integer[] {0}).distinctBy(new Function<Integer, Integer>() {
             public Integer apply(Integer value) { return value; } });
+    }
+
+    @SuppressWarnings("unchecked")
+    @Test
+    public void chain2() {
         U.chain(new String[] {""}).union();
         U.chain(new String[] {""}).intersection();
         U.chain(new String[] {""}).difference();