10h30/odin-javascript-exercises
1
0
Fork 0
mirror of https://github.com/10h30/odin-javascript-exercises.git synced 2026-07-18 14:13:24 +09:00
Code Issues Packages Projects Releases Wiki Activity

Javascript-exercises: 02_repeatString - Fix code on line 32

Browse Source 
The original code had "hey" as the seed string value and hence the comment on line-29 mentions "hey". But the last commit on the code changed "hey" to "odin".

We can either change instances of comment or the code. I propose changing the code instance from "odin" to "hey" for consistency.
This commit is contained in:
Bhagyesh Pathak
2024-11-13 19:37:26 +05:30
committed by GitHub
parent 203fd76a54
commit 2861e62954
1 changed files with 1 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
02_repeatString/repeatString.spec.js
+1 -1
View File
@@ -29,7 +29,7 @@ describe('repeatString', () => {
/*The .match(/((hey))/g).length is a regex that will count the number of heys /*The .match(/((hey))/g).length is a regex that will count the number of heys
in the result, which if your function works correctly will equal the number that in the result, which if your function works correctly will equal the number that
was randomly generated. */ was randomly generated. */
expect(repeatString('odin', number).match(/((odin))/g).length).toEqual(number); expect(repeatString('hey', number).match(/((hey))/g).length).toEqual(number);
}); });
test.skip('works with blank strings', () => { test.skip('works with blank strings', () => {
expect(repeatString('', 10)).toEqual(''); expect(repeatString('', 10)).toEqual('');