[fix] - heading-has-content: dangerouslySetInnerHTML should pass

Fixes #64

Author: Ethan Cohen

docs/rules/heading-has-content.md

@@ -49,6 +49,7 @@ function Foo(props) {
 ```jsx
 <h1>Heading Content!</h1>
 <h1><TextWrapper /><h1>
+<h1 dangerouslySetInnerHTML={{ __html: 'foo' }} />
 ```
 
 ### Fail

src/rules/heading-has-content.js

@@ -7,7 +7,7 @@
 // Rule Definition
 // ----------------------------------------------------------------------------
 
-import { elementType } from 'jsx-ast-utils';
+import { elementType, hasProp } from 'jsx-ast-utils';
 import isHiddenFromScreenReader from '../util/isHiddenFromScreenReader';
 
 const errorMessage =
@@ -49,7 +49,8 @@ module.exports = context => ({
         default:
           return false;
       }
-    });
+    }) || hasProp(node.attributes, 'dangerouslySetInnerHTML');
+
 
     if (isAccessible) {
       return;

tests/src/rules/heading-has-content.js

@@ -41,6 +41,7 @@ ruleTester.run('heading-has-content', rule, {
     { code: '<h1><Bar /></h1>', parserOptions },
     { code: '<h1>{foo}</h1>', parserOptions },
     { code: '<h1>{foo.bar}</h1>', parserOptions },
+    { code: '<h1 dangerouslySetInnerHTML={{ __html: "foo" }} />', parserOptions },
   ],
   invalid: [
     { code: '<h1 />', errors: [expectedError], parserOptions },