Modified files for exercise TheOdinProject#5. Completed exercise TheOdinProject#5.

Author: ibrahimcode85

05_sumAll/sumAll.js

@@ -1,5 +1,31 @@
-const sumAll = function() {
+const sumAll = function(start, end) {
+    
+    //only number datatype
+    if (typeof start !== 'number' || typeof end !== 'number') {
+        return 'ERROR';
+    }
+    
+    //only positive number
+    if (start < 0 || end < 0) {
+        return 'ERROR';
+    };
 
+    if (start > end){
+        startSum = end;
+        endSum = start;
+
+    } else {
+        startSum = start;
+        endSum  = end;
+    }
+
+    sumNum = 0;
+
+    for (let num = startSum; num <= endSum; num++ ) {
+        sumNum = sumNum + num;
+    };
+
+    return sumNum;
 };
 
 // Do not edit below this line

05_sumAll/sumAll.spec.js

@@ -4,19 +4,19 @@ describe('sumAll', () => {
   test('sums numbers within the range', () => {
     expect(sumAll(1, 4)).toEqual(10);
   });
-  test.skip('works with large numbers', () => {
+  test('works with large numbers', () => {
     expect(sumAll(1, 4000)).toEqual(8002000);
   });
-  test.skip('works with larger number first', () => {
+  test('works with larger number first', () => {
     expect(sumAll(123, 1)).toEqual(7626);
   });
-  test.skip('returns ERROR with negative numbers', () => {
+  test('returns ERROR with negative numbers', () => {
     expect(sumAll(-10, 4)).toEqual('ERROR');
   });
-  test.skip('returns ERROR with non-number parameters', () => {
+  test('returns ERROR with non-number parameters', () => {
     expect(sumAll(10, "90")).toEqual('ERROR');
   });
-  test.skip('returns ERROR with non-number parameters', () => {
+  test('returns ERROR with non-number parameters', () => {
     expect(sumAll(10, [90, 1])).toEqual('ERROR');
   });
 });