BAEL-2770 Fixed code formatting and improved test cases

sskoli95 · sskoli95 · commit 16553936c7ae · 2019-04-22T20:48:49.000+05:30
diff --git a/core-java-arrays/src/main/java/com/baeldung/array/AddElementToEndOfArray.java b/core-java-arrays/src/main/java/com/baeldung/array/AddElementToEndOfArray.java
@@ -8,7 +8,7 @@ public class AddElementToEndOfArray {
     public Integer[] addElementUsingArraysCopyOf(Integer[] srcArray, int elementToAdd) {
         Integer[] destArray = Arrays.copyOf(srcArray, srcArray.length + 1);
 
-        destArray[destArray.length-1] = elementToAdd;
+        destArray[destArray.length - 1] = elementToAdd;
         return destArray;
     }
 
@@ -26,9 +26,9 @@ public Integer[] addElementUsingSystemArrayCopy(Integer[] srcArray, int elementT
 
         System.arraycopy(srcArray, 0, destArray, 0, srcArray.length);
 
-        destArray[destArray.length-1] = elementToAdd;
+        destArray[destArray.length - 1] = elementToAdd;
 
         return destArray;
     }
 
-}
+}
diff --git a/core-java-arrays/src/test/java/com/baeldung/array/AddElementToEndOfArrayUnitTest.java b/core-java-arrays/src/test/java/com/baeldung/array/AddElementToEndOfArrayUnitTest.java
@@ -3,44 +3,47 @@
 import org.junit.Before;
 import org.junit.Test;
 
-import static org.junit.Assert.assertEquals;
+import static org.junit.Assert.assertArrayEquals;
 
 public class AddElementToEndOfArrayUnitTest {
 
     AddElementToEndOfArray addElementToEndOfArray;
 
     @Before
-    public void init(){
+    public void init() {
         addElementToEndOfArray = new AddElementToEndOfArray();
     }
 
     @Test
-    public void givenSourceArrayAndElement_whenAddElementUsingArraysCopyIsInvoked_thenNewElementMustBeAdded(){
-        Integer[] sourceArray = {1,2,3,4};
+    public void givenSourceArrayAndElement_whenAddElementUsingArraysCopyIsInvoked_thenNewElementMustBeAdded() {
+        Integer[] sourceArray = {1, 2, 3, 4};
         int elementToAdd = 5;
 
         Integer[] destArray = addElementToEndOfArray.addElementUsingArraysCopyOf(sourceArray, elementToAdd);
 
-        assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
+        Integer[] expectedArray = {1, 2, 3, 4, 5};
+        assertArrayEquals(expectedArray, destArray);
     }
 
     @Test
-    public void givenSourceArrayAndElement_whenAddElementUsingArrayListIsInvoked_thenNewElementMustBeAdded(){
-        Integer[] sourceArray = {1,2,3,4};
+    public void givenSourceArrayAndElement_whenAddElementUsingArrayListIsInvoked_thenNewElementMustBeAdded() {
+        Integer[] sourceArray = {1, 2, 3, 4};
         int elementToAdd = 5;
 
         Integer[] destArray = addElementToEndOfArray.addElementUsingArrayList(sourceArray, elementToAdd);
 
-        assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
+        Integer[] expectedArray = {1, 2, 3, 4, 5};
+        assertArrayEquals(expectedArray, destArray);
     }
 
     @Test
-    public void givenSourceArrayAndElement_whenAddElementUsingSystemArrayCopyIsInvoked_thenNewElementMustBeAdded(){
-        Integer[] sourceArray = {1,2,3,4};
+    public void givenSourceArrayAndElement_whenAddElementUsingSystemArrayCopyIsInvoked_thenNewElementMustBeAdded() {
+        Integer[] sourceArray = {1, 2, 3, 4};
         int elementToAdd = 5;
 
         Integer[] destArray = addElementToEndOfArray.addElementUsingSystemArrayCopy(sourceArray, elementToAdd);
 
-        assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
+        Integer[] expectedArray = {1, 2, 3, 4, 5};
+        assertArrayEquals(expectedArray, destArray);
     }
 }

Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ public class AddElementToEndOfArray {`
`8`	`8`	`public Integer[] addElementUsingArraysCopyOf(Integer[] srcArray, int elementToAdd) {`
`9`	`9`	`Integer[] destArray = Arrays.copyOf(srcArray, srcArray.length + 1);`
`10`	`10`
`11`		`- destArray[destArray.length-1] = elementToAdd;`
	`11`	`+ destArray[destArray.length - 1] = elementToAdd;`
`12`	`12`	`return destArray;`
`13`	`13`	`}`
`14`	`14`
`@@ -26,9 +26,9 @@ public Integer[] addElementUsingSystemArrayCopy(Integer[] srcArray, int elementT`
`26`	`26`
`27`	`27`	`System.arraycopy(srcArray, 0, destArray, 0, srcArray.length);`
`28`	`28`
`29`		`- destArray[destArray.length-1] = elementToAdd;`
	`29`	`+ destArray[destArray.length - 1] = elementToAdd;`
`30`	`30`
`31`	`31`	`return destArray;`
`32`	`32`	`}`
`33`	`33`
`34`		`-}`
	`34`	`+}`