ApplicationCredential Tests and Config Samples

On-behalf-of: SAP nils.gondermann@sap.com

Author: gndrmnn

config/samples/openstack_v1alpha1_applicationcredential.yaml

@@ -5,10 +5,18 @@ metadata:
   name: applicationcredential-sample
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: Sample ApplicationCredential
-    # TODO(scaffolding): Add all fields the resource supports
+    name: application-credential
+    userID: "00000000000000000000000000000000"
+    unrestricted: False
+    secret: "Secret"
+    roles:
+      - name: "role-name"
+    accessRules:
+      - method: "GET"
+        service: "monitoring"
+    expiresAt: "2022-02-02T22:22:22.000000"

internal/controllers/applicationcredential/tests/applicationcredential-create-full/00-assert.yaml

@@ -7,7 +7,15 @@ status:
   resource:
     name: applicationcredential-create-full-override
     description: ApplicationCredential from "create full" test
-    # TODO(scaffolding): Add all fields the resource supports
+    unrestricted: true
+    secret: "SSSSSSSSSSSSSSSSSSSSSSSS"
+    roles:
+      - name: "role-name"
+        domainID: null
+    accessRules:
+      - method: "GET"
+        service: "monitoring"
+    expiresAt: "2022-02-02T22:22:22.000000"
   conditions:
   - type: Available
     status: "True"
@@ -25,4 +33,8 @@ resourceRefs:
       ref: applicationcredential
 assertAll:
     - celExpr: "applicationcredential.status.id != ''"
-    # TODO(scaffolding): Add more checks
+    - celExpr: "applicationcredential.status.resource.secret != ''"
+    - celExpr: "applicationcredential.status.resource.roles[0].id != ''"
+    - celExpr: "applicationcredential.status.resource.accessRules[0].id != ''"
+    - celExpr: "applicationcredential.status.resource.accessRules[0].path != ''"
+    - celExpr: "has(applicationcredential.status.resource.links)"

internal/controllers/applicationcredential/tests/applicationcredential-create-full/00-create-resource.yaml

@@ -5,11 +5,18 @@ metadata:
   name: applicationcredential-create-full
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     name: applicationcredential-create-full-override
     description: ApplicationCredential from "create full" test
-    # TODO(scaffolding): Add all fields the resource supports
+    userID: "00000000000000000000000000000000"
+    unrestricted: true
+    secret: "SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS"
+    roles:
+      - name: "role-name"
+    accessRules:
+      - method: "GET"
+        service: "monitoring"
+    expiresAt: "2022-02-02T22:22:22.000000"

internal/controllers/applicationcredential/tests/applicationcredential-create-minimal/00-assert.yaml

@@ -5,8 +5,10 @@ metadata:
   name: applicationcredential-create-minimal
 status:
   resource:
+    # Name should default to object name
     name: applicationcredential-create-minimal
-    # TODO(scaffolding): Add all fields the resource supports
+    unrestricted: false
+    expiresAt: null
   conditions:
   - type: Available
     status: "True"
@@ -24,4 +26,6 @@ resourceRefs:
       ref: applicationcredential
 assertAll:
     - celExpr: "applicationcredential.status.id != ''"
-    # TODO(scaffolding): Add more checks
+    - celExpr: "applicationcredential.status.resource.secret != ''"
+    - celExpr: "applicationcredential.status.resource.projectID != ''"
+    - celExpr: "!has(applicationcredential.status.resource.description)"

internal/controllers/applicationcredential/tests/applicationcredential-create-minimal/00-create-resource.yaml

@@ -5,10 +5,9 @@ metadata:
   name: applicationcredential-create-minimal
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
-  # TODO(scaffolding): Only add the mandatory fields. It's possible the resource
-  # doesn't have mandatory fields, in that case, leave it empty.
-  resource: {}
+  resource:
+    UserID: "00000000000000000000000000000000"
+    Secret: "Secret"

internal/controllers/applicationcredential/tests/applicationcredential-import-error/00-create-resources.yaml

@@ -5,24 +5,22 @@ metadata:
   name: applicationcredential-import-error-external-1
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: ApplicationCredential from "import error" test
-    # TODO(scaffolding): add any required field
+    userID: "00000000000000000000000000000000"
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: ApplicationCredential
 metadata:
   name: applicationcredential-import-error-external-2
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: ApplicationCredential from "import error" test
-    # TODO(scaffolding): add any required field
+    userID: "00000000000000000000000000000000"

internal/controllers/applicationcredential/tests/applicationcredential-import/00-import-resource.yaml

@@ -12,4 +12,4 @@ spec:
     filter:
       name: applicationcredential-import-external
       description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
-      # TODO(scaffolding): Add all fields supported by the filter
+      userID: "00000000000000000000000000000000"

internal/controllers/applicationcredential/tests/applicationcredential-import/01-assert.yaml

@@ -16,7 +16,7 @@ status:
   resource:
     name: applicationcredential-import-external-not-this-one
     description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
-    # TODO(scaffolding): Add fields necessary to match filter
+    userID: "00000000000000000000000000000000"
 ---
 apiVersion: openstack.k-orc.cloud/v1alpha1
 kind: ApplicationCredential

internal/controllers/applicationcredential/tests/applicationcredential-import/01-create-trap-resource.yaml

@@ -8,10 +8,9 @@ metadata:
   name: applicationcredential-import-external-not-this-one
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
-    # TODO(scaffolding): Add fields necessary to match filter
+    userID: "00000000000000000000000000000000"

internal/controllers/applicationcredential/tests/applicationcredential-import/02-create-resource.yaml

@@ -5,10 +5,9 @@ metadata:
   name: applicationcredential-import-external
 spec:
   cloudCredentialsRef:
-    # TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
     cloudName: openstack
     secretName: openstack-clouds
   managementPolicy: managed
   resource:
     description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
-    # TODO(scaffolding): Add fields necessary to match filter
+    userID: "00000000000000000000000000000000"