Skip to content
Open
Original file line number Diff line number Diff line change
@@ -1,37 +1,65 @@
// Implement a function getAngleType
//
// When given an angle in degrees, it should return a string indicating the type of angle:
// - "Acute angle" for angles greater than 0° and less than 90°
// - "Right angle" for exactly 90°
// - "Obtuse angle" for angles greater than 90° and less than 180°
// - "Straight angle" for exactly 180°
// - "Reflex angle" for angles greater than 180° and less than 360°
// - "Invalid angle" for angles outside the valid range.

// Assumption: The parameter is a valid number. (You do not need to handle non-numeric inputs.)

// Acceptance criteria:
// After you have implemented the function, write tests to cover all the cases, and
// execute the code to ensure all tests pass.

function getAngleType(angle) {
// TODO: Implement this function
if (angle <= 0 || angle >= 360) {
return "Invalid angle";
}
else if (angle == 90) {

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Whats the difference between == and === and why are you using == here?

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.

== converts and compares operands that are of different types and === always considers operands of different types to be different. In this case, angle == 90 which means LHS is compared with RHS and it is equal. I can also use ===.

return "Right angle";
}
else if (angle == 180) {
return "Straight angle";
}
else if (angle > 0 && angle < 90) {
return "Acute angle";
}
else if (angle > 90 && angle < 180) {
return "Obtuse angle";
}
else
return "Reflex 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;

// This helper function is written to make our assertions easier to read.
// If the actual output matches the target output, the test will pass
function assertEquals(actualOutput, targetOutput) {
console.assert(
actualOutput === targetOutput,
`Expected ${actualOutput} to equal ${targetOutput}`
);
}
const invalid = getAngleType(0);
assertEquals(invalid, "Invalid angle");
console.log (getAngleType(0));

const invalid0 = getAngleType(-10);
assertEquals(invalid, "Invalid angle");
console.log (getAngleType(-10));

const invalid1 = getAngleType(360);
assertEquals(invalid1, "Invalid angle");
console.log (getAngleType(360));

const invalid2 = getAngleType(900);
assertEquals(invalid2, "Invalid angle");
console.log (getAngleType(900));

// TODO: Write tests to cover all cases, including boundary and invalid cases.
// Example: Identify Right Angles
const right = getAngleType(90);
assertEquals(right, "Right angle");
console.log (getAngleType(90));

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

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

const obtuse = getAngleType(135);
assertEquals(obtuse, "Obtuse angle");
console.log (getAngleType(135));

const reflex = getAngleType(240);
assertEquals(reflex, "Reflex angle");
console.log (getAngleType(240));

module.exports = getAngleType;

Original file line number Diff line number Diff line change
Expand Up @@ -11,23 +11,66 @@
// execute the code to ensure all tests pass.

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

// 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;
return Math.abs(numerator) < Math.abs(denominator);
}

// Here's our helper again
function assertEquals(actualOutput, targetOutput) {
console.assert(
actualOutput === targetOutput,
`Expected ${actualOutput} to equal ${targetOutput}`
);
}

const case1 = isProperFraction(1, 2);
assertEquals(case1, true);
console.log(isProperFraction(1, 2));

const case2 = isProperFraction(3, 4);
assertEquals(case2, true);
console.log(isProperFraction(3, 4));

const case3 = isProperFraction(4, 4);
assertEquals(case3, false);
console.log(isProperFraction(4, 4));

const case4 = isProperFraction(8, 5);
assertEquals(case4, false);
console.log(isProperFraction(8, 5));

const case5 = isProperFraction(16, 22);
assertEquals(case5, true);
console.log(isProperFraction(16, 22));

const case6 = isProperFraction(0, 6);
assertEquals(case6, true);
console.log(isProperFraction(0, 6));

const case7 = isProperFraction(5, 0);
assertEquals(case7, false);
console.log(isProperFraction(5, 0));

const case8 = isProperFraction(-1, 2);
assertEquals(case8, true);
console.log(isProperFraction(-1, 2));

const case9 = isProperFraction(1, -2);
assertEquals(case9, true);
console.log(isProperFraction(1, -2));

module.exports = isProperFraction;

// 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.


// Here's our helper again
// TODO: Write tests to cover all cases.
// What combinations of numerators and denominators should you test?

// Example: 1/2 is a proper fraction
assertEquals(isProperFraction(1, 2), true);
// assertEquals(isProperFraction(1, 2), true);
119 changes: 106 additions & 13 deletions Sprint-3/1-implement-and-rewrite-tests/implement/3-get-card-value.js
Original file line number Diff line number Diff line change
Expand Up @@ -22,33 +22,126 @@
// execute the code to ensure all tests pass.

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

// The line below allows us to load the getCardValue function into tests in other files.
// This will be useful in the "rewrite tests with jest" step.
module.exports = getCardValue;
const validRank = [
"A",
"2",
"3",
"4",
"5",
"6",
"7",
"8",
"9",
"10",
"J",
"Q",
"K",
];
const validSuit = ["♠", "♥", "♦", "♣"];

if (!validRank.includes(rank) || !validSuit.includes(suit)) {
throw new Error("Invalid Card");
}

if (rank === "A") {
return 11;
} else if (rank === "J" || rank === "Q" || rank === "K") {
return 10;
} else return Number(rank);
}

// Helper functions to make our assertions easier to read.
function assertEquals(actualOutput, targetOutput) {
console.assert(
actualOutput === targetOutput,
`Expected ${actualOutput} to equal ${targetOutput}`
);
}

// TODO: Write tests to cover all outcomes, including throwing errors for invalid cards.
// Examples:
assertEquals(getCardValue("9♠"), 9);
const ace = getCardValue("A♠");
assertEquals(getCardValue("A♠"), 11);
console.log(getCardValue("A♠"));

const faceJ = getCardValue("J♣");
assertEquals(getCardValue("J♣"), 10);
console.log(getCardValue("J♣"));

const faceQ = getCardValue("Q♦");
assertEquals(getCardValue("Q♦"), 10);
console.log(getCardValue("Q♦"));

const faceK = getCardValue("K♦");
assertEquals(getCardValue("K♦"), 10);
console.log(getCardValue("K♦"));

const number5 = getCardValue("5♥");
assertEquals(getCardValue("5♥"), 5);
console.log(getCardValue("5♥"));

const number10 = getCardValue("10♥");
assertEquals(getCardValue("10♥"), 10);
console.log(getCardValue("10♥"));

try {
getCardValue("10");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}

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

// Handling invalid cards
try {
getCardValue("invalid");
getCardValue("10x");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}

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

// What other invalid card cases can you think of?
try {
getCardValue("Qxx");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}
try {
getCardValue("*");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}
try {
getCardValue("diamonds");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}
try {
getCardValue("♤");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}
try {
getCardValue("x");
console.error("Error was not thrown for invalid card 😢");
} catch (e) {
console.log("Error thrown for invalid card 🎉");
}


module.exports = getCardValue;
Original file line number Diff line number Diff line change
@@ -1,20 +1,28 @@
// This statement loads the getAngleType function you wrote in the implement directory.
// We will use the same function, but write tests for it using Jest in this file.
const getAngleType = require("../implement/1-get-angle-type");

// TODO: Write tests in Jest syntax to cover all cases/outcomes,
// including boundary and invalid cases.

// Case 1: Acute angles
test(`should return "Acute angle" when (0 < angle < 90)`, () => {
// Test various acute angles, including boundary cases
expect(getAngleType(1)).toEqual("Acute angle");
expect(getAngleType(45)).toEqual("Acute angle");
expect(getAngleType(89)).toEqual("Acute angle");
});
test(`should return "Right angle" when (angle == 90)`, () => {
expect(getAngleType(90)).toEqual("Right angle");
});
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");
});
test(`should return "Straight angle" when (angle == 180)`, () => {
expect(getAngleType(180)).toEqual("Straight angle");
});
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");
Comment on lines +12 to +22

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Good job on testing the border cases here

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!

});
test(`should return "Invalid angle" when (angle <= 0 || angle >= 360)`, () => {
expect(getAngleType(-1)).toEqual("Invalid angle");
expect(getAngleType(370)).toEqual("Invalid angle");
Comment on lines +25 to +26

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Which border cases can you test here? (Which numbers are closest to the condition?)

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.

I tested for the invalid case when (angle <= 0 || angle >= 360). So, when i considered -1 and 370, the closest number for this condition is -1 and 361. Because -1 is closest to 0 and 361 is closest to 360.

});

// Case 2: Right angle
// Case 3: Obtuse angles
// Case 4: Straight angle
// Case 5: Reflex angles
// Case 6: Invalid angles
Original file line number Diff line number Diff line change
Expand Up @@ -8,3 +8,27 @@ const isProperFraction = require("../implement/2-is-proper-fraction");
test(`should return false when denominator is zero`, () => {
expect(isProperFraction(1, 0)).toEqual(false);
});
test(`should return true when the function is proper fraction`, () => {
expect(isProperFraction(1, 2)).toEqual(true);
});
test(`should return false when the function is improper fraction`, () => {
expect(isProperFraction(8, 5)).toEqual(false);
});
test(`should return false when the numerator is equal to denominator`, () => {
expect(isProperFraction(4, 4)).toEqual(false);
});
test(`should return true when numerator is zero`, () => {
expect(isProperFraction(0, 1)).toEqual(true);
});
test(`should return false when numerator is negative`, () => {
expect(isProperFraction(-1, 1)).toEqual(false);
});
test(`should return false when denominator is negative`, () => {
expect(isProperFraction(1, -1)).toEqual(false);
});
test(`should return true when numerator is negative but not the same number as denominator`, () => {
expect(isProperFraction(-1, 2)).toEqual(true);
});
test(`should return true when denominator is negative but not the same number as denominator`, () => {
expect(isProperFraction(1, -2)).toEqual(true);
});
Comment on lines +23 to +34

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

What about values like -1/2 or 1/-2. What should the function return for them?

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.

I think for values like -1/2 or 1/-2, the function will also return false because proper fractions normally require positive numerator and denominator.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

I believe a proper fraction can also include negative values
In general, a common fraction is said to be a proper fraction if the absolute value of the fraction is strictly less than one—that is, if the fraction is greater than −1 and less than 1.

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.

Oh! now I am clear about the concept of proper fraction. So -1/2 or 1/-2 can be proper fraction cause both of these absolute value is 1/2 which is -1<1/2<1.

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Correct. Please adjust the tests accordingly.

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.

adjusted the tests.

Loading
Loading