Skip to content

test: poll the emulator to ensure it is ready before proceeding#747

Merged
olavloite merged 1 commit into
mainfrom
emulator-polling
Jun 11, 2026
Merged

test: poll the emulator to ensure it is ready before proceeding#747
olavloite merged 1 commit into
mainfrom
emulator-polling

Conversation

@olavloite

Copy link
Copy Markdown
Collaborator

Poll the Emulator to ensure it is actually running before proceeding with the test or sample.

@olavloite olavloite requested review from a team as code owners June 11, 2026 13:08

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request replaces static delays with a robust WaitUntilReady polling mechanism to verify that the Spanner emulator is fully started before returning. The reviewer feedback suggests removing an unused System.Net.Sockets import and avoiding modifying the global SPANNER_EMULATOR_HOST environment variable. Instead, specifying the host and port directly in the connection string is recommended to prevent potential race conditions during concurrent test execution.

Comment thread Google.Cloud.EntityFrameworkCore.Spanner.Samples/EmulatorRunner.cs Outdated
Comment thread Google.Cloud.EntityFrameworkCore.Spanner.Samples/EmulatorRunner.cs
Comment thread spanner-ado-net/spanner-ado-net-samples/EmulatorRunner.cs
@olavloite olavloite force-pushed the emulator-polling branch 3 times, most recently from 08ae93f to c586c19 Compare June 11, 2026 14:04
Poll the Emulator to ensure it is actually running before proceeding with the test
or sample.
@olavloite olavloite merged commit aa06873 into main Jun 11, 2026
23 checks passed
@olavloite olavloite deleted the emulator-polling branch June 11, 2026 16:27
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