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

…n number

fix htmlhint#1284

src/core/rules/tag-pair.ts

@@ -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
           )

test/rules/tag-pair.spec.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)