fix ignore range

cenfun · Jan 28, 2024 · f10a284 · f10a284
1 parent c6d6b41
commit f10a284
Show file tree

Hide file tree

Showing 4 changed files with 10 additions and 10 deletions.
diff --git a/lib/converter/ignore.js b/lib/converter/ignore.js
@@ -42,7 +42,7 @@ const addNextIgnore = (list, content, item, locator) => {
     const nextItem = {
         type: 'next',
         n,
-        startOffset: item.end,
+        startOffset: item.start,
         endOffset
     };
 
@@ -93,7 +93,7 @@ const getIgnoredRanges = (locator, options) => {
         if (ignoreStart) {
             // v8 ignore stop
             if (content === 'stop') {
-                ignoreStart.endOffset = start;
+                ignoreStart.endOffset = end;
                 ignoreStart = null;
             }
             return;
@@ -104,8 +104,8 @@ const getIgnoredRanges = (locator, options) => {
             // add first for sort by start
             ignoreStart = {
                 type: 'start',
-                startOffset: end,
-                endOffset: end
+                startOffset: start,
+                endOffset: start
             };
             list.push(ignoreStart);
             return;

diff --git a/test/mock/src/ignore/v8-ignore-next.js b/test/mock/src/ignore/v8-ignore-next.js
@@ -1,21 +1,21 @@
 const a = 99
 const b = true ? 1 /* v8 ignore next */ : 2
-if (true) {
+if (a + b) {
   console.info('covered')
 /* v8 ignore next 3 */
 } else {
   console.info('uncovered')
 }
 
 /* v8 ignore next */
-if (false) console.info('uncovered')
+if (a < b) console.info('uncovered')
 
 /* v8 ignore next 3 */
 function notExecuted () {
 
 }
 
-if (true) {
+if (a + b) {
   console.info('covered')
 } else { /* c8 ignore next */
   console.info('uncovered')

diff --git a/test/mock/src/ignore/v8-ignore-return.js b/test/mock/src/ignore/v8-ignore-return.js
@@ -1,6 +1,6 @@
 function fn() {
   return true;
-  /* v8 ignore next */
+      /* v8 ignore next */
   console.log('never runs');
 }
 

diff --git a/test/mock/src/ignore/v8-ignore-start-stop.js b/test/mock/src/ignore/v8-ignore-start-stop.js
@@ -1,6 +1,6 @@
 const a = 99
 const b = true ? 1 : 2
-if (true) {
+if (a + b) {
   console.info('covered')
 /* v8 ignore start */
 } else {
@@ -14,7 +14,7 @@ function notExecuted () {
 }
 /* v8 ignore stop */
 
-if (true) {
+if (a + b) {
   console.info('covered')
 } else { /* v8 ignore start */
   console.info('uncovered')