Andy51011
diff --git a/‎sumAll/sumAll.js
Lines changed: 19 additions & 2 deletions b/‎sumAll/sumAll.js
Lines changed: 19 additions & 2 deletions
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,22 @@
-const sumAll = function() {
-
+const sumAll = function(val1, val2) { 
+    if (typeof val1 !== "number" || typeof val2 !== "number") {
+        return "ERROR"
+    }
+    if (val1 < 0 || val2 < 0) {
+        return "ERROR"
+    }
+    var sum = 0
+    if (val1 < val2) {
+        for (var i = val1; i <= val2; i++) {
+            sum += i
+        }
+    }
+    if (val1 > val2) {
+        for (var i = val2; i <= val1; i++) {
+            sum += i
+        }
+    }
+  return sum
 }
 
 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');
   });
 });