-
Notifications
You must be signed in to change notification settings - Fork 6
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
Added integration test for print and add command, updated ug #156
Added integration test for print and add command, updated ug #156
Conversation
Codecov Report
@@ Coverage Diff @@
## master #156 +/- ##
============================================
+ Coverage 32.70% 34.49% +1.79%
- Complexity 355 378 +23
============================================
Files 103 103
Lines 2382 2380 -2
Branches 301 302 +1
============================================
+ Hits 779 821 +42
+ Misses 1537 1487 -50
- Partials 66 72 +6
Continue to review full report at Codecov.
|
docs/UserGuide.md
Outdated
### Saving data | ||
Data (all stocks in inventory in json) is automatically saved to | ||
`[root directory]/data/stockbook.json` when any of these commands is executed: | ||
* add | ||
* delete | ||
* note | ||
* notedelete | ||
* update | ||
* note |
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.
One of the note can be removed.
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, however do clear the extra "note" mentioned by @amylzting
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.
Overall LGTM! Do remove the note
commented by @amylzting.
// @Test | ||
// public void execute_duplicateStock_throwsCommandException() { | ||
// dog test case that causes some test to fail 1/2 the times??? what kind of sorcery is this? | ||
// Stock stockInList = model.getStockBook().getStockList().get(0); | ||
// assertCommandFailure(new AddCommand(stockInList), model, AddCommand.MESSAGE_DUPLICATE_STOCK); | ||
// } |
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.
Maybe consider removing this first so the code will be cleaner. Can always refer to this later by viewing GitHub commit history
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.
Alright will remove it!
No description provided.