test: fix e2e

Author: maiieul

packages/qwik/src/optimizer/src/plugins/rollup.ts

@@ -230,16 +230,15 @@ export function normalizeRollupOutputOptionsObject(
           return 'build/qwik-city.js';
         }
 
-        // Sanitize The path to use dashes instead of slashes, to keep the same folder strucutre as without debug:true.
+        // The chunk name can often be a path. We sanitize it to use dashes instead of slashes, to keep the same folder structure as without debug:true.
         // Besides, Rollup doesn't accept absolute or relative paths as inputs for the [name] placeholder for the same reason.
         const path = optimizer.sys.path;
         const relativePath = path.relative(optimizer.sys.cwd(), chunkInfo.name);
-        const sanitizedPath = relativePath
+        const sanitized = relativePath
           .replace(/^(\.\.\/)+/, '')
           .replace(/^\/+/, '')
           .replace(/\//g, '-');
-        chunkInfo.name = sanitizedPath;
-        return `build/[name].js`;
+        return `build/${sanitized}.js`;
       };
     }
     // client production output

packages/qwik/src/optimizer/src/plugins/vite.unit.ts

@@ -12,16 +12,28 @@ import {
 
 const cwd = process.cwd();
 
-const chunkInfoMock: Rollup.PreRenderedChunk = {
-  exports: [''],
-  name: 'chunk.tsx',
-  facadeModuleId: 'chunk.tsx',
-  isDynamicEntry: false,
-  isEntry: false,
-  isImplicitEntry: false,
-  moduleIds: ['chunk.tsx'],
-  type: 'chunk',
-};
+const chunkInfoMocks: Rollup.PreRenderedChunk[] = [
+  {
+    exports: [''],
+    name: 'chunk.tsx',
+    facadeModuleId: 'chunk.tsx',
+    isDynamicEntry: false,
+    isEntry: false,
+    isImplicitEntry: false,
+    moduleIds: ['chunk.tsx'],
+    type: 'chunk',
+  },
+  {
+    exports: [''],
+    name: '/Users/username/app/chunk.tsx',
+    facadeModuleId: '/Users/username/app/chunk.tsx',
+    isDynamicEntry: false,
+    isEntry: false,
+    isImplicitEntry: false,
+    moduleIds: ['/Users/username/app/chunk.tsx'],
+    type: 'chunk',
+  },
+];
 
 function mockOptimizerOptions(): OptimizerOptions {
   return {
@@ -88,8 +100,10 @@ test('command: serve, mode: development', async () => {
   assert.deepEqual(rollupOptions.input, normalizePath(resolve(cwd, 'src', 'entry.dev')));
 
   assert.deepEqual(outputOptions.assetFileNames, 'assets/[hash]-[name].[ext]');
-  assert.deepEqual(chunkFileNames(chunkInfoMock), 'build/[name].js');
-  assert.deepEqual(entryFileNames(chunkInfoMock), 'build/[name].js');
+  assert.deepEqual(chunkFileNames(chunkInfoMocks[0]), `build/chunk.tsx.js`);
+  assert.deepEqual(entryFileNames(chunkInfoMocks[0]), `build/chunk.tsx.js`);
+  assert.deepEqual(chunkFileNames(chunkInfoMocks[1]), 'build/username-app-chunk.tsx.js');
+  assert.deepEqual(entryFileNames(chunkInfoMocks[1]), 'build/username-app-chunk.tsx.js');
   assert.deepEqual(outputOptions.format, 'es');
 
   assert.deepEqual(build.dynamicImportVarsOptions?.exclude, [/./]);
@@ -167,8 +181,10 @@ test('command: build, mode: development', async () => {
   assert.deepEqual(rollupOptions.input, [normalizePath(resolve(cwd, 'src', 'root'))]);
 
   assert.deepEqual(outputOptions.assetFileNames, 'assets/[hash]-[name].[ext]');
-  assert.deepEqual(chunkFileNames(chunkInfoMock), 'build/[name].js');
-  assert.deepEqual(entryFileNames(chunkInfoMock), 'build/[name].js');
+  assert.deepEqual(chunkFileNames(chunkInfoMocks[0]), `build/chunk.tsx.js`);
+  assert.deepEqual(entryFileNames(chunkInfoMocks[0]), `build/chunk.tsx.js`);
+  assert.deepEqual(chunkFileNames(chunkInfoMocks[1]), 'build/username-app-chunk.tsx.js');
+  assert.deepEqual(entryFileNames(chunkInfoMocks[1]), 'build/username-app-chunk.tsx.js');
 
   assert.deepEqual(build.dynamicImportVarsOptions?.exclude, [/./]);
   assert.deepEqual(build.ssr, undefined);