fix: remove sourceMappingUrl from prepends and appends

Author: 3cp

spec/shared.spec.js

@@ -78,9 +78,9 @@ test('contentOrFile rejects wrong remote url', t => {
 test('contentOrFile reads local js file', t => {
   const path = 'a.js';
 
-  contentOrFile(path, {readFile: buildReadFile({'a.js': 'var a;'})})
+  contentOrFile(path, {readFile: buildReadFile({'a.js': 'var a;\n//# sourceMappingURL=abc'})})
   .then(
-    result => t.equal(result.contents, 'var a;'),
+    result => t.equal(result.contents, 'var a;\n'),
     err => t.fail(err.message)
   )
   .then(t.end);

src/shared.js

@@ -53,28 +53,31 @@ export function contentOrFile(pathOrContent, mock) {
     return Promise.reject(new Error('No content or file provided'));
   }
 
+  let p;
   // pathOrContent is a path
   if (pathOrContent.match(/^https?:\/\//)) {
     // remote url
-    return fetch(pathOrContent)
+    p = fetch(pathOrContent)
     .then(response => {
       if (response.ok) return response.text();
       else throw new Error(response.statusText)
     })
     .then(text => {
       ensureParsed(text);
       // pathOrContent is code
-      return Promise.resolve({contents: text});
+      return text;
     });
   } else if (pathOrContent.endsWith('.js')) {
-    return _readFile(pathOrContent)
-    .then(buffer => ({contents: buffer.toString()}));
+    p = _readFile(pathOrContent)
+    .then(buffer => buffer.toString());
   } else {
-    return new Promise(resolve => {
+    p = new Promise(resolve => {
       ensureParsed(pathOrContent);
-      resolve({contents: pathOrContent});
+      resolve(pathOrContent);
     });
   }
+
+  return p.then(text => ({contents: stripSourceMappingUrl(text)}));
 }
 
 export function generateHash(bufOrStr) {