Skip to content
This repository was archived by the owner on Nov 18, 2025. It is now read-only.

chore: fail faster in tests#1772

Merged
sofisl merged 11 commits intomainfrom
issue1769
Jun 18, 2025
Merged

chore: fail faster in tests#1772
sofisl merged 11 commits intomainfrom
issue1769

Conversation

@leahecole
Copy link
Contributor

@leahecole leahecole commented Jun 12, 2025

Fixes googleapis/google-cloud-node-core#177 - adds try/catch to tests using done to properly raise assertion errors quickly rather than having the test fail because it times out.

Separated browser tests into a separate issue

@product-auto-label product-auto-label bot added the size: l Pull request size is large. label Jun 12, 2025
@leahecole leahecole changed the title WIP: fail faster in tests chore: fail faster in tests Jun 12, 2025
@leahecole leahecole marked this pull request as ready for review June 12, 2025 20:53
@leahecole leahecole requested a review from a team June 12, 2025 20:53
@leahecole leahecole requested a review from a team as a code owner June 12, 2025 20:53
@leahecole leahecole added the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 13, 2025
@gcf-owl-bot gcf-owl-bot bot removed the owlbot:run Add this label to trigger the Owlbot post processor. label Jun 13, 2025
@sofisl sofisl merged commit e2b9410 into main Jun 18, 2025
28 checks passed
@sofisl sofisl deleted the issue1769 branch June 18, 2025 19:34
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

size: l Pull request size is large.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

cleanup: fail faster in tests using .done

3 participants