fix(sourcemaps): missing sourcemaps on debugger statement (#503)

davidbonnet · Apr 5, 2021 · 9822a2a · 9822a2a
1 parent d3e22f9
commit 9822a2a
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 5 deletions.
diff --git a/src/astring.js b/src/astring.js
@@ -476,7 +476,7 @@ export const GENERATOR = {
   }),
   ForOfStatement: ForInStatement,
   DebuggerStatement(node, state) {
-    state.write('debugger;' + state.lineEnd)
+    state.write('debugger;', node)
   },
   FunctionDeclaration: (FunctionDeclaration = function (node, state) {
     state.write(

diff --git a/src/tests/astring.js b/src/tests/astring.js
@@ -154,6 +154,10 @@ test('Source map generation', (assert) => {
     generate(ast, {
       sourceMap,
     })
+    assert.true(
+      sourceMap.mappings.length > 0,
+      `Expect ${path.basename(filename)} to have mappings`,
+    )
   })
 })
 

diff --git a/src/tests/fixtures/sourcemap-cases/strings.js b/src/tests/fixtures/sourcemap-cases/strings.js
diff --git a/src/tests/fixtures/syntax/debugger.js b/src/tests/fixtures/syntax/debugger.js
@@ -0,0 +1,2 @@
+debugger;
+"this string should be mapped to the second line";
Original file line number	Diff line number	Diff line change
		@@ -0,0 +1,2 @@
		debugger;
		"this string should be mapped to the second line";