processor: run tests as root in the container

- We need to install packages, compile ts, and move assets before running the actual testing code. - The `node` user doesn't have permessions to do anything of these tasks. Update scripts/clear_volumes_and_test_processor.sh Co-authored-by: Kaspar Emanuel <[email protected]>
kitspace · Oct 27, 2022 · 84d03ee · 84d03ee
1 parent a149c1f
commit 84d03ee
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 3 deletions.
diff --git a/docker-compose.override.yml b/docker-compose.override.yml
@@ -40,6 +40,7 @@ services:
       dockerfile: Dockerfile
 
   processor:
+    user: root
     build:
       context: processor/
       dockerfile: Dockerfile

diff --git a/processor/Dockerfile b/processor/Dockerfile
@@ -33,4 +33,3 @@ RUN addgroup --gid 1000 node && \
 
 USER node
 CMD ["node", "dist/src/server.js"]
-
diff --git a/scripts/clear_volumes_and_test_processor.sh b/scripts/clear_volumes_and_test_processor.sh
@@ -20,8 +20,11 @@ docker-compose down -v
 
 # you can pass arguments to mocha e.g. `-g multi`
 args="$(concatenate_args "$@")"
+
+# We need to install packages, compile ts, and move assets before running the actual testing code.
+# The `node` user doesn't have permission to do any of these tasks.
 docker-compose run \
-    -u node \
+    -u root \
     -e LOG_LEVEL=debug \
     -e DATA_DIR=/data/test \
-    processor sh -c "whoami && stat /data /gitea-data /app"
+    processor sh -c "yarn install && yarn tsc && yarn cp-assets && yarn cp-test-assets && yarn test ${args}"
Original file line number	Diff line number	Diff line change
Expand Up		@@ -33,4 +33,3 @@ RUN addgroup --gid 1000 node && \

		USER node
		CMD ["node", "dist/src/server.js"]