fivan18
diff --git a/‎sumAll/sumAll.js
Lines changed: 16 additions & 1 deletion b/‎sumAll/sumAll.js
Lines changed: 16 additions & 1 deletion
diff --git a/‎sumAll/sumAll.spec.js
Lines changed: 5 additions & 5 deletions b/‎sumAll/sumAll.spec.js
Lines changed: 5 additions & 5 deletions
@@ -1,5 +1,20 @@
-const sumAll = function() {
+const sumAll = function(...args) {
+    if(
+        isNaN(args[0]) || isNaN(args[1]) || 
+        args[0] < 0 || args[1] < 0 ||
+        typeof args[0] === "string" || typeof args[1] === "string"
+        ){
+        return 'ERROR';
+    }
 
+    args.sort((a,b) => a-b);
+
+    let count = 0;
+    for(let i = args[0]; i <= args[1]; i++){
+        count += i;
+    }
+    return count;
+    
 }
 
 module.exports = sumAll
@@ -4,19 +4,19 @@ describe('sumAll', function() {
   it('sums numbers within the range', function() {
     expect(sumAll(1, 4)).toEqual(10);
   });
-  xit('works with large numbers', function() {
+  it('works with large numbers', function() {
     expect(sumAll(1, 4000)).toEqual(8002000);
   });
-  xit('works with larger number first', function() {
+  it('works with larger number first', function() {
     expect(sumAll(123, 1)).toEqual(7626);
   });
-  xit('returns ERROR with negative numbers', function() {
+  it('returns ERROR with negative numbers', function() {
     expect(sumAll(-10, 4)).toEqual('ERROR');
   });
-  xit('returns ERROR with non-number parameters', function() {
+  it('returns ERROR with non-number parameters', function() {
     expect(sumAll(10, "90")).toEqual('ERROR');
   });
-  xit('returns ERROR with non-number parameters', function() {
+  it('returns ERROR with non-number parameters', function() {
     expect(sumAll(10, [90, 1])).toEqual('ERROR');
   });
 });