refactor: update three function and add tests for three files

Author: shreefAhmedM

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

@@ -1,5 +1,11 @@
-function countChar(stringOfCharacters, findCharacter) {
-  return 5
+function countChar(stringOfCharacters, findChar) {
+  let count = 0;
+  for (const char of stringOfCharacters) {
+    if (char === findChar) {
+      count++;
+    }
+  }
+  return count;
 }
 
 module.exports = countChar;

Sprint-3/2-practice-tdd/get-ordinal-number.js

@@ -1,5 +1,18 @@
 function getOrdinalNumber(num) {
-  return "1st";
+  let last2Digit = num % 100;
+  let las1Digit = num % 10;
+  if (last2Digit >= 11 && last2Digit <= 13) {
+    return num + "th";
+  }
+  if (las1Digit === 1) {
+    return num + "st";
+  } else if (las1Digit === 2) {
+    return num + "nd";
+  } else if (las1Digit === 3) {
+    return num + "rd";
+  } else {
+    return num + "th";
+  }
 }
 
 module.exports = getOrdinalNumber;

Sprint-3/2-practice-tdd/get-ordinal-number.test.js

@@ -11,3 +11,17 @@ const getOrdinalNumber = require("./get-ordinal-number");
 test("should return '1st' for 1", () => {
   expect(getOrdinalNumber(1)).toEqual("1st");
 });
+test("should return '22nd' for 22", () => {
+  expect(getOrdinalNumber(22)).toEqual("22nd");
+});
+
+test("should return '3th' for 3", () => {
+  expect(getOrdinalNumber(3)).toEqual("3rd");
+});
+test("should return '23rd' for 23", () => {
+  expect(getOrdinalNumber(23)).toEqual("23rd");
+});
+
+test("should return '100th' for 100", () => {
+  expect(getOrdinalNumber(100)).toEqual("100th");
+});

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

@@ -1,5 +1,7 @@
-function repeat() {
-  return "hellohellohello";
+function repeat(str, count) {
+  if (count < 0) {
+    return "Count must be non-negative";
+  }
+  return str.repeat(count);
 }
-
 module.exports = repeat;

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

@@ -21,12 +21,31 @@ test("should repeat the string count times", () => {
 // 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 str without repetition", () => {
+  const str = "shreef";
+  const count = 1;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toEqual("shreef");
+});
+
 // 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", () => {
+  const str = "ahmed";
+  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 or return an appropriate error message", () => {
+  const str = "jone";
+  const count = -5;
+  const repeatedStr = repeat(str, count);
+  expect(repeatedStr).toBe("Count must be non-negative");
+});