Apply changes asked by the reviewer

Author: mohsenzamanist

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

@@ -58,5 +58,5 @@ assertEquals(reflex, "Reflex angle");
 const invalid1 = getAngleType(400);
 assertEquals(invalid1, "Invalid angle");
 
-const invalid2 = getAngleType("400");
+const invalid2 = getAngleType("510");
 assertEquals(invalid2, "Invalid angle");

Sprint-3/1-implement-and-rewrite-tests/implement/2-is-proper-fraction.js

@@ -14,8 +14,7 @@ function isProperFraction(numerator, denominator) {
   numerator = Math.abs(numerator);
   denominator = Math.abs(denominator);
   if (numerator < denominator) return true;
-  else if (numerator > denominator) return false;
-  else if (numerator === denominator) return false;
+  return false;
 }
 
 // The line below allows us to load the isProperFraction function into tests in other files.

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

@@ -40,5 +40,6 @@ test("should identify reflex angle (180° < angle < 360)", () => {
 // Case 6: Invalid angles
 test("should identify invalid angle (0 > angle > 360)", () => {
   expect(getAngleType(0)).toEqual("Invalid angle");
+  expect(getAngleType(360)).toEqual("Invalid angle");
   expect(getAngleType(361)).toEqual("Invalid angle");
 });

Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/2-is-proper-fraction.test.js

@@ -17,8 +17,8 @@ test("should return false when both numerator and denominator are zero", () => {
   expect(isProperFraction(0, 0)).toEqual(false);
 });
 
-// Case 2: Identify Improper Fractions:
-test("should return false for improper fraction", () => {
+// Case 2: Identify proper Fractions:
+test("should return true for proper fraction", () => {
   expect(isProperFraction(2, 3)).toEqual(true);
 });
 

Sprint-3/1-implement-and-rewrite-tests/rewrite-tests-with-jest/3-get-card-value.test.js

@@ -35,6 +35,9 @@ test("should return 10 for face cards", () => {
 test("Should return 'Invalid card rank.' for invalid cards", () => {
   expect(() => getCardValue("KJ")).toThrow("Invalid card rank.");
   expect(() => getCardValue("AK")).toThrow("Invalid card rank.");
+  expect(() => getCardValue("  ")).toThrow("Invalid card rank.");
+  expect(() => getCardValue("S♠")).toThrow("Invalid card rank.");
+  expect(() => getCardValue("J♠♠")).toThrow("Invalid card rank.");
 });
 
 // Suggestion: Group the remaining test data into these categories: