From f573853be9c0350ce41c7f2c5ae1f5a1a0dbb28e Mon Sep 17 00:00:00 2001 From: reddforman Date: Wed, 9 Nov 2022 07:20:44 -0800 Subject: [PATCH] Add files via upload --- 05_sumAll/sumAll.js | 21 +++++++++++++++++++-- 05_sumAll/sumAll.spec.js | 10 +++++----- 2 files changed, 24 insertions(+), 7 deletions(-) diff --git a/05_sumAll/sumAll.js b/05_sumAll/sumAll.js index 00880c7..7239243 100644 --- a/05_sumAll/sumAll.js +++ b/05_sumAll/sumAll.js @@ -1,6 +1,23 @@ -const sumAll = function() { - +const sumAll = function(num1, num2) { + if (typeof num1 === "number" && typeof num2 === "number") { + smallNumber = Math.min(num1, num2); + bigNumber = Math.max(num1, num2); + sum = smallNumber; + for (let i = smallNumber; i <= bigNumber; i++) { + if (i > smallNumber) { + sum = sum + i; + } + } + if (sum >=0) { + return sum; + } else { + return "ERROR"; + } + } else { + return "ERROR"; + } }; // Do not edit below this line module.exports = sumAll; + diff --git a/05_sumAll/sumAll.spec.js b/05_sumAll/sumAll.spec.js index 1a9fb7c..a171e5f 100644 --- a/05_sumAll/sumAll.spec.js +++ b/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'); }); });