diff --git a/repeatString/repeatString.js b/repeatString/repeatString.js index 06f0400..50f34c7 100644 --- a/repeatString/repeatString.js +++ b/repeatString/repeatString.js @@ -5,10 +5,10 @@ const repeatString = function(string, number) { for (let i = 0; i < number; i++) { if (number < 0) { return 'ERROR'; - } else { - result += string; } + result += string; + } return result; diff --git a/repeatString/repeatString.spec.js b/repeatString/repeatString.spec.js index 7124484..d2e3daa 100644 --- a/repeatString/repeatString.spec.js +++ b/repeatString/repeatString.spec.js @@ -1,33 +1,33 @@ const repeatString = require('./repeatString') describe('repeatString', function() { - it('repeats the string', function() { - expect(repeatString('hey', 3)).toEqual('heyheyhey'); - }); - xit('repeats the string many times', function() { - expect(repeatString('hey', 10)).toEqual('heyheyheyheyheyheyheyheyheyhey'); - }); - xit('repeats the string 1 times', function() { - expect(repeatString('hey', 1)).toEqual('hey'); - }); - xit('repeats the string 0 times', function() { - expect(repeatString('hey', 0)).toEqual(''); - }); - xit('returns ERROR with negative numbers', function() { - expect(repeatString('hey', -1)).toEqual('ERROR'); - }); - xit('repeats the string a random amount of times', function () { - /*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); - }); - xit('works with blank strings', function() { - expect(repeatString('', 10)).toEqual(''); - }); -}); + it('repeats the string', function() { + expect(repeatString('hey', 3)).toEqual('heyheyhey'); + }); + it('repeats the string many times', function() { + expect(repeatString('hey', 10)).toEqual('heyheyheyheyheyheyheyheyheyhey'); + }); + xit('repeats the string 1 times', function() { + expect(repeatString('hey', 1)).toEqual('hey'); + }); + xit('repeats the string 0 times', function() { + expect(repeatString('hey', 0)).toEqual(''); + }); + xit('returns ERROR with negative numbers', function() { + expect(repeatString('hey', -1)).toEqual('ERROR'); + }); + xit('repeats the string a random amount of times', function() { + /*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); + }); + xit('works with blank strings', function() { + expect(repeatString('', 10)).toEqual(''); + }); +}); \ No newline at end of file