finished sumAll.js exercise
This commit is contained in:
parent
300930e15a
commit
1a7921ed56
|
@ -1,5 +1,17 @@
|
||||||
const sumAll = function() {
|
const sumAll = function(num1, num2) {
|
||||||
|
if (Math.sign(num1) === 1 && Math.sign(num2) === 1) {
|
||||||
|
if (Number.isInteger(num1) && Number.isInteger(num2)) {
|
||||||
|
let total = 0;
|
||||||
|
for (let i = num1; i < num2 + 1; i++) {
|
||||||
|
total += i;
|
||||||
|
}
|
||||||
|
return total;
|
||||||
|
} else {
|
||||||
|
return "ERROR";
|
||||||
|
}
|
||||||
|
} else {
|
||||||
|
return "ERROR";
|
||||||
|
}
|
||||||
}
|
}
|
||||||
|
|
||||||
module.exports = sumAll
|
module.exports = sumAll
|
|
@ -4,19 +4,19 @@ describe('sumAll', function() {
|
||||||
it('sums numbers within the range', function() {
|
it('sums numbers within the range', function() {
|
||||||
expect(sumAll(1, 4)).toEqual(10);
|
expect(sumAll(1, 4)).toEqual(10);
|
||||||
});
|
});
|
||||||
xit('works with large numbers', function() {
|
it('works with large numbers', function() {
|
||||||
expect(sumAll(1, 4000)).toEqual(8002000);
|
expect(sumAll(1, 4000)).toEqual(8002000);
|
||||||
});
|
});
|
||||||
xit('works with larger number first', function() {
|
xit('works with larger number first', function() {
|
||||||
expect(sumAll(123, 1)).toEqual(7626);
|
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');
|
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');
|
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');
|
expect(sumAll(10, [90, 1])).toEqual('ERROR');
|
||||||
});
|
});
|
||||||
});
|
});
|
Loading…
Reference in New Issue