Skip to content

Commit e36591a

Browse files
committed
Refactor test for multiple-consumer
Fix prefix double dash Signed-off-by: Mangirdas Judeikis <mangirdas@judeikis.lt> On-behalf-of: @SAP mangirdas.judeikis@sap.com
1 parent fc3edfe commit e36591a

11 files changed

Lines changed: 1653 additions & 843 deletions

File tree

backend/kubernetes/resources/namespace.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -61,7 +61,7 @@ func CreateNamespace(ctx context.Context, client client.Client, generateNamePref
6161
name := identityHash(identity)
6262
namespace := &corev1.Namespace{
6363
ObjectMeta: metav1.ObjectMeta{
64-
Name: fmt.Sprintf("%s-%s", generateNamePrefix, name),
64+
Name: fmt.Sprintf("%s%s", generateNamePrefix, name),
6565
Annotations: map[string]string{
6666
IdentityAnnotationKey: identity,
6767
AuthorAnnotationKey: author,

backend/kubernetes/resources/namespace_test.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ func TestCreateNamespace(t *testing.T) {
6060
},
6161
{
6262
name: "create new namespace with generateName already ending with dash",
63-
generateName: "test-ns-",
63+
generateName: "test-ns",
6464
identity: "test-id-456",
6565
author: "alice",
6666
wantErr: false,
@@ -98,7 +98,7 @@ func TestCreateNamespace(t *testing.T) {
9898

9999
expectedPrefix := tt.generateName
100100
expectedHash := expectedIdentityHash(tt.identity)
101-
expectedName := fmt.Sprintf("%s-%s", expectedPrefix, expectedHash)
101+
expectedName := fmt.Sprintf("%s%s", expectedPrefix, expectedHash)
102102

103103
if result.Name != expectedName {
104104
t.Errorf("CreateNamespace() name = %v, expected %v", result.Name, expectedName)

backend/options/options.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -102,7 +102,7 @@ func NewOptions() *Options {
102102

103103
ExtraOptions: ExtraOptions{
104104
Provider: "kubernetes",
105-
NamespacePrefix: "cluster",
105+
NamespacePrefix: "cluster-",
106106
PrettyName: "Backend",
107107
ConsumerScope: string(kubebindv1alpha2.NamespacedScope),
108108
ClusterScopedIsolation: string(kubebindv1alpha2.IsolationPrefixed),

deploy/examples/template-cowboys.yaml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -16,6 +16,3 @@ spec:
1616
labelSelector:
1717
matchLabels:
1818
app: cowboy
19-
namespaces:
20-
- name: wild-west
21-

deploy/examples/template-sheriffs.yaml

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,4 @@ spec:
2121
group: wildwest.dev
2222
jsonPath:
2323
name: spec.secretRefs.#.name
24-
namespace: spec.secretRefs.#.namespace
25-
namespaces:
26-
- name: wild-west
24+
namespace: spec.secretRefs.#.namespace

test/e2e/bind/happy-case_test.go

Lines changed: 636 additions & 431 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)