Update self-closing-comp to check html tags by default

Author: yannickcr

docs/rules/self-closing-comp.md

@@ -22,13 +22,13 @@ var Profile = <Hello name="John"><img src="picture.png" /></Hello>;
 
 ## Rule Options
 
-The rule can take one argument to select types of tags, which should be self-closed when this is possible. By default only custom components tags should be self-closed.
+The rule can take one argument to select types of tags, which should be self-closed when this is possible. By default custom components tags and html tags should be self-closed.
 
 ```js
 ...
 "self-closing-comp": ["error", {
   "component": true,
-  "html": false
+  "html": true
 }]
 ...
 ```

lib/rules/self-closing-comp.js

@@ -24,7 +24,7 @@ module.exports = {
           type: 'boolean'
         },
         html: {
-          default: false,
+          default: true,
           type: 'boolean'
         }
       },
@@ -55,7 +55,7 @@ module.exports = {
     }
 
     function isShouldBeSelfClosed(node) {
-      var configuration = context.options[0] || {component: true};
+      var configuration = context.options[0] || {component: true, html: true};
       return (
         configuration.component && isComponent(node) ||
         configuration.html && isTagName(node.name.name)

tests/lib/rules/self-closing-comp.js

@@ -27,9 +27,6 @@ ruleTester.run('self-closing-comp', rule, {
 
   valid: [
     {
-      code: 'var contentContainer = <div className="content"></div>;',
-      parserOptions: parserOptions
-    }, {
       code: 'var HelloJohn = <Hello name="John" />;',
       parserOptions: parserOptions
     }, {
@@ -50,10 +47,6 @@ ruleTester.run('self-closing-comp', rule, {
     }, {
       code: 'var HelloJohn = <Hello name="John">&nbsp;</Hello>;',
       parserOptions: parserOptions
-    }, {
-      code: 'var contentContainer = <div className="content"></div>;',
-      options: [],
-      parserOptions: parserOptions
     }, {
       code: 'var HelloJohn = <Hello name="John" />;',
       options: [],
@@ -113,6 +106,19 @@ ruleTester.run('self-closing-comp', rule, {
 
   invalid: [
     {
+      code: 'var contentContainer = <div className="content"></div>;',
+      parserOptions: parserOptions,
+      errors: [{
+        message: 'Empty components are self-closing'
+      }]
+    }, {
+      code: 'var contentContainer = <div className="content"></div>;',
+      options: [],
+      parserOptions: parserOptions,
+      errors: [{
+        message: 'Empty components are self-closing'
+      }]
+    }, {
       code: 'var HelloJohn = <Hello name="John"></Hello>;',
       parserOptions: parserOptions,
       errors: [{