Skip to content

Commit ccb7a54

Browse files
committed
revise function and test for getAngleType
1 parent c67d63d commit ccb7a54

3 files changed

Lines changed: 18 additions & 7 deletions

File tree

‎implement-and-rewrite-tests/implement-tests-with-node-test/1-get-angle-type.test.js‎

Lines changed: 7 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,11 @@ test("Classifies reflex angle", () => {
2626
});
2727

2828
test("Classifies boundary and invalid angles", () => {
29-
assert.equal(getAngleType(365), "Invalid angle");
30-
assert.equal(getAngleType(-10), "Invalid angle");
29+
assert.equal(getAngleType(365), "invalid angle");
30+
assert.equal(getAngleType(-10), "invalid angle");
31+
assert.equal(getAngleType(0), "invalid angle");
32+
assert.equal(getAngleType(""), "invalid angle");
33+
assert.equal(getAngleType("angle"), "invalid angle");
34+
assert.equal(getAngleType("null"), "invalid angle")
35+
3136
});

‎implement-and-rewrite-tests/implement/1-get-angle-type.js‎

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,10 @@
1717
// execute the code to ensure all tests pass.
1818

1919
function getAngleType(angle) {
20-
if (angle > 180 && angle < 360) {
20+
if (angle > 360 || angle <= 0 || angle === "" || isNaN(angle)) {
21+
return "invalid angle"
22+
}
23+
else if (angle > 180 && angle < 360) {
2124
return "Reflex angle";
2225
} else if (angle === 180) {
2326
return "Straight angle";
@@ -28,7 +31,7 @@ function getAngleType(angle) {
2831
} else if (angle < 90 && angle > 0) {
2932
return "Acute angle";
3033
}
31-
return "Invalid angle";
34+
3235
}
3336

3437
export { getAngleType };

‎implement-and-rewrite-tests/rewrite-tests-with-jest/1-get-angle-type.test.js‎

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -34,7 +34,10 @@ test(`should return "Reflex angle" when (angle > 180 && angle < 360)`, () => {
3434
});
3535

3636
// Case 6: Invalid angles
37-
test(`should return "Invalid angle" when (angel > 360 || angle < 1)`, () => {
38-
expect(getAngleType(-10)).toEqual("Invalid angle");
39-
expect(getAngleType(365)).toEqual("Invalid angle");
37+
test(`should return "Invalid value" when (angel > 360 || angle <= 0 || angel = "")`, () => {
38+
expect(getAngleType(-10)).toEqual("invalid angle");
39+
expect(getAngleType(365)).toEqual("invalid angle");
40+
expect(getAngleType()).toEqual("invalid angle");
41+
expect(getAngleType(null)).toEqual("invalid angle");
42+
expect(getAngleType("angle")).toEqual("invalid angle");
4043
});

0 commit comments

Comments
 (0)