Repository navigation
London | 26 ITP SEP | Russom Gebremeskel | Sprint 2 | Implement and Rewrite Tests #1700
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
russom-g
wants to merge
68
commits into
CodeYourFuture:main
Choose a base branch
from
russom-g:coursework/sprint-2-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
68 commits
Select commit
Hold shift + click to select a range
301ea63
List edge cases for test.
2501561
Update list.
63e477b
Add an if condition to check midnight
a8e1b2d
Add another case to test midnight
9e2cef8
Add test case 00:01 to 12:01 am
2ba2e34
Add condition for checking midday 12 pm
06d5f91
Add test case for 01:00 to 01:00 am
3dc6da6
Add test case for 11:59 to 11:59 am
1e6cb55
Add test case for 12:01 to 12:01 pm
61b2ce7
Add test case for 13:00 to 1:00 pm
0407869
Add test case fo 23:59 to 11:59 pm
06d3f9a
Add padStart method to hours
598f744
Add more description.
e6fde3b
Add padStart method to convert 23:00 to 01:00pm
c2f743c
Edit comments
e7ff9c7
Inline code directly into return statement
751fb78
Change am and pm to lower case and update description.
4707f5a
Unreachable code.
f504687
Remove unreachable and redundant code.
d9197c4
Remove unused code
9494c3d
Update function to work for any input.
582699c
Format code with prettier.
b806b41
Stop tracking files
fcf27e7
Stop tracking files
04f544f
Restore files unrelated to this PR
4815e87
Add if condition for checking right angle.
9dc9e1f
Write test for Acute angle.
10b0d0f
Write test for Obtuse angle.
cccf002
Add if condition for checking Acute angle.
ed2729e
Add if condition for checking Obtuse angle.
4fa3e6c
Write test for invalid angles
ea5fa0f
Add final if condition for checking invalid angle
6102a89
Add boundary case test for acute angle.
717584d
Add if condition for checking invalid inputs.
fed8ddd
Add boundary test cases for multiple angles.
bb815ef
Add = operator for checking 360 degrees is included
c8fc9dd
Add boundary test cases.
3a565e9
Add angle type boundary tests
c80ed87
Add angle type boundary tests
38c28b3
Implement if condition to check propr function.
f12c635
Delete not needed test.
48b8dae
Write cases for testing proper and improper fractions.
d5091de
Update tests description.
fa07e40
Implement if condition to check card value
651191c
Check valid card ranks
russom-g 53d733c
Add test cases for single-digit card
russom-g 699f8c0
Add test cases for invalid card
russom-g 63b1d20
Add test case for Obtuse angle
russom-g c0bc7b7
Add test cases for Straight, Reflex, and Invalid angles.
russom-g eb68334
Write multiple test cases for proper and improper fraction
russom-g 3d93bc4
Add more test cases for Ace and face cards
russom-g b9732cb
Write test cases for cards number 2 -10
russom-g 66f6482
Correct spelling error
russom-g 7f0c6c8
Write test cases for invalid cards
russom-g 0a2de27
Format code with prettier.
russom-g 3e40397
Fix spelling error.
russom-g ad4fae0
Fix spelling error.
russom-g b8ba7fd
Format code with prettier.
russom-g 0c75ac6
Restore dead-code files to match main
russom-g fb40ba0
Remove = bug from reflex angle condition
russom-g 2b13cd7
Update reflex angle test max limit to 359
russom-g 3dad527
Update reflex angle test to 359
russom-g dc6fa1d
Update test for negative numbers
russom-g 72cf361
Add if condition for checking negative numbers.
russom-g 533ec51
Change test cases with negative numbers to return false.
russom-g c3fd5c6
Add an array of suits to check for valid suits.
russom-g 2d476b1
Add more test case for testing invalid card
russom-g 9cd8008
Add more test case for testing invalid cards.
russom-g 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
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
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.
I don't think this program is completely correct. Consider
card = "AA", would the result be the same as expected?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.
I added suits and validSuits variables to fix the bug.
Many thanks
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.
Much better now.