Merge remote-tracking branch 'upstream/master' into tag-failures

Conflicts:
	lib/index.js

Author: AsaAyers

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

@@ -12,6 +12,7 @@ var assertAccessibility = (tagName, props, children) => {
 
   var propTests;
   for (var propName in props) {
+    if (props[propName] === null || props[propName] === undefined) continue;
     propTests = assertions.props[propName] || [];
     for (key in propTests)
       if (propTests[key] && !propTests[key].test(tagName, props, children))