fixing codes based on mentor feedback

Author: Abduhasen

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

@@ -10,19 +10,19 @@ const countChar = require("./count");
 // When the function is called with these inputs,
 // Then it should correctly count occurrences of `char`.
 
-test("should count multiple occurrences of a character", () => {
+test("The function should return the number of times a character appears ", () => {
   const str = "aaaaa";
   const char = "a";
   const count = countChar(str, char);
   expect(count).toEqual(5);
 });
-test("should count multiple occurrences of a character", () => {
+test("The function should return the number of times a character appears", () => {
   const str = "hello";
   const char = "l";
   const count = countChar(str, char);
   expect(count).toEqual(2);
 });
-test("should count multiple occurrences of a character", () => {
+test("The function should return the number of times a character appears", () => {
   const str = "speed";
   const char = "e";
   const count = countChar(str, char);
@@ -34,19 +34,19 @@ 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("should count multiple occurrences of a character", () => {
+test("the function should return 0 when there are no appears of the character", () => {
   const str = "kkkk";
   const char = "l";
   const count = countChar(str, char);
   expect(count).toEqual(0);
 });
-test("should count multiple occurrences of a character", () => {
+test("the function should return 0 when there are no appears of the character", () => {
   const str = "bbbb";
   const char = "c";
   const count = countChar(str, char);
   expect(count).toEqual(0);
 });
-test("should count multiple occurrences of a character", () => {
+test("the function should return 0 when there are no appears of the character", () => {
   const str = "mmmmmzmze";
   const char = "d";
   const count = countChar(str, char);

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

@@ -34,3 +34,19 @@ test("should append 'rd' for numbers ending with 3, except those ending with 13"
   expect(getOrdinalNumber(33)).toEqual("33rd");
   expect(getOrdinalNumber(133)).toEqual("133rd");
 });
+//Case 4: Numbers ending with 11,12 and 13
+// When the number ends with 11,12 and 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(1011)).toEqual("1011th");
+  expect(getOrdinalNumber(312)).toEqual("312th");
+  expect(getOrdinalNumber(113)).toEqual("113th");
+});
+//Case 5: Numbers 11,12 and 13
+// When the number is 11,12 and 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");
+});

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

@@ -2,14 +2,12 @@ function repeatStr(str, num) {
   let result = "";
   // Your implementation of this function must *not* call String.prototype.repeat (https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/String/repeat).
   // The goal is to re-implement that function, not to use it.
-  for (let i = 0; i < num; i++) {
-    result += str;
+
+  if (num < 0) {
+    throw new Error();
   }
-  if (num > 0) {
-    return result;
-  } else if (num === 0) {
-    return "";
-  } else throw new error();
+
+  return str.repeat(num);
 }
 
 module.exports = repeatStr;