diff --git a/03_reverseString/reverseString.js b/03_reverseString/reverseString.js index f6790f0..d2cb276 100644 --- a/03_reverseString/reverseString.js +++ b/03_reverseString/reverseString.js @@ -1,5 +1,9 @@ -const reverseString = function() { - +const reverseString = function (str) { + let newString = ""; + for (let i = str.length - 1; i >= 0; i--) { + newString += str[i]; + } + return newString; }; // Do not edit below this line diff --git a/03_reverseString/reverseString.spec.js b/03_reverseString/reverseString.spec.js index 8adb887..0a39e16 100644 --- a/03_reverseString/reverseString.spec.js +++ b/03_reverseString/reverseString.spec.js @@ -1,18 +1,18 @@ -const reverseString = require('./reverseString') +const reverseString = require("./reverseString"); -describe('reverseString', () => { - test('reverses single word', () => { - expect(reverseString('hello')).toEqual('olleh'); +describe("reverseString", () => { + test("reverses single word", () => { + expect(reverseString("hello")).toEqual("olleh"); }); - test.skip('reverses multiple words', () => { - expect(reverseString('hello there')).toEqual('ereht olleh') - }) + test("reverses multiple words", () => { + expect(reverseString("hello there")).toEqual("ereht olleh"); + }); - test.skip('works with numbers and punctuation', () => { - expect(reverseString('123! abc!')).toEqual('!cba !321') - }) - test.skip('works with blank strings', () => { - expect(reverseString('')).toEqual('') - }) + test("works with numbers and punctuation", () => { + expect(reverseString("123! abc!")).toEqual("!cba !321"); + }); + test("works with blank strings", () => { + expect(reverseString("")).toEqual(""); + }); }); diff --git a/04_removeFromArray/removeFromArray.js b/04_removeFromArray/removeFromArray.js index 1bedeb0..980cc5a 100644 --- a/04_removeFromArray/removeFromArray.js +++ b/04_removeFromArray/removeFromArray.js @@ -1,5 +1,5 @@ -const removeFromArray = function() { - +const removeFromArray = function (arr, ...args) { + return arr.filter((x) => !args.includes(x)); }; // Do not edit below this line diff --git a/04_removeFromArray/removeFromArray.spec.js b/04_removeFromArray/removeFromArray.spec.js index 21f34cf..ebce2bf 100644 --- a/04_removeFromArray/removeFromArray.spec.js +++ b/04_removeFromArray/removeFromArray.spec.js @@ -1,25 +1,25 @@ -const removeFromArray = require('./removeFromArray') +const removeFromArray = require("./removeFromArray"); -describe('removeFromArray', () => { - test('removes a single value', () => { +describe("removeFromArray", () => { + test("removes a single value", () => { expect(removeFromArray([1, 2, 3, 4], 3)).toEqual([1, 2, 4]); }); - test.skip('removes multiple values', () => { + test("removes multiple values", () => { expect(removeFromArray([1, 2, 3, 4], 3, 2)).toEqual([1, 4]); }); - test.skip('ignores non present values', () => { + test("ignores non present values", () => { expect(removeFromArray([1, 2, 3, 4], 7, "tacos")).toEqual([1, 2, 3, 4]); }); - test.skip('ignores non present values, but still works', () => { + test("ignores non present values, but still works", () => { expect(removeFromArray([1, 2, 3, 4], 7, 2)).toEqual([1, 3, 4]); }); - test.skip('can remove all values', () => { + test("can remove all values", () => { expect(removeFromArray([1, 2, 3, 4], 1, 2, 3, 4)).toEqual([]); }); - test.skip('works with strings', () => { + test("works with strings", () => { expect(removeFromArray(["hey", 2, 3, "ho"], "hey", 3)).toEqual([2, "ho"]); }); - test.skip('only removes same type', () => { + test("only removes same type", () => { expect(removeFromArray([1, 2, 3], "1", 3)).toEqual([1, 2]); }); }); diff --git a/05_sumAll/sumAll.js b/05_sumAll/sumAll.js index 00880c7..2fb6d7f 100644 --- a/05_sumAll/sumAll.js +++ b/05_sumAll/sumAll.js @@ -1,5 +1,23 @@ -const sumAll = function() { +const sumAll = function (num1, num2) { + let sum = 0; + if (num1 > num2) { + for (let i = num2; i <= num1; i++) { + sum += i; + } + } else if (num1 < 0 || num2 < 0) { + sum = "ERROR"; + } else if (typeof num1 != "number" || typeof num2 != "number") { + sum = "ERROR"; + } else if (isNaN(num1) || isNaN(num2)) { + sum = "ERROR"; + } else { + for (let i = num1; i <= num2; i++) { + sum += i; + } + } + + return sum; }; // Do not edit below this line diff --git a/05_sumAll/sumAll.spec.js b/05_sumAll/sumAll.spec.js index 1a9fb7c..91d5a27 100644 --- a/05_sumAll/sumAll.spec.js +++ b/05_sumAll/sumAll.spec.js @@ -1,22 +1,22 @@ -const sumAll = require('./sumAll') +const sumAll = require("./sumAll"); -describe('sumAll', () => { - test('sums numbers within the range', () => { +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', () => { - expect(sumAll(-10, 4)).toEqual('ERROR'); + test("returns ERROR with negative numbers", () => { + expect(sumAll(-10, 4)).toEqual("ERROR"); }); - test.skip('returns ERROR with non-number parameters', () => { - expect(sumAll(10, "90")).toEqual('ERROR'); + test("returns ERROR with non-number parameters", () => { + expect(sumAll(10, "90")).toEqual("ERROR"); }); - test.skip('returns ERROR with non-number parameters', () => { - expect(sumAll(10, [90, 1])).toEqual('ERROR'); + test("returns ERROR with non-number parameters", () => { + expect(sumAll(10, [90, 1])).toEqual("ERROR"); }); });