17914: Improves cypress test launch script. 17914-peer-federation-login-fix
authorLucas Di Pentima <lucas.dipentima@curii.com>
Tue, 1 Feb 2022 16:09:47 +0000 (13:09 -0300)
committerLucas Di Pentima <lucas.dipentima@curii.com>
Tue, 1 Feb 2022 16:09:47 +0000 (13:09 -0300)
Instead of modifying the in-repo config file to point to the correct
vocabulary temp path, make a copy of it and pass that to arvados boot.
This will avoid committing the arvados_config.yml file by accident.

Arvados-DCO-1.1-Signed-off-by: Lucas Di Pentima <lucas.dipentima@curii.com>

tools/run-integration-tests.sh

index bf4c3ba4c1478c0d315df52992464ee5fee48ed5..6a6177a4a27791798f39b7d92be9d3a60bc08dc4 100755 (executable)
@@ -105,12 +105,14 @@ echo "Installing dev dependencies..."
 ~/go/bin/arvados-server install -type test || exit 1
 
 echo "Launching arvados in test mode..."
-VOC_DIR=$(mktemp -d | cut -d \/ -f3) # Removes the /tmp/ part
-cp ${VOCABULARY_CONF} /tmp/${VOC_DIR}/voc.json
-sed -i "s/VocabularyPath: \".*\"/VocabularyPath: \"\/tmp\/${VOC_DIR}\/voc.json\"/" ${ARVADOS_CONF}
+TMPSUBDIR=$(mktemp -d -p /tmp | cut -d \/ -f3) # Removes the /tmp/ part for the regex below
+TMPDIR=/tmp/${TMPSUBDIR}
+cp ${VOCABULARY_CONF} ${TMPDIR}/voc.json
+cp ${ARVADOS_CONF} ${TMPDIR}/arvados.yml
+sed -i "s/VocabularyPath: \".*\"/VocabularyPath: \"\/tmp\/${TMPSUBDIR}\/voc.json\"/" ${TMPDIR}/arvados.yml
 coproc arvboot (~/go/bin/arvados-server boot \
     -type test \
-    -config ${ARVADOS_CONF} \
+    -config ${TMPDIR}/arvados.yml \
     -no-workbench1 \
     -own-temporary-database \
     -timeout 20m 2> ${ARVADOS_LOG})