Merge pull request #522 from MaoShizhong/repeat-string-test

02_repeatString: Replace regex test with built-in repeat method
This commit is contained in:
Josh Smith
2025-02-10 20:04:47 -07:00
committed by GitHub
2 changed files with 19 additions and 25 deletions
+10 -12
View File
@@ -24,19 +24,17 @@ describe('repeatString', () => {
expect(repeatString('goodbye', -1)).toEqual('ERROR'); expect(repeatString('goodbye', -1)).toEqual('ERROR');
}); });
test.skip('repeats the string a random amount of times', function () { test.skip('repeats the string a random amount of times', function () {
/*The number is generated by using Math.random to get a value from between /* The number is generated by using Math.random to get a value from between
0 to 1, when this is multiplied by 1000 and rounded down with Math.floor it 0 to 1, when this is multiplied by 1000 and rounded down with Math.floor it
equals a number between 0 to 999 (this number will change everytime you run equals a number between 0 to 999 (this number will change everytime you run
the test).*/ the test).
// DO NOT use Math.floor(Math.random() * 1000) in your code, DO NOT use Math.floor(Math.random() * 1000) in your code,
// this test generates a random number, then passes it into your code with a function parameter. this test generates a random number, then passes it into your code with a function parameter.
// If this doesn't make sense, you should go read about functions here: https://www.theodinproject.com/paths/foundations/courses/foundations/lessons/fundamentals-part-3 If this doesn't make sense, you should go read about functions here:
const number = Math.floor(Math.random() * 1000) https://www.theodinproject.com/paths/foundations/courses/foundations/lessons/fundamentals-part-3 */
/*The .match(/((hey))/g).length is a regex that will count the number of heys const number = Math.floor(Math.random() * 1000);
in the result, which if your function works correctly will equal the number that expect(repeatString('hey', number)).toBe('hey'.repeat(number));
was randomly generated. */
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('');
@@ -24,21 +24,17 @@ describe('repeatString', () => {
expect(repeatString('goodbye', -1)).toEqual('ERROR'); expect(repeatString('goodbye', -1)).toEqual('ERROR');
}); });
test('repeats the string a random amount of times', function () { test('repeats the string a random amount of times', function () {
/*The number is generated by using Math.random to get a value from between /* The number is generated by using Math.random to get a value from between
0 to 1, when this is multiplied by 1000 and rounded down with Math.floor it 0 to 1, when this is multiplied by 1000 and rounded down with Math.floor it
equals a number between 0 to 999 (this number will change everytime you run equals a number between 0 to 999 (this number will change everytime you run
the test).*/ the test).
// DO NOT use Math.floor(Math.random() * 1000) in your code, DO NOT use Math.floor(Math.random() * 1000) in your code,
// this test generates a random number, then passes it into your code with a function parameter. this test generates a random number, then passes it into your code with a function parameter.
// If this doesn't make sense, you should go read about functions here: https://www.theodinproject.com/paths/foundations/courses/foundations/lessons/fundamentals-part-3 If this doesn't make sense, you should go read about functions here:
https://www.theodinproject.com/paths/foundations/courses/foundations/lessons/fundamentals-part-3 */
const number = Math.floor(Math.random() * 1000); const number = Math.floor(Math.random() * 1000);
/*The .match(/((hey))/g).length is a regex that will count the number of heys expect(repeatString('hey', number)).toBe('hey'.repeat(number));
in the result, which if your function works correctly will equal the number that
was randomly generated. */
expect(repeatString('hey', number).match(/((hey))/g).length).toEqual(
number
);
}); });
test('works with blank strings', () => { test('works with blank strings', () => {
expect(repeatString('', 10)).toEqual(''); expect(repeatString('', 10)).toEqual('');