From f7238090c921e1c8c7a0e211ee5e45faef6de5e8 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Fri, 31 Oct 2025 18:25:17 +0000 Subject: [PATCH 01/19] Sprint-3-implement 1-get-angle-type.js all assertions and necessary functions added --- .../implement/1-get-angle-type.js | 27 +++++++++++++++---- 1 file changed, 22 insertions(+), 5 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js b/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js index ca1dfe7f2..f696075de 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js @@ -10,10 +10,23 @@ function getAngleType(angle) { if (angle === 90) { return "Right angle"; + + } else if (angle < 90){ + return "Acute angle"; + } + else if (angle > 90 && angle <180){ + return "Obtuse angle"; + } + else if (angle === 180){ + return "Straight angle"; } + else if (angle > 180 && angle < 360){ + return "Reflex angle"; + } + } + // Run the tests, work out what Case 2 is testing, and implement the required code here. - // Then keep going for the other cases, one at a time. -} + // Then keep going for the other cases, one at a time.} // The line below allows us to load the getAngleType function into tests in other files. // This will be useful in the "rewrite tests with jest" step. @@ -50,14 +63,18 @@ assertEquals(acute, "Acute angle"); // When the angle is greater than 90 degrees and less than 180 degrees, // Then the function should return "Obtuse angle" const obtuse = getAngleType(120); -// ====> write your test here, and then add a line to pass the test in the function above +assertEquals(obtuse, "Obtuse angle"); // Case 4: Identify Straight Angles: // When the angle is exactly 180 degrees, // Then the function should return "Straight angle" -// ====> write your test here, and then add a line to pass the test in the function above +const straightangle = getAngleType(180); +assertEquals(straightangle, "Straight angle"); // Case 5: Identify Reflex Angles: // When the angle is greater than 180 degrees and less than 360 degrees, // Then the function should return "Reflex angle" -// ====> write your test here, and then add a line to pass the test in the function above \ No newline at end of file +const reflex = getAngleType(270); +assertEquals(reflex, "Reflex angle"); + +// Sprint-3-implement 1-get-angle-type.js all assertions and necessary functions added \ No newline at end of file From ac087578ad33c3c72de942dcd8504ef5de129cf8 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Fri, 31 Oct 2025 19:01:49 +0000 Subject: [PATCH 02/19] // Sprint-3-implement 1-is-proper-fraction.js all assertions and necessary functions added --- .../implement/2-is-proper-fraction.js | 24 ++++++++++++++++++- 1 file changed, 23 insertions(+), 1 deletion(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js index a4739af77..b0a3e43b4 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js @@ -11,6 +11,18 @@ function isProperFraction(numerator, denominator) { if (numerator < denominator) { return true; } + else if (numerator > denominator){ + return false; + } + else if (numerator === denominator) { + return false; + } + else if (numerator === 0){ + retrurn (false); + } + else if (denominator === 0){ + return (false); + } } // The line below allows us to load the isProperFraction function into tests in other files. @@ -46,14 +58,24 @@ assertEquals(improperFraction, false); // target output: true // Explanation: The fraction -4/7 is a proper fraction because the absolute value of the numerator (4) is less than the denominator (7). The function should return true. const negativeFraction = isProperFraction(-4, 7); -// ====> complete with your assertion +assertEquals(negativeFraction, true); // Equal Numerator and Denominator check: // Input: numerator = 3, denominator = 3 // target output: false // Explanation: The fraction 3/3 is not a proper fraction because the numerator is equal to the denominator. The function should return false. const equalFraction = isProperFraction(3, 3); +assertEquals(equalFraction, false); // ====> complete with your assertion // Stretch: // What other scenarios could you test for? + +const numeratorZero = isProperFraction(0, 5); +assertEquals(numeratorZero, true); + +const denominatorrZero = isProperFraction(5, 0); +assertEquals(denominatorrZero, false); + +// Sprint-3-implement 1-is-proper-fraction.js all assertions and necessary functions added + From 2c28a87fd54425977f13f1db381cb651bed8c0f5 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Fri, 31 Oct 2025 19:31:09 +0000 Subject: [PATCH 03/19] Corrected an error in the code --- .../implement/2-is-proper-fraction.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js index b0a3e43b4..eed7de8b8 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js @@ -18,7 +18,7 @@ function isProperFraction(numerator, denominator) { return false; } else if (numerator === 0){ - retrurn (false); + retrurn (true); } else if (denominator === 0){ return (false); From 5d979ed5087e39cbdef9faf69be8b0882049ebfc Mon Sep 17 00:00:00 2001 From: UbunMen Date: Fri, 31 Oct 2025 19:33:30 +0000 Subject: [PATCH 04/19] Modified -implement -is-proper-fraction.js --- .../implement/3-get-card-value.js | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js index 266525d1b..d60ba1cbd 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js @@ -28,7 +28,9 @@ function assertEquals(actualOutput, targetOutput) { } // Acceptance criteria: -// Given a card string in the format "A♠" (representing a card in blackjack - the last character will always be an emoji for a suit, and all characters before will be a number 2-10, or one letter of J, Q, K, A), +// Given a card string in the format "A♠" (representing a card in blackjack +// - the last character will always be an emoji for a suit, +// and all characters before will be a number 2-10, or one letter of J, Q, K, A), // When the function getCardValue is called with this card string as input, // Then it should return the numerical card value const aceofSpades = getCardValue("A♠"); From 5f2e94622f467401e90fa6520fd514bc291ae310 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Sat, 1 Nov 2025 16:44:21 +0000 Subject: [PATCH 05/19] Sprint-3 -3-get-card-value.js Functions, assertion and invalid rank tested --- .../implement/3-get-card-value.js | 30 +++++++++++++++++++ 1 file changed, 30 insertions(+) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js index d60ba1cbd..ae11725cc 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js @@ -8,9 +8,20 @@ // write one test at a time, and make it pass, build your solution up methodically // just make one change at a time -- don't rush -- programmers are deep and careful thinkers function getCardValue(card) { + + const rank = card.slice(0,-1); if (rank === "A") { return 11; } + else if (["K", "J", "Q", "10"].includes(rank)){ + return 10; + } + else if (!isNaN(rank )) { + return Number(rank); + } + else { + throw new Error ("Invalid Card"); + } } // The line below allows us to load the getCardValue function into tests in other files. @@ -41,19 +52,38 @@ assertEquals(aceofSpades, 11); // When the function is called with such a card, // Then it should return the numeric value corresponding to the rank (e.g., "5" should return 5). const fiveofHearts = getCardValue("5♥"); +assertEquals(fiveofHearts, 5); // ====> write your test here, and then add a line to pass the test in the function above + + // Handle Face Cards (J, Q, K): // Given a card with a rank of "10," "J," "Q," or "K", // When the function is called with such a card, // Then it should return the value 10, as these cards are worth 10 points each in blackjack. +const faceCards = getCardValue("J♥"); +assertEquals(faceCards, 10); + +const tenOfClubs = getCardValue("10♣"); +assertEquals(tenOfClubs, 10); + // Handle Ace (A): // Given a card with a rank of "A", // When the function is called with an Ace, // Then it should, by default, assume the Ace is worth 11 points, which is a common rule in blackjack. +const aceCards = getCardValue("A♣"); +assertEquals(aceCards, 11); + // Handle Invalid Cards: // Given a card with an invalid rank (neither a number nor a recognized face card), // When the function is called with such a card, // Then it should throw an error indicating "Invalid card rank." + +try { + getCardValue("Z♣"); // invalid card + console.log("Test failed: no error thrown"); +} catch (error) { + assertEquals(error.message, "Invalid Card"); +} \ No newline at end of file From ba5537df13288a43d46a01c0a68425011d533243 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Sat, 1 Nov 2025 17:24:37 +0000 Subject: [PATCH 06/19] Sprint-3 rewrite-tests-with jest 1-get-angle-type-test.js tests conducted --- .../1-get-angle-type.test.js | 19 +++++++++++++++++++ 1 file changed, 19 insertions(+) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js index 4a92a3e82..6d507fb0b 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js @@ -13,14 +13,33 @@ test("should identify right angle (90°)", () => { // When the angle is less than 90 degrees, // Then the function should return "Acute angle" +test("should identify actue angle (40°)", () => { + expect(getAngleType(40)).toEqual("Acute angle"); +}); + // Case 3: Identify Obtuse Angles: // When the angle is greater than 90 degrees and less than 180 degrees, // Then the function should return "Obtuse angle" +test("should identify obtuse angle (120°)", () => { + expect(getAngleType(120)).toEqual("Obtuse angle"); +}); + // Case 4: Identify Straight Angles: // When the angle is exactly 180 degrees, // Then the function should return "Straight angle" +test("should identify straight angle (180°)", () => { + expect(getAngleType(180)).toEqual("Straight angle"); +}); + // Case 5: Identify Reflex Angles: // When the angle is greater than 180 degrees and less than 360 degrees, // Then the function should return "Reflex angle" + +test("should identify reflex angle (240)", () => { + expect(getAngleType(240)).toEqual("Reflex angle"); +}); + + +// Sprint-3 rewrite-tests-with jest 1-get-angle-type-test.js tests conducted \ No newline at end of file From 26e660eb26315456015e9da813edbfc15dc82679 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Sat, 1 Nov 2025 17:38:37 +0000 Subject: [PATCH 07/19] Modified Typo --- .../implement/2-is-proper-fraction.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js index eed7de8b8..115971ebf 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js @@ -18,7 +18,7 @@ function isProperFraction(numerator, denominator) { return false; } else if (numerator === 0){ - retrurn (true); + return (true); } else if (denominator === 0){ return (false); From 49307429a0881064ccf0619836a13b1b539feb47 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Sat, 1 Nov 2025 17:42:48 +0000 Subject: [PATCH 08/19] // Sprint-3 ewrite-tests-with-jest/2-is-proper-fraction.test.js. rewrote tests in jest --- .../2-is-proper-fraction.test.js | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js index caf08d15b..4534a80b8 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js @@ -7,7 +7,18 @@ test("should return true for a proper fraction", () => { }); // Case 2: Identify Improper Fractions: +test("should return false for an improper fraction", () => { + expect(isProperFraction(5, 3)).toEqual(false); +}); // Case 3: Identify Negative Fractions: +test("should return true for a negative fraction", () => { + expect(isProperFraction(-2, 3)).toEqual(true); +}); // Case 4: Identify Equal Numerator and Denominator: +test("should return false for equal numerator and denominator", () => { + expect(isProperFraction(3, 3)).toEqual(false); +}); + +// Sprint-3 ewrite-tests-with-jest/2-is-proper-fraction.test.js. rewrote tests in jest \ No newline at end of file From c68a37cd89a4c66b85632ee9bef3dc5787dd10c5 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Sat, 1 Nov 2025 18:01:22 +0000 Subject: [PATCH 09/19] Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote tests using jest --- .../3-get-card-value.test.js | 17 +++++++++++++++++ 1 file changed, 17 insertions(+) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index 04418ff72..3852fb1d9 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -8,6 +8,23 @@ test("should return 11 for Ace of Spades", () => { }); // Case 2: Handle Number Cards (2-10): +test("should number cards for (2-10)", () => { + const aceofSpades = getCardValue("5♠"); + expect(aceofSpades).toEqual(5); +}); + // Case 3: Handle Face Cards (J, Q, K): +test("should return 10 for face cards", () => { + const aceofSpades = getCardValue("Q♠"); + expect(aceofSpades).toEqual(10); +}); // Case 4: Handle Ace (A): +// test("should return 11 for Ace of Spades", () => { +// const aceofSpades = getCardValue("A♠"); +// expect(aceofSpades).toEqual(11); +// }); // Case 5: Handle Invalid Cards: +test("should throw an error for invalid cards", () => { + expect(() => getCardValue("Z♠")).toThrow("Invalid Card"); +}); +// Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote tests using jest From b57c40460c69716911bb141fcbaeaf9cbdfa7381 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 5 Nov 2025 21:40:45 +0000 Subject: [PATCH 10/19] Few modifications done. --- .../implement/3-get-card-value.js | 8 +++++--- .../rewrite-tests-with-jest/3-get-card-value.test.js | 1 + 2 files changed, 6 insertions(+), 3 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js index ae11725cc..23d355506 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js @@ -51,8 +51,8 @@ assertEquals(aceofSpades, 11); // Given a card with a rank between "2" and "9", // When the function is called with such a card, // Then it should return the numeric value corresponding to the rank (e.g., "5" should return 5). -const fiveofHearts = getCardValue("5♥"); -assertEquals(fiveofHearts, 5); +const numberCards = getCardValue("5♥"); +assertEquals(numberCards, 5); // ====> write your test here, and then add a line to pass the test in the function above @@ -86,4 +86,6 @@ try { console.log("Test failed: no error thrown"); } catch (error) { assertEquals(error.message, "Invalid Card"); -} \ No newline at end of file +} + +// Functions, assertion and invalid rank tested \ No newline at end of file diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index 3852fb1d9..6f0db25ea 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -28,3 +28,4 @@ test("should throw an error for invalid cards", () => { expect(() => getCardValue("Z♠")).toThrow("Invalid Card"); }); // Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote tests using jest +// Few modifications done. From 0a1ed8cb760253b163af053117e3473b9a8daa0f Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 5 Nov 2025 22:16:15 +0000 Subject: [PATCH 11/19] Little change made --- .../rewrite-tests-with-jest/3-get-card-value.test.js | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index 6f0db25ea..e5c034801 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -9,8 +9,8 @@ test("should return 11 for Ace of Spades", () => { // Case 2: Handle Number Cards (2-10): test("should number cards for (2-10)", () => { - const aceofSpades = getCardValue("5♠"); - expect(aceofSpades).toEqual(5); + const aceofSpades = getCardValue("6♠"); + expect(aceofSpades).toEqual(6); }); // Case 3: Handle Face Cards (J, Q, K): From 9abc5a9f667038b4e213a4734b1152916f772df7 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Mon, 10 Nov 2025 15:09:42 +0000 Subject: [PATCH 12/19] 2-is-proper-fraction.js Updated the function and has been able to handle possible edge cases. --- .../implement/2-is-proper-fraction.js | 36 ++++++++++--------- 1 file changed, 20 insertions(+), 16 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js index 115971ebf..0478752f3 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js @@ -8,23 +8,14 @@ // write one test at a time, and make it pass, build your solution up methodically function isProperFraction(numerator, denominator) { - if (numerator < denominator) { - return true; - } - else if (numerator > denominator){ - return false; - } - else if (numerator === denominator) { + // Make sure denominator cannot be zero + if (denominator === 0) { return false; } - else if (numerator === 0){ - return (true); - } - else if (denominator === 0){ - return (false); - } -} + // Check if the absolute value of the numerator is smaller than denominator + return Math.abs(numerator) < Math.abs(denominator); +} // The line below allows us to load the isProperFraction function into tests in other files. // This will be useful in the "rewrite tests with jest" step. module.exports = isProperFraction; @@ -57,8 +48,8 @@ assertEquals(improperFraction, false); // Input: numerator = -4, denominator = 7 // target output: true // Explanation: The fraction -4/7 is a proper fraction because the absolute value of the numerator (4) is less than the denominator (7). The function should return true. -const negativeFraction = isProperFraction(-4, 7); -assertEquals(negativeFraction, true); +const negativeFraction = isProperFraction(-4, 3); +assertEquals(negativeFraction, false); // Equal Numerator and Denominator check: // Input: numerator = 3, denominator = 3 @@ -77,5 +68,18 @@ assertEquals(numeratorZero, true); const denominatorrZero = isProperFraction(5, 0); assertEquals(denominatorrZero, false); +// Expected: true (|-2| < |5|) +const reviewerCase2 = isProperFraction(-2, 5); +assertEquals(reviewerCase2, true); + +// Expected: false (|-1| = |1|) +const reviewerCase3 = isProperFraction(-1, 1); +assertEquals(reviewerCase3, false); + +// Expected: true (|-2| < |-3|) +const reviewerCase4 = isProperFraction(-2, -3); +assertEquals(reviewerCase4, true); + // Sprint-3-implement 1-is-proper-fraction.js all assertions and necessary functions added +// Updated the function and has been able to handle possible edge cases. From 608a309cd73c8e67d0e62c09f39f636a2519e7fc Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 15:40:04 +0000 Subject: [PATCH 13/19] Functions, assertions, and invalid ranks tested. Modified the if statement to handle valid numeric literals such as '2.1' and '002'. --- .../implement/3-get-card-value.js | 41 +++++++++------ Sprint-3/2-practice-tdd/count.test.js | 50 +++++++++++++++++++ 2 files changed, 77 insertions(+), 14 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js index 23d355506..8171896d7 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js @@ -9,19 +9,16 @@ // just make one change at a time -- don't rush -- programmers are deep and careful thinkers function getCardValue(card) { - const rank = card.slice(0,-1); - if (rank === "A") { - return 11; - } - else if (["K", "J", "Q", "10"].includes(rank)){ - return 10; - } - else if (!isNaN(rank )) { - return Number(rank); - } - else { - throw new Error ("Invalid Card"); - } + const rank = card.slice(0, -1); + const validRanks = ["A", "K", "Q", "J", "10", "9", "8", "7", "6", "5", "4", "3", "2"]; + + if (!validRanks.includes(rank)) { + throw new Error("Invalid Card"); + } + + if (rank === "A") return 11; + if (["K", "Q", "J", "10"].includes(rank)) return 10; + return Number(rank); } // The line below allows us to load the getCardValue function into tests in other files. @@ -88,4 +85,20 @@ try { assertEquals(error.message, "Invalid Card"); } -// Functions, assertion and invalid rank tested \ No newline at end of file +try { + getCardValue("2.1♥"); // decimal — invalid + console.log("Test failed: no error thrown for 2.1♥"); +} catch (error) { + assertEquals(error.message, "Invalid Card"); +} + +try { + getCardValue("0002♦"); // padded number — invalid + console.log("Test failed: no error thrown for 0002♦"); +} catch (error) { + assertEquals(error.message, "Invalid Card"); +} + +// Functions, assertion and invalid rank tested, and modified the if statment +// to hadnle valid numeric literals such as "2.1", "002" "Functions, assertions, and invalid ranks tested. +// Modified the if statement to handle valid numeric literals such as '2.1' and '002'. \ No newline at end of file diff --git a/Sprint-3/2-practice-tdd/count.test.js b/Sprint-3/2-practice-tdd/count.test.js index 42baf4b4b..2ff0dccfb 100644 --- a/Sprint-3/2-practice-tdd/count.test.js +++ b/Sprint-3/2-practice-tdd/count.test.js @@ -22,3 +22,53 @@ test("should count multiple occurrences of a character", () => { // And a character char that does not exist within the case-sensitive str, // When the function is called with these inputs, // Then it should return 0, indicating that no occurrences of the char were found in the case-sensitive str. + +test("should return 0 when character does not exist in string", () => { + const str = "Code Your Future"; + const char = "g"; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + +// Scenario: Empty string + +test("should return 0 when string is empty", () => { + const str = ""; + const char = "a"; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + +// Scenario: String is a space + +test("should return 0 when string is space", () => { + const str = " "; + const char = "a"; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + +// Scenario: String is + +// Scenario: Empty character + +test("should return 0 when character is empty", () => { + const str = "my code"; + const char = ""; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + +// Scenario: char is empty character + +test("should return 0 when character is empty string ", () => { + const str = "my code"; + const char = " "; + const count = countChar(str, char); + expect(count).toEqual(0); +}); + + + +// Scenario: charcter not string + From b0eefecd1aa8370ccafad9bc5f401ae1e1aa0d65 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 15:56:40 +0000 Subject: [PATCH 14/19] Testing modified to cover broader scenarios that cover all possible cases. --- .../1-get-angle-type.test.js | 22 ++++++++++++------- 1 file changed, 14 insertions(+), 8 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js index 6d507fb0b..ce2b2cc96 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js @@ -13,16 +13,20 @@ test("should identify right angle (90°)", () => { // When the angle is less than 90 degrees, // Then the function should return "Acute angle" -test("should identify actue angle (40°)", () => { - expect(getAngleType(40)).toEqual("Acute angle"); +test("should identify acute angles (0 < angle < 90)", () => { + expect(getAngleType(10)).toEqual("Acute angle"); + expect(getAngleType(45)).toEqual("Acute angle"); + expect(getAngleType(89.999)).toEqual("Acute angle"); }); // Case 3: Identify Obtuse Angles: // When the angle is greater than 90 degrees and less than 180 degrees, // Then the function should return "Obtuse angle" -test("should identify obtuse angle (120°)", () => { - expect(getAngleType(120)).toEqual("Obtuse angle"); +test("should identify obtuse angles (90 < angle < 180)", () => { + expect(getAngleType(110)).toEqual("Obtuse angle"); + expect(getAngleType(145)).toEqual("Obtuse angle"); + expect(getAngleType(169.99)).toEqual("Obtuse angle"); }); // Case 4: Identify Straight Angles: @@ -37,9 +41,11 @@ test("should identify straight angle (180°)", () => { // When the angle is greater than 180 degrees and less than 360 degrees, // Then the function should return "Reflex angle" -test("should identify reflex angle (240)", () => { - expect(getAngleType(240)).toEqual("Reflex angle"); +test("should identify reflex angles (180 < angle < 360)", () => { + expect(getAngleType(180.5)).toEqual("Reflex angle"); + expect(getAngleType(270)).toEqual("Reflex angle"); + expect(getAngleType(359.999)).toEqual("Reflex angle"); }); - -// Sprint-3 rewrite-tests-with jest 1-get-angle-type-test.js tests conducted \ No newline at end of file +// Sprint-3 rewrite-tests-with jest 1-get-angle-type-test.js tests conducted +// Testing modified to cover broader scenarios that cover all possible cases. \ No newline at end of file From 6041f592553f9d6f06aa5d4c27c9c5a4d5b96dde Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 16:15:43 +0000 Subject: [PATCH 15/19] Corrected a misleading comment --- .../implement/1-get-angle-type.js | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js b/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js index f696075de..c2ab8dc7d 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js +++ b/Sprint-3/1-implement-and-rewrite-tests/implement/1-get-angle-type.js @@ -23,6 +23,10 @@ function getAngleType(angle) { else if (angle > 180 && angle < 360){ return "Reflex angle"; } + else { + throw new Error("Invalid angle"); +} + } // Run the tests, work out what Case 2 is testing, and implement the required code here. From e84da292517b5f0bd57efade5423236ac64b0f94 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 16:27:17 +0000 Subject: [PATCH 16/19] Completed the incomplete description and added a test for multiple values such as boundr values 2 and 10 in line 12-17 --- .../rewrite-tests-with-jest/3-get-card-value.test.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index e5c034801..887f0b8f5 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -8,9 +8,13 @@ test("should return 11 for Ace of Spades", () => { }); // Case 2: Handle Number Cards (2-10): -test("should number cards for (2-10)", () => { - const aceofSpades = getCardValue("6♠"); - expect(aceofSpades).toEqual(6); +test("should return number cards for (2-10)", () => { + const twoOfSpades = getCardValue("2♠"); + expect(twoOfSpades).toEqual(2); + const sixOfHearts = getCardValue("6♥"); + expect(sixOfHearts).toEqual(6); + const tenOfDiamonds = getCardValue("10♦"); + expect(tenOfDiamonds).toEqual(10); }); // Case 3: Handle Face Cards (J, Q, K): @@ -29,3 +33,4 @@ test("should throw an error for invalid cards", () => { }); // Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote tests using jest // Few modifications done. +// Completed incomplete description and added a test for multiple values such as boundr values 2 and 10 in line 12-17 \ No newline at end of file From f8bbdf04d09104aeaa7ee1c0bb972cb99aac3118 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 16:33:28 +0000 Subject: [PATCH 17/19] Removed misleading description --- .../rewrite-tests-with-jest/3-get-card-value.test.js | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index 887f0b8f5..8559afec7 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -2,7 +2,7 @@ // We will use the same function, but write tests for it using Jest in this file. const getCardValue = require("../implement/3-get-card-value"); -test("should return 11 for Ace of Spades", () => { +test("should return 11 for an Ace", () => { const aceofSpades = getCardValue("A♠"); expect(aceofSpades).toEqual(11); }); From 2934b0ea8179a1939716651968afc44c46ac7cc7 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 16:38:30 +0000 Subject: [PATCH 18/19] Added test coverage cases --- .../rewrite-tests-with-jest/3-get-card-value.test.js | 11 ++++++++--- 1 file changed, 8 insertions(+), 3 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js index 8559afec7..6518b6774 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js @@ -3,10 +3,12 @@ const getCardValue = require("../implement/3-get-card-value"); test("should return 11 for an Ace", () => { - const aceofSpades = getCardValue("A♠"); - expect(aceofSpades).toEqual(11); + expect(getCardValue("A♠")).toEqual(11); + expect(getCardValue("A♥")).toEqual(11); + }); + // Case 2: Handle Number Cards (2-10): test("should return number cards for (2-10)", () => { const twoOfSpades = getCardValue("2♠"); @@ -33,4 +35,7 @@ test("should throw an error for invalid cards", () => { }); // Sprint-3 rewrite-tests-with-jest/2-get-card-value.test.js. rewrote tests using jest // Few modifications done. -// Completed incomplete description and added a test for multiple values such as boundr values 2 and 10 in line 12-17 \ No newline at end of file +// Completed incomplete description and added a test +// for multiple values such as boundr values 2 and 10 in line 12-17 + +// Added test coverage cases \ No newline at end of file From 74b4cc12befc4630473a6a682e2906de733a7db7 Mon Sep 17 00:00:00 2001 From: UbunMen Date: Wed, 12 Nov 2025 16:39:59 +0000 Subject: [PATCH 19/19] Modified --- .../2-is-proper-fraction.test.js | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-) diff --git a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js index 4534a80b8..a68934763 100644 --- a/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js +++ b/Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js @@ -12,7 +12,7 @@ test("should return false for an improper fraction", () => { }); // Case 3: Identify Negative Fractions: -test("should return true for a negative fraction", () => { +test("should return true for a proper fraction with negative numerator", () => { expect(isProperFraction(-2, 3)).toEqual(true); }); @@ -21,4 +21,12 @@ test("should return false for equal numerator and denominator", () => { expect(isProperFraction(3, 3)).toEqual(false); }); -// Sprint-3 ewrite-tests-with-jest/2-is-proper-fraction.test.js. rewrote tests in jest \ No newline at end of file +test("should return false for a negative improper fraction", () => { + expect(isProperFraction(-4, 3)).toEqual(false); +}); + + +// Sprint-3 ewrite-tests-with-jest/2-is-proper-fraction.test.js. rewrote tests in jest +// Changed the misleading comment "should return true for a negative fraction" +// to "should return true for a proper fraction with negative numerator" +// Modified \ No newline at end of file