Update cleanup script and remove max parallel

YarnSpinnerTool · Mar 14, 2024 · 0eea1b6 · 0eea1b6
1 parent 84c20c3
commit 0eea1b6
Showing 1 changed file with 17 additions and 6 deletions.
diff --git a/.github/workflows/test_sh.yml b/.github/workflows/test_sh.yml
@@ -19,7 +19,7 @@ jobs:
     runs-on: self-hosted
     strategy:
       fail-fast: false
-      max-parallel: 1 # Only run one at a time, to prevent license contention
+      # max-parallel: 1 # Only run one at a time, to prevent license contention
       matrix:
         projectPath:
           - YarnSpinner
@@ -77,13 +77,24 @@ jobs:
                 -v ./output:/output \
                 --hostname YS-Linux-Build \
                 yarnspinner/unity-${{ matrix.unityVersion }} \
-                /bin/bash -c 'unity-editor -runTests -batchmode -projectPath /project/ -testResults /output/TestResults-EditMode.xml -testPlatform EditMode -logFile -; chmod -R 777 /project/ /output/'
-    
+                /bin/bash -c 'unity-editor -runTests -batchmode -projectPath /project/ -testResults /output/TestResults-EditMode.xml -testPlatform EditMode -logFile -'
+                
       - name: Upload test results
         uses: actions/upload-artifact@v2
         if: always()
         with:
-            name: Test results (edit + play, ${{ matrix.unityVersion }}-${{ matrix.targetPlatform }}
-            # path: ${{ steps.testRunner.outputs.artifactsPath }}
-            path: ./output
+          name: Test results (edit + play, ${{ matrix.unityVersion }}-${{ matrix.targetPlatform }}
+          # path: ${{ steps.testRunner.outputs.artifactsPath }}
+          path: ./output
+
+      - name: Cleanup
+        if: always()
+        run: |
+            docker run \
+                --run \
+                -v ./${{ matrix.projectPath }}:/project
+                -v ./output:/output
+                yarnspinner/unity-${{ matrix.unityVersion }} \
+                /bin/bash -c 'rm -rf /project/* /output/*'
+