Simplify version handling of browser-ui-test

Author: GuillaumeGomez

dockerfiles/Dockerfile-gui-tests

@@ -69,13 +69,13 @@ WORKDIR /build
 
 RUN mkdir out
 
+COPY ../package.json /build/package.json
+
 # For now, we need to use `--unsafe-perm=true` to go around an issue when npm tries
 # to create a new folder. For reference:
 # https://github.com/puppeteer/puppeteer/issues/375
 #
 # We also specify the version in case we need to update it to go around cache limitations.
-RUN npm install -g [email protected] --unsafe-perm=true
-
-EXPOSE 3000
+RUN npm install -g --unsafe-perm=true /build
 
 CMD ["node", "/build/out/gui-tests/tester.js"]

dockerfiles/run-gui-tests.sh

@@ -32,11 +32,12 @@ fi
 
 . .env
 
+set +e # We disable the "exit right away if command failed" setting.
 cargo run -- start-web-server &
 SERVER_PID=$!
 
 # status="docker run . -v `pwd`:/build/out:ro gui_tests"
-docker compose run gui_tests
+docker compose run --rm gui_tests
 status=$?
 kill $SERVER_PID
 exit $status