Merge pull request #6 from M-Bujak/05_sumAll

solved 05

05_sumAll/sumAll.js

@@ -1,5 +1,24 @@
-const sumAll = function() {
+const sumAll = function () {
+    if (typeof(arguments[0]) === 'number' && typeof(arguments[1]) === 'number' && arguments[0] >= 0 && arguments[1] >= 0) {
+        let biggerNum = 0, smallerNum = 0;
+        if (arguments[0] > arguments[1]) {
+            biggerNum = arguments[0];
+            smallerNum = arguments[1];
+        } else {
+            biggerNum = arguments[1];
+            smallerNum = arguments[0];
+        }
 
+        let result = 0;
+
+        for (let i = smallerNum; i <= biggerNum; i++) {
+            result += i;
+        }
+
+        return result;
+    }
+
+    return 'ERROR';
 };
 
 // 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');
   });
 });