test: remove jsx plugin from worker playground (#11177)

vitejs · Dec 6, 2022 · 51ceb4e · 51ceb4e
1 parent 014e4aa
commit 51ceb4e
Show file tree

Hide file tree

Showing 10 changed files with 26 additions and 156 deletions.
diff --git a/playground/worker/__tests__/es/es-worker.spec.ts b/playground/worker/__tests__/es/es-worker.spec.ts
@@ -57,7 +57,7 @@ describe.runIf(isBuild)('build', () => {
   test('inlined code generation', async () => {
     const assetsDir = path.resolve(testDir, 'dist/es/assets')
     const files = fs.readdirSync(assetsDir)
-    expect(files.length).toBe(27)
+    expect(files.length).toBe(28)
     const index = files.find((f) => f.includes('main-module'))
     const content = fs.readFileSync(path.resolve(assetsDir, index), 'utf-8')
     const worker = files.find((f) => f.includes('my-worker'))

diff --git a/playground/worker/modules/test-plugin.js b/playground/worker/modules/test-plugin.js
@@ -0,0 +1 @@
+export const bundleWithPlugin = 'worker bundle with plugin fail.   '
diff --git a/playground/worker/modules/test-plugin.tsx b/playground/worker/modules/test-plugin.tsx
diff --git a/playground/worker/package.json b/playground/worker/package.json
@@ -25,8 +25,5 @@
   },
   "dependencies": {
     "@vitejs/test-dep-to-optimize": "file:./dep-to-optimize"
-  },
-  "devDependencies": {
-    "@vitejs/plugin-vue-jsx": "^3.0.0-alpha.0"
   }
 }
diff --git a/playground/worker/vite.config-es.js b/playground/worker/vite.config-es.js
@@ -1,5 +1,5 @@
-const vueJsx = require('@vitejs/plugin-vue-jsx')
 const vite = require('vite')
+const workerPluginTestPlugin = require('./worker-plugin-test-plugin')
 
 module.exports = vite.defineConfig({
   base: '/es/',
@@ -11,7 +11,7 @@ module.exports = vite.defineConfig({
   },
   worker: {
     format: 'es',
-    plugins: [vueJsx()],
+    plugins: [workerPluginTestPlugin()],
     rollupOptions: {
       output: {
         assetFileNames: 'assets/worker_asset-[name].[ext]',
@@ -31,6 +31,7 @@ module.exports = vite.defineConfig({
     },
   },
   plugins: [
+    workerPluginTestPlugin(),
     {
       name: 'resolve-format-es',
 

diff --git a/playground/worker/vite.config-iife.js b/playground/worker/vite.config-iife.js
@@ -1,5 +1,5 @@
-const vueJsx = require('@vitejs/plugin-vue-jsx')
 const vite = require('vite')
+const workerPluginTestPlugin = require('./worker-plugin-test-plugin')
 
 module.exports = vite.defineConfig({
   base: '/iife/',
@@ -11,7 +11,7 @@ module.exports = vite.defineConfig({
   worker: {
     format: 'iife',
     plugins: [
-      vueJsx(),
+      workerPluginTestPlugin(),
       {
         name: 'config-test',
         config() {
@@ -46,4 +46,5 @@ module.exports = vite.defineConfig({
       },
     },
   },
+  plugins: [workerPluginTestPlugin()],
 })
diff --git a/playground/worker/vite.config-relative-base.js b/playground/worker/vite.config-relative-base.js
@@ -1,6 +1,6 @@
 const path = require('node:path')
-const vueJsx = require('@vitejs/plugin-vue-jsx')
 const vite = require('vite')
+const workerPluginTestPlugin = require('./worker-plugin-test-plugin')
 
 module.exports = vite.defineConfig({
   base: './',
@@ -11,7 +11,7 @@ module.exports = vite.defineConfig({
   },
   worker: {
     format: 'es',
-    plugins: [vueJsx()],
+    plugins: [workerPluginTestPlugin()],
     rollupOptions: {
       output: {
         assetFileNames: 'worker-assets/worker_asset-[name]-[hash].[ext]',
@@ -34,6 +34,7 @@ module.exports = vite.defineConfig({
     baseRoute: '/relative-base/',
   },
   plugins: [
+    workerPluginTestPlugin(),
     {
       name: 'resolve-format-es',
       transform(code, id) {

diff --git a/playground/worker/vite.config-sourcemap.js b/playground/worker/vite.config-sourcemap.js
@@ -1,5 +1,5 @@
-const vueJsx = require('@vitejs/plugin-vue-jsx')
 const vite = require('vite')
+const workerPluginTestPlugin = require('./worker-plugin-test-plugin')
 
 module.exports = vite.defineConfig((sourcemap) => {
   sourcemap = process.env.WORKER_MODE || sourcemap
@@ -17,7 +17,7 @@ module.exports = vite.defineConfig((sourcemap) => {
     },
     worker: {
       format: 'iife',
-      plugins: [vueJsx()],
+      plugins: [workerPluginTestPlugin()],
       rollupOptions: {
         output: {
           assetFileNames: 'assets/[name]-worker_asset[hash].[ext]',
@@ -39,5 +39,6 @@ module.exports = vite.defineConfig((sourcemap) => {
         },
       },
     },
+    plugins: [workerPluginTestPlugin()],
   }
 })
diff --git a/playground/worker/worker-plugin-test-plugin.js b/playground/worker/worker-plugin-test-plugin.js
@@ -0,0 +1,12 @@
+module.exports = () => ({
+  name: 'plugin-for-worker',
+  transform(code, id) {
+    if (id.includes('worker/modules/test-plugin.js')) {
+      return {
+        // keep length for sourcemap
+        code: code.replace('plugin fail.   ', 'plugin success!'),
+        map: null,
+      }
+    }
+  },
+})
diff --git a/pnpm-lock.yaml b/pnpm-lock.yaml