Merge pull request TheOdinProject#524 from RedViperPT/issue_javascript-exercises_05_sumAll

JoshDevHub · web-flow · commit d4421b055f6c · 2025-02-11T16:03:10.000-07:00
05_sumAll: Fix tests to require correct range summation
diff --git a/05_sumAll/solution/sumAll-solution.spec.js b/05_sumAll/solution/sumAll-solution.spec.js
@@ -2,7 +2,7 @@ const sumAll = require('./sumAll-solution');
 
 describe('sumAll', () => {
   test('sums numbers within the range', () => {
-    expect(sumAll(1, 4)).toEqual(10);
+    expect(sumAll(2, 4)).toEqual(9);
   });
   test('works with large numbers', () => {
     expect(sumAll(1, 4000)).toEqual(8002000);
diff --git a/05_sumAll/sumAll.spec.js b/05_sumAll/sumAll.spec.js
@@ -2,7 +2,7 @@ const sumAll = require('./sumAll')
 
 describe('sumAll', () => {
   test('sums numbers within the range', () => {
-    expect(sumAll(1, 4)).toEqual(10);
+    expect(sumAll(2, 4)).toEqual(9);
   });
   test.skip('works with large numbers', () => {
     expect(sumAll(1, 4000)).toEqual(8002000);
