Skip to content

Align missing namespace test with stock ARM errors - #12940

Draft
willdavsmith wants to merge 1 commit into
radius-project:mainfrom
willdavsmith:willdavsmith-stock-error-assertion
Draft

Align missing namespace test with stock ARM errors#12940
willdavsmith wants to merge 1 commit into
radius-project:mainfrom
willdavsmith:willdavsmith-stock-error-assertion

Conversation

@willdavsmith

Copy link
Copy Markdown
Contributor

📊 Application Graph Diff

Comparing mainwilldavsmith-stock-error-assertion

✅ No application graph changes detected. The application model is identical between main and willdavsmith-stock-error-assertion.

graph TD
    classDef added fill:#dafbe1,stroke:#1a7f37,stroke-width:2px,color:#1a7f37
    classDef removed fill:#ffebe9,stroke:#cf222e,stroke-width:2px,color:#cf222e
    classDef modified fill:#fff8c5,stroke:#bf8700,stroke-width:2px,color:#9a6700
    classDef unchanged fill:#f6f8fa,stroke:#d1d9e0,stroke-width:1px,color:#656d76
    repo_radius_state_container["repo-radius-state-container\ncontainers"]:::unchanged
Loading

Summary

Update the modern UDT recipe pack missing-namespace functional test to expect the standard stock ARM deployment error hierarchy: DeploymentFailedResourceDeploymentFailureBadRequest.

This preserves the existing namespace-missing message assertion and aligns the test with the stock Azure.Deployments.Engine semantics introduced by azure-octo/deployment-engine#637. It does not change Deployment Engine behavior or deprecated Applications.* assertions.

Validation

  • gofmt -w test/functional-portable/dynamicrp/noncloud/resources/recipepacks_test.go
  • go test ./test/functional-portable/dynamicrp/noncloud/resources -run '^$'

Expect the stock ARM ResourceDeploymentFailure wrapper around the existing BadRequest detail.

Copilot-Session: 816c511c-1b15-47e4-b936-34e2f7bf2c1c
Signed-off-by: willdavsmith <willdavsmith@gmail.com>
@radius-functional-tests

radius-functional-tests Bot commented Sep 9, 2026

Copy link
Copy Markdown

Radius functional test overview

🔍 Go to test action run

Click here to see the test run details
Name Value
Repository willdavsmith/radius
Commit ref 6496918
Unique ID func0a8d28f735
Image tag pr-func0a8d28f735
  • Dapr: 1.14.4
  • Azure KeyVault CSI driver: 1.4.2
  • Azure Workload identity webhook: 1.3.0
  • Bicep recipe location ghcr.io/radius-project/dev/test/testrecipes/test-bicep-recipes/<name>:pr-func0a8d28f735
  • Terraform recipe location http://tf-module-server.radius-test-tf-module-server.svc.cluster.local/<name>.zip (in cluster)
  • applications-rp test image location: ghcr.io/radius-project/dev/applications-rp:pr-func0a8d28f735
  • dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-func0a8d28f735
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-func0a8d28f735
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func0a8d28f735
  • deployment-engine test image location: ghcr.io/radius-project/deployment-engine:latest

Test Status

⌛ Building Radius and pushing container images for functional tests...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting ucp-cloud functional tests...
⌛ Starting corerp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded

@codecov

codecov Bot commented Sep 9, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 59.82%. Comparing base (a0654b1) to head (6496918).

Additional details and impacted files
@@           Coverage Diff           @@
##             main   #12940   +/-   ##
=======================================
  Coverage   59.82%   59.82%           
=======================================
  Files         779      779           
  Lines       46058    46058           
=======================================
  Hits        27555    27555           
  Misses      18503    18503           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant