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

Implement per-service test setup #1985

Merged
merged 102 commits into from
Jan 15, 2025

Conversation

LarryOsterman
Copy link
Member

Add support for per-service/per-package test setup and cleanup scripts, and implement scripts for the azure_core_amqp package.

Note that this includes a gratuitous change to lib.rs to trigger the test script.

CoPilot Summary

This pull request introduces several updates to the testing scripts and setup for the azure_core_amqp package. The most important changes involve adding a working directory parameter, integrating setup and cleanup scripts for package tests, and specific setup for the azure_core_amqp package.

Updates to testing scripts:

  • eng/pipelines/templates/steps/test-packages.yml: Added -WorkingDirectory '$(Build.SourcesDirectory)' argument to the test packages step.

  • eng/scripts/Test-Packages.ps1:

    • Added workingDirectory parameter to the script.
    • Included logic to run Test-Setup.ps1 and Test-Cleanup.ps1 scripts if they exist. [1] [2]

Setup and Cleanup scripts:

  • sdk/core/Test-Setup.ps1:

    • Added script to set up the test environment for the azure_core_amqp package, including cloning the repository, building the test broker, and starting it.
  • sdk/core/Test-Cleanup.ps1:

    • Added script to clean up the test environment for the azure_core_amqp package, including stopping the test broker.

Minor code changes:

eng/scripts/Test-Packages.ps1 Outdated Show resolved Hide resolved
eng/scripts/Test-Packages.ps1 Outdated Show resolved Hide resolved
sdk/core/Test-Setup.ps1 Outdated Show resolved Hide resolved
sdk/core/Test-Setup.ps1 Outdated Show resolved Hide resolved
@LarryOsterman
Copy link
Member Author

/azp run rust - pr

Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@LarryOsterman LarryOsterman requested a review from heaths January 15, 2025 19:37
@LarryOsterman LarryOsterman requested a review from hallipr January 15, 2025 21:21
sdk/core/azure_core_amqp/README.md Outdated Show resolved Hide resolved
sdk/core/azure_core_amqp/README.md Outdated Show resolved Hide resolved
sdk/core/azure_core_amqp/README.md Outdated Show resolved Hide resolved
sdk/core/azure_core_amqp/README.md Outdated Show resolved Hide resolved
@LarryOsterman LarryOsterman merged commit 6d89f3a into Azure:main Jan 15, 2025
13 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.

3 participants