+change sourcemap testing way

Author: timofei-iatsenko

cases/babel/reference/build.js.map

@@ -0,0 +1,10 @@
+module.exports = [
+  {
+    original: { source: 'webpack:///file-to-annotate.js', line: 33, column: 2 },
+    generated: { line: 84, column: 3},
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.js', line: 41, column: 0 },
+    generated: { line: 95, column: 0 },
+  },
+];

cases/babel/reference/sourcemap-checkpoints.js

@@ -0,0 +1,10 @@
+module.exports = [
+  {
+    original: { source: 'webpack:///file-to-annotate.js', line: 24, column: 5 },
+    generated: { line: 71, column: 3},
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.js', line: 11, column: 5 },
+    generated: { line: 58, column: 3 },
+  },
+];

cases/simple/reference/build.js.map

@@ -0,0 +1,14 @@
+module.exports = [
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 6, column: 0 },
+    generated: { line: 50, column: 0},
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 16, column: 5 },
+    generated: { line: 58, column: 0 },
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 30, column: 3 },
+    generated: { line: 70, column: 0 },
+  },
+];

cases/simple/reference/sourcemap-checkpoints.js

@@ -0,0 +1,14 @@
+module.exports = [
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 6, column: 0 },
+    generated: { line: 1, column: 303},
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 16, column: 5 },
+    generated: { line: 1, column: 502 },
+  },
+  {
+    original: { source: 'webpack:///file-to-annotate.ts', line: 30, column: 3 },
+    generated: { line: 1, column: 253 },
+  },
+];

cases/typescript/reference/build.js.map

@@ -5,7 +5,7 @@ var SourceMapGenerator = require('source-map').SourceMapGenerator;
 var normalizePath = require('normalize-path');
 
 function loadPlugins(pluginNames) {
-  var pluginNames = pluginNames || [];
+  pluginNames = pluginNames || [];
   return pluginNames.map(function(name) {
     return require(name);
   });

cases/typescript/reference/sourcemap-checkpoints.js

@@ -3,13 +3,13 @@
 const test = require('tape');
 const webpack = require('webpack');
 const fs = require('fs');
-const crlf = require('crlf-helper');
+const SourceMapConsumer = require('source-map').SourceMapConsumer;
 
 test.createStream()
     .pipe(require('tap-spec')())
     .pipe(process.stdout);
 
-const cases = ['typescript', 'babel', 'simple', 'uglifyjs'];
+const cases = ['typescript','simple', 'babel', 'uglifyjs'];
 
 for (let testCase of cases) {
     test('Acceptance tests. Case ' + testCase, (t) => {
@@ -35,24 +35,43 @@ for (let testCase of cases) {
 
             t.equal(actualSource, expectedSource, 'Test annotated source');
 
-            const actualMap = prepareMap(fs.readFileSync(folder + '/dist/build.js.map', 'utf8'));
-            const expectedMap = prepareMap(fs.readFileSync(folder + '/reference/build.js.map', 'utf8'));
-
-          t.deepEqual(actualMap.sourcesContent, expectedMap.sourcesContent, 'Test source map sourceContent');
-          t.deepEqual(actualMap.sources, expectedMap.sources, 'Test source map sources');
-          t.equal(actualMap.mappings, expectedMap.mappings, 'Test source map mappings');
+            const actualMap = fs.readFileSync(folder + '/dist/build.js.map', 'utf8');
+            testMap(t, actualMap, require(folder + '/reference/sourcemap-checkpoints'));
         });
 
-        t.plan(4);
+        t.plan(2);
     });
 }
 
-function prepareMap(content){
-  const map = JSON.parse(content.replace(/webpack\/bootstrap [\d\w]+/g, 'webpack/bootstrap [hash]')); // remove hash from map
+/**
+ *
+ * @param t
+ * @param content
+ * @param {array<{original, generated}>} checkpoints
+ */
+function testMap(t, content, checkpoints){
+  t.test('Check source map cases', (t) => {
+    const rawMap = JSON.parse(content);
+    const map = new SourceMapConsumer(rawMap);
+
+    const sources = rawMap.sources.map((source) => {
+      const matches = source.match(/[^/]+\..+$/);
+      return matches ?  matches[0] : source;
+    });
+
+    t.equal(sources.length, uniq(sources).length, 'No duplicates in sourcemap sources');
 
-  map.sourcesContent = map.sourcesContent.map((sourceContent) => {
-    return crlf.setLineEnding(sourceContent, 'LF');
+    for(let point of checkpoints) {
+      const result = map.generatedPositionFor(point.original);
+
+      t.deepEqual({line: result.line, column: result.column}, point.generated);
+    }
   });
 
-  return map;
+  t.end();
+}
+
+
+function uniq(a) {
+  return Array.from(new Set(a));
 }