Merge pull request #24804 from storybookjs/norbert/testbuild-no-sourc…

…emaps TestBuild: No sourcemaps for test builds
storybookjs · Nov 10, 2023 · 176017d · 176017d
2 parents 4f6ac8d + e347ebf
commit 176017d
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/code/builders/builder-vite/src/build.ts b/code/builders/builder-vite/src/build.ts
@@ -12,7 +12,7 @@ export async function build(options: Options) {
     build: {
       outDir: options.outputDir,
       emptyOutDir: false, // do not clean before running Vite build - Storybook has already added assets in there!
-      sourcemap: true,
+      sourcemap: !options.test,
       rollupOptions: {
         // Do not try to bundle the storybook runtime, it is copied into the output dir after the build.
         external: ['./sb-preview/runtime.js'],

diff --git a/code/builders/builder-webpack5/src/preview/iframe-webpack.config.ts b/code/builders/builder-webpack5/src/preview/iframe-webpack.config.ts
@@ -227,7 +227,7 @@ export default async (
     name: 'preview',
     mode: isProd ? 'production' : 'development',
     bail: isProd,
-    devtool: 'cheap-module-source-map',
+    devtool: options.test ? false : 'cheap-module-source-map',
     entry: entries,
     output: {
       path: resolve(process.cwd(), outputDir),