Updating App Config test resources - #48769
Conversation
|
Azure Pipelines: Successfully started running 1 pipeline(s). 9 pipeline(s) were filtered out due to trigger conditions. There may be pipelines that require an authorized user to comment /azp run to run. |
There was a problem hiding this comment.
Pull request overview
Updates App Configuration test resources to use RBAC-only authentication and tolerate role-assignment propagation delays.
Changes:
- Modernizes the ARM template and removes unused connection-string/key resources.
- Adds live-test readiness checks with bounded exponential backoff.
- Adds unit coverage for success, retry, error, and timeout paths.
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
sdk/appconfiguration/test-resources.json |
Updates resource provisioning and outputs. |
sdk/appconfiguration/azure-appconfiguration/tests/conftest.py |
Adds RBAC readiness fixture. |
sdk/appconfiguration/azure-appconfiguration/tests/test_readiness.py |
Tests readiness behavior. |
sdk/appconfiguration/azure-appconfiguration-provider/tests/conftest.py |
Orders setup after readiness and uses test credentials. |
sdk/appconfiguration/azure-appconfiguration-provider/tests/test_readiness.py |
Tests provider readiness behavior. |
💡 Add a code-review agent skill for context-aware, tailored reviews. Learn more in the docs.
|
One or more custom setup steps configured for this repository failed during this Copilot code review run: Setup steps run before each review. If the review above is missing context, or no review was posted at all, the failing step above may be the cause. See the workflow run for failure details, fix your setup steps configuration, and re-request a review. Note You can configure setup steps for Copilot code review separately from Copilot cloud agent with a |
Description
Updates the test structure of the python libraries.