Skip to content

test: added tests for edge cases - #74

Merged
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:test/objective-coverage
Nov 16, 2025
Merged

test: added tests for edge cases#74
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:test/objective-coverage

Conversation

@fredbi

@fredbi fredbi commented Nov 16, 2025

Copy link
Copy Markdown
Member
  • fixed panic case when attempting to set a non-assignable value
  • clarified example and limitation regarding embedded structs

Change type

Please select: 🆕 New feature or enhancement|🔧 Bug fix'|📃 Documentation update

Short description

Fixes

Full description

Checklist

  • I have signed all my commits with my name and email (see DCO. This does not require a PGP-signed commit
  • I have rebased and squashed my work, so only one commit remains
  • I have added tests to cover my changes.
  • I have properly enriched go doc comments in code.
  • I have properly documented any breaking change.
@codecov

codecov Bot commented Nov 16, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 82.53%. Comparing base (3f0fe76) to head (3abc29e).
⚠️ Report is 1 commits behind head on master.
✅ All tests successful. No failed tests found.

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #74      +/-   ##
==========================================
+ Coverage   79.00%   82.53%   +3.53%     
==========================================
  Files           2        2              
  Lines         281      292      +11     
==========================================
+ Hits          222      241      +19     
+ Misses         40       35       -5     
+ Partials       19       16       -3     

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

* fixed panic case when attempting to set a non-assignable value
* clarified example and limitation regarding embedded structs

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@fredbi
fredbi force-pushed the test/objective-coverage branch from dc201c7 to 3abc29e Compare November 16, 2025 15:08
@fredbi
fredbi merged commit 61efbb7 into go-openapi:master Nov 16, 2025
19 checks passed
@fredbi
fredbi deleted the test/objective-coverage branch November 16, 2025 15:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant