Merge pull request #2099 from yuwenma/stricter-check-projectnumber

fix: replace mockresourcemanager generated Project number
GoogleCloudPlatform · Jun 22, 2024 · 93d018f · 93d018f
2 parents 404ed0a + c092454
commit 93d018f
Show file tree

Hide file tree

Showing 4 changed files with 11 additions and 3 deletions.
diff --git a/...urcemanager/v1beta1/project/projectinfolder/_generated_object_projectinfolder.golden.yaml b/...urcemanager/v1beta1/project/projectinfolder/_generated_object_projectinfolder.golden.yaml
@@ -25,5 +25,5 @@ status:
     reason: UpToDate
     status: "True"
     type: Ready
-  number: "2727742225"
+  number: ${projectNumber}
   observedGeneration: 3
diff --git a/...c/resourcemanager/v1beta1/project/projectinorg/_generated_object_projectinorg.golden.yaml b/...c/resourcemanager/v1beta1/project/projectinorg/_generated_object_projectinorg.golden.yaml
@@ -25,5 +25,5 @@ status:
     reason: UpToDate
     status: "True"
     type: Ready
-  number: "2713651918"
+  number: ${projectNumber}
   observedGeneration: 3
diff --git a/...oject/projectmovedfoldertofolder/_generated_object_projectmovedfoldertofolder.golden.yaml b/...oject/projectmovedfoldertofolder/_generated_object_projectmovedfoldertofolder.golden.yaml
@@ -24,5 +24,5 @@ status:
     reason: UpToDate
     status: "True"
     type: Ready
-  number: "2589592254"
+  number: ${projectNumber}
   observedGeneration: 3
diff --git a/tests/e2e/normalize.go b/tests/e2e/normalize.go
@@ -102,12 +102,20 @@ func normalizeKRMObject(u *unstructured.Unstructured, project testgcp.GCPProject
 	visitor.sortSlices.Insert(".spec.access")
 	visitor.sortSlices.Insert(".spec.nodeConfig.oauthScopes")
 
+	if u.GetKind() == "Project" {
+		// For some tests that talk to the Mock Resource Manager, the Project object's ProjectID and ProjectNumber are dynamcially generated.
+		// We do not want to overrride this with the default mocked Project "mock-project".
+		visitor.replacePaths[".status.number"] = "${projectNumber}"
+	}
+
 	visitor.stringTransforms = append(visitor.stringTransforms, func(path string, s string) string {
 		return strings.ReplaceAll(s, project.ProjectID, "${projectId}")
 	})
+
 	visitor.stringTransforms = append(visitor.stringTransforms, func(path string, s string) string {
 		return strings.ReplaceAll(s, fmt.Sprintf("%d", project.ProjectNumber), "${projectNumber}")
 	})
+
 	visitor.stringTransforms = append(visitor.stringTransforms, func(path string, s string) string {
 		return strings.ReplaceAll(s, uniqueID, "${uniqueId}")
 	})