Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Cosmos] run cosmos emulator tests during PRs #2207

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

analogrelay
Copy link
Member

This PR adds some logic to the azure_data_cosmos CI pipeline to run tests against the Cosmos emulator. This is largely based on similar logic in the Go SDK.

To implement this, I added an AdditionalStages parameter to archetype-sdk-client.yml, again mirroring what I saw in the Go SDK, which places the steps after the default pack/test/analyzer steps but BEFORE any release steps.

We install the Windows emulator primarily because that's the logic that already exists in eng/common. There is a Linux emulator, that runs as a Docker image, but I wanted to be consistent here.

In addition, I tweaked recording::start to activate tracing even when test proxy wasn't in use. This allows us to use RUST_LOG to turn tracing on for live tests.

@github-actions github-actions bot added Azure.Core The azure_core crate Cosmos The azure_cosmos crate labels Feb 20, 2025
@analogrelay
Copy link
Member Author

I see now that the PR builds aren't quite structured the same. I'm looking in to an approach using the Test-Setup.ps1 file

@heaths
Copy link
Member

heaths commented Feb 20, 2025

I see now that the PR builds aren't quite structured the same. I'm looking in to an approach using the Test-Setup.ps1 file

Work with @hallipr and @LarryOsterman if needed. They devised that system.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Azure.Core The azure_core crate Cosmos The azure_cosmos crate
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants