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 DeleteTaskCommand test cases #81

Merged

Conversation

mingyi456
Copy link

No description provided.

…i456/tp into branch-Delete-Task-Command

* 'branch-Delete-Task-Command' of https://github.com/mingyi456/tp:
  Update DeleteCommandParser.java
  Fix test cases checkstyle for real
  Fix tests checkstyle
  Fix test cases
  Fix checkstyle again
  Fix checkstyle
  fix: Remove copy URL button
  fix: Trailing whitespace in HelpWindow.fxml
  add: User command list to help window
  update: Change help window URL
  update: Change help window to have a clickable hyperlink to open guide
  Add basic task list GUI
@codecov-commenter
Copy link

codecov-commenter commented Oct 24, 2021

Codecov Report

Merging #81 (c603e16) into master (f696bcd) will increase coverage by 1.05%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff              @@
##             master      #81      +/-   ##
============================================
+ Coverage     60.95%   62.00%   +1.04%     
- Complexity      504      513       +9     
============================================
  Files           100      100              
  Lines          2000     2000              
  Branches        197      197              
============================================
+ Hits           1219     1240      +21     
+ Misses          715      697      -18     
+ Partials         66       63       -3     
Impacted Files Coverage Δ
src/main/java/seedu/address/model/task/Task.java 78.57% <0.00%> (+3.57%) ⬆️
...rc/main/java/seedu/address/model/ModelManager.java 85.59% <0.00%> (+5.93%) ⬆️
.../seedu/address/model/task/filters/TaskFilters.java 75.00% <0.00%> (+10.00%) ⬆️
src/main/java/seedu/address/model/TaskList.java 77.77% <0.00%> (+11.11%) ⬆️
...address/logic/commands/task/DeleteTaskCommand.java 60.00% <0.00%> (+45.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 f696bcd...c603e16. Read the comment docs.

@mingyi456 mingyi456 added this to the v1.3 milestone Oct 24, 2021
@mingyi456 mingyi456 linked an issue Oct 24, 2021 that may be closed by this pull request
* 'master' of https://github.com/AY2122S1-CS2103-F09-2/tp:
  Minor tweaks to grammar and documentation detail
  Fix DeveloperGuide typos
  Edit DG to include `task delete`
  Remove trailing whitespaces, fix checkstyle
  Update DevGuide with better TaskFilter explanations
  Edit running sentence
  Fix grammar mistakes
  Add edit task DG
@wlren wlren merged commit edf0114 into AY2122S1-CS2103-F09-2:master Oct 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add test cases for Task Delete
4 participants