search

AY2122S2-CS2103T-T09-3 / tp

MIT License
0 stars 4 forks source link

Remove unused class #281

Closed ghost closed 2 years ago

ghost commented 2 years ago

Let me know if i miss out any more classes

codecov-commenter commented 2 years ago

Codecov Report

Merging #281 (1d764f9) into master (34924f3) will increase coverage by 0.07%. The diff coverage is n/a.

@@             Coverage Diff              @@
##             master     #281      +/-   ##
============================================
+ Coverage     60.51%   60.58%   +0.07%     
  Complexity      462      462              
============================================
  Files            89       88       -1     
  Lines          1712     1710       -2     
  Branches        194      194              
============================================
  Hits           1036     1036              
+ Misses          584      582       -2     
  Partials         92       92

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 34924f3...1d764f9. Read the comment docs.

ghost commented 2 years ago

Could you add back JsonUtilTest.java, ModelManagerTest.java, NameContainsKeywordPredicateTest.java and UserPrefTest.java? I think this files are test that we missed out when doing our test cases. Pretty sure this are test classes we missed out.

I added back the files.