Skip to content

Migrate application graph test to new Radius.Core/* types#12436

Open
lakshmimsft wants to merge 1 commit into
mainfrom
feat/migrate-applicationgraph-test
Open

Migrate application graph test to new Radius.Core/* types#12436
lakshmimsft wants to merge 1 commit into
mainfrom
feat/migrate-applicationgraph-test

Conversation

@lakshmimsft

Copy link
Copy Markdown
Contributor

Summary

Migrates the application-graph functional test from  Applications.Core/*  to the new resource types and the  2025-08-01-preview  API.

Reason for change

graph api for new types updated after previous round of migration of tests.

File change summary

Bicep ( corerp-resources-application-graph.bicep ):  Radius.Core/applications  + two  Radius.Compute/containers , each with an  environment  reference and the new plural  containers / readinessProbe  schema. The front→back connection now uses a resource-ID source ( backendContainer.id ) instead of a URL, matching the new connection model.

• Test ( application_test.go ): uses the preview-env  PreSetup , the new  validation  resource-type constants ( CoreApplicationsResource ,  ComputeContainersResource ), and the  v20250801preview   GetGraph  client.

• Assertions are now dynamic and portable: validates resource types, the  application / environment  references, and the front↔back connection edge in both directions.

@lakshmimsft
lakshmimsft requested a review from nithyatsu July 14, 2026 22:49
@github-actions

Copy link
Copy Markdown

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

None

Signed-off-by: lakshmimsft <ljavadekar@microsoft.com>
@lakshmimsft
lakshmimsft force-pushed the feat/migrate-applicationgraph-test branch from 6a54987 to 457f8b1 Compare July 14, 2026 22:54
@lakshmimsft
lakshmimsft marked this pull request as ready for review July 14, 2026 22:54
@lakshmimsft
lakshmimsft requested review from a team as code owners July 14, 2026 22:54
Copilot AI review requested due to automatic review settings July 14, 2026 22:54
@radius-functional-tests

radius-functional-tests Bot commented Jul 14, 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 radius-project/radius
Commit ref 457f8b1
Unique ID func39b94ded10
Image tag pr-func39b94ded10
  • 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-func39b94ded10
  • 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-func39b94ded10
  • dynamic-rp test image location: ghcr.io/radius-project/dev/dynamic-rp:pr-func39b94ded10
  • controller test image location: ghcr.io/radius-project/dev/controller:pr-func39b94ded10
  • ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-func39b94ded10
  • 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...
✅ Container images build succeeded
⌛ Publishing Bicep Recipes for functional tests...
✅ Recipe publishing succeeded
⌛ Starting corerp-cloud functional tests...
⌛ Starting ucp-cloud functional tests...
✅ ucp-cloud functional tests succeeded
✅ ucp-cloud functional tests succeeded
✅ corerp-cloud functional tests succeeded
❌ corerp-cloud functional test failed. Please check the logs for more details

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

This pull request migrates the Core RP “application graph” functional test from legacy Applications.Core/* resources to the new Radius.Core/* / Radius.Compute/* resource types using the 2025-08-01-preview API, aligning the test suite with the updated graph model for the new providers.

Changes:

  • Updated the test Bicep template to deploy Radius.Core/applications plus two Radius.Compute/containers, including the new containers object schema and readinessProbe.httpGet, and switched the front→back connection source to a resource ID (backendContainer.id).
  • Reworked the Go test to use preview environment pre-setup, new validation resource-type constants, and the v20250801preview graph client, replacing the static JSON fixture with portable/dynamic assertions.
  • Removed the obsolete graph output fixture JSON now that assertions are derived dynamically from the graph response.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.

File Description
test/functional-portable/corerp/noncloud/resources/testdata/corerp-resources-application-graph.bicep Migrates the deployed resources to Radius.Core/* + Radius.Compute/* and updates container schema/connection source to match the new graph model.
test/functional-portable/corerp/noncloud/resources/testdata/corerp-resources-application-graph-out.json Removes the legacy static graph fixture no longer used by the test.
test/functional-portable/corerp/noncloud/resources/application_test.go Updates the test to use preview env setup and the v20250801preview graph client with dynamic, cluster-portable assertions.

@github-actions

Copy link
Copy Markdown

Unit Tests

    2 files  ±0    453 suites  ±0   7m 27s ⏱️ +4s
5 816 tests ±0  5 814 ✅ ±0  2 💤 ±0  0 ❌ ±0 
7 019 runs  ±0  7 017 ✅ ±0  2 💤 ±0  0 ❌ ±0 

Results for commit 457f8b1. ± Comparison against base commit a6941d7.

@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 53.33%. Comparing base (a6941d7) to head (457f8b1).

Additional details and impacted files
@@            Coverage Diff             @@
##             main   #12436      +/-   ##
==========================================
- Coverage   53.34%   53.33%   -0.02%     
==========================================
  Files         756      756              
  Lines       49040    49040              
==========================================
- Hits        26162    26154       -8     
- Misses      20451    20455       +4     
- Partials     2427     2431       +4     

☔ 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants