From d67e7a3ecb7597b3f5516f62ec178961340df1df Mon Sep 17 00:00:00 2001 From: vishpant76 Date: Tue, 7 May 2024 00:10:50 +0530 Subject: [PATCH] 03_reverseString - update punctuation test case to include comma (#467) --- 03_reverseString/reverseString.spec.js | 16 +++++++++------- .../solution/reverseString-solution.spec.js | 4 +++- 2 files changed, 12 insertions(+), 8 deletions(-) diff --git a/03_reverseString/reverseString.spec.js b/03_reverseString/reverseString.spec.js index 8adb887..6ade95b 100644 --- a/03_reverseString/reverseString.spec.js +++ b/03_reverseString/reverseString.spec.js @@ -1,4 +1,4 @@ -const reverseString = require('./reverseString') +const reverseString = require('./reverseString'); describe('reverseString', () => { test('reverses single word', () => { @@ -6,13 +6,15 @@ describe('reverseString', () => { }); test.skip('reverses multiple words', () => { - expect(reverseString('hello there')).toEqual('ereht olleh') - }) + expect(reverseString('hello there')).toEqual('ereht olleh'); + }); test.skip('works with numbers and punctuation', () => { - expect(reverseString('123! abc!')).toEqual('!cba !321') - }) + expect(reverseString('123! abc! Hello, Odinite.')).toEqual( + '.etinidO ,olleH !cba !321' + ); + }); test.skip('works with blank strings', () => { - expect(reverseString('')).toEqual('') - }) + expect(reverseString('')).toEqual(''); + }); }); diff --git a/03_reverseString/solution/reverseString-solution.spec.js b/03_reverseString/solution/reverseString-solution.spec.js index 629e071..bdf9ea6 100644 --- a/03_reverseString/solution/reverseString-solution.spec.js +++ b/03_reverseString/solution/reverseString-solution.spec.js @@ -10,7 +10,9 @@ describe('reverseString', () => { }); test('works with numbers and punctuation', () => { - expect(reverseString('123! abc!')).toEqual('!cba !321'); + expect(reverseString('123! abc! Hello, Odinite.')).toEqual( + '.etinidO ,olleH !cba !321' + ); }); test('works with blank strings', () => { expect(reverseString('')).toEqual('');