Skip to content

Commit 42d2625

Browse files
committed
Make sure to expose exact same passive port range. Local tests tend to fail
if not done like that. Added a bit more robustness in the test scripts to not fail so easily if services are not fully initialized yet when make test is run e.g. from CI.
1 parent 929618f commit 42d2625

11 files changed

+20
-11
lines changed

docker-compose_development.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -274,7 +274,7 @@ services:
274274
# NOTE: compose does not support shell-style replacement so we can't use
275275
#- "${FTPS_PASV_PORTS/:/-}:${FTPS_PASV_PORTS/:/-}"
276276
# hard-code them for now.
277-
- "8100-8399"
277+
- "8100-8399:8100-8399"
278278
networks:
279279
default:
280280
aliases:

docker-compose_development_gdp.yml

+1-1
Original file line numberDiff line numberDiff line change
@@ -278,7 +278,7 @@ services:
278278
# NOTE: compose does not support shell-style replacement so we can't use
279279
#- "${FTPS_PASV_PORTS/:/-}:${FTPS_PASV_PORTS/:/-}"
280280
# hard-code them for now.
281-
- "8100-8399"
281+
- "8100-8399:8100-8399"
282282
networks:
283283
default:
284284
aliases:

tests/test-10-migrid-https-frontpage.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
https://${DOMAIN}:${PUBLIC_HTTPS_PORT} \
910
-k \
1011
-s \

tests/test-11-migrid-https-migoid-public.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
https://${MIGOID_DOMAIN}:${PUBLIC_HTTPS_PORT}/public/index.html \
910
-k \
1011
-s \

tests/test-12-migrid-https-sid-login-page.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
https://${SID_DOMAIN}:${SID_HTTPS_PORT}/cgi-sid/login.py \
910
-k \
1011
-s \

tests/test-14-migrid-https-migoid-login-page.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
https://${OPENID_DOMAIN}:${OPENID_PORT}/openid/login \
910
-k \
1011
-s \

tests/test-16-migrid-https-migoid-login-submit.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
--data-urlencode user=${MIG_TEST_USER} \
910
--data-urlencode password=${MIG_TEST_USER_PASSWORD} \
1011
--data-urlencode success_to=/openid/id/ \

tests/test-20-migrid-webdavs-read.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
--user "${MIG_TEST_USER}:${MIG_TEST_USER_PASSWORD}" \
910
https://${WEBDAVS_DOMAIN}:${DAVS_PORT}/welcome.txt \
1011
-k \

tests/test-30-migrid-sftp-read.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
--user "${MIG_TEST_USER}:${MIG_TEST_USER_PASSWORD}" \
910
sftp://${SFTP_DOMAIN}:${SFTP_PORT}/welcome.txt \
1011
-k \

tests/test-31-migrid-sftpsubsys-read.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
--user "${MIG_TEST_USER}:${MIG_TEST_USER_PASSWORD}" \
910
sftp://${SFTP_DOMAIN}:${SFTP_SUBSYS_PORT}/welcome.txt \
1011
-k \

tests/test-40-migrid-ftps-read.sh

+2-1
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,8 @@ source default
44

55
${DOCKER:?} run --rm --network host --dns 127.0.0.1 curlimages/curl:8.7.1 \
66
--connect-timeout 1 \
7-
--retry 1 \
7+
--retry-connrefused \
8+
--retry 3 \
89
--user "${MIG_TEST_USER}:${MIG_TEST_USER_PASSWORD}" \
910
ftp://${FTPS_DOMAIN}:${FTPS_CTRL_PORT}/welcome.txt \
1011
--ssl-reqd \

0 commit comments

Comments
 (0)