Use built-in .repeat() method instead of manual implementation; add one more test and verify all tests with the new function

Author: AhmadHmedann

Sprint-3/2-practice-tdd/repeat.js

@@ -1,10 +1,24 @@
 function repeat(myString, repeatNumber) {
   if (repeatNumber < 0) return "Invalid Input must be a positive number";
-  let str = "";
-  for (let i = 0; i < repeatNumber; i++) {
-    str += myString;
-  }
-  return str;
+  //if (repeatNumber < 0) throw new Error("Repeat count must be a positive number");
+  //  if I use this instead return how I can test it with jest??
+  return myString.repeat(repeatNumber);
 }
 
 module.exports = repeat;
+
+// Note:
+// When I wrote this code, I didn’t know there was already a built-in method for this.
+// Please review this implementation as well.
+//if (repeatNumber < 0) throw new Error("Repeat count must be a positive number"); if I use this instead return how I can test it with jest??
+
+// function repeat(myString, repeatNumber) {
+//   if (repeatNumber < 0) return "Invalid Input must be a positive number";
+// let str = "";
+// for (let i = 0; i < repeatNumber; i++) {
+//   str += myString;
+// }
+// return str;
+// }
+
+//if (repeatNumber < 0) throw new Error("Repeat count must be a positive number");

Sprint-3/2-practice-tdd/repeat.test.js

@@ -43,9 +43,21 @@ test("should return empty when the count is 0", () => {
 // Given a target string str and a negative integer count,
 // When the repeat function is called with these inputs,
 // Then it should throw an error or return an appropriate error message, as negative counts are not valid.
-test("should return empty when the count is 0", () => {
+test("should return empty when the count is negative", () => {
   const str = "CYF";
   const count = -2;
   const repeatedStr = repeat(str, count);
   expect(repeatedStr).toEqual("Invalid Input must be a positive number");
 });
+test("should repeat the string five times when the count is 5", () => {
+  const str = "CYF";
+  const count = 5;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toEqual("CYFCYFCYFCYFCYF");
+});
+test("should return an empty string when the str is empty whatever was count equal", () => {
+  const str = "";
+  const count = 10;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toEqual("");
+});