Skip to content

feat: eventually with / CollecT - #90

Merged
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:fix/eventuallyWith
Apr 15, 2026
Merged

feat: eventually with / CollecT#90
fredbi merged 1 commit into
go-openapi:masterfrom
fredbi:fix/eventuallyWith

Conversation

@fredbi

@fredbi fredbi commented Apr 15, 2026

Copy link
Copy Markdown
Member

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.
* fix(condition): in async testing, allow condition goroutine to exit
  * allows condition to Goexit without exiting the whole assertion
* feat(condition): CollectT now distinguishes between FailNow() and Cancel()
  * FailNow() exits the condition, Cancel() interrupts the whole
    assertion
* regenerated code with examples and doc strings

* fixes go-openapi#90

Follow-up PRs:

1. Update documentation and code examples
2. Enhance / better report to user in migration tool

Signed-off-by: Frederic BIDON <fredbi@yahoo.com>
@fredbi
fredbi force-pushed the fix/eventuallyWith branch from b5ff5ba to e53eb5b Compare April 15, 2026 21:41
@codecov

codecov Bot commented Apr 15, 2026

Copy link
Copy Markdown

Codecov Report

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

Additional details and impacted files
@@            Coverage Diff             @@
##           master      #90      +/-   ##
==========================================
- Coverage   91.32%   91.28%   -0.05%     
==========================================
  Files          93       93              
  Lines       12041    12056      +15     
==========================================
+ Hits        10997    11005       +8     
- Misses        829      836       +7     
  Partials      215      215              

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

@fredbi
fredbi merged commit b635c75 into go-openapi:master Apr 15, 2026
23 checks passed
@fredbi
fredbi deleted the fix/eventuallyWith branch April 15, 2026 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant