test: install Node dependencies as a separate step

supabase · Dec 10, 2024 · f392d56 · f392d56
1 parent cdcff01
commit f392d56
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/.github/workflows/elixir.yml b/.github/workflows/elixir.yml
@@ -143,6 +143,7 @@ jobs:
       - uses: actions/setup-node@v4
         with:
           node-version: 'lts/*'
+          cache-dependency-path: 'test/integration/js/'
       - name: Set up Rust
         uses: dtolnay/rust-toolchain@v1
         with:
@@ -172,6 +173,8 @@ jobs:
         run: docker-compose -f ./docker-compose.db.yml up -d
       - name: Start epmd
         run: epmd -daemon
+      - name: Install Node dependencies
+        run: 'cd test/integration/js/; npm install'
       - name: Run tests
         run: mix test --only integration --trace
 

diff --git a/test/integration/js/postgres/test.js b/test/integration/js/postgres/test.js
@@ -81,6 +81,6 @@ function exit() {
         ? console.log('🎉')
         : console.error('⚠️', 'Not good')
 
-  !process.exitCode && (!success || only || ignored) && (process.exitCode = 1)
+  if (!success || only || ignored) { process.exit(1) } else { process.exit(0) }
 }