changed condition rule in function getOrdinalNumber() and added new test cases and added throw error

Author: M-Abdoon

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

@@ -1,6 +1,6 @@
 function getOrdinalNumber(num) {
-  if (typeof num !== "number" || num <= 0) {
-    return "Input must be a positive integer.";
+  if (!Number.isInteger(num) || num <= 0) {
+    throw new Error("Input must be a positive integer.");
   }
 
   const lastTwoDigits = num % 100;
@@ -23,5 +23,5 @@ function getOrdinalNumber(num) {
   return num;  
 }
 
-console.log(getOrdinalNumber(21));    // "1st"
+//console.log(getOrdinalNumber(21));    // "1st"
 module.exports = getOrdinalNumber;

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

@@ -60,4 +60,20 @@ test("should return correct ordinal suffixes for big numbers", () => {
     expect( getOrdinalNumber(4712) ).toEqual("4712th");
     expect( getOrdinalNumber(10003) ).toEqual("10003rd");
     expect( getOrdinalNumber(10012) ).toEqual("10012th");
+});
+
+
+// Case 6: dealing with invalid inputs
+// test("should return error message", () => {
+//     expect( getOrdinalNumber(NaN) ).toEqual("Input must be a positive integer.");
+//     expect( getOrdinalNumber(Infinity) ).toEqual("Input must be a positive integer.");
+//     expect( getOrdinalNumber("hello") ).toEqual("Input must be a positive integer.");
+//     expect( getOrdinalNumber([0, 3]) ).toEqual("Input must be a positive integer.");
+// });
+
+test("should throw an error stating that input must be a positive integer", () => {
+    expect(() => getOrdinalNumber(NaN)).toThrow("Input must be a positive integer.");
+    expect(() => getOrdinalNumber(Infinity)).toThrow("Input must be a positive integer.");
+    expect(() => getOrdinalNumber("hello")).toThrow("Input must be a positive integer.");
+    expect(() => getOrdinalNumber([0, 1, 2])).toThrow("Input must be a positive integer.");
 });