Refactor getOrdinalNumber function for clarity and improve test cases for better coverage

Author: Payman IB

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

@@ -1,11 +1,11 @@
 function getOrdinalNumber(num) {
   const suffixes = ["th", "st", "nd", "rd"];
-  const date = num % 100;
+  const lastTwoDigits = num % 100;
 
-  if (date >= 11 && date <= 13) {
+  if (lastTwoDigits >= 11 && lastTwoDigits <= 13) {
     return num + "th";
   }
-  const suffix = suffixes[(date % 10)] || "th";
+  const suffix = suffixes[(lastTwoDigits % 10)] || "th";
   return num + suffix;
 }
 

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

@@ -24,11 +24,16 @@ test("append 'rd' to numbers ending in 3, except those ending in 13", () => {
   expect( getOrdinalNumber(33) ).toEqual("33rd");
     expect( getOrdinalNumber(133) ).toEqual("133rd");
 });   
-test("append 'th' to numbers ending in 4, except those ending in 14", () => {
+test("append 'th' to numbers ending in 4", () => {
   expect(getOrdinalNumber(4)).toEqual("4th");
   expect( getOrdinalNumber(24) ).toEqual("24th");
     expect( getOrdinalNumber(134) ).toEqual("134th");
 });   
 test("should return '11th' for 11", () => {
   expect(getOrdinalNumber(11)).toEqual("11th");
-});
+});
+test("append 'th' to numbers which are not ending in 1, 2, 3", () => {
+  expect(getOrdinalNumber(5)).toEqual("5th");
+  expect( getOrdinalNumber(29) ).toEqual("29th");
+    expect( getOrdinalNumber(138) ).toEqual("138th");
+});