Merge pull request TheOdinProject#432 from damon314159/patch-1

CouchofTomato · web-flow · commit d740f2de0922 · 2024-02-05T15:14:54.000Z
04_removeFromArrray: missing test case
diff --git a/04_removeFromArray/removeFromArray.spec.js b/04_removeFromArray/removeFromArray.spec.js
@@ -7,6 +7,9 @@ describe('removeFromArray', () => {
   test.skip('removes multiple values', () => {
     expect(removeFromArray([1, 2, 3, 4], 3, 2)).toEqual([1, 4]);
   });
+  test.skip('removes multiple of the same value', () => {
+    expect(removeFromArray([1, 2, 2, 3], 2)).toEqual([1, 3]);
+  });
   test.skip('ignores non present values', () => {
     expect(removeFromArray([1, 2, 3, 4], 7, "tacos")).toEqual([1, 2, 3, 4]);
   });
diff --git a/04_removeFromArray/solution/removeFromArray-solution.spec.js b/04_removeFromArray/solution/removeFromArray-solution.spec.js
@@ -7,6 +7,9 @@ describe('removeFromArray', () => {
   test('removes multiple values', () => {
     expect(removeFromArray([1, 2, 3, 4], 3, 2)).toEqual([1, 4]);
   });
+  test('removes multiple of the same value', () => {
+    expect(removeFromArray([1, 2, 2, 3], 2)).toEqual([1, 3]);
+  });
   test('ignores non present values', () => {
     expect(removeFromArray([1, 2, 3, 4], 7, 'tacos')).toEqual([1, 2, 3, 4]);
   });
