From 097b43cc9de21696f1cb670b8c8261364ef9624f Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Fri, 3 Jul 2026 10:03:30 +0100 Subject: [PATCH 1/6] Add tests for character counting function: handle no occurrences, empty string, and case sensitivity --- Sprint-3/2-practice-tdd/count.test.js | 32 +++++++++++++++++++++++++++ 1 file changed, 32 insertions(+) diff --git a/Sprint-3/2-practice-tdd/count.test.js b/Sprint-3/2-practice-tdd/count.test.js index 179ea0ddf7..a1d3638f9e 100644 --- a/Sprint-3/2-practice-tdd/count.test.js +++ b/Sprint-3/2-practice-tdd/count.test.js @@ -22,3 +22,35 @@ 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 return 0 when character does not occur", () => { + const str = "hello"; + const char = "z"; + const count = countChar(str, char); + expect(count).toEqual(0); +}); +// Scenario: Empty string +// Given an empty string `str`, +// And any character `char`, +// When the function is called with these inputs, +// Then it should return 0. + +test("should return 0 for an empty string", () => { + const str = ""; + const char = "a"; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + +// Scenario: Case sensitivity +// Given a string `str` containing both upper and lower case letters, +// And a character `char` matching only one case, +// When the function is called with these inputs, +// Then it should only count exact case matches. + +test("should be case-sensitive", () => { + const str = "Banana"; + const char = "a"; + const count = countChar(str, char); + expect(count).toEqual(3); +}); From 817103941de140f953c204aab8b709a00cf8ec38 Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Fri, 3 Jul 2026 10:05:56 +0100 Subject: [PATCH 2/6] Implement character counting logic in countChar function --- Sprint-3/2-practice-tdd/count.js | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/Sprint-3/2-practice-tdd/count.js b/Sprint-3/2-practice-tdd/count.js index 95b6ebb7d4..3b3e6d2b40 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 char of stringOfCharacters) { + if (char === findCharacter) { + count++; + } + } + return count; } module.exports = countChar; From ddd628dc8178660424553382d948451086f60708 Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Tue, 7 Jul 2026 04:04:25 +0100 Subject: [PATCH 3/6] Add tests for getOrdinalNumber function: cover cases for 2, 3, and other numbers --- .../2-practice-tdd/get-ordinal-number.test.js | 29 +++++++++++++++++++ 1 file changed, 29 insertions(+) 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 adfa58560f..82a6315e99 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,32 @@ 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 2: Numbers ending with 2 (but not 12) +// When the number ends with 2, except those ending with 12, +// Then the function should return a string by appending "nd" to the number. +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(102)).toEqual("102nd"); +}); + +// 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 "rd" to the number. +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(103)).toEqual("103rd"); +}); + +// Case 4: All other numbers, including 11, 12, and 13 +// When the number doesn't fall into the above categories, +// Then the function should return a string by appending "th" to the number. +test("should append 'th' for all other numbers, including 11, 12, and 13", () => { + expect(getOrdinalNumber(4)).toEqual("4th"); + expect(getOrdinalNumber(11)).toEqual("11th"); + expect(getOrdinalNumber(12)).toEqual("12th"); + expect(getOrdinalNumber(13)).toEqual("13th"); + expect(getOrdinalNumber(111)).toEqual("111th"); + expect(getOrdinalNumber(0)).toEqual("0th"); +}); From 3798b1bae16fb608244e5124a82f95500dbf50a9 Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Tue, 7 Jul 2026 04:41:08 +0100 Subject: [PATCH 4/6] Add tests for repeatStr function: cover cases for count of 1, 0, and negative values --- Sprint-3/2-practice-tdd/repeat-str.test.js | 22 ++++++++++++++++++++++ 1 file changed, 22 insertions(+) diff --git a/Sprint-3/2-practice-tdd/repeat-str.test.js b/Sprint-3/2-practice-tdd/repeat-str.test.js index a3fc1196c4..f3cad56789 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.test.js +++ b/Sprint-3/2-practice-tdd/repeat-str.test.js @@ -21,12 +21,34 @@ test("should repeat the string count times", () => { // When the repeatStr function is called with these inputs, // Then it should return the original `str` without repetition. +test("should return the original 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: // 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 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: // 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 throw an error when count is negative", () => { + const str = "hello"; + const count = -1; + expect(() => repeatStr(str, count)).toThrow( + "Count must be a non-negative integer" + ); +}); From 6a5a66cbad3a9a89b077df1ae9ea107ea815e8ab Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Tue, 7 Jul 2026 04:48:28 +0100 Subject: [PATCH 5/6] Implement repeatStr function to re-create string repetition logic without using String.prototype.repeat --- Sprint-3/2-practice-tdd/repeat-str.js | 15 ++++++++++++--- 1 file changed, 12 insertions(+), 3 deletions(-) diff --git a/Sprint-3/2-practice-tdd/repeat-str.js b/Sprint-3/2-practice-tdd/repeat-str.js index 2af0a2cea7..1a451ef7c9 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.js +++ b/Sprint-3/2-practice-tdd/repeat-str.js @@ -1,7 +1,16 @@ function repeatStr() { // 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. - return "hellohellohello"; -} + function repeatStr(str, count) { + if (count < 0) { + throw new Error("Count must be a non-negative integer"); + } + let result = ""; + for (let i = 0; i < count; i++) { + result += str; + } + return result; + } -module.exports = repeatStr; + module.exports = repeatStr; +} From 76eabc91bdbccc1d5a991146e99542ab421555b3 Mon Sep 17 00:00:00 2001 From: TTiamiyu Date: Tue, 7 Jul 2026 05:04:24 +0100 Subject: [PATCH 6/6] Refactor repeatStr function: streamline implementation and maintain non-use of String.prototype.repeat --- Sprint-3/2-practice-tdd/repeat-str.js | 26 ++++++++++++-------------- 1 file changed, 12 insertions(+), 14 deletions(-) diff --git a/Sprint-3/2-practice-tdd/repeat-str.js b/Sprint-3/2-practice-tdd/repeat-str.js index 1a451ef7c9..50a9ee2b8e 100644 --- a/Sprint-3/2-practice-tdd/repeat-str.js +++ b/Sprint-3/2-practice-tdd/repeat-str.js @@ -1,16 +1,14 @@ -function repeatStr() { - // 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. - function repeatStr(str, count) { - if (count < 0) { - throw new Error("Count must be a non-negative integer"); - } - let result = ""; - for (let i = 0; i < count; i++) { - result += str; - } - return result; +// Your implementation of this function must *not* call String.prototype.repeat (https://developer...) +// The goal is to re-implement that function, not to use it. +function repeatStr(str, count) { + if (count < 0) { + throw new Error("Count must be a non-negative integer"); } - - module.exports = repeatStr; + let result = ""; + for (let i = 0; i < count; i++) { + result += str; + } + return result; } + +module.exports = repeatStr;