Skip to content
Open
Original file line number Diff line number Diff line change
Expand Up @@ -16,8 +16,25 @@

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.
// This will be useful in the "rewrite tests with jest" step.
module.exports = getAngleType;
Expand All @@ -35,3 +52,12 @@ function assertEquals(actualOutput, targetOutput) {
// Example: Identify Right Angles
const right = getAngleType(90);
assertEquals(right, "Right angle");
assertEquals(getAngleType(45), "Acute angle")
assertEquals(getAngleType(120), "Obtuse angle");
assertEquals(getAngleType(180), "Straight angle");
assertEquals(getAngleType(270), "Reflex angle")
assertEquals(getAngleType(-10), "Invalid angle")
assertEquals(getAngleType(0), "Invalid angle");
assertEquals(getAngleType(360), "Invalid angle");
assertEquals(getAngleType(400), "Invalid angle");

Original file line number Diff line number Diff line change
Expand Up @@ -12,8 +12,25 @@

function isProperFraction(numerator, denominator) {
// TODO: Implement this function
if (denominator <= 0) {
return false;
}
if (numerator < 0) {
return false;
}
return numerator < denominator;
Comment on lines +15 to +21

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What authoritative source do you base your definition of a proper fraction on?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thank you @cjyuan for your feedback. I have added authoritative source and made necessary adjustments.

}

/* A proper fraction is defined as a fraction where:
- The numerator is less than the denominator.
- The denominator is positive.

Source: https://www.mathsisfun.com/proper-fractions.html
Comment on lines +26 to +28

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Note: The source does not actually say anything about negative numerator or denominator.

No change required because the spec also does not say anything about what to do with negative numbers.

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Noted. Thank you @cjyuan.

*/




// 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;
Expand All @@ -31,3 +48,24 @@ function assertEquals(actualOutput, targetOutput) {

// Example: 1/2 is a proper fraction
assertEquals(isProperFraction(1, 2), true);
// Proper fractions
assertEquals(isProperFraction(2, 3), true);
assertEquals(isProperFraction(3, 4), true);
assertEquals(isProperFraction(0, 5), true);

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

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

// Negative numerators
assertEquals(isProperFraction(-1, 2), false);

// Negative denominators
assertEquals(isProperFraction(1, -2), false);

// Both negative
assertEquals(isProperFraction(-3, -2), false);

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

function getCardValue(card) {
// TODO: Implement this function
const validRanks = [
"A",
"2",
"3",
"4",
"5",
"6",
"7",
"8",
"9",
"10",
"J",
"Q",
"K",
];
const validSuits = ["♠", "♥", "♦", "♣"];

// Suit is always the last character
const suit = card.slice(-1);

// Rank is everything before the suit
const rank = card.slice(0, -1);

// Validate rank and suit
if (!validRanks.includes(rank) || !validSuits.includes(suit)) {
throw new Error("Invalid card");
}

// Convert rank to value
if (rank === "A") return 11;
if (["J", "Q", "K"].includes(rank)) return 10;

// Number card
return Number(rank);
}

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

// Handling invalid cards
try {
Expand All @@ -52,3 +92,30 @@ try {
}

// What other invalid card cases can you think of?
try {
getCardValue("1♠"); // invalid rank
console.error("Error was not thrown for invalid rank");
} catch (e) {
console.log("Error thrown for invalid rank 🎉");
}

try {
getCardValue("A?"); // invalid suit
console.error("Error was not thrown for invalid suit");
} catch (e) {
console.log("Error thrown for invalid suit 🎉");
}

try {
getCardValue("10"); // missing suit
console.error("Error was not thrown for missing suit");
} catch (e) {
console.log("Error thrown for missing suit 🎉");
}

try {
getCardValue(""); // empty string
console.error("Error was not thrown for empty string");
} catch (e) {
console.log("Error thrown for empty string 🎉");
}
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,48 @@ test(`should return "Acute angle" when (0 < angle < 90)`, () => {
});

// Case 2: Right angle
test(`should return "Right angle" when angle = 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(120)).toEqual("Obtuse angle");
expect(getAngleType(179)).toEqual("Obtuse angle");
});

// Case 4: Straight angle
test(`should return "Straight angle" when angle = 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" for angles outside valid range (1-359)`, () => {

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Could the value of angle be a decimal number? For example, is 0.5 a valid angle?

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Thanks for pointing this out. I hadn't considered decimal angles, but I've now added tests to cover fractional values like 0.5.

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

What about the valid range?

... outside valid range (1-359)

expect(getAngleType(-10)).toEqual("Invalid angle");
expect(getAngleType(0)).toEqual("Invalid angle");
expect(getAngleType(360)).toEqual("Invalid angle");
expect(getAngleType(400)).toEqual("Invalid angle");
});

// Case 7: Decimal angles
test(`should correctly classify decimal angles`, () => {
// valid decimal angles
expect(getAngleType(0.5)).toEqual("Acute angle");
expect(getAngleType(89.9)).toEqual("Acute angle");
expect(getAngleType(90.1)).toEqual("Obtuse angle");
expect(getAngleType(179.9)).toEqual("Obtuse angle");
expect(getAngleType(180.1)).toEqual("Reflex angle");
expect(getAngleType(359.9)).toEqual("Reflex angle");

// invalid decimal angles
expect(getAngleType(-0.5)).toEqual("Invalid angle");
expect(getAngleType(360.1)).toEqual("Invalid angle");
});
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,36 @@ const isProperFraction = require("../implement/2-is-proper-fraction");

// TODO: Write tests in Jest syntax to cover all combinations of positives, negatives, zeros, and other categories.

// Special case: numerator is zero
test(`should return false when denominator is zero`, () => {
expect(isProperFraction(1, 0)).toEqual(false);
// Denominator zero
test("should return false when denominator is zero", () => {
expect(isProperFraction(1, 0)).toBe(false);
});

// Proper fractions
test("should return true for proper fractions (numerator < denominator)", () => {
expect(isProperFraction(1, 2)).toBe(true);
expect(isProperFraction(2, 3)).toBe(true);
expect(isProperFraction(3, 4)).toBe(true);
expect(isProperFraction(0, 5)).toBe(true); // zero numerator is allowed
});

// Improper fractions
test("should return false for improper fractions (numerator >= denominator)", () => {
expect(isProperFraction(5, 5)).toBe(false);
expect(isProperFraction(7, 3)).toBe(false);
});

// Negative numerators
test("should return false when numerator is negative", () => {
expect(isProperFraction(-1, 2)).toBe(false);
});

// Negative denominators
test("should return false when denominator is negative", () => {
expect(isProperFraction(1, -2)).toBe(false);
});

// Both negative
test("should return false when both numerator and denominator are negative", () => {
expect(isProperFraction(-3, -2)).toBe(false);
});
Original file line number Diff line number Diff line change
Expand Up @@ -7,13 +7,39 @@ const getCardValue = require("../implement/3-get-card-value");
// Case 1: Ace (A)
test(`Should return 11 when given an ace card`, () => {
expect(getCardValue("A♠")).toEqual(11);
expect(getCardValue("A♥")).toEqual(11);
expect(getCardValue("A♦")).toEqual(11);
expect(getCardValue("A♣")).toEqual(11);
});

// Suggestion: Group the remaining test data into these categories:
// Number Cards (2-10)
// Face Cards (J, Q, K)
// Invalid Cards

// Case 2: Number cards (2–10)
test("should return correct values for number cards", () => {
expect(getCardValue("2♠")).toEqual(2);
expect(getCardValue("7♦")).toEqual(7);
expect(getCardValue("10♥")).toEqual(10);
});

// Case 3: 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);
});

// Case 4: Invalid cards
test("should throw an error for invalid card values", () => {
expect(() => getCardValue("1♠")).toThrowError();
expect(() => getCardValue("Z♦")).toThrowError();
expect(() => getCardValue("")).toThrowError();
expect(() => getCardValue("♠")).toThrowError();
});


// To learn how to test whether a function throws an error as expected in Jest,
// please refer to the Jest documentation:
// https://jestjs.io/docs/expect#tothrowerror
Expand Down
Loading