Skip to content

test(bigquery): accept cancelled outcome in job cancel acceptance tests - #36657

Merged
quartzmo merged 1 commit into
googleapis:mainfrom
quartzmo:test-bigquery-cancel-race
Oct 23, 2026
Merged

quartzmo merged 1 commit into
googleapis:mainfrom
quartzmo:test-bigquery-cancel-race

Conversation

@quartzmo

Copy link
Copy Markdown
Member

The "creates and cancels jobs" and "creates, cancels and gets a job with location" acceptance tests cancel a load job and then assert that it didn't fail. That only holds if the job finishes before the cancel lands. BigQuery now honors the cancel, so the job ends with an error result whose reason is "stopped". This change accepts either outcome: a successful job, or one that was stopped by the cancellation. Any other error still fails the test.

Two acceptance tests cancel a load job and then assert it didn't fail,
which only holds if the job finishes before the cancel lands. BigQuery
now honors the cancel (errorResult reason "stopped"), so the tests fail
on main. Accept either a successful job or a cancellation.

@torreypayne torreypayne left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nice catch!

@quartzmo
quartzmo merged commit 97e8439 into googleapis:main Oct 23, 2026
19 checks passed
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