Skip to content
This repository has been archived by the owner on Jun 19, 2024. It is now read-only.

chore: coverage #174

Draft
wants to merge 3 commits into
base: main
Choose a base branch
from
Draft

Conversation

PhearZero
Copy link

@PhearZero PhearZero commented Aug 10, 2023

Description

Increase coverage in codebase

Checklist

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

  • Code compiles correctly

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

  • All tests passing

  • Extended the README / documentation, if necessary

  • closes Increase coverage by 80% #99

@codecov
Copy link

codecov bot commented Aug 10, 2023

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

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

@PhearZero
Copy link
Author

@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 PhearZero changed the title chore: Coverage chore: coverage Aug 11, 2023
@aorumbayev
Copy link
Contributor

@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 ;)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase coverage by 80%
2 participants