Skip to content

test: increase coverage thresholds on portfolio domain#1811

Merged
ItsANameToo merged 92 commits into
feat/mainsailfrom
test/domains-portfolio-coverage
May 29, 2026
Merged

test: increase coverage thresholds on portfolio domain#1811
ItsANameToo merged 92 commits into
feat/mainsailfrom
test/domains-portfolio-coverage

Conversation

@goga-m
Copy link
Copy Markdown
Contributor

@goga-m goga-m commented Apr 22, 2026

Summary

Closes https://app.clickup.com/t/86e0bj61v

Coverage threshold increase after merging the subsequent PRs

#1862
#1867
#1866
#1865
#1864
#1863

Checklist

  • My changes look good in both light AND dark mode
  • The change is not hardcoded to a single network, but has multi-asset in mind
  • I checked my changes for obvious issues, debug statements and commented code
  • Documentation (if necessary)
  • Tests (if necessary)
  • Ready to be merged

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 22, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
arkvault Ready Ready Preview May 29, 2026 11:40am

Request Review

Comment on lines -492 to +495
COVERAGE_THRESHOLD_LINES: 77.55
COVERAGE_THRESHOLD_FUNCTIONS: 73.98
COVERAGE_THRESHOLD_STATEMENTS: 77.69
COVERAGE_THRESHOLD_BRANCHES: 69.85
COVERAGE_THRESHOLD_LINES: 97.57
COVERAGE_THRESHOLD_FUNCTIONS: 94.9
COVERAGE_THRESHOLD_STATEMENTS: 97.46
COVERAGE_THRESHOLD_BRANCHES: 94.31
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There are new lines left uncovered after the recent dep updates on coverage libraries, but can cover them on a second batch as this PR got already big

@goga-m goga-m changed the title test: increase coverage on portfolio domain test: increase coverage thresholds on portfolio domain May 29, 2026
@goga-m goga-m marked this pull request as ready for review May 29, 2026 11:44
@ItsANameToo ItsANameToo merged commit 7d56f4a into feat/mainsail May 29, 2026
41 of 45 checks passed
@ItsANameToo ItsANameToo deleted the test/domains-portfolio-coverage branch May 29, 2026 11:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants