solution exercise 5
This commit is contained in:
parent
1afddb852e
commit
0721e9d022
|
@ -1,6 +1,27 @@
|
||||||
const sumAll = function() {
|
const sumAll = function(num1, num2) {
|
||||||
|
let total = 0;
|
||||||
|
let a = [];
|
||||||
|
let b = [];
|
||||||
|
if (num1 < num2) {
|
||||||
|
a = num1;
|
||||||
|
b = num2;
|
||||||
|
}else {
|
||||||
|
a = num2;
|
||||||
|
b = num1;
|
||||||
|
}
|
||||||
|
if (a <0 || typeof a == 'string' || typeof b == 'string' || typeof a == 'object' || typeof b == 'object'){
|
||||||
|
console.log('ERROR');
|
||||||
|
return 'ERROR'
|
||||||
|
|
||||||
};
|
}
|
||||||
|
for (i = a; i <= b; i++){
|
||||||
|
total += i;
|
||||||
|
|
||||||
|
}
|
||||||
|
// console.log (a)
|
||||||
|
console.log(total)
|
||||||
|
return total
|
||||||
|
};
|
||||||
|
sumAll(5, 7)
|
||||||
// Do not edit below this line
|
// Do not edit below this line
|
||||||
module.exports = sumAll;
|
module.exports = sumAll;
|
||||||
|
|
|
@ -4,19 +4,19 @@ describe('sumAll', () => {
|
||||||
test('sums numbers within the range', () => {
|
test('sums numbers within the range', () => {
|
||||||
expect(sumAll(1, 4)).toEqual(10);
|
expect(sumAll(1, 4)).toEqual(10);
|
||||||
});
|
});
|
||||||
test.skip('works with large numbers', () => {
|
test('works with large numbers', () => {
|
||||||
expect(sumAll(1, 4000)).toEqual(8002000);
|
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);
|
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');
|
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');
|
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');
|
expect(sumAll(10, [90, 1])).toEqual('ERROR');
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in New Issue