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

Undo feature tests #222

Merged

Conversation

hanstirtaputra
Copy link

closes #220

@hanstirtaputra hanstirtaputra added priority.High Must do type:Testing Any improvement to test coverage of the project. labels Oct 27, 2020
@hanstirtaputra hanstirtaputra added this to the v1.4 milestone Oct 27, 2020
@hanstirtaputra hanstirtaputra self-assigned this Oct 27, 2020
@codecov-io
Copy link

Codecov Report

Merging #222 into master will increase coverage by 0.36%.
The diff coverage is 0.00%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master     #222      +/-   ##
============================================
+ Coverage     54.74%   55.10%   +0.36%     
- Complexity      977      986       +9     
============================================
  Files           204      204              
  Lines          3860     3860              
  Branches        448      448              
============================================
+ Hits           2113     2127      +14     
+ Misses         1622     1606      -16     
- Partials        125      127       +2     
Impacted Files Coverage Δ Complexity Δ
...java/seedu/homerce/logic/commands/UndoCommand.java 14.28% <0.00%> (+14.28%) 2.00 <0.00> (+2.00)
...va/seedu/homerce/model/manager/HistoryManager.java 92.85% <ø> (+7.14%) 9.00 <0.00> (+1.00)
...e/logic/commands/service/DeleteServiceCommand.java 13.04% <0.00%> (+13.04%) 2.00% <0.00%> (+2.00%)
...rc/main/java/seedu/homerce/model/undo/History.java 100.00% <0.00%> (+33.33%) 3.00% <0.00%> (+2.00%)
...erce/logic/commands/service/AddServiceCommand.java 45.45% <0.00%> (+45.45%) 2.00% <0.00%> (+2.00%)

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 779b7ed...bd06148. Read the comment docs.

@RuiFengg RuiFengg merged commit 666fd40 into AY2021S1-CS2103T-W13-3:master Oct 31, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
priority.High Must do type:Testing Any improvement to test coverage of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add tests for undo feature
3 participants