passed all test cases

pom.xml

@@ -21,6 +21,12 @@
             <artifactId>junit</artifactId>
             <version>4.12</version>
         </dependency>
+        <dependency>
+            <groupId>com.zipcodewilmington</groupId>
+            <artifactId>arrayutility</artifactId>
+            <version>1.0-SNAPSHOT</version>
+            <scope>test</scope>
+        </dependency>
     </dependencies>
 
 

src/main/java/com/zipcodewilmington/arrayutility/ArrayUtility.java

@@ -1,5 +1,6 @@
 package com.zipcodewilmington.arrayutility;
 
+import java.lang.reflect.Array;
 import java.util.ArrayList;
 import java.util.Arrays;
 
@@ -8,13 +9,13 @@ import java.util.Arrays;
  */
 public class ArrayUtility<S> {
     ArrayList<S> newlist=new ArrayList<S>();
-
+    S[] inputArray;
     public ArrayUtility(S[] inputArray) {
-
+        this.inputArray=inputArray;
         this.newlist.addAll(Arrays.asList(inputArray));
+        //S[] temp = (S[]) new Object[length];(creating a new generic array)
     }
     public Integer countDuplicatesInMerge(S[] inputArray, S valueToEvaluate ) {
-        // Display array elements
         Integer count=0;
         newlist.addAll(Arrays.asList(inputArray));
         for (S element : newlist) {
@@ -43,7 +44,7 @@ public class ArrayUtility<S> {
     public int getNumberOfOccurrences(S valueToEvaluate)
     {
         int countOccurence=0;
-        for(S element :newlist)
+        for(S element :inputArray)
         {
             if(element.equals(valueToEvaluate))
             {
@@ -52,21 +53,22 @@ public class ArrayUtility<S> {
         }
 
         return countOccurence;
+
     }
 
-   /* public S[] removeValue(S valueToRemove)
+    public S[] removeValue(S valueToRemove)
     {
-        ArrayList<S> resultList=new ArrayList<S>();
-
-        for(S element : newlist)
-        {
-            if(!element.equals(valueToRemove))
-            {
-                resultList.add(valueToRemove);
+        Integer removed = getNumberOfOccurrences(valueToRemove);
+        S[] array = (S[])Array.newInstance(inputArray.getClass().getComponentType(), inputArray.length-removed);
+        int count = 0;
+        for (int i = 0; i < inputArray.length; i++) {
+            if (!inputArray[i].equals(valueToRemove)) {
+                array[count] = inputArray[i];
+                count++;
             }
         }
-        S[] newArray=new (S[])[resultList.size()];
+        return array;
 
+    }
 
-    }*/
 }

src/test/java/com/zipcodewilmington/arrayutility/ArrayUtilityTestSuite.java

@@ -1,19 +1,19 @@
-//package com.zipcodewilmington.arrayutility;
-//
-//import org.junit.runner.RunWith;
-//import org.junit.runners.Suite;
-//
-///**
-// * Created by leon on 3/1/18.
-// * The purpose of this class is to test all classes within this package
-// */
-//@RunWith(Suite.class)
-//
-//@Suite.SuiteClasses({
-//        CountDuplicatesInMergeTest.class,
-//        GetMostCommonFromMergeTest.class,
-//        GetNumberOfOccurrencesTest.class,
-//        RemoveValueTest.class
-//})
-//public class ArrayUtilityTestSuite {
-//}
+package com.zipcodewilmington.arrayutility;
+
+import org.junit.runner.RunWith;
+import org.junit.runners.Suite;
+
+/**
+ * Created by leon on 3/1/18.
+ * The purpose of this class is to test all classes within this package
+ */
+@RunWith(Suite.class)
+
+@Suite.SuiteClasses({
+        CountDuplicatesInMergeTest.class,
+        GetMostCommonFromMergeTest.class,
+        GetNumberOfOccurrencesTest.class,
+        RemoveValueTest.class
+})
+public class ArrayUtilityTestSuite {
+}

src/test/java/com/zipcodewilmington/arrayutility/RemoveValueTest.java

@@ -1,78 +1,78 @@
-//package com.zipcodewilmington.arrayutility;
-//
+package com.zipcodewilmington.arrayutility;
+
 //import com.zipcodewilmington.UnitTestingUtils;
-//import org.junit.Test;
-//
-///**
-// * Created by leon on 3/1/18.
-// * The purpose of this class is to thoroughly test the method removeValue()
-// */
-//public class RemoveValueTest {
-//    @Test
-//    public void integerTest() {
-//        // Given
-//        Integer valueToRemove = 7;
-//        Integer[] expected = {11, 2, 8, 4, 5, 0, 9, 8};
-//        Integer[] inputArray = {11, 2, valueToRemove, 8, 4, 5, valueToRemove, 0, 9, 8, valueToRemove};
-//        ArrayUtility<Integer> arrayUtility = new ArrayUtility<Integer>(inputArray);
-//
-//
-//        // When
-//        Integer[] actual = arrayUtility.removeValue(valueToRemove);
-//
-//        // Then
-//        UnitTestingUtils.assertArrayEquality(expected, actual);
-//    }
-//
-//    @Test
-//    public void longTest() {
-//        // Given
-//        Long valueToRemove = 7L;
-//        Long[] expected = {12L, 2L, 8L, 4L, 5L, 0L, 9L, 8L};
-//        Long[] inputArray = {12L, 2L, valueToRemove, 8L, 4L, 5L, valueToRemove, 0L, 9L, 8L, valueToRemove};
-//        ArrayUtility<Long> arrayUtility = new ArrayUtility<Long>(inputArray);
-//
-//
-//        // When
-//        Long[] actual = arrayUtility.removeValue(valueToRemove);
-//
-//        // Then
-//        UnitTestingUtils.assertArrayEquality(expected, actual);
-//    }
-//
-//
-//    @Test
-//    public void stringTest() {
-//        // Given
-//        String valueToRemove = "7";
-//        String[] expected = {"13", "2", "8", "4", "5", "0", "9", "8"};
-//        String[] inputArray = {"13", "2", valueToRemove, "8", "4", "5", valueToRemove, "0", "9", "8", valueToRemove};
-//        ArrayUtility<String> arrayUtility = new ArrayUtility<String>(inputArray);
-//
-//
-//        // When
-//        String[] actual = arrayUtility.removeValue(valueToRemove);
-//
-//        // Then
-//        UnitTestingUtils.assertArrayEquality(expected, actual);
-//    }
-//
-//
-//
-//    @Test
-//    public void objectTest() {
-//        // Given
-//        Object valueToRemove = "7";
-//        Object[] expected = {"41", "2", "8", "4", "5", "0", "9", "8"};
-//        Object[] inputArray = {"41", "2", valueToRemove, "8", "4", "5", valueToRemove, "0", "9", "8", valueToRemove};
-//        ArrayUtility<Object> arrayUtility = new ArrayUtility<Object>(inputArray);
-//
-//
-//        // When
-//        Object[] actual = arrayUtility.removeValue(valueToRemove);
-//
-//        // Then
-//        UnitTestingUtils.assertArrayEquality(expected, actual);
-//    }
-//
-//}
+import org.junit.Test;
+
+/**
+ * Created by leon on 3/1/18.
+ * The purpose of this class is to thoroughly test the method removeValue()
+ */
+public class RemoveValueTest {
+    @Test
+    public void integerTest() {
+        // Given
+        Integer valueToRemove = 7;
+        Integer[] expected = {11, 2, 8, 4, 5, 0, 9, 8};
+        Integer[] inputArray = {11, 2, valueToRemove, 8, 4, 5, valueToRemove, 0, 9, 8, valueToRemove};
+        ArrayUtility<Integer> arrayUtility = new ArrayUtility<Integer>(inputArray);
+
+
+        // When
+        Integer[] actual = arrayUtility.removeValue(valueToRemove);
+
+        // Then
+        UnitTestingUtils.assertArrayEquality(expected, actual);
+    }
+
+    @Test
+    public void longTest() {
+        // Given
+        Long valueToRemove = 7L;
+        Long[] expected = {12L, 2L, 8L, 4L, 5L, 0L, 9L, 8L};
+        Long[] inputArray = {12L, 2L, valueToRemove, 8L, 4L, 5L, valueToRemove, 0L, 9L, 8L, valueToRemove};
+        ArrayUtility<Long> arrayUtility = new ArrayUtility<Long>(inputArray);
+
+
+        // When
+        Long[] actual = arrayUtility.removeValue(valueToRemove);
+
+        // Then
+        UnitTestingUtils.assertArrayEquality(expected, actual);
+    }
+
+
+    @Test
+    public void stringTest() {
+        // Given
+        String valueToRemove = "7";
+        String[] expected = {"13", "2", "8", "4", "5", "0", "9", "8"};
+        String[] inputArray = {"13", "2", valueToRemove, "8", "4", "5", valueToRemove, "0", "9", "8", valueToRemove};
+        ArrayUtility<String> arrayUtility = new ArrayUtility<String>(inputArray);
+
+
+        // When
+        String[] actual = arrayUtility.removeValue(valueToRemove);
+
+        // Then
+        UnitTestingUtils.assertArrayEquality(expected, actual);
+    }
+
+
+
+    @Test
+    public void objectTest() {
+        // Given
+        Object valueToRemove = "7";
+        Object[] expected = {"41", "2", "8", "4", "5", "0", "9", "8"};
+        Object[] inputArray = {"41", "2", valueToRemove, "8", "4", "5", valueToRemove, "0", "9", "8", valueToRemove};
+        ArrayUtility<Object> arrayUtility = new ArrayUtility<Object>(inputArray);
+
+
+        // When
+        Object[] actual = arrayUtility.removeValue(valueToRemove);
+
+        // Then
+        UnitTestingUtils.assertArrayEquality(expected, actual);
+    }
+
+}