Skip to content

Commit ea9c4b2

Browse files
ewoutpewoutp
committed
Fixed unit tests
1 parent ee5dcb5 commit ea9c4b2

File tree

4 files changed

+102
-82
lines changed

4 files changed

+102
-82
lines changed

‎pkg/deployment/pod_creator_agent_args_test.go‎

Lines changed: 24 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -54,9 +54,9 @@ func TestCreateArangodArgsAgent(t *testing.T) {
5454
assert.Equal(t,
5555
[]string{
5656
"--agency.activate=true",
57-
"--agency.endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
58-
"--agency.endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
59-
"--agency.my-address=tcp://name-agent-a1.name-int.ns.svc:8529",
57+
"--agency.endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
58+
"--agency.endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
59+
"--agency.my-address=ssl://name-agent-a1.name-int.ns.svc:8529",
6060
"--agency.size=3",
6161
"--agency.supervision=true",
6262
"--cluster.my-id=a1",
@@ -65,16 +65,18 @@ func TestCreateArangodArgsAgent(t *testing.T) {
6565
"--log.level=INFO",
6666
"--log.output=+",
6767
"--server.authentication=true",
68-
"--server.endpoint=tcp://[::]:8529",
68+
"--server.endpoint=ssl://[::]:8529",
6969
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
7070
"--server.statistics=false",
7171
"--server.storage-engine=rocksdb",
72+
"--ssl.ecdh-curve=",
73+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
7274
},
7375
cmdline,
7476
)
7577
}
7678

77-
// Default+TLS deployment
79+
// Default+TLS disabled deployment
7880
{
7981
apiObject := &api.ArangoDeployment{
8082
ObjectMeta: metav1.ObjectMeta{
@@ -84,7 +86,7 @@ func TestCreateArangodArgsAgent(t *testing.T) {
8486
Spec: api.DeploymentSpec{
8587
Mode: api.DeploymentModeCluster,
8688
TLS: api.TLSSpec{
87-
CASecretName: "test-ca",
89+
CASecretName: "None",
8890
},
8991
},
9092
}
@@ -98,9 +100,9 @@ func TestCreateArangodArgsAgent(t *testing.T) {
98100
assert.Equal(t,
99101
[]string{
100102
"--agency.activate=true",
101-
"--agency.endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
102-
"--agency.endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
103-
"--agency.my-address=ssl://name-agent-a1.name-int.ns.svc:8529",
103+
"--agency.endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
104+
"--agency.endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
105+
"--agency.my-address=tcp://name-agent-a1.name-int.ns.svc:8529",
104106
"--agency.size=3",
105107
"--agency.supervision=true",
106108
"--cluster.my-id=a1",
@@ -109,12 +111,10 @@ func TestCreateArangodArgsAgent(t *testing.T) {
109111
"--log.level=INFO",
110112
"--log.output=+",
111113
"--server.authentication=true",
112-
"--server.endpoint=ssl://[::]:8529",
114+
"--server.endpoint=tcp://[::]:8529",
113115
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
114116
"--server.statistics=false",
115117
"--server.storage-engine=rocksdb",
116-
"--ssl.ecdh-curve=",
117-
"--ssl.keyfile=/secrets/tls/tls.keyfile",
118118
},
119119
cmdline,
120120
)
@@ -143,9 +143,9 @@ func TestCreateArangodArgsAgent(t *testing.T) {
143143
assert.Equal(t,
144144
[]string{
145145
"--agency.activate=true",
146-
"--agency.endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
147-
"--agency.endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
148-
"--agency.my-address=tcp://name-agent-a1.name-int.ns.svc:8529",
146+
"--agency.endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
147+
"--agency.endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
148+
"--agency.my-address=ssl://name-agent-a1.name-int.ns.svc:8529",
149149
"--agency.size=3",
150150
"--agency.supervision=true",
151151
"--cluster.my-id=a1",
@@ -154,9 +154,11 @@ func TestCreateArangodArgsAgent(t *testing.T) {
154154
"--log.level=INFO",
155155
"--log.output=+",
156156
"--server.authentication=false",
157-
"--server.endpoint=tcp://[::]:8529",
157+
"--server.endpoint=ssl://[::]:8529",
158158
"--server.statistics=false",
159159
"--server.storage-engine=mmfiles",
160+
"--ssl.ecdh-curve=",
161+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
160162
},
161163
cmdline,
162164
)
@@ -184,9 +186,9 @@ func TestCreateArangodArgsAgent(t *testing.T) {
184186
assert.Equal(t,
185187
[]string{
186188
"--agency.activate=true",
187-
"--agency.endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
188-
"--agency.endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
189-
"--agency.my-address=tcp://name-agent-a1.name-int.ns.svc:8529",
189+
"--agency.endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
190+
"--agency.endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
191+
"--agency.my-address=ssl://name-agent-a1.name-int.ns.svc:8529",
190192
"--agency.size=3",
191193
"--agency.supervision=true",
192194
"--cluster.my-id=a1",
@@ -195,10 +197,12 @@ func TestCreateArangodArgsAgent(t *testing.T) {
195197
"--log.level=INFO",
196198
"--log.output=+",
197199
"--server.authentication=true",
198-
"--server.endpoint=tcp://[::]:8529",
200+
"--server.endpoint=ssl://[::]:8529",
199201
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
200202
"--server.statistics=false",
201203
"--server.storage-engine=rocksdb",
204+
"--ssl.ecdh-curve=",
205+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
202206
"--foo1",
203207
"--foo2",
204208
},

‎pkg/deployment/pod_creator_coordinator_args_test.go‎

Lines changed: 28 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -53,27 +53,29 @@ func TestCreateArangodArgsCoordinator(t *testing.T) {
5353
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupCoordinators, apiObject.Spec.Coordinators, agents, "id1")
5454
assert.Equal(t,
5555
[]string{
56-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
57-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
58-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
59-
"--cluster.my-address=tcp://name-coordinator-id1.name-int.ns.svc:8529",
56+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
57+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
58+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
59+
"--cluster.my-address=ssl://name-coordinator-id1.name-int.ns.svc:8529",
6060
"--cluster.my-id=id1",
6161
"--cluster.my-role=COORDINATOR",
6262
"--database.directory=/data",
6363
"--foxx.queues=true",
6464
"--log.level=INFO",
6565
"--log.output=+",
6666
"--server.authentication=true",
67-
"--server.endpoint=tcp://[::]:8529",
67+
"--server.endpoint=ssl://[::]:8529",
6868
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
6969
"--server.statistics=true",
7070
"--server.storage-engine=rocksdb",
71+
"--ssl.ecdh-curve=",
72+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
7173
},
7274
cmdline,
7375
)
7476
}
7577

76-
// Default+TLS deployment
78+
// Default+TLS disabled deployment
7779
{
7880
apiObject := &api.ArangoDeployment{
7981
ObjectMeta: metav1.ObjectMeta{
@@ -83,7 +85,7 @@ func TestCreateArangodArgsCoordinator(t *testing.T) {
8385
Spec: api.DeploymentSpec{
8486
Mode: api.DeploymentModeCluster,
8587
TLS: api.TLSSpec{
86-
CASecretName: "test-ca",
88+
CASecretName: "None",
8789
},
8890
},
8991
}
@@ -96,23 +98,21 @@ func TestCreateArangodArgsCoordinator(t *testing.T) {
9698
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupCoordinators, apiObject.Spec.Coordinators, agents, "id1")
9799
assert.Equal(t,
98100
[]string{
99-
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
100-
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
101-
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
102-
"--cluster.my-address=ssl://name-coordinator-id1.name-int.ns.svc:8529",
101+
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
102+
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
103+
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
104+
"--cluster.my-address=tcp://name-coordinator-id1.name-int.ns.svc:8529",
103105
"--cluster.my-id=id1",
104106
"--cluster.my-role=COORDINATOR",
105107
"--database.directory=/data",
106108
"--foxx.queues=true",
107109
"--log.level=INFO",
108110
"--log.output=+",
109111
"--server.authentication=true",
110-
"--server.endpoint=ssl://[::]:8529",
112+
"--server.endpoint=tcp://[::]:8529",
111113
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
112114
"--server.statistics=true",
113115
"--server.storage-engine=rocksdb",
114-
"--ssl.ecdh-curve=",
115-
"--ssl.keyfile=/secrets/tls/tls.keyfile",
116116
},
117117
cmdline,
118118
)
@@ -139,20 +139,22 @@ func TestCreateArangodArgsCoordinator(t *testing.T) {
139139
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupCoordinators, apiObject.Spec.Coordinators, agents, "id1")
140140
assert.Equal(t,
141141
[]string{
142-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
143-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
144-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
145-
"--cluster.my-address=tcp://name-coordinator-id1.name-int.ns.svc:8529",
142+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
143+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
144+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
145+
"--cluster.my-address=ssl://name-coordinator-id1.name-int.ns.svc:8529",
146146
"--cluster.my-id=id1",
147147
"--cluster.my-role=COORDINATOR",
148148
"--database.directory=/data",
149149
"--foxx.queues=true",
150150
"--log.level=INFO",
151151
"--log.output=+",
152152
"--server.authentication=false",
153-
"--server.endpoint=tcp://[::]:8529",
153+
"--server.endpoint=ssl://[::]:8529",
154154
"--server.statistics=true",
155155
"--server.storage-engine=rocksdb",
156+
"--ssl.ecdh-curve=",
157+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
156158
},
157159
cmdline,
158160
)
@@ -180,21 +182,23 @@ func TestCreateArangodArgsCoordinator(t *testing.T) {
180182
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupCoordinators, apiObject.Spec.Coordinators, agents, "id1")
181183
assert.Equal(t,
182184
[]string{
183-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
184-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
185-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
186-
"--cluster.my-address=tcp://name-coordinator-id1.name-int.ns.svc:8529",
185+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
186+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
187+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
188+
"--cluster.my-address=ssl://name-coordinator-id1.name-int.ns.svc:8529",
187189
"--cluster.my-id=id1",
188190
"--cluster.my-role=COORDINATOR",
189191
"--database.directory=/data",
190192
"--foxx.queues=true",
191193
"--log.level=INFO",
192194
"--log.output=+",
193195
"--server.authentication=true",
194-
"--server.endpoint=tcp://[::]:8529",
196+
"--server.endpoint=ssl://[::]:8529",
195197
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
196198
"--server.statistics=true",
197199
"--server.storage-engine=mmfiles",
200+
"--ssl.ecdh-curve=",
201+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
198202
"--foo1",
199203
"--foo2",
200204
},

‎pkg/deployment/pod_creator_dbserver_args_test.go‎

Lines changed: 28 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -53,27 +53,29 @@ func TestCreateArangodArgsDBServer(t *testing.T) {
5353
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupDBServers, apiObject.Spec.DBServers, agents, "id1")
5454
assert.Equal(t,
5555
[]string{
56-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
57-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
58-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
59-
"--cluster.my-address=tcp://name-dbserver-id1.name-int.ns.svc:8529",
56+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
57+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
58+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
59+
"--cluster.my-address=ssl://name-dbserver-id1.name-int.ns.svc:8529",
6060
"--cluster.my-id=id1",
6161
"--cluster.my-role=PRIMARY",
6262
"--database.directory=/data",
6363
"--foxx.queues=false",
6464
"--log.level=INFO",
6565
"--log.output=+",
6666
"--server.authentication=true",
67-
"--server.endpoint=tcp://[::]:8529",
67+
"--server.endpoint=ssl://[::]:8529",
6868
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
6969
"--server.statistics=true",
7070
"--server.storage-engine=rocksdb",
71+
"--ssl.ecdh-curve=",
72+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
7173
},
7274
cmdline,
7375
)
7476
}
7577

76-
// Default+TLS deployment
78+
// Default+TLS disabled deployment
7779
{
7880
apiObject := &api.ArangoDeployment{
7981
ObjectMeta: metav1.ObjectMeta{
@@ -83,7 +85,7 @@ func TestCreateArangodArgsDBServer(t *testing.T) {
8385
Spec: api.DeploymentSpec{
8486
Mode: api.DeploymentModeCluster,
8587
TLS: api.TLSSpec{
86-
CASecretName: "test-ca",
88+
CASecretName: "None",
8789
},
8890
},
8991
}
@@ -96,23 +98,21 @@ func TestCreateArangodArgsDBServer(t *testing.T) {
9698
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupDBServers, apiObject.Spec.DBServers, agents, "id1")
9799
assert.Equal(t,
98100
[]string{
99-
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
100-
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
101-
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
102-
"--cluster.my-address=ssl://name-dbserver-id1.name-int.ns.svc:8529",
101+
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
102+
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
103+
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
104+
"--cluster.my-address=tcp://name-dbserver-id1.name-int.ns.svc:8529",
103105
"--cluster.my-id=id1",
104106
"--cluster.my-role=PRIMARY",
105107
"--database.directory=/data",
106108
"--foxx.queues=false",
107109
"--log.level=INFO",
108110
"--log.output=+",
109111
"--server.authentication=true",
110-
"--server.endpoint=ssl://[::]:8529",
112+
"--server.endpoint=tcp://[::]:8529",
111113
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
112114
"--server.statistics=true",
113115
"--server.storage-engine=rocksdb",
114-
"--ssl.ecdh-curve=",
115-
"--ssl.keyfile=/secrets/tls/tls.keyfile",
116116
},
117117
cmdline,
118118
)
@@ -139,20 +139,22 @@ func TestCreateArangodArgsDBServer(t *testing.T) {
139139
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupDBServers, apiObject.Spec.DBServers, agents, "id1")
140140
assert.Equal(t,
141141
[]string{
142-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
143-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
144-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
145-
"--cluster.my-address=tcp://name-dbserver-id1.name-int.ns.svc:8529",
142+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
143+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
144+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
145+
"--cluster.my-address=ssl://name-dbserver-id1.name-int.ns.svc:8529",
146146
"--cluster.my-id=id1",
147147
"--cluster.my-role=PRIMARY",
148148
"--database.directory=/data",
149149
"--foxx.queues=false",
150150
"--log.level=INFO",
151151
"--log.output=+",
152152
"--server.authentication=false",
153-
"--server.endpoint=tcp://[::]:8529",
153+
"--server.endpoint=ssl://[::]:8529",
154154
"--server.statistics=true",
155155
"--server.storage-engine=rocksdb",
156+
"--ssl.ecdh-curve=",
157+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
156158
},
157159
cmdline,
158160
)
@@ -180,21 +182,23 @@ func TestCreateArangodArgsDBServer(t *testing.T) {
180182
cmdline := createArangodArgs(apiObject, apiObject.Spec, api.ServerGroupDBServers, apiObject.Spec.DBServers, agents, "id1")
181183
assert.Equal(t,
182184
[]string{
183-
"--cluster.agency-endpoint=tcp://name-agent-a1.name-int.ns.svc:8529",
184-
"--cluster.agency-endpoint=tcp://name-agent-a2.name-int.ns.svc:8529",
185-
"--cluster.agency-endpoint=tcp://name-agent-a3.name-int.ns.svc:8529",
186-
"--cluster.my-address=tcp://name-dbserver-id1.name-int.ns.svc:8529",
185+
"--cluster.agency-endpoint=ssl://name-agent-a1.name-int.ns.svc:8529",
186+
"--cluster.agency-endpoint=ssl://name-agent-a2.name-int.ns.svc:8529",
187+
"--cluster.agency-endpoint=ssl://name-agent-a3.name-int.ns.svc:8529",
188+
"--cluster.my-address=ssl://name-dbserver-id1.name-int.ns.svc:8529",
187189
"--cluster.my-id=id1",
188190
"--cluster.my-role=PRIMARY",
189191
"--database.directory=/data",
190192
"--foxx.queues=false",
191193
"--log.level=INFO",
192194
"--log.output=+",
193195
"--server.authentication=true",
194-
"--server.endpoint=tcp://[::]:8529",
196+
"--server.endpoint=ssl://[::]:8529",
195197
"--server.jwt-secret=$(ARANGOD_JWT_SECRET)",
196198
"--server.statistics=true",
197199
"--server.storage-engine=mmfiles",
200+
"--ssl.ecdh-curve=",
201+
"--ssl.keyfile=/secrets/tls/tls.keyfile",
198202
"--foo1",
199203
"--foo2",
200204
},

0 commit comments

Comments
 (0)