From d65f68874b30cd04dd55fdcab5fb0b31953a8e1f Mon Sep 17 00:00:00 2001 From: Oussama Mouggal Date: Mon, 23 Feb 2026 14:18:40 +0000 Subject: [PATCH 1/5] Implement countChar function to count character occurrences and add test for zero occurrences --- Sprint-3/2-practice-tdd/count.js | 10 ++++++++-- Sprint-3/2-practice-tdd/count.test.js | 4 ++++ 2 files changed, 12 insertions(+), 2 deletions(-) diff --git a/Sprint-3/2-practice-tdd/count.js b/Sprint-3/2-practice-tdd/count.js index 95b6ebb7d..1afc228a9 100644 --- a/Sprint-3/2-practice-tdd/count.js +++ b/Sprint-3/2-practice-tdd/count.js @@ -1,5 +1,11 @@ function countChar(stringOfCharacters, findCharacter) { - return 5 -} + let count = 0; + for (let i = 0; i < stringOfCharacters.length; i++) { + if (stringOfCharacters[i] === findCharacter) { + count++; + } + } + return count; +} module.exports = countChar; diff --git a/Sprint-3/2-practice-tdd/count.test.js b/Sprint-3/2-practice-tdd/count.test.js index 179ea0ddf..45c9bd9c8 100644 --- a/Sprint-3/2-practice-tdd/count.test.js +++ b/Sprint-3/2-practice-tdd/count.test.js @@ -18,6 +18,10 @@ test("should count multiple occurrences of a character", () => { }); // Scenario: No Occurrences +test("should return 0 when the character does not occur in the string", () => { +const count = countChar("hello", "z"); +expect(count).toEqual(0); +}); // Given the input string `str`, // And a character `char` that does not exist within `str`. // When the function is called with these inputs, From a4fe3352b36f2e45ed71dbae1eb4339b1a7ceb2f Mon Sep 17 00:00:00 2001 From: Oussama Mouggal Date: Mon, 23 Feb 2026 15:19:58 +0000 Subject: [PATCH 2/5] Implement getOrdinalNumber function and add comprehensive tests for ordinal suffixes --- Sprint-3/2-practice-tdd/get-ordinal-number.js | 18 ++++++++++++++++- .../2-practice-tdd/get-ordinal-number.test.js | 20 +++++++++++++++++++ 2 files changed, 37 insertions(+), 1 deletion(-) diff --git a/Sprint-3/2-practice-tdd/get-ordinal-number.js b/Sprint-3/2-practice-tdd/get-ordinal-number.js index f95d71db1..6d61445bf 100644 --- a/Sprint-3/2-practice-tdd/get-ordinal-number.js +++ b/Sprint-3/2-practice-tdd/get-ordinal-number.js @@ -1,5 +1,21 @@ function getOrdinalNumber(num) { - return "1st"; + const lastDigit = num % 10; + const lastTwoDigits = num % 100; + + if (lastTwoDigits >= 11 && lastTwoDigits <= 13) { + return num + "th"; + } + + switch (lastDigit) { + case 1: + return num + "st"; + case 2: + return num + "nd"; + case 3: + return num + "rd"; + default: + return num + "th"; + } } module.exports = getOrdinalNumber; diff --git a/Sprint-3/2-practice-tdd/get-ordinal-number.test.js b/Sprint-3/2-practice-tdd/get-ordinal-number.test.js index adfa58560..f59623827 100644 --- a/Sprint-3/2-practice-tdd/get-ordinal-number.test.js +++ b/Sprint-3/2-practice-tdd/get-ordinal-number.test.js @@ -18,3 +18,23 @@ test("should append 'st' for numbers ending with 1, except those ending with 11" expect(getOrdinalNumber(21)).toEqual("21st"); expect(getOrdinalNumber(131)).toEqual("131st"); }); +test("should append 'nd' for numbers ending with 2, except those ending with 12", () => { + expect(getOrdinalNumber(2)).toEqual("2nd"); + expect(getOrdinalNumber(22)).toEqual("22nd"); + expect(getOrdinalNumber(132)).toEqual("132nd"); +}); +test("should append 'rd' for numbers ending with 3, except those ending with 13", () => { + expect(getOrdinalNumber(3)).toEqual("3rd"); + expect(getOrdinalNumber(23)).toEqual("23rd"); + expect(getOrdinalNumber(133)).toEqual("133rd"); +}); +test("should append 'th' for numbers ending with 4-9, and those ending with 11-13", () => { + expect(getOrdinalNumber(4)).toEqual("4th"); + expect(getOrdinalNumber(11)).toEqual("11th"); + expect(getOrdinalNumber(12)).toEqual("12th"); + expect(getOrdinalNumber(13)).toEqual("13th"); + expect(getOrdinalNumber(14)).toEqual("14th"); + expect(getOrdinalNumber(24)).toEqual("24th"); + expect(getOrdinalNumber(112)).toEqual("112th"); + expect(getOrdinalNumber(113)).toEqual("113th"); +}); From c6721c6306173bb58f7d25816e3f1724ddfae776 Mon Sep 17 00:00:00 2001 From: Oussama Mouggal Date: Mon, 23 Feb 2026 15:41:12 +0000 Subject: [PATCH 3/5] Implement repeatStr function with error handling and add tests for various cases --- Sprint-3/2-practice-tdd/repeat-str.js | 11 ++++++++--- Sprint-3/2-practice-tdd/repeat-str.test.js | 21 +++++++++++++++++++++ 2 files changed, 29 insertions(+), 3 deletions(-) diff --git a/Sprint-3/2-practice-tdd/repeat-str.js b/Sprint-3/2-practice-tdd/repeat-str.js index 3838c7b00..e6d61da89 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.js +++ b/Sprint-3/2-practice-tdd/repeat-str.js @@ -1,5 +1,10 @@ -function repeatStr() { - return "hellohellohello"; +function repeatStr(str, count) { + if (count < 0) { + throw new Error("Count must be non-negative"); + } + if (count === 0) { + return ""; + } + return str.repeat(count); } - module.exports = repeatStr; diff --git a/Sprint-3/2-practice-tdd/repeat-str.test.js b/Sprint-3/2-practice-tdd/repeat-str.test.js index a3fc1196c..166cbd6fc 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.test.js +++ b/Sprint-3/2-practice-tdd/repeat-str.test.js @@ -15,7 +15,28 @@ test("should repeat the string count times", () => { const repeatedStr = repeatStr(str, count); expect(repeatedStr).toEqual("hellohellohello"); }); +// Case: handle count of 1: +test("should return the string when count is 1", () => { + const str = "hello"; + const count = 1; + const repeatedStr = repeatStr(str, count); + expect(repeatedStr).toEqual("hello"); +}); + +// Case: Handle count of 0: +test("should return an empty string when count is 0", () => { + const str = "hello"; + const count = 0; + const repeatedStr = repeatStr(str, count); + expect(repeatedStr).toEqual(""); +}); +// Case: Handle negative count: +test("should throw an error when count is negative", () => { + const str = "hello"; + const count = -1; + expect(() => repeatStr(str, count)).toThrow(); +}); // Case: handle count of 1: // Given a target string `str` and a `count` equal to 1, // When the repeatStr function is called with these inputs, From a2c2560c943f62130ddb622da2ecb98e661fe9b2 Mon Sep 17 00:00:00 2001 From: Oussama Mouggal Date: Tue, 24 Feb 2026 09:57:07 +0000 Subject: [PATCH 4/5] improved the code and added more test cases --- .../2-practice-tdd/get-ordinal-number.test.js | 25 ++++++++++++++----- 1 file changed, 19 insertions(+), 6 deletions(-) diff --git a/Sprint-3/2-practice-tdd/get-ordinal-number.test.js b/Sprint-3/2-practice-tdd/get-ordinal-number.test.js index f59623827..2620f04b8 100644 --- a/Sprint-3/2-practice-tdd/get-ordinal-number.test.js +++ b/Sprint-3/2-practice-tdd/get-ordinal-number.test.js @@ -18,23 +18,36 @@ test("should append 'st' for numbers ending with 1, except those ending with 11" expect(getOrdinalNumber(21)).toEqual("21st"); expect(getOrdinalNumber(131)).toEqual("131st"); }); + +test("should append 'th' for numbers ending with 11", () => { + expect(getOrdinalNumber(11)).toEqual("11th"); + expect(getOrdinalNumber(111)).toEqual("111th"); +}); + test("should append 'nd' for numbers ending with 2, except those ending with 12", () => { expect(getOrdinalNumber(2)).toEqual("2nd"); expect(getOrdinalNumber(22)).toEqual("22nd"); expect(getOrdinalNumber(132)).toEqual("132nd"); }); + +test("should append 'th' for numbers ending with 12", () => { + expect(getOrdinalNumber(12)).toEqual("12th"); + expect(getOrdinalNumber(112)).toEqual("112th"); +}); + test("should append 'rd' for numbers ending with 3, except those ending with 13", () => { expect(getOrdinalNumber(3)).toEqual("3rd"); expect(getOrdinalNumber(23)).toEqual("23rd"); expect(getOrdinalNumber(133)).toEqual("133rd"); }); -test("should append 'th' for numbers ending with 4-9, and those ending with 11-13", () => { - expect(getOrdinalNumber(4)).toEqual("4th"); - expect(getOrdinalNumber(11)).toEqual("11th"); - expect(getOrdinalNumber(12)).toEqual("12th"); + +test("should append 'th' for numbers ending with 13", () => { expect(getOrdinalNumber(13)).toEqual("13th"); + expect(getOrdinalNumber(113)).toEqual("113th"); +}); + +test("should append 'th' for numbers ending with 4-9", () => { + expect(getOrdinalNumber(4)).toEqual("4th"); expect(getOrdinalNumber(14)).toEqual("14th"); expect(getOrdinalNumber(24)).toEqual("24th"); - expect(getOrdinalNumber(112)).toEqual("112th"); - expect(getOrdinalNumber(113)).toEqual("113th"); }); From 0daeef7e47e1218039047c55a575c10bb5f25aef Mon Sep 17 00:00:00 2001 From: Oussama Mouggal Date: Tue, 24 Feb 2026 10:01:49 +0000 Subject: [PATCH 5/5] I added the error message to improve the code and the test --- Sprint-3/2-practice-tdd/repeat-str.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Sprint-3/2-practice-tdd/repeat-str.test.js b/Sprint-3/2-practice-tdd/repeat-str.test.js index 166cbd6fc..5c2f3505e 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.test.js +++ b/Sprint-3/2-practice-tdd/repeat-str.test.js @@ -35,7 +35,7 @@ test("should return an empty string when count is 0", () => { test("should throw an error when count is negative", () => { const str = "hello"; const count = -1; - expect(() => repeatStr(str, count)).toThrow(); + expect(() => repeatStr(str, count)).toThrow("Count must be non-negative"); }); // Case: handle count of 1: // Given a target string `str` and a `count` equal to 1,