🔧 Skip asset routing tests for parallel execution safety on multiple Terraform versions

Author: slntopp

.github/workflows/test.yml

@@ -65,6 +65,11 @@ jobs:
           - '1.11.*'
           - '1.12.*'
           - '1.13.*'
+        include:
+          # Org-scoped asset routing tests mutate shared state, so only run them
+          # on a single Terraform version to avoid parallel conflicts.
+          - terraform: '1.13.*'
+            run_asset_routing_tests: true
     steps:
       - uses: actions/checkout@de0fac2e4500dabe0009e67214ff5f5447ce83dd # v6.0.2
       - uses: actions/setup-go@4a3601121dd01d1626a1e23e37211e3254c1c06c # v6.4.0
@@ -79,6 +84,7 @@ jobs:
       - env:
           TF_ACC: "1"
           MONDOO_CONFIG_BASE64: ${{ secrets.MONDOO_CONFIG_BASE64 }}
+          RUN_ASSET_ROUTING_TESTS: ${{ matrix.run_asset_routing_tests }}
         run: go test -v -cover ./internal/provider/
         timeout-minutes: 10
 

internal/provider/asset_routing_rule_resource_test.go

@@ -5,12 +5,16 @@ package provider
 
 import (
 	"fmt"
+	"os"
 	"testing"
 
 	"github.com/hashicorp/terraform-plugin-testing/helper/resource"
 )
 
 func TestAccAssetRoutingRuleResource(t *testing.T) {
+	if os.Getenv("RUN_ASSET_ROUTING_TESTS") != "true" {
+		t.Skip("skipping: asset routing tests only run on a single TF version to avoid parallel conflicts")
+	}
 	orgID, err := getOrgId()
 	if err != nil {
 		t.Skip("skipping: no org-scoped service account available")
@@ -57,6 +61,9 @@ func TestAccAssetRoutingRuleResource(t *testing.T) {
 }
 
 func TestAccAssetRoutingRuleResourceWithLabel(t *testing.T) {
+	if os.Getenv("RUN_ASSET_ROUTING_TESTS") != "true" {
+		t.Skip("skipping: asset routing tests only run on a single TF version to avoid parallel conflicts")
+	}
 	orgID, err := getOrgId()
 	if err != nil {
 		t.Skip("skipping: no org-scoped service account available")

internal/provider/asset_routing_table_resource_test.go

@@ -5,12 +5,16 @@ package provider
 
 import (
 	"fmt"
+	"os"
 	"testing"
 
 	"github.com/hashicorp/terraform-plugin-testing/helper/resource"
 )
 
 func TestAccAssetRoutingTableResource(t *testing.T) {
+	if os.Getenv("RUN_ASSET_ROUTING_TESTS") != "true" {
+		t.Skip("skipping: asset routing tests only run on a single TF version to avoid parallel conflicts")
+	}
 	orgID, err := getOrgId()
 	if err != nil {
 		t.Skip("skipping: no org-scoped service account available")