Vite & vitest (#313)

* update deps (matrix crdt)

* add force

* fix matrix test server conf

* disable test

* wip: migrate to vite

* vitest

* small fixes

* fix build

* add lint

* fix star:local

* fix test

* clean deps

* fix ci

* add gh ci timeout

* small fixes

* several vite fixes

* improve build configs and playwright

* fix pw

* update index

* fix uris

* try pw fix

* try fix ci

* fix build

* undo pw

* faster pw?

* always collect traces

* fix icons

* update cache

* test pw cache

* playwright no deps

* run pw

* fix deps

* dont run

* run pw

* fix pw

* disable lerna hoist and improve playwright (#314)

* no hoist

* fix patch package

* fix build

* npm ci root

* no test

* update pw version

* fix overrides

* fix build

* update packages

* force

* run tests

* add comment

* improve playwright

* playwright preview

* increase build heap space

* comments

* fix revert

Author: YousefED

.github/workflows/build.yaml

@@ -6,6 +6,7 @@ jobs:
   build:
     name: Build
     runs-on: ubuntu-latest
+    timeout-minutes: 20
     steps:
       - uses: actions/checkout@master
 
@@ -15,11 +16,12 @@ jobs:
           node-version: 16.x
 
       # doesn't seem to cause significant speed up, probably because it still needs to install deps
-      # - name: cache playwright
-      #   uses: actions/cache@v2
-      #   with:
-      #     path: ~/.cache/ms-playwright
-      #     key: pw-${{ runner.os }}-${{ hashFiles('./package-lock.json') }}
+      - name: cache playwright
+        id: playwright-cache
+        uses: actions/cache@v2
+        with:
+          path: ~/.cache/ms-playwright
+          key: pw3-${{ runner.os }}-${{ hashFiles('./packages/**/package-lock.json') }}
 
       # doesn't seem to cause significant speed up, maybe not right conf?
       # - name: cache lerna
@@ -37,7 +39,7 @@ jobs:
         with:
           # npm cache files are stored in `~/.npm` on Linux/macOS
           path: ~/.npm
-          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('./package-lock.json') }}
+          key: ${{ runner.os }}-build-${{ env.cache-name }}-${{ hashFiles('./packages/**/package-lock.json') }}
           restore-keys: |
             ${{ runner.os }}-build-${{ env.cache-name }}-
             ${{ runner.os }}-build-
@@ -55,8 +57,8 @@ jobs:
       - name: Check docker logs
         run: docker logs synapse
 
-      - name: Install Dependencies
-        run: npm run install-lerna
+      - name: NPM CI
+        run: npm ci
 
       - name: Bootstrap packages
         run: npm run bootstrap
@@ -69,21 +71,38 @@ jobs:
         env:
           CI: true
 
-      - name: Lint (we don't build React in CI, but at least lint it)
+      - name: Install Playwright
+        run: npx playwright install --with-deps
+        # if: steps.playwright-cache.outputs.cache-hit != 'true'
+        # env:
+        # PLAYWRIGHT_BROWSERS_PATH: 0 # https://github.com/microsoft/playwright/blob/main/docs/src/ci.md#caching-browsers
+
+      - name: Lint
         run: npm run lint
         env:
           CI: true
 
-      - name: Run server
-        run: npm run start:local & npx wait-on http://localhost:3000
+      # Option A: don't build in CI, but test against dev server
+      # (with Vite, building in CI is now fast enough so we can go for option B)
+
+      # - name: Run dev server
+      #   # Wait on config file needed for vite dev server: https://github.com/jeffbski/wait-on/issues/78
+      #   run: npm run start:local & npx wait-on http://localhost:5173 -c ./packages/editor/wait-on.conf.json
+      #   env:
+      #     CI: true
+
+      # - name: Run Playwright tests (against dev server)
+      #   run: npm run playwright:dev
+
+      # Option B: actually build and run react code and run tests against that
+      - name: Build and run preview server
+        # Wait on config file needed for vite dev server: https://github.com/jeffbski/wait-on/issues/78
+        run: npm run start:preview & npx wait-on http://localhost:4173 -c ./packages/editor/wait-on.conf.json
         env:
           CI: true
 
-      - name: Install Playwright
-        run: npm run install-playwright
-
-      - name: Run Playwright tests
-        run: npm run playwright
+      - name: Run Playwright tests (against preview server)
+        run: npm run playwright:preview
 
       - uses: actions/upload-artifact@v2
         if: always()

.gitignore

@@ -9,6 +9,7 @@ types
 
 # testing
 coverage
+playwright-report
 
 # production
 /build

.vscode/extensions.json

@@ -4,6 +4,6 @@
   "recommendations": [
     "esbenp.prettier-vscode",
     "dbaeumer.vscode-eslint",
-    "orta.vscode-jest"
+    "ZixuanChen.vitest-explorer"
   ]
 }

.vscode/settings.json

@@ -8,5 +8,12 @@
   },
   "search.exclude": {
     "packages/editor/public/types": true
-  }
+  },
+  "vitest.exclude": [
+    "**/node_modules/**",
+    "**/dist/**",
+    "**/cypress/**",
+    "**/.{idea,git,cache,output,temp}/**",
+    "**/end-to-end/**"
+  ]
 }