Refactor repeat function to handle input validation and add comprehensive test cases

Author: Baba05206

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

@@ -1,5 +1,8 @@
-function repeat() {
-  return "hellohellohello";
+function repeat(str, count) {
+  if (typeof str !== "string") throw new Error("Input must be a string");
+  if (typeof count !== "number" || count < 0)
+    throw new Error("Count must be a non-negative number");
+  return str.repeat(count);
 }
 
 module.exports = repeat;

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

@@ -20,13 +20,41 @@ test("should repeat the string count times", () => {
 // Given a target string str and a count equal to 1,
 // When the repeat function is called with these inputs,
 // Then it should return the original str without repetition, ensuring that a count of 1 results in no repetition.
+test("should return the original string when count is 1", () => {
+  const str = "world";
+  const count = 1;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toEqual("world");
+});
 
 // case: Handle Count of 0:
 // Given a target string str and a count equal to 0,
 // When the repeat function is called with these inputs,
 // Then it should return an empty string, ensuring that a count of 0 results in an empty output.
+test("should return an empty string when count is 0", () => {
+  const str = "test";
+  const count = 0;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toEqual("");
+});
 
 // case: Negative Count:
 // 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 throw an error for a negative count", () => {
+  const str = "error";
+  const count = -2;
+  expect(() => repeat(str, count)).toThrow(
+    "Count must be a non-negative number"
+  );
+});
+// case: Non-String Input:
+// Given a non-string input for str (e.g., a number, object, or array) and a positive integer count,
+// When the repeat function is called with these inputs,
+// Then it should throw an error or return an appropriate error message, as the function is expected to handle only string inputs.
+test("should throw an error for non-string input", () => {
+  const str = 12345; // Non-string input
+  const count = 2;
+  expect(() => repeat(str, count)).toThrow("Input must be a string");
+});