chore: fix visual tests

CONTRIBUTING.md

@@ -54,7 +54,7 @@ Currently we need two shells to develop the live-editor demo.
 ./scripts/watch-for-browser.sh
 ```
 
-2. Start the demo. When the CLI stops rolling, open `http://localhost:3001/demo/live-editor/` in
+2. Start the demo. When the CLI stops rolling, open `https://localhost:3001/demo/live-editor/` in
    your browser and you will see the editor page. It will reload once `demo/src` or its dependency packages change.
 
 ```sh
@@ -72,5 +72,5 @@ pnpm run compile
 Build demo and docs site.
 
 ```sh
-pnpm run build-site 
+pnpm run build-site
 ```

demo/package.json

@@ -10,7 +10,7 @@
     "cypress:open": "cypress open",
     "local-test:visual": "percy exec -- cypress run",
     "test:visual": "percy exec -- cypress run --record",
-    "ci:visual": "start-server-and-test serve http://localhost:3001/demo/preview/ test:visual"
+    "ci:visual": "start-server-and-test serve http://localhost:3001/demo/preview/ local-test:visual"
   },
   "dependencies": {
     "@codemirror/basic-setup": "^0.20.0",
@@ -54,7 +54,7 @@
     "cypress": "^13.3.2",
     "less": "^4.1.3",
     "postcss": "^8.4.16",
-    "start-server-and-test": "^2.0.0",
+    "start-server-and-test": "^2.0.4",
     "tailwindcss": "^3.1.8",
     "typescript": "^5.0.0",
     "vite": "^5.0.10",

demo/vite.config.ts

@@ -11,6 +11,8 @@ const mode = process.env.NODE_ENV === 'production' ? 'production' : 'development
 
 const serverAllow = process.env.SERVER_ALLOW || '..'
 
+const enableHttps = process.env.ENABLE_HTTPS === 'true'
+
 // https://vitejs.dev/config/
 export default defineConfig({
   plugins: [
@@ -37,9 +39,11 @@ export default defineConfig({
       strategies: 'injectManifest',
       workbox: {},
     }),
-    ViteMkcert({
-      savePath: './certs', // save the generated certificate into certs directory
-    }),
+    enableHttps
+      ? ViteMkcert({
+          savePath: './certs', // save the generated certificate into certs directory
+        })
+      : null,
   ],
   base: BASE,
   mode,
@@ -48,10 +52,14 @@ export default defineConfig({
       // Allow serving files from one level up to the project root
       allow: [serverAllow],
     },
-    https: {
-      cert: './certs/cert.pem',
-      key: './certs/dev.pem'
-    }
+    ...(enableHttps
+      ? {
+          https: {
+            cert: './certs/cert.pem',
+            key: './certs/dev.pem',
+          },
+        }
+      : null),
   },
   build: {
     rollupOptions: {