-
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
GUI/invalid messages #272
GUI/invalid messages #272
Conversation
Codecov Report
@@ Coverage Diff @@
## master #272 +/- ##
============================================
- Coverage 44.16% 44.07% -0.09%
- Complexity 521 523 +2
============================================
Files 128 128
Lines 2690 2709 +19
Branches 279 282 +3
============================================
+ Hits 1188 1194 +6
- Misses 1439 1453 +14
+ Partials 63 62 -1
Continue to review full report at Codecov.
|
@@ -11,7 +13,7 @@ | |||
* Contains utility methods for populating {@code AddressBook} with sample data. | |||
*/ | |||
public class SampleDataUtil { | |||
public static Student[] getSamplePersons() { |
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.
Thanks for refactoring names!
/** | ||
* Sets the week and topic | ||
*/ | ||
private void setWeekAndTopic() { |
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.
Awesome way to calculate how many weeks it has been since the start of the semester 👍
@@ -284,7 +321,7 @@ private void handleSourceAcademy() { | |||
private void handleUserGuide() { | |||
try { | |||
Desktop.getDesktop() | |||
.browse(URI.create("https://ay2021s1-cs2103t-w11-2.github.io/tp/UserGuide.html#quick-start")); | |||
.browse(URI.create(USER_GUIDE_URL)); |
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.
nice abstraction
@@ -11,9 +16,37 @@ | |||
public static final Quest TEST_QUEST = new Quest(TEST_QUEST_TITLE, TEST_QUEST_DEADLINE); | |||
|
|||
public static final Quest TEST_QUEST_COPY = new Quest(TEST_QUEST_TITLE, TEST_QUEST_DEADLINE); | |||
public static final Quest RUNIC_CARPETS = new QuestBuilder().withTitle("Runic Carpets") |
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.
Thumbs up for creating tests!
Fixes #32
Fixes #241
Fixes #270