Refactor getCardValue tests to group categories and add invalid card cases

Author: arunkumarakilan

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

@@ -8,31 +8,35 @@ const getCardValue = require("../implement/3-get-card-value");
 test(`Should return 11 when given an ace card`, () => {
   expect(getCardValue("A♠")).toEqual(11);
 });
-test(`should return 9 when given a 9♠ card`, () => {
-  expect(getCardValue("9♠")).toEqual(9);
-});
-test(`should return 10 when given a face card K♦`, () => {
-  expect(getCardValue("K♦")).toEqual(10);
+//   Number Cards (2-10)
+test("should return the value of number cards (2–10)", () => {
+  expect(getCardValue("2♣")).toEqual(2);
+  expect(getCardValue("5♦")).toEqual(5);
+  expect(getCardValue("9♥")).toEqual(9);
+  expect(getCardValue("10♠")).toEqual(10);
 });
-test(`should return 10 when given a 10`, () => {
-  expect(getCardValue("10♥")).toEqual(10);
+//   Face Cards (J, Q, K)
+test("should return 10 for face cards", () => {
+  expect(getCardValue("J♣")).toEqual(10);
+  expect(getCardValue("Q♦")).toEqual(10);
+  expect(getCardValue("K♥")).toEqual(10);
 });
-test(`should throw error for invalid Ranks`, () => {
+//   Invalid Cards
+test("should throw error for invalid ranks", () => {
   expect(() => getCardValue("11♠")).toThrow();
 });
-test(`should throw error for invalid Suits`, () => {
+
+test("should throw error for invalid suits", () => {
   expect(() => getCardValue("A?")).toThrow();
 });
-test(`should throw error for missing Suits`, () => {
+
+test("should throw error for missing suits", () => {
   expect(() => getCardValue("A")).toThrow();
 });
-test(`should throw error for non String input`, () => {
-  expect(() => getCardValue("10")).toThrow();
+
+test("should throw error for non-string input", () => {
+  expect(() => getCardValue(10)).toThrow();
 });
-// Suggestion: Group the remaining test data into these categories:
-//   Number Cards (2-10)
-//   Face Cards (J, Q, K)
-//   Invalid Cards
 
 // To learn how to test whether a function throws an error as expected in Jest,
 // please refer to the Jest documentation: