AY2122S2-CS2103T-T17-1 / tp

About Tinner
https://ay2122s2-cs2103t-t17-1.github.io/tp/
MIT License
1 stars 5 forks source link

Fix same company name and different spacing #288

Closed BrendonLau closed 2 years ago

BrendonLau commented 2 years ago

Update now addCompany n/ test test = addCompany n/testtest previously, different number of spaces are deciding factor in determing is 2 companies are the same.

codecov-commenter commented 2 years ago

Codecov Report

Merging #288 (8ec724f) into master (a28ee01) will increase coverage by 0.01%. The diff coverage is 100.00%.

@@             Coverage Diff              @@
##             master     #288      +/-   ##
============================================
+ Coverage     68.13%   68.15%   +0.01%     
  Complexity      694      694              
============================================
  Files           107      107              
  Lines          2200     2201       +1     
  Branches        258      258              
============================================
+ Hits           1499     1500       +1     
  Misses          607      607              
  Partials         94       94              
Impacted Files Coverage Δ
...n/java/seedu/tinner/model/company/CompanyName.java 81.81% <100.00%> (+1.81%) :arrow_up:

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 a28ee01...8ec724f. Read the comment docs.