Skip to content

Commit 562e2d1

Browse files
adding negative logic to fraction exercise and tests
1 parent 92ada3e commit 562e2d1

2 files changed

Lines changed: 15 additions & 6 deletions

File tree

‎implement-and-rewrite-tests/implement-tests-with-node-test/2-is-proper-fraction.test.js‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -40,4 +40,8 @@ test("positive numerator and negative denumerator should return false", () => {
4040
assert.equal(isProperFraction(5, -16), false);
4141
});
4242

43+
test("negative numerator smaller then a negative denumerator should return false", () => {
44+
assert.equal(isProperFraction(-50 , -16), false);
45+
});
46+
4347

‎implement-and-rewrite-tests/implement/2-is-proper-fraction.js‎

Lines changed: 11 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -14,16 +14,21 @@
1414
// execute the code to ensure all tests pass.
1515

1616
export function isProperFraction(numerator, denominator) {
17-
if (Number.isNaN(numerator) || Number.isNaN(denominator)|| denominator===0) {
18-
return false;
19-
}
17+
if (
18+
Number.isNaN(numerator) ||
19+
Number.isNaN(denominator) ||
20+
denominator === 0
21+
) {
22+
return false;
23+
}
2024
if (!Number.isInteger(numerator) || !Number.isInteger(denominator)) {
2125
return false;
2226
}
23-
if (numerator >= denominator ) {
27+
if (numerator >= denominator) {
2428
return false;
2529
}
30+
if (numerator < 0 && denominator < 0) {
31+
if (Math.abs(numerator) > Math.abs(denominator)) return false;
32+
}
2633
return true;
27-
28-
2934
}

0 commit comments

Comments
 (0)