Changed let to const and added some comments.
I changed the let to a const and added some explanation to how the test is conducted in comments included in the test.
This commit is contained in:
parent
312103dc4f
commit
fb5677ee8f
1 changed files with 8 additions and 1 deletions
|
@ -17,7 +17,14 @@ describe('repeatString', function() {
|
||||||
expect(repeatString('hey', -1)).toEqual('ERROR');
|
expect(repeatString('hey', -1)).toEqual('ERROR');
|
||||||
});
|
});
|
||||||
xit('repeats the string a random amount of times', function () {
|
xit('repeats the string a random amount of times', function () {
|
||||||
let number = Math.floor(Math.random() * 1000)
|
/*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
|
||||||
|
equals a number between 0 to 999 (this number will change everytime you run
|
||||||
|
the test).*/
|
||||||
|
const number = Math.floor(Math.random() * 1000)
|
||||||
|
/*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 randomaly generated. */
|
||||||
expect(repeatString('hey', number).match(/((hey))/g).length).toEqual(number);
|
expect(repeatString('hey', number).match(/((hey))/g).length).toEqual(number);
|
||||||
});
|
});
|
||||||
});
|
});
|
||||||
|
|
Loading…
Reference in a new issue