Fix: no-invalid-template-root has false positive about empty templates (fixes #41)

Author: mysticatea

lib/rules/no-invalid-template-root.js

@@ -65,12 +65,6 @@ function create (context) {
           loc: extraElement.loc,
           message: 'The template root requires exactly one element.'
         })
-      } else if (rootElements.length === 0) {
-        context.report({
-          node,
-          loc: node.loc,
-          message: 'The template root requires exactly one element.'
-        })
       } else {
         for (const element of rootElements) {
           const tag = element.startTag
@@ -92,14 +86,6 @@ function create (context) {
             })
           }
         }
-        if (vIf) {
-          const tag = rootElements[0].startTag
-          context.report({
-            node: tag,
-            loc: tag.loc,
-            message: "The template root requires the next element which has 'v-else' directives if it has 'v-if' directives."
-          })
-        }
       }
     }
   }

tests/lib/rules/no-invalid-template-root.js

@@ -50,14 +50,21 @@ tester.run('no-invalid-template-root', rule, {
     {
       filename: 'test.vue',
       code: `<template>\n    <c1 v-if="1" />\n    <c2 v-else-if="1" />\n    <c3 v-else />\n</template>`
-    }
-  ],
-  invalid: [
+    },
     {
       filename: 'test.vue',
-      code: '<template>\n</template>',
-      errors: ['The template root requires exactly one element.']
+      code: '<template>\n</template>'
     },
+    {
+      filename: 'test.vue',
+      code: '<template><div v-if="foo"></div></template>'
+    },
+    {
+      filename: 'test.vue',
+      code: '<template><div v-if="foo"></div><div v-else-if="bar"></div></template>'
+    }
+  ],
+  invalid: [
     {
       filename: 'test.vue',
       code: '<template><div></div><div></div></template>',
@@ -88,16 +95,6 @@ tester.run('no-invalid-template-root', rule, {
       code: '<template><div v-for="x in list"></div></template>',
       errors: ["The template root disallows 'v-for' directives."]
     },
-    {
-      filename: 'test.vue',
-      code: '<template><div v-if="foo"></div></template>',
-      errors: ["The template root requires the next element which has 'v-else' directives if it has 'v-if' directives."]
-    },
-    {
-      filename: 'test.vue',
-      code: '<template><div v-if="foo"></div><div v-else-if="bar"></div></template>',
-      errors: ["The template root requires the next element which has 'v-else' directives if it has 'v-if' directives."]
-    },
     {
       filename: 'test.vue',
       code: '<template><slot></slot></template>',