-
Notifications
You must be signed in to change notification settings - Fork 5
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
Add birthday attribute and tests #44
Conversation
-implemented BirthdayTest
fail test coverage, add more tests or uwuhan |
…3T-F11-2/main into BirthdayAttribute
Improved coverage. |
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.
LGTM
// valid birthday | ||
assertTrue(Birthday.isValidBirthday("01-01-1980")); // birthday in 01-01-1980 | ||
assertTrue(Birthday.isValidBirthday("01-01-1970")); //birthday in 01-01-1979 | ||
|
||
assertTrue(new Birthday("01-01-1980").hashCode() == new Birthday("01-01-1980").hashCode()); | ||
assertTrue(new Birthday("01-01-1980").toString().equals("01-01-1980") == true); |
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.
abstract awayy
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.
LGTM
-implemented Birthday and BirthdayTest
A valid birthday satisfies the following:
-in the format "DD-MM-YYYY"
-input date is earlier than the current date
BirthdayTest includes a hard-coded test case for a birthday on 1st January 1980.