From 598ab5c937e541ee64f94a6ffb52bbfce5932c1c Mon Sep 17 00:00:00 2001 From: Bhagyesh Pathak <165566941+bhagyeshsp@users.noreply.github.com> Date: Mon, 3 Feb 2025 09:58:47 +0530 Subject: [PATCH] 02_repeatString/solution - Line-32, replace "odin" with "hey" As asked by @JoshDevHub in a discussion of a related PR, I made a similar change to this solution. In the earlier code, the comment description mentioned "hey" as a sample string but the code had "odin" string. This update addresses that tiny gap between sample comments and related code. More on: https://github.com/TheOdinProject/javascript-exercises/pull/505#pullrequestreview-2588692590 --- 02_repeatString/solution/repeatString-solution.spec.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/02_repeatString/solution/repeatString-solution.spec.js b/02_repeatString/solution/repeatString-solution.spec.js index 1cffa2e..a9ca13e 100644 --- a/02_repeatString/solution/repeatString-solution.spec.js +++ b/02_repeatString/solution/repeatString-solution.spec.js @@ -29,7 +29,7 @@ describe('repeatString', () => { /*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 was randomly generated. */ - expect(repeatString('odin', number).match(/((odin))/g).length).toEqual( + expect(repeatString('hey', number).match(/((hey))/g).length).toEqual( number ); });