fixing test descriptions (adding brief description should and when ) based on mentor feedback

Author: Abduhasen

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

@@ -22,7 +22,7 @@ test("should count multiple occurrences of a character", () => {
 // And a character `char` that does not exist within `str`.
 // When the function is called with these inputs,
 // Then it should return 0, indicating that no occurrences of `char` were found.
-test("it should return 0 because the character doesn't occurrences", () => {
+test("returns 0 when the character does not exist in the string", () => {
   const str = "The character have no match";
   const char = "z";
   const count = countChar(str, char);
@@ -33,7 +33,7 @@ test("it should return 0 because the character doesn't occurrences", () => {
 // And a character `char` that have 1 character exist in `str`.
 // When the function is called with these inputs,
 // Then it should return 1, indicating that 1 occurrences of `char` were found.
-test("it should return 1 because the character occurrences 1 time", () => {
+test("returns 1 when the character appears once in the string", () => {
   const str = "code your future";
   const char = "f";
   const count = countChar(str, char);

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

@@ -18,27 +18,38 @@ test("should append 'st' for numbers ending with 1, except those ending with 11"
   expect(getOrdinalNumber(21)).toEqual("21st");
   expect(getOrdinalNumber(131)).toEqual("131st");
 });
-// Case 1: Numbers ending with 1 (but not 11)
-// When the number ends with 1, except those ending with 11,
+// Case 2: Numbers ending with 2 (but not 12)
+// When the number ends with 1, except those ending with 12,
 // Then the function should return a string by appending "st" to the number.
 test("should append 'nd' for numbers ending with 2, except those ending with 12", () => {
   expect(getOrdinalNumber(2)).toEqual("2nd");
   expect(getOrdinalNumber(42)).toEqual("42nd");
   expect(getOrdinalNumber(1752)).toEqual("1752nd");
 });
-// Case 1: Numbers ending with 1 (but not 11)
-// When the number ends with 1, except those ending with 11,
+// Case 3: Numbers ending with 3 (but not 13)
+// When the number ends with 3, except those ending with 13,
 // Then the function should return a string by appending "st" to the number.
 test("should append 'rd' for numbers ending with 3, except those ending with 13", () => {
   expect(getOrdinalNumber(3)).toEqual("3rd");
   expect(getOrdinalNumber(83)).toEqual("83rd");
   expect(getOrdinalNumber(653)).toEqual("653rd");
 });
-// Case 1: Numbers ending with 1 (but not 11)
-// When the number ends with 1, except those ending with 11,
-// Then the function should return a string by appending "st" to the number.
-test("should append 'th' for numbers not ending with 1, 2 and 3 ", () => {
+// Case 4: Numbers ending with 4 through 9
+// When the number does not end with 1, 2 or 3,
+// Then the function should return a string by appending "th" to the number.
+test("should append 'th' for all other numbers that do not end with 1, 2 or 3", () => {
   expect(getOrdinalNumber(5)).toEqual("5th");
   expect(getOrdinalNumber(38)).toEqual("38th");
   expect(getOrdinalNumber(15678)).toEqual("15678th");
 });
+//Case 5: Numbers ending with 11, 12 and 13 (special exceptions)
+// When the number ends with 11, 12 or 13,
+// Then the function should return a string by appending "th" to the number.
+test("should append 'th' for numbers ending with 11, 12 and 13", () => {
+  expect(getOrdinalNumber(11)).toEqual("11th");
+  expect(getOrdinalNumber(12)).toEqual("12th");
+  expect(getOrdinalNumber(13)).toEqual("13th");
+  expect(getOrdinalNumber(111)).toEqual("111th");
+  expect(getOrdinalNumber(112)).toEqual("112th");
+  expect(getOrdinalNumber(113)).toEqual("113th");
+});

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

@@ -9,7 +9,7 @@ const repeatStr = require("./repeat-str");
 // When the repeatStr function is called with these inputs,
 // Then it should return a string that contains the original `str` repeated `count` times.
 
-test("should repeat the string count times", () => {
+test("should return the string the given number of count when count is a positive number", () => {
   let word = "hello";
   let times = 3;
   const repeatedStr = repeatStr(word, times);
@@ -20,7 +20,7 @@ test("should repeat the string count times", () => {
 // Given a target string `str` and a `count` equal to 1,
 // When the repeatStr function is called with these inputs,
 // Then it should return the original `str` without repetition.
-test("should repeat the string count times", () => {
+test("should return the original string without repetition when count is 1", () => {
   let word = "code your future";
   let times = 1;
   const repeatedStr = repeatStr(word, times);
@@ -30,7 +30,7 @@ test("should repeat the string count times", () => {
 // Given a target string `str` and a `count` equal to 0,
 // When the repeatStr function is called with these inputs,
 // Then it should return an empty string.
-test("should repeat the string is empty ", () => {
+test("should return an empty string when the number of times is 0", () => {
   let word = "hello";
   let times = 0;
   const repeatedStr = repeatStr(word, times);
@@ -40,7 +40,7 @@ test("should repeat the string is empty ", () => {
 // Given a target string `str` and a negative integer `count`,
 // When the repeatStr function is called with these inputs,
 // Then it should throw an error, as negative counts are not valid.
-test("should repeat the string return negative number not allowed ", () => {
+test("should return a string 'negative number not allowed' when negative number passed", () => {
   let word = "hello";
   let times = -3;
   const repeatedStr = repeatStr(word, times);