Skip to content
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 more tests #118

Merged
merged 10 commits into from
Oct 23, 2020
Merged

Conversation

yyutong
Copy link

@yyutong yyutong commented Oct 23, 2020

No description provided.

@codecov-io
Copy link

Codecov Report

Merging #118 into master will increase coverage by 8.90%.
The diff coverage is 100.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #118      +/-   ##
============================================
+ Coverage     28.82%   37.73%   +8.90%     
- Complexity      184      240      +56     
============================================
  Files            81       81              
  Lines          1332     1333       +1     
  Branches        117      117              
============================================
+ Hits            384      503     +119     
+ Misses          907      771     -136     
- Partials         41       59      +18     
Impacted Files Coverage Δ Complexity Δ
.../seedu/address/logic/parser/ExpenseBookParser.java 60.00% <100.00%> (+60.00%) 8.00 <0.00> (+8.00)
.../java/seedu/address/model/expense/Description.java 44.44% <0.00%> (ø) 2.00% <0.00%> (+1.00%)
...main/java/seedu/address/model/expense/Expense.java 32.07% <0.00%> (+3.77%) 10.00% <0.00%> (+1.00%)
.../java/seedu/address/model/ExpenseModelManager.java 30.76% <0.00%> (+11.53%) 7.00% <0.00%> (+3.00%)
...edu/address/logic/commands/DescriptionCommand.java 25.00% <0.00%> (+25.00%) 2.00% <0.00%> (+2.00%)
...eedu/address/logic/commands/AddExpenseCommand.java 50.00% <0.00%> (+50.00%) 6.00% <0.00%> (+6.00%)
...dress/logic/parser/DeleteExpenseCommandParser.java 50.00% <0.00%> (+50.00%) 2.00% <0.00%> (+2.00%)
...c/main/java/seedu/address/logic/parser/Prefix.java 58.33% <0.00%> (+58.33%) 5.00% <0.00%> (+5.00%)
.../seedu/address/logic/parser/ViewCommandParser.java 66.66% <0.00%> (+66.66%) 2.00% <0.00%> (+2.00%)
... and 9 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 7092568...ece5d78. Read the comment docs.

@yyutong yyutong self-assigned this Oct 23, 2020
@yyutong yyutong added this to the v1.3 milestone Oct 23, 2020
@yyutong yyutong added priority.Medium Nice to have type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new labels Oct 23, 2020
@yyutong yyutong merged commit 709482b into AY2021S1-CS2103T-W10-1:master Oct 23, 2020
@yyutong yyutong deleted the branch-sort-command branch October 23, 2020 07:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.Medium Nice to have type.Task Something that needs to be done, but not a story, bug, or an epic. e.g. Move testing code into a new
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants