Skip to content

Commit c377e35

Browse files
author
Brandon Dail
committed
Add expectation for non-functional listener warning in EventPluginHub
1 parent 40f3c08 commit c377e35

File tree

1 file changed

+10
-0
lines changed

1 file changed

+10
-0
lines changed

src/renderers/__tests__/EventPluginHub-test.js

Lines changed: 10 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -16,11 +16,13 @@ jest.mock('isEventSupported');
1616
describe('EventPluginHub', () => {
1717
var React;
1818
var ReactTestUtils;
19+
var ReactDOMFeatureFlags;
1920

2021
beforeEach(() => {
2122
jest.resetModules();
2223
React = require('react');
2324
ReactTestUtils = require('react-dom/test-utils');
25+
ReactDOMFeatureFlags = require('ReactDOMFeatureFlags');
2426
});
2527

2628
it('should prevent non-function listeners, at dispatch', () => {
@@ -34,6 +36,14 @@ describe('EventPluginHub', () => {
3436
}).toThrowError(
3537
'Expected onClick listener to be a function, instead got type string',
3638
);
39+
if (ReactDOMFeatureFlags.useFiber) {
40+
expectDev(console.error.calls.count()).toBe(1);
41+
expectDev(console.error.calls.argsFor(0)[0]).toContain(
42+
'Expected onClick listener to be a function, instead got type string',
43+
);
44+
} else {
45+
expectDev(console.error.calls.count()).toBe(0);
46+
}
3747
});
3848

3949
it('should not prevent null listeners, at dispatch', () => {

0 commit comments

Comments
 (0)