moving steps to a better location

CircleCI-Public · Dec 28, 2023 · 4fb78a5 · 4fb78a5
1 parent 935a545
commit 4fb78a5
Show file tree

Hide file tree

Showing 2 changed files with 72 additions and 30 deletions.
diff --git a/.circleci/test-deploy.yml b/.circleci/test-deploy.yml
@@ -203,27 +203,51 @@ workflows:
           app-dir: "~/project/sample"
           cache-version: v4
           test-results-for: jest
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/pnpm-lock.yaml
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-yarn-jest-test-job
           app-dir: "~/project/sample"
           cache-version: v4
           test-results-for: jest
           pkg-manager: yarn
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/pnpm-lock.yaml
       - node/test:
           filters: *filters
           name: node-pnpm-jest-test-job
           app-dir: "~/project/sample"
           cache-version: v4
           test-results-for: jest
           pkg-manager: pnpm
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-npm-mocha-test-job
           app-dir: "~/project/sample"
           cache-version: v4
           test-results-for: mocha
           run-command: testmocha
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/pnpm-lock.yaml
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-yarn-mocha-test-job
@@ -232,6 +256,12 @@ workflows:
           test-results-for: mocha
           pkg-manager: yarn
           run-command: testmocha
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/pnpm-lock.yaml
       - node/test:
           filters: *filters
           name: node-pnpm-mocha-test-job
@@ -240,6 +270,12 @@ workflows:
           test-results-for: mocha
           pkg-manager: pnpm
           run-command: testmocha
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-yarn-mocha-with-test-result-path-job
@@ -249,37 +285,73 @@ workflows:
           pkg-manager: yarn
           run-command: testmocha
           test-results-path: sample/test-results.xml
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-test-results-file-job
           app-dir: "~/project/sample"
           cache-version: v4
           test-results-path: sample/other-junit.xml
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/pnpm-lock.yaml
+                  rm ~/project/sample/yarn.lock
       - node/test:
           filters: *filters
           name: node-test-no-junit
           app-dir: "~/project/sample"
           cache-version: v4
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/pnpm-lock.yaml
+                  rm ~/project/sample/yarn.lock
       - node/run:
           filters: *filters
           name: node-run-npm-job
           app-dir: "~/project/sample"
           cache-version: v4
           npm-run: build
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/pnpm-lock.yaml
+                  rm ~/project/sample/yarn.lock
       - node/run:
           filters: *filters
           name: node-run-yarn-job
           app-dir: "~/project/sample"
           cache-version: v5
           pkg-manager: yarn
           yarn-run: build
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/pnpm-lock.yaml
       - node/run:
           filters: *filters
           name: node-run-pnpm-job
           app-dir: "~/project/sample"
           cache-version: v5
           pkg-manager: pnpm
           pnpm-run: build
+          setup:
+            - run:
+                name: Remove other lock files
+                command: |
+                  rm ~/project/sample/package-lock.json
+                  rm ~/project/sample/pnpm-lock.yaml
       - integration-test-override-ci:
           filters: *filters
       - integration-test-override-ci-windows:

diff --git a/src/jobs/test.yml b/src/jobs/test.yml
@@ -69,36 +69,6 @@ executor:
 steps:
     - checkout
     - steps: << parameters.setup >>
-    - when:
-        condition:
-            and:
-                - equal: [ npm, << parameters.pkg-manager >> ]
-        steps:
-            - run:
-                name: Remove other lock files
-                command: |
-                    rm ~/project/sample/pnpm-lock.yaml
-                    rm ~/project/sample/yarn.lock
-    - when:
-        condition:
-            and:
-                - equal: [ pnpm, << parameters.pkg-manager >> ]
-        steps:
-            - run:
-                name: Remove other lock files
-                command: |
-                    rm ~/project/sample/package-lock.json
-                    rm ~/project/sample/yarn.lock
-    - when:
-        condition:
-            and:
-                - equal: [ yarn, << parameters.pkg-manager >> ]
-        steps:
-            - run:
-                name: Remove other lock files
-                command: |
-                    rm ~/project/sample/package-lock.json
-                    rm ~/project/sample/pnpm-lock.yaml
     - install-packages:
           app-dir: <<parameters.app-dir>>
           pkg-manager: <<parameters.pkg-manager>>