Update test case to check for Forbidden instead of NotFound

embik · embik · commit 00bd786713a8 · 2025-03-18T11:11:07.000+01:00
On-behalf-of: SAP &lt;marvin.beckers@sap.com&gt;
Signed-off-by: Marvin Beckers &lt;marvin@kubermatic.com&gt;
diff --git a/test/e2e/virtual/apiexport/authorizer_test.go b/test/e2e/virtual/apiexport/authorizer_test.go
@@ -465,11 +465,11 @@ metadata:
 		if err == nil {
 			return false, "expected error, got none"
 		}
-		if apierrors.IsNotFound(err) {
+		if apierrors.IsForbidden(err) {
 			return true, ""
 		}
-		return false, fmt.Sprintf("expected a not-found error, but got %v", err)
-	}, wait.ForeverTestTimeout, 100*time.Millisecond, "expected service-provider-2-admin to get a not-found for shadowed cowboy resources")
+		return false, fmt.Sprintf("expected a forbidden error, but got %v", err)
+	}, wait.ForeverTestTimeout, 100*time.Millisecond, "expected service-provider-2-admin to get a forbidden for shadowed cowboy resources")
 }
 
 var scheme *runtime.Scheme

Original file line number	Diff line number	Diff line change
`@@ -465,11 +465,11 @@ metadata:`
`465`	`465`	`if err == nil {`
`466`	`466`	`return false, "expected error, got none"`
`467`	`467`	`}`
`468`		`- if apierrors.IsNotFound(err) {`
	`468`	`+ if apierrors.IsForbidden(err) {`
`469`	`469`	`return true, ""`
`470`	`470`	`}`
`471`		`- return false, fmt.Sprintf("expected a not-found error, but got %v", err)`
`472`		`- }, wait.ForeverTestTimeout, 100*time.Millisecond, "expected service-provider-2-admin to get a not-found for shadowed cowboy resources")`
	`471`	`+ return false, fmt.Sprintf("expected a forbidden error, but got %v", err)`
	`472`	`+ }, wait.ForeverTestTimeout, 100*time.Millisecond, "expected service-provider-2-admin to get a forbidden for shadowed cowboy resources")`
`473`	`473`	`}`
`474`	`474`
`475`	`475`	`var scheme *runtime.Scheme`