-
-
Notifications
You must be signed in to change notification settings - Fork 389
London | 26-ITP-May | Liridona Shehu | Sprint 3 | Implement and rewrite #1474
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
Open
shehu-dona
wants to merge
27
commits into
CodeYourFuture:main
Choose a base branch
from
shehu-dona:coursework/sprint-3-implement-and-rewrite
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
27 commits
Select commit
Hold shift + click to select a range
7fd7710
capitalize function, str declaration
dona-shehu 88d76b4
Declaration and scope
dona-shehu b6def45
Function parameters
dona-shehu d06f9a3
Add return a * b
dona-shehu c15a48e
Sum function fixed return
dona-shehu 66ddc97
Function with no parameter fixed
dona-shehu b82f302
Calculate BMI
dona-shehu 491605b
Capitalize function
dona-shehu b8a684c
Convert pence to pound function
dona-shehu 095f72d
Removed comments
dona-shehu 0a4c587
Add 2p example
dona-shehu 4aa639f
Time format pad()
dona-shehu 892bb92
Fixed variable declaration for bodyMassIndex
dona-shehu 4bc146e
implement, 1-get-angle-type.
dona-shehu 4ccf833
implement, 2-is-proper-fraction
dona-shehu 4b20e69
Implement, 3-get-card-value
dona-shehu 6f270d2
Add comments to tests
dona-shehu 0799ea5
Update error message
dona-shehu 956f635
Add tests in Jest syntax
dona-shehu 5cec9c4
Add test in Jest syntax
dona-shehu 1096cc2
Used array to validate rank & suit
dona-shehu 6caf04c
Add test in Jest syntax
dona-shehu fdd00ab
Revert accidental changes to Sprint-2 files
dona-shehu 31e40fa
Added default formatter, and more test cases.
dona-shehu 68ce473
Removed last unnecessary else.
dona-shehu bfe4374
Updated the comments.
dona-shehu 188448b
Removed unnecessary comments.
dona-shehu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -13,8 +13,43 @@ test(`should return "Acute angle" when (0 < angle < 90)`, () => { | |
| expect(getAngleType(89)).toEqual("Acute angle"); | ||
| }); | ||
|
|
||
|
|
||
| // Case 2: Right angle | ||
| test(`should return "Right angle" when ( angle = 90)`, () => { | ||
| // Test various right angles, including boundary cases | ||
| expect(getAngleType(90)).toEqual("Right angle"); | ||
|
|
||
| }); | ||
|
|
||
|
|
||
| // Case 3: Obtuse angles | ||
| test(`should return "Obtuse angle" when (90 < angle < 180)`, () => { | ||
| // Test various obtuse angles, including boundary cases | ||
| expect(getAngleType(91)).toEqual("Obtuse angle"); | ||
| expect(getAngleType(120)).toEqual("Obtuse angle"); | ||
| expect(getAngleType(179)).toEqual("Obtuse angle"); | ||
| }); | ||
|
|
||
|
|
||
| // Case 4: Straight angle | ||
| test(`should return "Straight angle" when ( angle = 180)`, () => { | ||
| // Test various straight angles, including boundary cases | ||
| expect(getAngleType(180)).toEqual("Straight angle"); | ||
| }); | ||
|
|
||
|
|
||
| // Case 5: Reflex angles | ||
| test(`should return "Reflex angle" when (180 < angle < 360)`, () => { | ||
| // Test various reflex angles, including boundary cases | ||
| expect(getAngleType(181)).toEqual("Reflex angle"); | ||
| expect(getAngleType(220)).toEqual("Reflex angle"); | ||
| expect(getAngleType(359)).toEqual("Reflex angle"); | ||
| }); | ||
|
|
||
|
|
||
| // Case 6: Invalid angles | ||
| test(`should return "Invalid angle" when (180 < angle < 360)`, () => { | ||
| // Test various invalid angles, including boundary cases | ||
| expect(getAngleType(0)).toEqual("Invalid angle"); | ||
| expect(getAngleType(360)).toEqual("Invalid angle"); | ||
|
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. You could add 2 test cases that are not boundary cases
Author
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. Added more test cases that are not boundary cases. |
||
| }); | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
Thats a nice way of checking if the input is valid. By expliciting stating all allowed values that code is easy to understand