Fix: don't use internal APIs of ESLint (fixes #171) (#172)

* Fix: don't use internal APIs of ESLint (fixes #171)

* Chore: add tests with old ESLint

Author: mysticatea

circle.yml

@@ -4,12 +4,12 @@ machine:
 
 dependencies:
   pre:
-    - nvm install 7
+    - nvm install 6
     - nvm install 8
 
 test:
   override:
-    - npm test
+    - nvm use 4 && npm test
     - nvm use 6 && npm test
-    - nvm use 7 && npm test
     - nvm use 8 && npm test
+    - nvm use 8 && npm i eslint@3.18.0 --no-save && npm run -s test:base

lib/rules/attribute-hyphenation.js

@@ -43,7 +43,7 @@ function create (context) {
   // Public
   // ----------------------------------------------------------------------
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     VAttribute (node) {
       if (!utils.isCustomComponent(node.parent.parent)) return
 
@@ -53,8 +53,6 @@ function create (context) {
       reportIssue(node, name)
     }
   })
-
-  return {}
 }
 
 module.exports = {

lib/rules/html-end-tags.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     VElement (node) {
       const name = node.name
       const isVoid = utils.isHtmlVoidElementName(name)
@@ -48,8 +48,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-no-self-closing.js

@@ -22,7 +22,7 @@ const utils = require('../utils')
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VElement' (node) {
       if (utils.isSvgElementNode(node)) {
         return
@@ -42,8 +42,6 @@ function create (context) {
       })
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-quotes.js

@@ -27,7 +27,7 @@ function create (context) {
   const quoteChar = double ? '"' : "'"
   const quoteName = double ? 'double quotes' : 'single quotes'
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VAttribute[value!=null]' (node) {
       const text = sourceCode.getText(node.value)
       const firstChar = text[0]
@@ -41,8 +41,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/html-self-closing.js

@@ -89,7 +89,7 @@ function create (context) {
   const sourceCode = context.getSourceCode()
   const options = parseOptions(context.options[0])
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VElement' (node) {
       const elementType = getElementType(node)
       const mode = options[elementType]
@@ -132,8 +132,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/max-attributes-per-line.js

@@ -71,7 +71,7 @@ module.exports = {
     const singlelinemMaximum = configuration.singleline
     const canHaveFirstLine = configuration.allowFirstLine
 
-    utils.registerTemplateBodyVisitor(context, {
+    return utils.defineTemplateBodyVisitor(context, {
       'VStartTag' (node) {
         const numberOfAttributes = node.attributes.length
 
@@ -155,7 +155,5 @@ module.exports = {
 
       return propsPerLine
     }
-
-    return {}
   }
 }

lib/rules/mustache-interpolation-spacing.js

@@ -65,7 +65,7 @@ module.exports = {
     // Public
     // ----------------------------------------------------------------------
 
-    utils.registerTemplateBodyVisitor(context, {
+    return utils.defineTemplateBodyVisitor(context, {
       'VExpressionContainer[expression!=null]' (node) {
         const tokens = template.getTokens(node, {
           includeComments: true,
@@ -85,7 +85,5 @@ module.exports = {
         }
       }
     })
-
-    return { }
   }
 }

lib/rules/no-confusing-v-for-v-if.js

@@ -37,7 +37,7 @@ function isUsingIterationVar (vIf) {
  * @returns {Object} AST event handlers.
  */
 function create (context) {
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     "VAttribute[directive=true][key.name='if']" (node) {
       const element = node.parent.parent
 
@@ -50,8 +50,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------

lib/rules/no-duplicate-attributes.js

@@ -51,7 +51,7 @@ function create (context) {
     return directiveNames.has(name) || attributeNames.has(name)
   }
 
-  utils.registerTemplateBodyVisitor(context, {
+  return utils.defineTemplateBodyVisitor(context, {
     'VStartTag' () {
       directiveNames.clear()
       attributeNames.clear()
@@ -78,8 +78,6 @@ function create (context) {
       }
     }
   })
-
-  return {}
 }
 
 // ------------------------------------------------------------------------------