-
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 tests for quests, topics and greeting #303
Conversation
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.
Great additions, thank you! LGTM
@@ -18,6 +20,8 @@ public Topic() { | |||
* @param outline outline of Topic | |||
*/ | |||
public Topic(String week, String outline) { | |||
requireNonNull(week); |
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.
Great protective measure!
} | ||
|
||
@Test | ||
void getGreeting() { |
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.
Perhaps this test method signature could be setGreeting_validString_success()?
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.
👍
private final List<Quest> questListTest = new ArrayList<>(); | ||
|
||
@Test | ||
void add_nullQuest_throwsNullPointerException() { |
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.
This type of checking is indeed really important, thank you for writing these tests cases!
Fixes #168