Skip to content

Respect [CancelAfter] budgets in the acceptance testing framework - #7940

Merged
danielmarbach merged 1 commit into
masterfrom
acceptance-testing-cancel-token
Sep 10, 2026
Merged

danielmarbach merged 1 commit into
masterfrom
acceptance-testing-cancel-token

Conversation

@danielmarbach

@danielmarbach danielmarbach commented Sep 10, 2026 •

Copy link
Copy Markdown
Contributor

@danielmarbach

Copy link
Copy Markdown
Contributor Author

@ramonsmits I rebased on master and did remove the TimeoutInfinite logic because it should no longer be necessary. I also did add another test verifying passing the token still works

@danielmarbach
danielmarbach force-pushed the acceptance-testing-cancel-token branch from 2afb3c9 to b75b168 Compare September 10, 2026 10:28
@danielmarbach
danielmarbach enabled auto-merge (squash) September 10, 2026 10:33
@danielmarbach
danielmarbach merged commit d2b5ed9 into master Sep 10, 2026
4 checks passed
@danielmarbach
danielmarbach deleted the acceptance-testing-cancel-token branch September 10, 2026 10:36
@danielmarbach danielmarbach changed the title Bound acceptance tests by the test's CancelAfter budget instead of the fixed 90 s Done limit Honor the test's cancellation token when running acceptance tests Sep 10, 2026
@danielmarbach danielmarbach changed the title Honor the test's cancellation token when running acceptance tests Respect [CancelAfter] budgets in the acceptance testing framework Sep 10, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants