tod
/
odin-default-js-exercises
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Ignores non-present values

This commit is contained in:
Mohammed Nabeel 2020-07-15 18:26:03 +03:00
parent 4250c06303
commit 1aa228d42c
2 changed files with 4 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
removeFromArray/removeFromArray.js
View File

@ -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;

2
removeFromArray/removeFromArray.spec.js
View File

@ -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() {