-
-
Notifications
You must be signed in to change notification settings - Fork 261
West Midlands | 25 Sep ITP | Iswat Bello | Sprint 3 | Implement and rewrite tests #800
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from 10 commits
f17635b
3766d27
f9eea26
cfda58c
b7a2183
f028f06
fec9455
0032765
69ddec4
8c717df
028ec7d
6bcf17a
00515f5
31446ec
121ebe4
b46a6d3
273a36a
48f2143
9705d12
5c5e00b
a66ab62
3b9f415
5e0b672
8daafbd
050e86a
df7ccfb
46aee35
47809af
cfecb6c
9a88261
7a4e526
5411c34
3b5e67d
bbde7ad
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,8 +8,17 @@ | |
| // write one test at a time, and make it pass, build your solution up methodically | ||
| // just make one change at a time -- don't rush -- programmers are deep and careful thinkers | ||
| function getCardValue(card) { | ||
| if (rank === "A") { | ||
| card = card.substring(0 , card.length -1); // Remove the suit emoji | ||
|
||
| console.log(card); | ||
|
||
| if (card === "A") { | ||
| return 11; | ||
| } if (card === "J" || card === "Q" || card === "K" || card === "10") { | ||
| return 10; | ||
| } else if (card >= 2 && card <=9) { | ||
cjyuan marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| card = Number(card); | ||
| return card; | ||
| } else { | ||
| return "Invalid card rank."; | ||
| } | ||
| } | ||
|
|
||
|
|
@@ -28,7 +37,8 @@ function assertEquals(actualOutput, targetOutput) { | |
| } | ||
| // Acceptance criteria: | ||
|
|
||
| // Given a card string in the format "A♠" (representing a card in blackjack - the last character will always be an emoji for a suit, and all characters before will be a number 2-10, or one letter of J, Q, K, A), | ||
| // Given a card string in the format "A♠" (representing a card in blackjack - the last character will always be an emoji for a suit, | ||
| // and all characters before will be a number 2-10, or one letter of J, Q, K, A), | ||
| // When the function getCardValue is called with this card string as input, | ||
| // Then it should return the numerical card value | ||
| const aceofSpades = getCardValue("A♠"); | ||
|
|
@@ -40,11 +50,23 @@ assertEquals(aceofSpades, 11); | |
| // Then it should return the numeric value corresponding to the rank (e.g., "5" should return 5). | ||
| const fiveofHearts = getCardValue("5♥"); | ||
| // ====> write your test here, and then add a line to pass the test in the function above | ||
| assertEquals(fiveofHearts, 5); | ||
|
|
||
| // Handle Face Cards (J, Q, K): | ||
| // Given a card with a rank of "10," "J," "Q," or "K", | ||
| // When the function is called with such a card, | ||
| // Then it should return the value 10, as these cards are worth 10 points each in blackjack. | ||
| const jackOfDiamonds = getCardValue("J♦"); | ||
| assertEquals(jackOfDiamonds, 10); | ||
|
|
||
| const queenOfClubs = getCardValue("Q♣"); | ||
| assertEquals(queenOfClubs, 10); | ||
|
|
||
| const kingOfHearts = getCardValue("K♥"); | ||
| assertEquals(kingOfHearts, 10); | ||
|
|
||
| const tenOfSpades = getCardValue("10♠"); | ||
| assertEquals(tenOfSpades, 10); | ||
|
|
||
| // Handle Ace (A): | ||
| // Given a card with a rank of "A", | ||
|
|
@@ -55,3 +77,7 @@ const fiveofHearts = getCardValue("5♥"); | |
| // Given a card with an invalid rank (neither a number nor a recognized face card), | ||
| // When the function is called with such a card, | ||
| // Then it should throw an error indicating "Invalid card rank." | ||
|
|
||
|
|
||
| const invalidRandomCard = getCardValue("15♠"); | ||
| //assertEquals(invalidRandomCard, "Invalid card rank."); | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -12,15 +12,26 @@ test("should identify right angle (90°)", () => { | |
| // Case 2: Identify Acute Angles: | ||
| // When the angle is less than 90 degrees, | ||
| // Then the function should return "Acute angle" | ||
| test("should identify acute angle (<90°)", () => { | ||
| expect(getAngleType(50)).toEqual("Acute angle"); | ||
| }); | ||
|
|
||
| // Case 3: Identify Obtuse Angles: | ||
| // When the angle is greater than 90 degrees and less than 180 degrees, | ||
| // Then the function should return "Obtuse angle" | ||
| test("should identiify obtuse angle (>90° and <180°)", () => { | ||
| expect(getAngleType(120)).toEqual("Obtuse angle"); | ||
| }); | ||
|
|
||
| // Case 4: Identify Straight Angles: | ||
| // When the angle is exactly 180 degrees, | ||
| // Then the function should return "Straight angle" | ||
|
|
||
| test("should identify straight angle (180°)", () => { | ||
| expect(getAngleType(180)).toEqual("Straight angle"); | ||
| }); | ||
| // Case 5: Identify Reflex Angles: | ||
| // When the angle is greater than 180 degrees and less than 360 degrees, | ||
| // Then the function should return "Reflex angle" | ||
| test("should identify reflex angle (>180° and <360°)", () => { | ||
| expect(getAngleType(250)).toEqual("Reflex angle"); | ||
| }); | ||
|
Comment on lines
42
to
47
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. To make a test more robust, we could specify multiple |
||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -2,12 +2,21 @@ | |
| // We will use the same function, but write tests for it using Jest in this file. | ||
| const isProperFraction = require("../implement/2-is-proper-fraction"); | ||
|
|
||
| test("should return true for a proper fraction", () => { | ||
| test("should return true for a proper fractions", () => { | ||
| expect(isProperFraction(2, 3)).toEqual(true); | ||
| }); | ||
|
|
||
| // Case 2: Identify Improper Fractions: | ||
| test("should return false for improper fractions", () => { | ||
| expect(isProperFraction(7, 3)).toEqual(false); | ||
| }) | ||
|
|
||
| // Case 3: Identify Negative Fractions: | ||
| test("should return false for negative fractions", () => { | ||
| expect(isProperFraction(-4, 9)).toEqual(true); | ||
| }) | ||
|
||
|
|
||
| // Case 4: Identify Equal Numerator and Denominator: | ||
| test("should return false for equal numerator and denominator", () => { | ||
| expect(isProperFraction(3, 3)).toEqual(false); | ||
| }) | ||
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -8,6 +8,22 @@ test("should return 11 for Ace of Spades", () => { | |
| }); | ||
|
|
||
| // Case 2: Handle Number Cards (2-10): | ||
| test("should return the numeric value for number cards from 2 to 10", () => { | ||
| const numberCard = getCardValue("7♥"); | ||
| expect(numberCard).toEqual(7); | ||
| }) | ||
|
Comment on lines
11
to
15
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Could consider testing multiple values, especially the boundary cases "2♥" and "10♥". |
||
| // Case 3: Handle Face Cards (J, Q, K): | ||
| test("should return 10 for face cards", () => { | ||
| const faceCard = getCardValue("J♥"); | ||
| expect(faceCard).toEqual(10); | ||
| }) | ||
| // Case 4: Handle Ace (A): | ||
| test("should return 11 for ace card", () => { | ||
| const aceCard = getCardValue("A♥"); | ||
| expect (aceCard).toEqual(11); | ||
| }) | ||
| // Case 5: Handle Invalid Cards: | ||
| test("should return invalid card rank for random numbers", () => { | ||
| const invalidRandomCard = getCardValue("85♠"); | ||
| expect(invalidRandomCard).toEqual("Invalid card rank."); | ||
| }); | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The function won't work properly if
numeratoranddenominatorare allowed to be negatives.For example -3 < 2 but -3/2 is not a proper fraction.