Skip to content

Commit ddb16d2

Browse files
committed
revised function and test for getAngleType.
1 parent 6f0f79f commit ddb16d2

3 files changed

Lines changed: 17 additions & 10 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
@@ -31,6 +31,11 @@ test("Classifies boundary and invalid angles", () => {
3131
assert.equal(getAngleType(0), "invalid angle");
3232
assert.equal(getAngleType(""), "invalid angle");
3333
assert.equal(getAngleType("angle"), "invalid angle");
34-
assert.equal(getAngleType("null"), "invalid angle")
35-
34+
assert.equal(getAngleType("null"), "invalid angle");
35+
assert.equal(getAngleType(360), "invalid angle");
36+
assert.equal(getAngleType(181), "Reflex angle");
37+
assert.equal(getAngleType(91), "Obtuse angle");
38+
assert.equal(getAngleType(179), "Obtuse angle");
39+
assert.equal(getAngleType(89), "Acute angle");
40+
assert.equal(getAngleType(1), "Acute angle");
3641
});

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

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

1919
function getAngleType(angle) {
20-
if (angle > 360 || angle <= 0 || angle === "" || isNaN(angle)) {
21-
return "invalid angle"
22-
}
23-
else if (angle > 180 && angle < 360) {
20+
if (angle >= 360 || angle <= 0 || angle === "" || isNaN(angle)) {
21+
return "invalid angle";
22+
} else if (angle > 180 && angle < 360) {
2423
return "Reflex angle";
2524
} else if (angle === 180) {
2625
return "Straight angle";
@@ -31,7 +30,6 @@ function getAngleType(angle) {
3130
} else if (angle < 90 && angle > 0) {
3231
return "Acute angle";
3332
}
34-
3533
}
3634

3735
export { getAngleType };

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

Lines changed: 7 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -18,9 +18,9 @@ test(`should return "Right angle" when (angle === 90)`, () => {
1818

1919
// Case 3: Obtuse angles
2020
test(`should return "Obtuse angle" when (angle < 180 && angle > 90)`, () => {
21-
expect(getAngleType(100)).toEqual("Obtuse angle");
21+
expect(getAngleType(91)).toEqual("Obtuse angle");
2222
expect(getAngleType(120)).toEqual("Obtuse angle");
23-
expect(getAngleType(175)).toEqual("Obtuse angle");
23+
expect(getAngleType(179)).toEqual("Obtuse angle");
2424
});
2525

2626
// Case 4: Straight angle
@@ -30,13 +30,17 @@ test(`should return "Straight angle" when (angle === 180)`, () => {
3030

3131
// Case 5: Reflex angles
3232
test(`should return "Reflex angle" when (angle > 180 && angle < 360)`, () => {
33+
expect(getAngleType(181)).toEqual("Reflex angle");
3334
expect(getAngleType(190)).toEqual("Reflex angle");
35+
expect(getAngleType(359)).toEqual("Reflex angle");
3436
});
3537

3638
// Case 6: Invalid angles
37-
test(`should return "Invalid value" when (angel > 360 || angle <= 0 || angel = "")`, () => {
39+
test(`should return "Invalid value" when (angel >= 360 || angle <= 0 || angel = "")`, () => {
3840
expect(getAngleType(-10)).toEqual("invalid angle");
3941
expect(getAngleType(365)).toEqual("invalid angle");
42+
expect(getAngleType(360)).toEqual("invalid angle");
43+
expect(getAngleType(0)).toEqual("invalid angle");
4044
expect(getAngleType()).toEqual("invalid angle");
4145
expect(getAngleType(null)).toEqual("invalid angle");
4246
expect(getAngleType("angle")).toEqual("invalid angle");

0 commit comments

Comments
 (0)