-
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
Add tests to increase code coverage for 'model' components #132
Conversation
To achieve 100% test coverage for Amount
To achieve 100% test coverage for Date
To achieve 100% test coverage for TransactionList
Use more realistic transactions instead of persons
To achieve 100% line coverage for Transaction
To achieve 100% test coverage for FinanceTracker
To achieve 100% test coverage for UserPrefs
Codecov Report
@@ Coverage Diff @@
## master #132 +/- ##
============================================
+ Coverage 71.44% 72.99% +1.54%
- Complexity 541 556 +15
============================================
Files 90 90
Lines 1653 1655 +2
Branches 161 161
============================================
+ Hits 1181 1208 +27
+ Misses 421 397 -24
+ Partials 51 50 -1
Continue to review full report at Codecov.
|
src/test/data/JsonSerializableFinanceTrackerTest/typicalTransactionsFinanceTracker.json
Show resolved
Hide resolved
src/main/java/ay2021s1_cs2103_w16_3/finesse/model/util/SampleDataUtil.java
Show resolved
Hide resolved
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!
To achieve 100% test coverage for SampleDataUtil
0b9a58e
to
a277a8a
Compare
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!
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
Changes:
TypicalTransactions
to be transactions instead of persons.Related to #115.