search

flatironinstitute / mcmc-monitor

Monitor MCMC runs in the browser
Other
35 stars 0 forks source link

Test spa branch #92

Open jsoules opened 1 year ago

jsoules commented 1 year ago

Apparently I never actually hit the PR button on this--oops!

I think it's pretty straightforward--just adding some tests for the rest of the changes from the prior PR #85.

codecov-commenter commented 1 year ago

Codecov Report

Merging #92 (a03bf55) into main (ef52102) will increase coverage by 2.39%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main      #92      +/-   ##
==========================================
+ Coverage   16.15%   18.55%   +2.39%     
==========================================
  Files          80       80              
  Lines        5854     5854              
  Branches      138      161      +23     
==========================================
+ Hits          946     1086     +140     
+ Misses       4908     4768     -140
Flag Coverage Δ
gui_units 23.90% <100.00%> (+3.17%) :arrow_up:
svc_units 2.27% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
src/MCMCMonitorDataManager/updateSequences.ts 100.00% <100.00%> (+19.04%) :arrow_up:
src/spaInterface/spaOutputsForRunIds.ts 100.00% <100.00%> (+66.12%) :arrow_up:
src/spaInterface/util.ts 100.00% <100.00%> (+63.15%) :arrow_up:

... and 6 files with indirect coverage changes

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more