completed sumAll

cotrones · cotrones · commit 2e85daf90554 · 2020-07-27T11:56:10.000-04:00
diff --git a/sumAll/sumAll.js b/sumAll/sumAll.js
@@ -1,5 +1,19 @@
-const sumAll = function() {
-
+const sumAll = function(x, y) {
+	let result = 0;
+	if (!(Number.isInteger(x)) || !(Number.isInteger(y)) || (x < 0) || (y < 0)) {
+		return 'ERROR';
+	} else {
+		if (x > y) {
+			for (let i = y; i <= x; i++) {
+				result += i;
+			}
+		} else {
+			for (let i = x; i <= y; i++) {
+				result += i;
+			}
+		}
+	}
+	return result;
 }
 
 module.exports = sumAll
diff --git a/sumAll/sumAll.spec.js b/sumAll/sumAll.spec.js
@@ -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');
   });
 });
