feat: inline sourcesContent for all npm package sourceMap

Author: 3cp

spec/fib.wasm spec/fixtures/fib.wasmspec/fib.wasm renamed to spec/fixtures/fib.wasm

@@ -0,0 +1,10 @@
+import _ from 'lodash';
+export default class X {
+  constructor() {
+    this.name = 'Hello';
+  }
+
+  description() {
+    return _.trim(this.name);
+  }
+}

spec/fixtures/map-file-comment.css

@@ -236,7 +236,7 @@ test('defaultNpmPackageLocator returns fileRead func rejects missing file for ex
 
 test('defaultNpmPackageLocator can read .wasm file into base64 string', t => {
   const defaultLocator = mockLocator(buildReadFile({
-    'node_modules/foo/fib.wasm': fs.readFileSync(path.join(__dirname, '..', 'fib.wasm'))
+    'node_modules/foo/fib.wasm': fs.readFileSync(path.join(__dirname, '..', 'fixtures', 'fib.wasm'))
   }));
 
   defaultLocator({name: 'foo'})

spec/map-file-comment.css.map spec/fixtures/map-file-comment.css.mapspec/map-file-comment.css.map renamed to spec/fixtures/map-file-comment.css.map

@@ -0,0 +1,25 @@
+import fs from 'fs';
+import path from 'path';
+import test from 'tape';
+import {getSourceMap} from '../src/shared';
+
+// no test in browser
+test('getSourceMap gets referenced sourceMap file content', t => {
+  const filePath = path.resolve(__dirname, 'fixtures', 'y.js');
+  const contents = fs.readFileSync(filePath, 'utf8');
+  const sourceMap = getSourceMap(contents, filePath);
+  t.ok(sourceMap);
+  t.deepEqual(sourceMap.sources, ['src/x.js']);
+  t.ok(sourceMap.sourcesContent);
+  t.equal(sourceMap.sourcesContent.length, 1);
+  t.ok(sourceMap.sourcesContent[0].includes('_.trim'));
+  t.end();
+});
+
+test('getSourceMap ignores missing sourceMap file', t => {
+  const filePath = path.resolve(__dirname, 'fixtures', 'missing.js');
+  const contents = fs.readFileSync(filePath, 'utf8');
+  const sourceMap = getSourceMap(contents, filePath);
+  t.notOk(sourceMap);
+  t.end();
+});

spec/fixtures/missing.js

@@ -148,7 +148,7 @@ test('getSourceMap ignores file without sourceMap', t => {
 
 test('getSourceMap gets inline sourceMap', t => {
   const contents = 'var a = 1;\n//# sourceMappingURL=data:application/json;base64,eyJ2ZXJzaW9uIjozLCJmaWxlIjoiYnVpbGQvZm9vLm1pbi5qcyIsInNvdXJjZXMiOlsic3JjL2Zvby5qcyJdLCJuYW1lcyI6W10sIm1hcHBpbmdzIjoiQUFBQSIsInNvdXJjZVJvb3QiOiIvIn0=';
-  const sourceMap = {"version":3,"file":"build/foo.min.js","sources":["src/foo.js"],"names":[],"mappings":"AAAA","sourceRoot":"/"};
+  const sourceMap = {"version":3,"file":"build/foo.min.js","sources":["src/foo.js"],"names":[],"mappings":"AAAA"};
   t.deepEqual(getSourceMap(contents, 'foo.js'), sourceMap);
   t.end();
 });
@@ -158,55 +158,3 @@ test('getSourceMap ignores broken inline sourceMap', t => {
   t.notOk(getSourceMap(contents, 'foo.js'));
   t.end();
 });
-
-if (typeof global.document === 'undefined') {
-  // no test in browser
-  test('getSourceMap gets referenced sourceMap file content', t => {
-    let contents = `
-  .header {
-    background: #444;
-    border: solid;
-    padding: 10px;
-    border-radius: 10px 5px 10px 5px;
-    color: #b4b472; }
-
-  #main li {
-    color: green;
-    margin: 10px;
-    padding: 10px;
-    font-size: 18px; }
-
-  `;
-    contents += '\n//# sourceMappingURL=map-file-comment.css.map\n';
-    const sourceMap = {
-      "version": "3",
-      "mappings": "AAAA,wBAAyB;EACvB,UAAU,EAAE,IAAI;EAChB,MAAM,EAAE,KAAK;EACb,OAAO,EAAE,IAAI;EACb,aAAa,EAAE,iBAAiB;EAChC,KAAK,EAAE,OAAkB;;AAG3B,wBAAyB;EACvB,OAAO,EAAE,IAAI;;ACTf,gBAAiB;EACf,UAAU,EAAE,IAAI;EAChB,KAAK,EAAE,MAAM;;AAGf,kBAAmB;EACjB,MAAM,EAAE,IAAI;EACZ,OAAO,EAAE,IAAI;EACb,UAAU,EAAE,KAAK;EACjB,aAAa,EAAE,GAAG;EAClB,KAAK,EAAE,KAAK;;AAEd,kBAAmB;EACjB,KAAK,EAAE,KAAK;;AAGd,mBAAoB;EAClB,KAAK,EAAE,KAAK;EACZ,MAAM,EAAE,IAAI;EACZ,OAAO,EAAE,IAAI;EACb,SAAS,EAAE,IAAI",
-      "sources": ["./client/sass/core.scss","./client/sass/main.scss"],
-      "file": "map-file-comment.css"
-    };
-
-    t.deepEqual(getSourceMap(contents, 'spec/map-file-comment.css'), sourceMap);
-    t.end();
-  });
-
-  test('getSourceMap ignores missing sourceMap file', t => {
-    let contents = `
-  .header {
-    background: #444;
-    border: solid;
-    padding: 10px;
-    border-radius: 10px 5px 10px 5px;
-    color: #b4b472; }
-
-  #main li {
-    color: green;
-    margin: 10px;
-    padding: 10px;
-    font-size: 18px; }
-
-  `;
-    contents += '\n//# sourceMappingURL=missing.css.map\n';
-    t.notOk(getSourceMap(contents, 'spec/map-file-comment.css'));
-    t.end();
-  });
-}