hellogabor
diff --git a/‎05_sumAll/sumAll.js
Lines changed: 31 additions & 1 deletion b/‎05_sumAll/sumAll.js
Lines changed: 31 additions & 1 deletion
diff --git a/‎05_sumAll/sumAll.spec.js
Lines changed: 5 additions & 5 deletions b/‎05_sumAll/sumAll.spec.js
Lines changed: 5 additions & 5 deletions
@@ -1,6 +1,36 @@
-const sumAll = function() {
+const sumAll = function(a, b) {
+    if (a > b) {
+        let c = a;
+        a = b;
+        b = c;
+    }
 
+    if (a < 0 || b < 0) {
+        return 'ERROR';
+    }
+
+    if ((typeof a !== 'number') || (typeof b !== 'number')) {
+        return 'ERROR';
+    }
+
+    let allNumbers = [];
+    let i = a;
+    while (i <= b) {
+        allNumbers.push(i);
+        i++;
+    }
+
+    let sum = 0;
+        allNumbers.forEach((number) => {
+        sum += number;
+        });
+    return sum;
 };
 
+/*     const sum = allNumbers.reduce((accumulator, currentValue) => 
+        accumulator + currentValue, 0);
+    return sum; */
+
+
 // Do not edit below this line
 module.exports = sumAll;
@@ -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');
   });
 });