Further updates

Author: libanj0161

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

@@ -24,18 +24,13 @@
 function getCardValue(card) {
   const suits = ["♠", "♥", "♦", "♣"]; // valid card suits
 
-  let suit = "";
-  let rank = card;
-
-  if (suits.includes(card.slice(-1))) {
-    suit = card.slice(-1); // gives me the last character and saves it in the suit variable.
-    rank = card.slice(0, -1); // gets beginning character up to (not including) the last character and saves it in the rank variable.
-  }
-
-  if (suit && !suits.includes(suit)) {
+  // must have a valid suit
+  if (!suits.includes(card.slice(-1))) {
     throw new Error("Error invalid card");
   }
 
+  const rank = card.slice(0, -1); // gets beginning character up to (not including) the last character and saves it in the rank variable.
+
   // if card is an ace return 11
   if (rank === "A") {
     return 11;

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

@@ -28,7 +28,7 @@ test(`Should return 10 when given an face card`, () => {
 });
 // Case 3: Number Cards (2-10)
 test(`Should return numerical value when given an number card`, () => {
-  expect(getCardValue("5")).toEqual(5);
+  expect(() => getCardValue("5")).toThrow("Error invalid card");
 });
 
 test(`Should return error if the card string is invalid`, () => {