finished removeFromArray.js exercise

This commit is contained in:
Denzel 2020-01-02 15:07:10 +08:00
parent 25625f266b
commit 300930e15a
2 changed files with 18 additions and 9 deletions

View File

@ -1,5 +1,14 @@
const removeFromArray = function() {
const removeFromArray = function(array, ...items) {
items.forEach(item => {
if (array.indexOf(item) >= 0) {
const index = array.indexOf(item);
array.splice(index, 1);
} else {
return
}
})
console.log(array);
return array;
}
module.exports = removeFromArray
module.exports = removeFromArray

View File

@ -4,19 +4,19 @@ describe('removeFromArray', function() {
it('removes a single value', function() {
expect(removeFromArray([1, 2, 3, 4], 3)).toEqual([1, 2, 4]);
});
xit('removes multiple values', 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() {
it('ignores non present values, but still works', function() {
expect(removeFromArray([1, 2, 3, 4], 7, 2)).toEqual([1, 3, 4]);
});
xit('can remove all values', function() {
it('can remove all values', function() {
expect(removeFromArray([1, 2, 3, 4], 1, 2, 3, 4)).toEqual([]);
});
xit('works with strings', function() {
it('works with strings', function() {
expect(removeFromArray(["hey", 2, 3, "ho"], "hey", 3)).toEqual([2, "ho"]);
});
});
});