fix: the line with the ignore comment itself should be skipped

istanbuljs · May 2, 2019 · cce4bd5 · cce4bd5
1 parent 7ce4bb1
commit cce4bd5
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 1 deletion.
diff --git a/lib/source.js b/lib/source.js
@@ -28,6 +28,7 @@ module.exports = class CovSource {
   _parseIgnoreNext (lineStr, line) {
     const testIgnoreNextLines = lineStr.match(/^\W*\/\* c8 ignore next (?<count>[0-9]+)? *\*\/\W*$/)
     if (testIgnoreNextLines) {
+      line.ignore = true
       if (testIgnoreNextLines.groups['count']) {
         return Number(testIgnoreNextLines.groups['count'])
       } else {

diff --git a/test/source.js b/test/source.js
@@ -42,7 +42,8 @@ describe('Source', () => {
       const a = 99
       `
       const source = new CovSource(sourceRaw, 0)
-      source.lines[2].ignore.should.equal(false)
+      source.lines[1].ignore.should.equal(false)
+      source.lines[2].ignore.should.equal(true)
       source.lines[3].ignore.should.equal(true)
     })
 
@@ -53,6 +54,7 @@ describe('Source', () => {
       const a = 99
       `
       const source = new CovSource(sourceRaw, 0)
+      source.lines[1].ignore.should.equal(true)
       source.lines[2].ignore.should.equal(true)
       source.lines[3].ignore.should.equal(true)
     })