Skip to content

Commit 1655393

Browse files
committed
BAEL-2770 Fixed code formatting and improved test cases
1 parent 9230192 commit 1655393

File tree

2 files changed

+17
-14
lines changed

2 files changed

+17
-14
lines changed

Diff for: core-java-arrays/src/main/java/com/baeldung/array/AddElementToEndOfArray.java

+3-3
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ public class AddElementToEndOfArray {
88
public Integer[] addElementUsingArraysCopyOf(Integer[] srcArray, int elementToAdd) {
99
Integer[] destArray = Arrays.copyOf(srcArray, srcArray.length + 1);
1010

11-
destArray[destArray.length-1] = elementToAdd;
11+
destArray[destArray.length - 1] = elementToAdd;
1212
return destArray;
1313
}
1414

@@ -26,9 +26,9 @@ public Integer[] addElementUsingSystemArrayCopy(Integer[] srcArray, int elementT
2626

2727
System.arraycopy(srcArray, 0, destArray, 0, srcArray.length);
2828

29-
destArray[destArray.length-1] = elementToAdd;
29+
destArray[destArray.length - 1] = elementToAdd;
3030

3131
return destArray;
3232
}
3333

34-
}
34+
}

Diff for: core-java-arrays/src/test/java/com/baeldung/array/AddElementToEndOfArrayUnitTest.java

+14-11
Original file line numberDiff line numberDiff line change
@@ -3,44 +3,47 @@
33
import org.junit.Before;
44
import org.junit.Test;
55

6-
import static org.junit.Assert.assertEquals;
6+
import static org.junit.Assert.assertArrayEquals;
77

88
public class AddElementToEndOfArrayUnitTest {
99

1010
AddElementToEndOfArray addElementToEndOfArray;
1111

1212
@Before
13-
public void init(){
13+
public void init() {
1414
addElementToEndOfArray = new AddElementToEndOfArray();
1515
}
1616

1717
@Test
18-
public void givenSourceArrayAndElement_whenAddElementUsingArraysCopyIsInvoked_thenNewElementMustBeAdded(){
19-
Integer[] sourceArray = {1,2,3,4};
18+
public void givenSourceArrayAndElement_whenAddElementUsingArraysCopyIsInvoked_thenNewElementMustBeAdded() {
19+
Integer[] sourceArray = {1, 2, 3, 4};
2020
int elementToAdd = 5;
2121

2222
Integer[] destArray = addElementToEndOfArray.addElementUsingArraysCopyOf(sourceArray, elementToAdd);
2323

24-
assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
24+
Integer[] expectedArray = {1, 2, 3, 4, 5};
25+
assertArrayEquals(expectedArray, destArray);
2526
}
2627

2728
@Test
28-
public void givenSourceArrayAndElement_whenAddElementUsingArrayListIsInvoked_thenNewElementMustBeAdded(){
29-
Integer[] sourceArray = {1,2,3,4};
29+
public void givenSourceArrayAndElement_whenAddElementUsingArrayListIsInvoked_thenNewElementMustBeAdded() {
30+
Integer[] sourceArray = {1, 2, 3, 4};
3031
int elementToAdd = 5;
3132

3233
Integer[] destArray = addElementToEndOfArray.addElementUsingArrayList(sourceArray, elementToAdd);
3334

34-
assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
35+
Integer[] expectedArray = {1, 2, 3, 4, 5};
36+
assertArrayEquals(expectedArray, destArray);
3537
}
3638

3739
@Test
38-
public void givenSourceArrayAndElement_whenAddElementUsingSystemArrayCopyIsInvoked_thenNewElementMustBeAdded(){
39-
Integer[] sourceArray = {1,2,3,4};
40+
public void givenSourceArrayAndElement_whenAddElementUsingSystemArrayCopyIsInvoked_thenNewElementMustBeAdded() {
41+
Integer[] sourceArray = {1, 2, 3, 4};
4042
int elementToAdd = 5;
4143

4244
Integer[] destArray = addElementToEndOfArray.addElementUsingSystemArrayCopy(sourceArray, elementToAdd);
4345

44-
assertEquals(elementToAdd, destArray[destArray.length-1].intValue());
46+
Integer[] expectedArray = {1, 2, 3, 4, 5};
47+
assertArrayEquals(expectedArray, destArray);
4548
}
4649
}

0 commit comments

Comments
 (0)