🐛 fuzzy sourcemap lookup

Author: philschatz

README.md

@@ -35,17 +35,14 @@ You can also generate LCOV Files for Coveralls or just HTML reports:
 ```txt
 Usage: css-coverage [options]
 
-Generate coverage info for a CSS file against an HTML file.
-This supports loading sourcemaps by using the sourceMappingURL=FILENAME.map CSS comment
+Generate coverage info for a CSS file against an HTML file. This supports loading sourcemaps by using the sourceMappingURL=FILENAME.map CSS comment
 
 Options:
-
-  -h, --help                    output usage information
-  --html [path/to/file.html]    path to a local HTML file
-  --css [path/to/file.css]      path to a local CSS file
-  --lcov [path/to/output.lcov]  the LCOV output file
-  --verbose                     verbose/debugging output
-  --ignore-source-map           disable loading the sourcemap if one is found
-  --cover-declarations          try to cover CSS declarations as well as selectors
-                                (best-effort, difficult with sourcemaps)
+  --html [path/to/file.html]               path to a local HTML file
+  --css [path/to/file.css]                 path to a local CSS file
+  --lcov [path/to/output.lcov]             the LCOV output file
+  --verbose                                verbose/debugging output
+  --ignore-source-map                      disable loading the sourcemap if one is found
+  --ignore-declarations [move-to,content]  A comma-separated list of declarations to ignore
+  -h, --help                               output usage information
 ```

package.json

@@ -3,8 +3,8 @@
   "version": "2.0.2",
   "scripts": {
     "pretest": "sass --source-map ./test/test.scss ./test/test.css",
-    "test": "./bin/css-coverage.js --html ./test/test.html --css ./test/test.css --lcov ./test/test.lcov --ignore-declarations 'move-to,move-foobar'",
-    "test-debug": "node --inspect-brk ./bin/css-coverage.js --cover-declarations --html ./test/test.html --css ./test/test.css --lcov ./test/test.lcov --ignore-declarations 'move-to,move-foobar'",
+    "test": "./bin/css-coverage.js                          --html ./test/test.html --css ./test/test.css --lcov ./test/test.lcov --ignore-declarations 'move-to,move-foobar'",
+    "test:debug": "node --inspect-brk ./bin/css-coverage.js --html ./test/test.html --css ./test/test.css --lcov ./test/test.lcov --ignore-declarations 'move-to,move-foobar'",
     "posttest": "standard --fix"
   },
   "dependencies": {

src/runCoverage.js

@@ -275,10 +275,14 @@ async function generateLcovStr (coverageOutput, supportedDeclarations) {
   }
 
   function getStartInfo (origStart, origEnd) {
-    const startInfo = sourceMapConsumer.originalPositionFor({ line: origStart.line, column: origStart.column - 1 })
+    let startInfo = sourceMapConsumer.originalPositionFor({ line: origStart.line, column: origStart.column - 1 })
     // const endInfo = sourceMapConsumer.originalPositionFor({line: origEnd.line, column: origEnd.column - 2})
 
-    // When there is no match, startInfo.source is null
+    // When there is no match, startInfo.source is null.
+    // Try fiddling with the column
+    if (!startInfo.source) {
+      startInfo = sourceMapConsumer.originalPositionFor({ line: origStart.line, column: origStart.column })
+    }
     if (!startInfo.source /* || startInfo.source !== endInfo.source */) {
       console.error('cssStart', JSON.stringify(origStart))
       origEnd && console.error('cssEnd', JSON.stringify(origEnd))