Merge pull request #17 from courseload/skip-null-props

Skip processing props that are null or undefined

Author: ryanflorence

lib/__tests__/index-test.js

@@ -35,6 +35,18 @@ describe('props', () => {
         });
       });
 
+      it('does not warn if onClick is null', () => {
+        doNotExpectWarning(assertions.props.onClick.NO_LABEL.msg, () => {
+          <div onClick={null}/>;
+        });
+      });
+
+      it('does not warn if onClick is undefined', () => {
+        doNotExpectWarning(assertions.props.onClick.NO_LABEL.msg, () => {
+          <div onClick={undefined}/>;
+        });
+      });
+
       it('does not warn if there is an aria-label', () => {
         doNotExpectWarning(assertions.props.onClick.NO_LABEL.msg, () => {
           <div aria-label="foo" onClick={k}/>;

lib/index.js

@@ -11,6 +11,7 @@ var assertAccessibility = (tagName, props, children, log) => {
 
   var propTests;
   for (var propName in props) {
+    if (props[propName] === null || props[propName] === undefined) continue;
     propTests = assertions.props[propName];
     if (propTests)
       for (key in propTests)
@@ -39,4 +40,3 @@ module.exports = (options) => {
     return _createElement.apply(this, arguments);
   };
 };
-