Skip to content

refactor test code#2300

Open
holyfuchs wants to merge 5 commits intomasterfrom
holyfuchs/refactor-test-code
Open

refactor test code#2300
holyfuchs wants to merge 5 commits intomasterfrom
holyfuchs/refactor-test-code

Conversation

@holyfuchs
Copy link
Member

Should be reviewed/merged after #2299

testCode was a ~300-line monolith mixing fork resolution, coverage setup, concurrency, and telemetry.
This refactor breaks it into single-responsibility helpers. No behaviour is changed.

@github-actions
Copy link

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

@codecov-commenter
Copy link

Codecov Report

❌ Patch coverage is 82.60870% with 36 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
internal/test/test.go 88.14% 16 Missing and 7 partials ⚠️
internal/settings/settings.go 0.00% 13 Missing ⚠️

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants