algoworld-swapper icon indicating copy to clipboard operation
algoworld-swapper copied to clipboard

chore: coverage

Open PhearZero opened this issue 2 years ago • 3 comments

Description

Increase coverage in codebase

Checklist

Please, make sure to comply with the checklist below before expecting review

  • [x] Code compiles correctly

  • [x] Created tests which fail without the change (if possible)

  • [x] All tests passing

  • [x] Extended the README / documentation, if necessary

  • closes #99

PhearZero avatar Aug 10 '23 01:08 PhearZero

Codecov Report

Merging #174 (95b0d97) into main (c19486d) will increase coverage by 1.37%. The diff coverage is 100.00%.

@@            Coverage Diff             @@
##             main     #174      +/-   ##
==========================================
+ Coverage   54.63%   56.01%   +1.37%     
==========================================
  Files          84       84              
  Lines        2028     2037       +9     
  Branches      188      189       +1     
==========================================
+ Hits         1108     1141      +33     
+ Misses        905      881      -24     
  Partials       15       15              
Files Changed Coverage Δ
src/utils/api/swaps/__utils__/testUtils.ts 100.00% <100.00%> (ø)

... and 2 files with indirect coverage changes

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

codecov[bot] avatar Aug 10 '23 01:08 codecov[bot]

@aorumbayev going to put more effort into coverage, just getting comfortable with requirements. Feel free to ping me on projects and I'll try my best to help out off-hours. 🍻

PhearZero avatar Aug 10 '23 01:08 PhearZero

@PhearZero missed this one completely thanks a lot! Will need to patch up unit tests, latest algosdk throws errors since i used fetch in node environment. Afterwards will merge ;)

aorumbayev avatar Dec 31 '23 15:12 aorumbayev