Add another tests for count test and change the function for repeat-str.

Author: B3-Bisrat

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

@@ -36,3 +36,50 @@ test("should return 0 when no occurrences of a character are found", () => {
   const count = countChar(str, char);
   expect(count).toEqual(0);
 });
+
+// Scenario: Empty String or Space
+// Given the input string `str`,
+// And a character `char` that is an empty string or a space.
+// When the function is called with these inputs,
+// Then it should handle these edge cases appropriately, returning 0 for an empty string and counting spaces correctly.
+
+test("should return 0 when the string is empty", () => {
+  const str = "";
+  const char = "a";
+  const count = countChar(str, char);
+  expect(count).toEqual(0);
+});
+
+test("should count characters correctly when the string contains spaces", () => {
+  const str = "a a a";
+  const char = "a";
+  const count = countChar(str, char);
+  expect(count).toEqual(3);
+});
+
+test("should count spaces correctly", () => {
+  const str = "a a a";
+  const char = " ";
+  const count = countChar(str, char);
+  expect(count).toEqual(2);
+});
+
+// Scenario: Uppercase and Lowercase Characters
+// Given the input string `str`,
+// And a character `char` that is either uppercase or lowercase.
+// When the function is called with these inputs,
+// Then it should be case-sensitive, counting occurrences of `char` based on its exact case.
+
+test("should be case-sensitive when searching for lowercase letters", () => {
+  const str = "Apple";
+  const char = "a";
+  const count = countChar(str, char);
+  expect(count).toEqual(0);
+});
+
+test("should be case-sensitive when searching for uppercase letters", () => {
+  const str = "Apple";
+  const char = "A";
+  const count = countChar(str, char);
+  expect(count).toEqual(1);
+});

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

@@ -1,16 +1,15 @@
 function repeatStr(stringOfCharacters, count) {
-  // Your implementation of this function must *not* call String.prototype.repeat (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat).
-  // The goal is to re-implement that function, not to use it.
-
-  if (count === 0) {
-    return "";
+  if (count < 0) {
+    throw new Error("Invalid input: count must be a non-negative integer");
   }
 
-  if (count > 0) {
-    return repeatStr(stringOfCharacters, count - 1) + stringOfCharacters;
+  let result = "";
+
+  for (let i = 0; i < count; i++) {
+    result += stringOfCharacters;
   }
 
-  throw new Error("Invalid input: count must be a non-negative integer");
+  return result;
 }
 
 module.exports = repeatStr;

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

@@ -4,6 +4,19 @@ const repeatStr = require("./repeat-str");
 // When the repeatStr function is called with these inputs,
 // Then it should:
 
+// Case: Handle negative count:
+// Given a target string `str` and a negative integer `count`,
+// When the repeatStr function is called with these inputs,
+// Then it should throw an error, as negative counts are not valid.
+
+test("should throw an error when count is negative", () => {
+  const str = "error";
+  const count = -2;
+  expect(() => repeatStr(str, count)).toThrow(
+    "Invalid input: count must be a non-negative integer"
+  );
+});
+
 // Case: handle multiple repetitions:
 // Given a target string `str` and a positive integer `count` greater than 1,
 // When the repeatStr function is called with these inputs,
@@ -39,16 +52,3 @@ test("should return an empty string when count is 0", () => {
   const repeatedStr = repeatStr(str, count);
   expect(repeatedStr).toEqual("");
 });
-
-// Case: Handle negative count:
-// Given a target string `str` and a negative integer `count`,
-// When the repeatStr function is called with these inputs,
-// Then it should throw an error, as negative counts are not valid.
-
-test("should throw an error when count is negative", () => {
-  const str = "error";
-  const count = -2;
-  expect(() => repeatStr(str, count)).toThrow(
-    "Invalid input: count must be a non-negative integer"
-  );
-});