Thanks to visit codestin.com
Credit goes to GitHub.com

Skip to content

test: add error tests for PureJSON and XML render types#4512

Open
aturzone wants to merge 1 commit intogin-gonic:masterfrom
aturzone:test/add-render-error-tests
Open

test: add error tests for PureJSON and XML render types#4512
aturzone wants to merge 1 commit intogin-gonic:masterfrom
aturzone:test/add-render-error-tests

Conversation

@aturzone
Copy link

@aturzone aturzone commented Feb 5, 2026

Add missing error handling tests for PureJSON and XML render types to improve test coverage. This addresses the TODO comment requesting error tests in the render package.

Pull Request Checklist

Please ensure your pull request meets the following requirements:

  • Open your pull request against the master branch.
  • All tests pass in available continuous integration systems (e.g., GitHub Actions).
  • Tests are added or modified as needed to cover code changes.
  • If the pull request introduces a new feature, the feature is documented in the docs/doc.md.

Thank you for contributing!

Add missing error handling tests for PureJSON and XML render types
to improve test coverage. This addresses the TODO comment requesting
error tests in the render package.

Co-Authored-By: Claude Opus 4.5 <[email protected]>
@codecov
Copy link

codecov bot commented Feb 5, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 99.01%. Comparing base (3dc1cd6) to head (10349b4).
⚠️ Report is 239 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #4512      +/-   ##
==========================================
- Coverage   99.21%   99.01%   -0.21%     
==========================================
  Files          42       46       +4     
  Lines        3182     3038     -144     
==========================================
- Hits         3157     3008     -149     
- Misses         17       21       +4     
- Partials        8        9       +1     
Flag Coverage Δ
?
--ldflags="-checklinkname=0" -tags sonic 99.00% <ø> (?)
-tags go_json 98.93% <ø> (?)
-tags nomsgpack 98.99% <ø> (?)
go-1.18 ?
go-1.19 ?
go-1.20 ?
go-1.21 ?
go-1.24 99.01% <ø> (?)
go-1.25 99.01% <ø> (?)
macos-latest 99.01% <ø> (-0.21%) ⬇️
ubuntu-latest 99.01% <ø> (-0.21%) ⬇️

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

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

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.

1 participant