fix: source map on dynamic import (#406)

privatenumber · Nov 22, 2023 · 7a0eb7e · 7a0eb7e
1 parent 0d54c99
commit 7a0eb7e
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/utils/transform/transform-dynamic-import.ts b/src/utils/transform/transform-dynamic-import.ts
@@ -52,7 +52,7 @@ export const transformDynamicImport = (
 		 * The performance hit on this is very high
 		 * Since we're only transforming import()s, I think this may be overkill
 		 */
-		// hires: 'boundary',
+		hires: 'boundary',
 	}) as SourceMap;
 
 	return {

diff --git a/tests/specs/smoke.ts b/tests/specs/smoke.ts
@@ -79,9 +79,10 @@ export default 1;
 `;
 
 const sourcemap = {
+	// Adding the dynamic import helps test the import transformation's source map
 	test: (
 		extension: string,
-	) => `const { stack } = new Error(); const searchString = 'index.${extension}:SOURCEMAP_LINE'; assert(stack.includes(searchString), \`Expected \${searchString} in stack: \${stack}\`)`,
+	) => `import('fs');\nconst { stack } = new Error(); const searchString = 'index.${extension}:SOURCEMAP_LINE'; assert(stack.includes(searchString), \`Expected \${searchString} in stack: \${stack}\`)`,
 	tag: (
 		strings: TemplateStringsArray,
 		...values: string[]