Merge pull request #1932 from romainmenke/fix-warning-end-index--inve…

…ntive-numbat-fbada42105 Fix resulting `end` position when `endIndex` is `0` for `rangeBy`
postcss · Mar 20, 2024 · c1ad8fb · c1ad8fb
2 parents 0fd1d86 + b45e7e9
commit c1ad8fb
Show file tree

Hide file tree

Showing 2 changed files with 19 additions and 1 deletion.
diff --git a/lib/node.js b/lib/node.js
@@ -241,7 +241,7 @@ class Node {
           column: opts.end.column,
           line: opts.end.line
         }
-      } else if (opts.endIndex) {
+      } else if (typeof opts.endIndex === 'number') {
         end = this.positionInside(opts.endIndex)
       } else if (opts.index) {
         end = this.positionInside(opts.index + 1)

diff --git a/test/warning.test.ts b/test/warning.test.ts
@@ -85,6 +85,24 @@ test('gets range from node without end', () => {
   is(warning.endColumn, 2)
 })
 
+test('gets range from node with endIndex 3', () => {
+  let root = parse('a{}')
+  let warning = new Warning('text', { node: root.first, index: 0, endIndex: 3 })
+  is(warning.line, 1)
+  is(warning.column, 1)
+  is(warning.endLine, 1)
+  is(warning.endColumn, 4)
+})
+
+test('gets range from node with endIndex 0', () => {
+  let root = parse('a{}')
+  let warning = new Warning('text', { node: root.first, index: 0, endIndex: 0 })
+  is(warning.line, 1)
+  is(warning.column, 1)
+  is(warning.endLine, 1)
+  is(warning.endColumn, 2)
+})
+
 test('gets range from word', () => {
   let root = parse('a b{}')
   let warning = new Warning('text', { node: root.first, word: 'b' })