Report functional test pass only when all subtests finish

[willdavsmith]

Description

• Change functional test logic to only report complete when all subtests (shared, samples, msgrp, etc.) are complete.

Type of change

• This pull request fixes a bug in Radius and has an approved issue (issue link required).

Fixes: #6939

[radius-functional-tests]

Radius functional test overview

🔍 Go to test action run

Name	Value
Repository	willdavsmith/radius
Commit ref	6b95370
Unique ID	87477f9a20
Image tag	pr-87477f9a20

Click here to see the list of tools in the current test run

• gotestsum 1.10.0
• KinD: v0.20.0
• Dapr: 1.12.0
• Azure KeyVault CSI driver: 1.4.2
• Azure Workload identity webhook: 1.1.0
• Bicep recipe location ghcr.io/radius-project/dev/test/functional/shared/recipes/<name>:pr-87477f9a20
• 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-87477f9a20
• controller test image location: ghcr.io/radius-project/dev/controller:pr-87477f9a20
• ucp test image location: ghcr.io/radius-project/dev/ucpd:pr-87477f9a20
• 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 shared functional tests...
⌛ Starting msgrp functional tests...
⌛ Starting daprrp functional tests...
⌛ Starting datastoresrp functional tests...
⌛ Starting samples functional tests...
⌛ Starting kubernetes functional tests...
✅ msgrp functional tests succeeded
✅ samples functional tests succeeded
✅ kubernetes functional tests succeeded
✅ ucp functional tests succeeded
✅ daprrp functional tests succeeded
✅ datastoresrp functional tests succeeded
✅ shared functional tests succeeded

[rynowak]

Maybe a dumb question about this about this ... does this work?

Since tests is a matrix are we going to end up with the same issue because each entry in the matrix will set test_status?

Would using needs.tests.status do the same thing? https://github.com/orgs/community/discussions/26822#discussioncomment-3253537