diff --git a/removeFromArray/removeFromArray.js b/removeFromArray/removeFromArray.js index d69f9e1..6d882c0 100644 --- a/removeFromArray/removeFromArray.js +++ b/removeFromArray/removeFromArray.js @@ -7,7 +7,9 @@ const removeFromArray = function(...args) { // Remove all the remaining elements from the array for (let i = 1; i < args.length; i++) { pos = myArray.indexOf(args[i]); - myArray.splice(pos, 1); + if (pos >= 0) { //Is it a valid argument + myArray.splice(pos, 1); + } } return myArray; diff --git a/removeFromArray/removeFromArray.spec.js b/removeFromArray/removeFromArray.spec.js index 60903a8..a1f499e 100644 --- a/removeFromArray/removeFromArray.spec.js +++ b/removeFromArray/removeFromArray.spec.js @@ -7,7 +7,7 @@ describe('removeFromArray', function() { it('removes multiple values', function() { expect(removeFromArray([1, 2, 3, 4], 3, 2)).toEqual([1, 4]); }); - xit('ignores non present values', function() { + it('ignores non present values', function() { expect(removeFromArray([1, 2, 3, 4], 7, "tacos")).toEqual([1, 2, 3, 4]); }); xit('ignores non present values, but still works', function() {