tilenmars
diff --git a/‎04_removeFromArray/removeFromArray.js
Lines changed: 5 additions & 5 deletions b/‎04_removeFromArray/removeFromArray.js
Lines changed: 5 additions & 5 deletions
diff --git a/‎04_removeFromArray/removeFromArray.spec.js
Lines changed: 5 additions & 5 deletions b/‎04_removeFromArray/removeFromArray.spec.js
Lines changed: 5 additions & 5 deletions
@@ -1,10 +1,10 @@
 const removeFromArray = function(arr, del) {
-    
-    //del -= 1;
     let index;
-    for (let i = 1; i < arguments.length; i++) {
-        index = arr.indexOf(arguments[i]);
-        arr.splice(index, 1);  
+    for (let i = 1; i <= arguments.length; i++) {
+        if (arr.includes(arguments[i])) {
+            index = arr.indexOf(arguments[i]);
+            arr.splice(index, 1);  
+        }
     }
     return arr;
     };
 
@@ -7,19 +7,19 @@ describe('removeFromArray', () => {
   test('removes multiple values', () => {
     expect(removeFromArray([1, 2, 3, 4], 3, 2)).toEqual([1, 4]);
   });
-  test.skip('ignores non present values', () => {
+  test('ignores non present values', () => {
     expect(removeFromArray([1, 2, 3, 4], 7, "tacos")).toEqual([1, 2, 3, 4]);
   });
-  test.skip('ignores non present values, but still works', () => {
+  test('ignores non present values, but still works', () => {
     expect(removeFromArray([1, 2, 3, 4], 7, 2)).toEqual([1, 3, 4]);
   });
-  test.skip('can remove all values', () => {
+  test('can remove all values', () => {
     expect(removeFromArray([1, 2, 3, 4], 1, 2, 3, 4)).toEqual([]);
   });
-  test.skip('works with strings', () => {
+  test('works with strings', () => {
     expect(removeFromArray(["hey", 2, 3, "ho"], "hey", 3)).toEqual([2, "ho"]);
   });
-  test.skip('only removes same type', () => {
+  test('only removes same type', () => {
     expect(removeFromArray([1, 2, 3], "1", 3)).toEqual([1, 2]);
   });
 });