Skip to content
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,22 @@
// execute the code to ensure all tests pass.

function getAngleType(angle) {
// TODO: Implement this function
if (angle > 0 && angle < 90) {
return "Acute angle";
}
if (angle === 90) {
return "Right angle";
}
if (angle > 90 && angle < 180) {
return "Obtuse angle";
}
if (angle === 180) {
return "Straight angle";
}
if (angle > 180 && angle < 360) {
return "Reflex angle";
}
return "Invalid angle";
}

// The line below allows us to load the getAngleType function into tests in other files.
Expand All @@ -35,3 +50,24 @@ function assertEquals(actualOutput, targetOutput) {
// Example: Identify Right Angles
const right = getAngleType(90);
assertEquals(right, "Right angle");

const acute = getAngleType(45);
assertEquals(acute, "Acute angle");

const obtuse = getAngleType(120);
assertEquals(obtuse, "Obtuse angle");

const straight = getAngleType(180);
assertEquals(straight, "Straight angle");

const reflex = getAngleType(270);
assertEquals(reflex, "Reflex angle");

const invalidZero = getAngleType(0);
assertEquals(invalidZero, "Invalid angle");

const invalidNegative = getAngleType(-45);
assertEquals(invalidNegative, "Invalid angle");

const invalidOver = getAngleType(361);
assertEquals(invalidOver, "Invalid angle");
Original file line number Diff line number Diff line change
Expand Up @@ -12,6 +12,11 @@

function isProperFraction(numerator, denominator) {
// TODO: Implement this function
if (denominator === 0) {
return false;
}

return Math.abs(numerator) < Math.abs(denominator);
}

// The line below allows us to load the isProperFraction function into tests in other files.
Expand All @@ -31,3 +36,23 @@ function assertEquals(actualOutput, targetOutput) {

// Example: 1/2 is a proper fraction
assertEquals(isProperFraction(1, 2), true);

// Equal numbers → will not be proper
assertEquals(isProperFraction(2, 2), false);

// Improper fractions
assertEquals(isProperFraction(5, 4), false);
assertEquals(isProperFraction(10, 3), false);

// Zero numerator → will be proper
assertEquals(isProperFraction(0, 5), true);

// Negative numbers
assertEquals(isProperFraction(-1, 2), true);
assertEquals(isProperFraction(1, -2), true);
assertEquals(isProperFraction(-3, -2), false);

// Denominator zero
assertEquals(isProperFraction(1, 0), false);
assertEquals(isProperFraction(0, 0), false);

Original file line number Diff line number Diff line change
Expand Up @@ -23,6 +23,22 @@

function getCardValue(card) {
// TODO: Implement this function
const rank = card.slice(0, -1);
const suit = card.slice(-1);

const validSuits = ["♠", "♥", "♦", "♣"];
const faceCards = ["J", "Q", "K"];
const numberCards = ["2", "3", "4", "5", "6", "7", "8", "9", "10"];

if (!validSuits.includes(suit)) {
throw new Error("Invalid card suit");
}

if (rank === "A") return 11;
if (faceCards.includes(rank)) return 10;
if (numberCards.includes(rank)) return Number(rank);

throw new Error("Invalid card rank");
}

// The line below allows us to load the getCardValue function into tests in other files.
Expand All @@ -40,13 +56,30 @@ function assertEquals(actualOutput, targetOutput) {
// TODO: Write tests to cover all outcomes, including throwing errors for invalid cards.
// Examples:
assertEquals(getCardValue("9♠"), 9);
assertEquals(getCardValue("5♥"), 5);
assertEquals(getCardValue("K♦"), 10);
assertEquals(getCardValue("A♣"), 11);

// Handling invalid cards
try {
getCardValue("invalid");
console.error("Error was not thrown for completely invalid card");
} catch (error) {}
try {
getCardValue("1♠");
console.error("Error was not thrown for invalid card rank");
} catch (error) {
assertEquals(error.message, "Invalid card rank");
}

// This line will not be reached if an error is thrown as expected
console.error("Error was not thrown for invalid card");
} catch (e) {}

// What other invalid card cases can you think of?
try {
getCardValue("5X");
console.error("Error was not thrown for invalid card suit");
} catch (error) {
assertEquals(error.message, "Invalid card suit");
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,30 @@ test(`should return "Acute angle" when (0 < angle < 90)`, () => {
});

// Case 2: Right angle
test(`should return "Right angle" when angle is 90`, () => {
expect(getAngleType(90)).toEqual("Right angle");
});
// Case 3: Obtuse angles
test(`should return "Obtuse angle" when (90 < angle < 180)`, () => {
expect(getAngleType(91)).toEqual("Obtuse angle");
expect(getAngleType(135)).toEqual("Obtuse angle");
expect(getAngleType(179)).toEqual("Obtuse angle");
});

// Case 4: Straight angle
test(`should return "Straight angle" when angle is 180`, () => {
expect(getAngleType(180)).toEqual("Straight angle");
});
// Case 5: Reflex angles
test(`should return "Reflex angle" when (180 < angle < 360)`, () => {
expect(getAngleType(181)).toEqual("Reflex angle");
expect(getAngleType(270)).toEqual("Reflex angle");
expect(getAngleType(359)).toEqual("Reflex angle");
});
// Case 6: Invalid angles
test(`should return "Invalid angle" when angle is <= 0 or >= 360`, () => {
expect(getAngleType(0)).toEqual("Invalid angle");
expect(getAngleType(-10)).toEqual("Invalid angle");
expect(getAngleType(360)).toEqual("Invalid angle");
expect(getAngleType(400)).toEqual("Invalid angle");
});
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,40 @@ const isProperFraction = require("../implement/2-is-proper-fraction");
test(`should return false when denominator is zero`, () => {
expect(isProperFraction(1, 0)).toEqual(false);
});

// Proper fractions (positive numbers
test(`should return true when numerator is smaller than denominator`, () => {
expect(isProperFraction(1, 2)).toEqual(true);
expect(isProperFraction(3, 4)).toEqual(true);
});

// Equal numbers → not proper
test(`should return false when numerator equals denominator`, () => {
expect(isProperFraction(2, 2)).toEqual(false);
});

// Improper fractions
test(`should return false when numerator is greater than denominator`, () => {
expect(isProperFraction(5, 4)).toEqual(false);
expect(isProperFraction(10, 3)).toEqual(false);
});

// Zero numerator
test(`should return true when numerator is zero and denominator is non-zero`, () => {
expect(isProperFraction(0, 5)).toEqual(true);
});

// Negative numbers
test("should return true when |numerator| < |denominator| and signs differ", () => {
expect(isProperFraction(-1, 2)).toBe(true);
expect(isProperFraction(1, -2)).toBe(true);
});

test("should return false when |numerator| > |denominator| and both are negative", () => {
expect(isProperFraction(-3, -2)).toBe(false);
});
Comment on lines +13 to +42
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We can use pseudo-code and notations like abs(...) or | ... | in the descriptions to more concisely describe the conditions (the "when" part).

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test("|n| < |d| with negative signs → true, |n| > |d| → false", () => {
expect(isProperFraction(-1, 2)).toBe(true);
expect(isProperFraction(1, -2)).toBe(true);
expect(isProperFraction(-3, -2)).toBe(false);
});

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

With this category, when one of the tests fail, it may not be easy to find out whether the function fails to identify proper fraction or improper fraction.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

test("|n| < |d| with mixed negative signs → true", () => {
expect(isProperFraction(-1, 2)).toBe(true);
expect(isProperFraction(1, -2)).toBe(true);
});

test("|n| > |d| with both negative → false", () => {
expect(isProperFraction(-3, -2)).toBe(false);
});

A strategy of separating tests by logic as much as possible helps pinpoint the test failure point better?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

  • Strategy is sound.

  • The format of the test description is quite uncommon. Normal practice is to describe them in the form:
    should <expected_behavior> when <condition>.
    For example, "should return true when |numerator| < |denominator|" (more natural to English-speaking people)


// Both zero
test(`should return false when both numerator and denominator are zero`, () => {
expect(isProperFraction(0, 0)).toEqual(false);
});
Original file line number Diff line number Diff line change
Expand Up @@ -11,8 +11,27 @@ test(`Should return 11 when given an ace card`, () => {

// Suggestion: Group the remaining test data into these categories:
// Number Cards (2-10)
test(`Should return the correct number for number cards`, () => {
expect(getCardValue("2♠")).toEqual(2);
expect(getCardValue("9♥")).toEqual(9);
expect(getCardValue("10♦")).toEqual(10);
});
// Face Cards (J, Q, K)

test(Should return 10 for face cards, () => {
expect(getCardValue("J♣")).toEqual(10);
expect(getCardValue("Q♦")).toEqual(10);
expect(getCardValue("K♠")).toEqual(10);
});
// Invalid Cards
test(`Should throw an error for invalid card strings`, () => {
const invalidCards = ["", "A", "11♠", "1♠", "B♠", "10X", "♠A", "invalid"];

for (const badCard of invalidCards) {
expect(() => getCardValue(badCard)).toThrowError();
}
});


// To learn how to test whether a function throws an error as expected in Jest,
// please refer to the Jest documentation:
Expand Down
Loading