Skip to content

Commit 6a374ef

Browse files
committed
test: Update test using ClientConfig
Signed-off-by: Viet Nguyen Duc <[email protected]>
1 parent 0666391 commit 6a374ef

File tree

3 files changed

+16
-21
lines changed

3 files changed

+16
-21
lines changed

tests/SeleniumTests/__init__.py

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -18,6 +18,7 @@
1818
SELENIUM_GRID_PORT = os.environ.get('SELENIUM_GRID_PORT', '4444')
1919
SELENIUM_GRID_USERNAME = os.environ.get('SELENIUM_GRID_USERNAME', None)
2020
SELENIUM_GRID_PASSWORD = os.environ.get('SELENIUM_GRID_PASSWORD', None)
21+
CHART_CERT_PATH = os.environ.get('CHART_CERT_PATH', None)
2122
SELENIUM_GRID_TEST_HEADLESS = os.environ.get('SELENIUM_GRID_TEST_HEADLESS', 'false').lower() == 'true'
2223
SELENIUM_ENABLE_MANAGED_DOWNLOADS = os.environ.get('SELENIUM_ENABLE_MANAGED_DOWNLOADS', 'true').lower() == 'true'
2324
WEB_DRIVER_WAIT_TIMEOUT = int(os.environ.get('WEB_DRIVER_WAIT_TIMEOUT', 60))
@@ -37,7 +38,8 @@
3738
keep_alive=True,
3839
timeout=3600,
3940
username=SELENIUM_GRID_USERNAME,
40-
password=SELENIUM_GRID_PASSWORD
41+
password=SELENIUM_GRID_PASSWORD,
42+
ca_certs=CHART_CERT_PATH,
4143
)
4244

4345
if TEST_NODE_RELAY == 'Android':
@@ -167,7 +169,7 @@ def setUp(self):
167169
start_time = time.time()
168170
self.driver = webdriver.Remote(
169171
options=options,
170-
command_executor="%s://%s:%s" % (SELENIUM_GRID_PROTOCOL,SELENIUM_GRID_HOST,SELENIUM_GRID_PORT),
172+
command_executor=SELENIUM_GRID_URL,
171173
client_config=CLIENT_CONFIG
172174
)
173175
end_time = time.time()
@@ -196,7 +198,7 @@ def setUp(self):
196198
start_time = time.time()
197199
self.driver = webdriver.Remote(
198200
options=options,
199-
command_executor="%s://%s:%s" % (SELENIUM_GRID_PROTOCOL,SELENIUM_GRID_HOST,SELENIUM_GRID_PORT),
201+
command_executor=SELENIUM_GRID_URL,
200202
client_config=CLIENT_CONFIG
201203
)
202204
end_time = time.time()
@@ -230,7 +232,7 @@ def setUp(self):
230232
start_time = time.time()
231233
self.driver = webdriver.Remote(
232234
options=options,
233-
command_executor="%s://%s:%s" % (SELENIUM_GRID_PROTOCOL,SELENIUM_GRID_HOST,SELENIUM_GRID_PORT),
235+
command_executor=SELENIUM_GRID_URL,
234236
client_config=CLIENT_CONFIG
235237
)
236238
end_time = time.time()

tests/bootstrap.sh

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,6 @@ fi
1616

1717
python3 -m pip install docker requests chardet | grep -v 'Requirement already satisfied'
1818

19-
if [ "${SELENIUM_GRID_PROTOCOL}" = "https" ]; then
20-
export REQUESTS_CA_BUNDLE="${CHART_CERT_PATH}"
21-
fi
22-
2319
python3 test.py $1
2420
ret_code=$?
2521

tests/charts/make/chart_test.sh

Lines changed: 10 additions & 13 deletions
Original file line numberDiff line numberDiff line change
@@ -67,11 +67,9 @@ cleanup() {
6767
echo "Logs for pod $pod"
6868
kubectl logs -n ${SELENIUM_NAMESPACE} $pod --all-containers > tests/tests/pod_logs_${pod}.txt
6969
done
70-
if [ "${SKIP_CLEANUP}" = "false" ]; then
70+
if [ "${SKIP_CLEANUP}" = "false" ] || [ "${CI:-false}" = "true" ]; then
7171
echo "Clean up chart release and namespace"
7272
helm delete ${RELEASE_NAME} --namespace ${SELENIUM_NAMESPACE} --wait
73-
kubectl patch ns ${SELENIUM_NAMESPACE} -p '{"metadata":{"finalizers":null}}'
74-
kubectl delete namespace ${SELENIUM_NAMESPACE} --wait=false
7573
fi
7674
}
7775

@@ -251,8 +249,6 @@ if [ "${SECURE_INGRESS_ONLY_GENERATE}" = "true" ] && [ "${RENDER_HELM_TEMPLATE_O
251249
--set tls.ingress.defaultSANList[0]=${SELENIUM_GRID_HOST} \
252250
--set tls.ingress.defaultIPList[0]=$(hostname -I | awk '{print $1}') \
253251
"
254-
kubectl get secret ${SELENIUM_TLS_SECRET_NAME} -n ${SELENIUM_NAMESPACE} -o jsonpath="{.data.tls\.crt}" | base64 -d > ./tests/tests/tls.crt
255-
CHART_CERT_PATH="./tests/tests/tls.crt"
256252
fi
257253

258254
if [ "${SECURE_INGRESS_ONLY_DEFAULT}" = "true" ]; then
@@ -280,13 +276,14 @@ if [ "${SECURE_USE_EXTERNAL_CERT}" = "true" ] && [ "${RENDER_HELM_TEMPLATE_ONLY}
280276
--set ingress.nginx.sslSecret="${SELENIUM_NAMESPACE}/${EXTERNAL_TLS_SECRET_NAME}" \
281277
"
282278
cert_dir="./tests/tests"
283-
ADD_IP_ADDRESS=hostname ./${CHART_PATH}/certs/gen-cert-helper.sh -d ${cert_dir}
284-
kubectl delete secret -n ${SELENIUM_NAMESPACE} ${EXTERNAL_TLS_SECRET_NAME} --ignore-not-found=true
285-
kubectl create secret generic -n ${SELENIUM_NAMESPACE} ${EXTERNAL_TLS_SECRET_NAME} \
286-
--from-file=tls.crt=${cert_dir}/tls.crt \
287-
--from-file=tls.key=${cert_dir}/tls.key \
288-
--from-file=server.jks=${cert_dir}/server.jks \
289-
--from-file=server.pass=${cert_dir}/server.pass
279+
if [ ! -f "./tests/tests/tls.crt" ]; then
280+
ADD_IP_ADDRESS=hostname ./${CHART_PATH}/certs/gen-cert-helper.sh -d ${cert_dir}
281+
kubectl create secret generic -n ${SELENIUM_NAMESPACE} ${EXTERNAL_TLS_SECRET_NAME} \
282+
--from-file=tls.crt=${cert_dir}/tls.crt \
283+
--from-file=tls.key=${cert_dir}/tls.key \
284+
--from-file=server.jks=${cert_dir}/server.jks \
285+
--from-file=server.pass=${cert_dir}/server.pass
286+
fi
290287
CHART_CERT_PATH="./tests/tests/tls.crt"
291288
fi
292289

@@ -408,7 +405,7 @@ if [ "${TEST_UPGRADE_CHART}" = "true" ]; then
408405
exit 0
409406
fi
410407

411-
if [ "${SECURE_INGRESS_ONLY_GENERATE}" = "true" ]; then
408+
if [ "${SECURE_INGRESS_ONLY_GENERATE}" = "true" ] && [ "${RENDER_HELM_TEMPLATE_ONLY}" != "true" ]; then
412409
kubectl get secret ${SELENIUM_TLS_SECRET_NAME} -n ${SELENIUM_NAMESPACE} -o jsonpath="{.data.tls\.crt}" | base64 -d > ./tests/tests/tls.crt
413410
CHART_CERT_PATH="./tests/tests/tls.crt"
414411
fi

0 commit comments

Comments
 (0)