fix(tag-pair): point tag-pair error message to correct line and colum… (#1503)

* fix(tag-pair): point tag-pair error message to correct line and column number

fix #1284

* Update /dist/

---------

Co-authored-by: Christian Oliff <[email protected]>

Author: bebehr

dist/core/rules/tag-pair.js

@@ -1155,6 +1155,7 @@
 	                stack.push({
 	                    tagName: tagName,
 	                    line: event.line,
+	                    col: event.col,
 	                    raw: event.raw,
 	                });
 	            }
@@ -1174,7 +1175,7 @@
 	                }
 	                if (arrTags.length > 0) {
 	                    const lastEvent = stack[stack.length - 1];
-	                    reporter.error(`Tag must be paired, missing: [ ${arrTags.join('')} ], start tag match failed [ ${lastEvent.raw} ] on line ${lastEvent.line}.`, event.line, event.col, this, event.raw);
+	                    reporter.error(`Tag must be paired, missing: [ ${arrTags.join('')} ], start tag match failed [ ${lastEvent.raw} ] on line ${lastEvent.line}.`, lastEvent.line || event.line, lastEvent.col || event.col, this, event.raw);
 	                }
 	                stack.length = pos;
 	            }

dist/htmlhint.js

@@ -16,6 +16,7 @@ export default {
         stack.push({
           tagName: tagName,
           line: event.line,
+          col: event.col,
           raw: event.raw,
         })
       }
@@ -46,8 +47,8 @@ export default {
             )} ], start tag match failed [ ${lastEvent.raw} ] on line ${
               lastEvent.line
             }.`,
-            event.line,
-            event.col,
+            lastEvent.line || event.line,
+            lastEvent.col || event.col,
             this,
             event.raw
           )

dist/htmlhint.min.js

@@ -12,7 +12,7 @@ describe(`Rules: ${ruleId}`, () => {
     expect(messages.length).toBe(2)
     expect(messages[0].rule.id).toBe(ruleId)
     expect(messages[0].line).toBe(1)
-    expect(messages[0].col).toBe(9)
+    expect(messages[0].col).toBe(5)
     expect(messages[1].rule.id).toBe(ruleId)
     expect(messages[1].line).toBe(1)
     expect(messages[1].col).toBe(20)
@@ -24,6 +24,17 @@ describe(`Rules: ${ruleId}`, () => {
     expect(messages[0].col).toBe(9)
   })
 
+  it('No end tag should result in an error with correct line number and column of the start tag', () => {
+    const code = '<div>\r\n  <h1>\r\n    <p>aaa</p>\r\n</div>'
+    const messages = HTMLHint.verify(code, ruleOptions)
+    expect(messages.length).toBe(1)
+    expect(messages[0].rule.id).toBe(ruleId)
+    expect(messages[0].line).not.toBe(4)
+    expect(messages[0].col).not.toBe(1)
+    expect(messages[0].line).toBe(2)
+    expect(messages[0].col).toBe(3)
+  })
+
   it('No start tag should result in an error', () => {
     const code = '</div>'
     const messages = HTMLHint.verify(code, ruleOptions)