Made changes based on comments received.

Author: A-Moiz

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

@@ -1,14 +1,15 @@
 function getOrdinalNumber(num) {
-  let stringNum = num.toString();
-  let lastDigit = stringNum[stringNum.length - 1];
-  if (lastDigit === "1" && num % 100 !== 11) {
-    return stringNum + "st";
-  } else if (lastDigit === "2" && num % 100 !== 12) {
-    return stringNum + "nd";
-  } else if (lastDigit === "3" && num % 100 !== 13) {
-    return stringNum + "rd";
+  const lastDigit = num % 10;
+  const lastTwoDigits = num % 100;
+
+  if (lastDigit === 1 && lastTwoDigits !== 11) {
+    return num + "st";
+  } else if (lastDigit === 2 && lastTwoDigits !== 12) {
+    return num + "nd";
+  } else if (lastDigit === 3 && lastTwoDigits !== 13) {
+    return num + "rd";
   } else {
-    return stringNum + "th";
+    return num + "th";
   }
 }
 

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

@@ -41,4 +41,7 @@ test("should append 'rd' for numbers ending with 3, except those ending with 13"
 test("should append 'th' for all other numbers", () => {
   expect(getOrdinalNumber(4)).toEqual("4th");
   expect(getOrdinalNumber(11)).toEqual("11th");
+  expect(getOrdinalNumber(12)).toEqual("12th");
+  expect(getOrdinalNumber(35)).toEqual("35th");
+  expect(getOrdinalNumber(100)).toEqual("100th");
 });

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

@@ -1,12 +1,9 @@
 function repeatStr(str, count) {
-  if (count === 0) {
-    return "";
-  } else if (count > 0) {
-    return str.repeat(count);
-  } else {
+  if (count < 0 || !Number.isInteger(count)) {
     throw new Error("Count must be a non-negative integer");
   }
-  return "";
+
+  return count > 0 ? str.repeat(count) : "";
 }
 
 module.exports = repeatStr;

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

@@ -49,3 +49,11 @@ test("should throw an error when count is negative", () => {
     repeatStr(str, count);
   }).toThrow("Count must be a non-negative integer");
 });
+
+test("should throw an error when count is not a number", () => {
+  const str = "error";
+  const count = "-2";
+  expect(() => {
+    repeatStr(str, count);
+  }).toThrow("Count must be a non-negative integer");
+});