process: add coverage tests for sourceMapFromDataUrl method

PR-URL: #30319 Reviewed-By: Anna Henningsen <anna@addaleax.net> Reviewed-By: Ben Coe <bencoe@gmail.com> Reviewed-By: Ruben Bridgewater <ruben@bridgewater.de> Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com> Reviewed-By: Masashi Hirano <shisama07@gmail.com>
nodejs · Dec 1, 2019 · 75cb026 · 75cb026
1 parent 0828154
commit 75cb026
Show file tree

Hide file tree

Showing 3 changed files with 36 additions and 0 deletions.
diff --git a/test/fixtures/source-map/inline-base64-json-error.js b/test/fixtures/source-map/inline-base64-json-error.js
diff --git a/test/fixtures/source-map/inline-base64-type-error.js b/test/fixtures/source-map/inline-base64-type-error.js
diff --git a/test/parallel/test-source-map.js b/test/parallel/test-source-map.js
@@ -116,6 +116,38 @@ function nextdir() {
   );
 }
 
+// base64 encoding error does not crash application.
+{
+  const coverageDirectory = nextdir();
+  const output = spawnSync(process.execPath, [
+    require.resolve('../fixtures/source-map/inline-base64-type-error.js')
+  ], { env: { ...process.env, NODE_V8_COVERAGE: coverageDirectory } });
+  assert.strictEqual(output.status, 0);
+  assert.strictEqual(output.stderr.toString(), '');
+  const sourceMap = getSourceMapFromCache(
+    'inline-base64-type-error.js',
+    coverageDirectory
+  );
+
+  assert.strictEqual(sourceMap.data, null);
+}
+
+// JSON error does not crash application.
+{
+  const coverageDirectory = nextdir();
+  const output = spawnSync(process.execPath, [
+    require.resolve('../fixtures/source-map/inline-base64-json-error.js')
+  ], { env: { ...process.env, NODE_V8_COVERAGE: coverageDirectory } });
+  assert.strictEqual(output.status, 0);
+  assert.strictEqual(output.stderr.toString(), '');
+  const sourceMap = getSourceMapFromCache(
+    'inline-base64-json-error.js',
+    coverageDirectory
+  );
+
+  assert.strictEqual(sourceMap.data, null);
+}
+
 // Does not apply source-map to stack trace if --experimental-modules
 // is not set.
 {