AlexKMarshall / regMan

CRM to manage registrations to summer camps
MIT License
2 stars 0 forks source link

Instruments controller test #95

Closed maylynn-ng closed 4 years ago

maylynn-ng commented 4 years ago

catch error tests passing in isolation, issues with mock leaking - afterEach and clearAllMocks behaving strangely. Will come back to this issue.

codecov[bot] commented 4 years ago

Codecov Report

Merging #95 into master will increase coverage by 5.20%. The diff coverage is 94.11%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master      #95      +/-   ##
==========================================
+ Coverage   41.78%   46.99%   +5.20%     
==========================================
  Files          46       47       +1     
  Lines         615      632      +17     
  Branches       56       56              
==========================================
+ Hits          257      297      +40     
+ Misses        331      308      -23     
  Partials       27       27              
Impacted Files Coverage Δ
server/routes/public-router.js 0.00% <0.00%> (ø)
server/controllers/InstrumentControllerMockData.js 100.00% <100.00%> (ø)
server/controllers/InstrumentsController.js 80.00% <0.00%> (+80.00%) :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 125e004...2290cdf. Read the comment docs.