From f74860d12d8e3933571e08e45d01c4d7f7883a12 Mon Sep 17 00:00:00 2001 From: Don Date: Wed, 22 Jun 2022 09:41:50 -0400 Subject: [PATCH] Finished the reverseString activity --- 03_reverseString/reverseString.js | 8 ++++++-- 03_reverseString/reverseString.spec.js | 6 +++--- 2 files changed, 9 insertions(+), 5 deletions(-) diff --git a/03_reverseString/reverseString.js b/03_reverseString/reverseString.js index f6790f0..0990003 100644 --- a/03_reverseString/reverseString.js +++ b/03_reverseString/reverseString.js @@ -1,5 +1,9 @@ -const reverseString = function() { - +const reverseString = function(string) { + reversedArray = [] + for (let i = string.length - 1; i >= 0; i--) { + reversedArray.push(string[i]) + } + return reversedArray.join('') }; // Do not edit below this line diff --git a/03_reverseString/reverseString.spec.js b/03_reverseString/reverseString.spec.js index 8adb887..b51c50e 100644 --- a/03_reverseString/reverseString.spec.js +++ b/03_reverseString/reverseString.spec.js @@ -5,14 +5,14 @@ describe('reverseString', () => { expect(reverseString('hello')).toEqual('olleh'); }); - test.skip('reverses multiple words', () => { + test('reverses multiple words', () => { expect(reverseString('hello there')).toEqual('ereht olleh') }) - test.skip('works with numbers and punctuation', () => { + test('works with numbers and punctuation', () => { expect(reverseString('123! abc!')).toEqual('!cba !321') }) - test.skip('works with blank strings', () => { + test('works with blank strings', () => { expect(reverseString('')).toEqual('') }) });