Skip to content

Commit 8fdf485

Browse files
committed
ApplicationCredential Tests and Config Samples
On-behalf-of: SAP [email protected]
1 parent 1f3f9b9 commit 8fdf485

19 files changed

Lines changed: 48 additions & 145 deletions

File tree

config/samples/openstack_v1alpha1_applicationcredential.yaml

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,18 @@ metadata:
55
name: applicationcredential-sample
66
spec:
77
cloudCredentialsRef:
8-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
98
cloudName: openstack
109
secretName: openstack-clouds
1110
managementPolicy: managed
1211
resource:
1312
description: Sample ApplicationCredential
14-
# TODO(scaffolding): Add all fields the resource supports
13+
name: application-credential
14+
userID: "00000000000000000000000000000000"
15+
unrestricted: False
16+
secret: "Secret"
17+
roles:
18+
- name: "role-name"
19+
accessRules:
20+
- method: "GET"
21+
service: "monitoring"
22+
expiresAt: "2022-02-02T22:22:22.000000"

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

Lines changed: 11 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,14 @@ status:
77
resource:
88
name: applicationcredential-create-full-override
99
description: ApplicationCredential from "create full" test
10-
# TODO(scaffolding): Add all fields the resource supports
10+
unrestricted: true
11+
roles:
12+
- name: "reader"
13+
accessRules:
14+
- method: "GET"
15+
service: "compute"
16+
path: "/v2.1/servers"
17+
expiresAt: "2033-03-03T22:22:22Z"
1118
conditions:
1219
- type: Available
1320
status: "True"
@@ -25,4 +32,6 @@ resourceRefs:
2532
ref: applicationcredential
2633
assertAll:
2734
- celExpr: "applicationcredential.status.id != ''"
28-
# TODO(scaffolding): Add more checks
35+
- celExpr: "applicationcredential.status.resource.roles[0].id != ''"
36+
- celExpr: "applicationcredential.status.resource.accessRules[0].id != ''"
37+
- celExpr: "has(applicationcredential.status.resource.links)"

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

Lines changed: 10 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,11 +5,19 @@ metadata:
55
name: applicationcredential-create-full
66
spec:
77
cloudCredentialsRef:
8-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
98
cloudName: openstack
109
secretName: openstack-clouds
1110
managementPolicy: managed
1211
resource:
1312
name: applicationcredential-create-full-override
1413
description: ApplicationCredential from "create full" test
15-
# TODO(scaffolding): Add all fields the resource supports
14+
userID: "token"
15+
unrestricted: true
16+
secret: "SSSSSSSSSSSSSSSSSSSSSSSSSSSSSSSS"
17+
roles:
18+
- name: "reader"
19+
accessRules:
20+
- method: "GET"
21+
service: "compute"
22+
path: "/v2.1/servers"
23+
expiresAt: "2033-03-03T22:22:22Z"

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

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,9 @@ metadata:
55
name: applicationcredential-create-minimal
66
status:
77
resource:
8+
# Name should default to object name
89
name: applicationcredential-create-minimal
9-
# TODO(scaffolding): Add all fields the resource supports
10+
unrestricted: false
1011
conditions:
1112
- type: Available
1213
status: "True"
@@ -24,4 +25,5 @@ resourceRefs:
2425
ref: applicationcredential
2526
assertAll:
2627
- celExpr: "applicationcredential.status.id != ''"
27-
# TODO(scaffolding): Add more checks
28+
- celExpr: "applicationcredential.status.resource.projectID != ''"
29+
- celExpr: "!has(applicationcredential.status.resource.description)"

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

Lines changed: 3 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,10 +5,9 @@ metadata:
55
name: applicationcredential-create-minimal
66
spec:
77
cloudCredentialsRef:
8-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
98
cloudName: openstack
109
secretName: openstack-clouds
1110
managementPolicy: managed
12-
# TODO(scaffolding): Only add the mandatory fields. It's possible the resource
13-
# doesn't have mandatory fields, in that case, leave it empty.
14-
resource: {}
11+
resource:
12+
userID: "token"
13+
secret: "Secret"

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

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -5,24 +5,24 @@ metadata:
55
name: applicationcredential-import-error-external-1
66
spec:
77
cloudCredentialsRef:
8-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
98
cloudName: openstack
109
secretName: openstack-clouds
1110
managementPolicy: managed
1211
resource:
1312
description: ApplicationCredential from "import error" test
14-
# TODO(scaffolding): add any required field
13+
userID: "token"
14+
secret: "abc"
1515
---
1616
apiVersion: openstack.k-orc.cloud/v1alpha1
1717
kind: ApplicationCredential
1818
metadata:
1919
name: applicationcredential-import-error-external-2
2020
spec:
2121
cloudCredentialsRef:
22-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
2322
cloudName: openstack
2423
secretName: openstack-clouds
2524
managementPolicy: managed
2625
resource:
2726
description: ApplicationCredential from "import error" test
28-
# TODO(scaffolding): add any required field
27+
userID: "token"
28+
secret: "def"

internal/controllers/applicationcredential/tests/applicationcredential-import-error/01-import-resource.yaml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -11,3 +11,4 @@ spec:
1111
import:
1212
filter:
1313
description: ApplicationCredential from "import error" test
14+
userID: "token"

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,4 @@ spec:
1212
filter:
1313
name: applicationcredential-import-external
1414
description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
15-
# TODO(scaffolding): Add all fields supported by the filter
15+
userID: "token"

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

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,6 @@ status:
1616
resource:
1717
name: applicationcredential-import-external-not-this-one
1818
description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
19-
# TODO(scaffolding): Add fields necessary to match filter
2019
---
2120
apiVersion: openstack.k-orc.cloud/v1alpha1
2221
kind: ApplicationCredential

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

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -8,10 +8,10 @@ metadata:
88
name: applicationcredential-import-external-not-this-one
99
spec:
1010
cloudCredentialsRef:
11-
# TODO(scaffolding): Use openstack-admin if the resource needs admin credentials to be created
1211
cloudName: openstack
1312
secretName: openstack-clouds
1413
managementPolicy: managed
1514
resource:
1615
description: ApplicationCredential applicationcredential-import-external from "applicationcredential-import" test
17-
# TODO(scaffolding): Add fields necessary to match filter
16+
userID: "token"
17+
secret: "bar"

0 commit comments

Comments
 (0)