Made changes in the TDD Test as per review

Author: dcostaprakash

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

@@ -28,3 +28,15 @@ test("should return 0 when the character is not found", () => {
   const count = countChar(str, char);
   expect(count).toEqual(0);
 });
+test("should return 0 when the character is case sensitive", () => {
+  const str = "Hello";
+  const char = "h";
+  const count = countChar(str, char);
+  expect(count).toEqual(0);
+});
+test("should count multiple occurrences of a symbols", () => {
+  const str = "Brilliant!!!";
+  const char = "!";
+  const count = countChar(str, char);
+  expect(count).toEqual(3);
+});

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

@@ -22,30 +22,31 @@ test("should append 'st' for numbers ending with 1, except those ending with 11"
 // Case 2: Numbers ending with 2 (but not 12)
 // When the number ends with 2, except for those ending with 12,
 // Then the function should return a string by appending "nd" to the number.
- test("returns 'nd' for 2, 22, 102", () => {
-    expect(getOrdinalNumber(2)).toBe("2nd");
-    expect(getOrdinalNumber(22)).toBe("22nd");
-    expect(getOrdinalNumber(102)).toBe("102nd");
-  });
+test("appends 'nd' for numbers ending with 2, except those ending with 12", () => {
+  expect(getOrdinalNumber(2)).toBe("2nd");
+  expect(getOrdinalNumber(22)).toBe("22nd");
+  expect(getOrdinalNumber(102)).toBe("102nd");
+});
 
 // Case 3: Numbers ending with 3 (but not 13)
 // When the number ends with 3, except for those ending with 13,
 // Then the function should return a string by appending "rd" to the number.
-  test("returns 'rd' for 3, 23, 103", () => {
-    expect(getOrdinalNumber(3)).toBe("3rd");
-    expect(getOrdinalNumber(23)).toBe("23rd");
-    expect(getOrdinalNumber(103)).toBe("103rd");
-  });
+test("appends 'rd' for numbers ending in 3, except those ending in 13", () => {
+  expect(getOrdinalNumber(3)).toBe("3rd");
+  expect(getOrdinalNumber(23)).toBe("23rd");
+  expect(getOrdinalNumber(103)).toBe("103rd");
+});
 
 // Case 4: All other numbers including umbers ending with 11, 12 or 13
 // These are for all other numbers including special cases and should always end with "th",
 // Then the function should return a string by appending "th" to the number.
-test("returns 'th' for all other numbers including those ending with 11, 12, 13", () => {
-    expect(getOrdinalNumber(11)).toBe("11th");
-    expect(getOrdinalNumber(12)).toBe("12th");
-    expect(getOrdinalNumber(13)).toBe("13th");
-    expect(getOrdinalNumber(10)).toBe("10th");
-    expect(getOrdinalNumber(99)).toBe("99th");
-    expect(getOrdinalNumber(100)).toBe("100th");
-
-  });
+test("returns 'th' for all numbers ending with 4-9 or 0 including numbers ending with 11, 12 and 13", () => {
+  expect(getOrdinalNumber(11)).toBe("11th");
+  expect(getOrdinalNumber(12)).toBe("12th");
+  expect(getOrdinalNumber(13)).toBe("13th");
+  expect(getOrdinalNumber(10)).toBe("10th");
+  expect(getOrdinalNumber(5)).toBe("5th");
+  expect(getOrdinalNumber(8)).toBe("8th");
+  expect(getOrdinalNumber(99)).toBe("99th");
+  expect(getOrdinalNumber(100)).toBe("100th");
+});