Skip to content

Fixed DocumentAI flakey tests#17470

Merged
melinath merged 2 commits into
GoogleCloudPlatform:mainfrom
melinath:fix-documentai-tests
May 8, 2026
Merged

Fixed DocumentAI flakey tests#17470
melinath merged 2 commits into
GoogleCloudPlatform:mainfrom
melinath:fix-documentai-tests

Conversation

@melinath
Copy link
Copy Markdown
Member

@melinath melinath commented May 7, 2026

Added a short sleep to test_check_destroy.

#17321 "broke" these tests by fixing the URL computation in test check destroy. I'm not able to reproduce the failure locally but I'm hopeful that adding a short sleep will help here (similar to #17465, which I was able to reproduce.)

Release Note Template for Downstream PRs (will be copied)

See Write release notes for guidance.


Added a short sleep to test_check_destroy
@melinath melinath requested a review from zli82016 May 7, 2026 22:37
@modular-magician
Copy link
Copy Markdown
Collaborator

modular-magician commented May 7, 2026

Hi there, I'm the Modular magician. I've detected the following information about your changes for commit 4357f0d:

Diff report

Your PR generated the following diffs in downstream repositories:

Repository Diff Link Changes
google provider View Diff 2 files changed, 8 insertions(+), 2 deletions(-)
google-beta provider View Diff 2 files changed, 8 insertions(+), 2 deletions(-)

Test report

Analytics

Total Tests Passed Skipped Affected
5 5 0 0
Affected Service Packages
  • documentai

Learn how VCR tests work


Step 1: Replaying Mode

🟢 All tests passed in Replaying mode! No Recording was needed.

View the build log

@melinath, @zli82016 VCR tests complete for 4357f0d!

@melinath melinath added this pull request to the merge queue May 8, 2026
Merged via the queue into GoogleCloudPlatform:main with commit 5b9f0d7 May 8, 2026
23 checks passed
@melinath melinath deleted the fix-documentai-tests branch May 8, 2026 00:10
inadenenko pushed a commit to inadenenko/magic-modules that referenced this pull request May 12, 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.

3 participants