Solution
This commit is contained in:
parent
25c0e43491
commit
f5fea2adf3
|
@ -1,5 +1,14 @@
|
|||
const removeFromArray = function() {
|
||||
let removeFromArray = function(...args) {
|
||||
let array = args[0];
|
||||
|
||||
const newArray = [];
|
||||
|
||||
array.forEach((item) => {
|
||||
if (!args.includes(item)) {
|
||||
newArray.push(item);
|
||||
}
|
||||
});
|
||||
return newArray;
|
||||
};
|
||||
|
||||
module.exports = removeFromArray
|
||||
|
|
|
@ -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"]);
|
||||
});
|
||||
});
|
||||
|
|
Loading…
Reference in New Issue