[test] Bundling fixtures should not override source build with publis…

…hed build (#26657)
mui · Jun 8, 2021 · 3f4c804 · 3f4c804
1 parent a28a5e7
commit 3f4c804
Show file tree

Hide file tree

Showing 2 changed files with 30 additions and 22 deletions.
diff --git a/.circleci/config.yml b/.circleci/config.yml
@@ -422,11 +422,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture node-esm
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: |
@@ -449,11 +450,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture next-webpack4
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: yarn start
@@ -472,11 +474,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture next-webpack5
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: yarn start
@@ -495,11 +498,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture create-react-app
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: yarn start
@@ -518,11 +522,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture snowpack
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: yarn start
@@ -541,11 +546,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture vite
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: yarn start
@@ -564,11 +570,12 @@ jobs:
       - run:
           name: Prepare fixture
           command: |
-            node ../../scripts/useBuildFromSource.js .
             node ../../scripts/createFixture esbuild
       - run:
           name: Install dependencies
-          command: yarn
+          command: |
+            yarn
+            node ../../scripts/useBuildFromSource.js .
       - run:
           name: Test fixture
           command: |

diff --git a/test/bundling/README.md b/test/bundling/README.md
@@ -11,15 +11,16 @@ The created file might need some manual adjustment since not every edge case is
 1. Use the node version you want to use (e.g. `nvm use 14.0.0`)
 1. Prepare the package.json
    - to test a Pull Request
-     - checkout branch
-     - `yarn`
-     - `yarn lerna run build --scope "@material-ui/*"`
-     - `cd` to fixture
-     - `node ../../scripts/useBuildFromSource.js .`
+     1. checkout branch
+     1. `yarn`
+     1. `yarn lerna run build --scope "@material-ui/*"`
+     1. `cd` to fixture
+     1. `yarn install`
+     1. `node ../../scripts/useBuildFromSource.js .`
    - to test a published npm dist tag (e.g. `latest` or `next`) on npm
-     - adjust the dependencies in the package.json accordingly
-1. `cd` to fixture
-1. `yarn install`
+     1. `cd` to fixture
+     1. adjust the dependencies in the package.json accordingly
+     1. `yarn install`
 1. `yarn start` should exit with 0
 
 ### In CI