Regression v11.1.0 -> v11.2.0
See original GitHub issueBasic info:
- Node.js version: v6.11.2
- jsdom version: v11.2.0
Minimal reproduction case
git clone git@github.com:callemall/material-ui.git
cd material-ui
git checkout a1a17995202db7f669451ad41690fc5e126a770c
yarn
yarn test:unit
or even better: https://github.com/callemall/material-ui/commit/a1a17995202db7f669451ad41690fc5e126a770c
Error: Uncaught [TypeError: Illegal invocation] at reportException (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/helpers/runtime-script-errors.js:63:24) at invokeEventListeners (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:211:9) at HTMLLIElementImpl._dispatch (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/events/EventTarget-impl.js:111:7) at Object.exports.fireFocusEventWithTargetAdjustment (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/helpers/focusing.js:73:10) at HTMLLIElementImpl.focus (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/nodes/HTMLElement-impl.js:65:14) at HTMLLIElement.focus (/tmp/material-ui/node_modules/jsdom/lib/jsdom/living/generated/HTMLElement.js:49:21) at Object.Menu._this.handleEnter [as onEnter] (/tmp/material-ui/src/Menu/Menu.js:113:27) at Object.Popover._this.handleEnter [as onEnter] (/tmp/material-ui/src/internal/Popover.js:228:18) at Object.Grow._this.handleEnter [as onEnter] (/tmp/material-ui/src/transitions/Grow.js:81:18) at Transition.performEnter (/tmp/material-ui/src/internal/Transition.js:235:15) at Transition.componentDidMount (/tmp/material-ui/src/internal/Transition.js:167:12)
How does similar code behave in browsers?
It’s working on IE11, Firefox 45, Safari 10.1, Edge 14, Chrome 49, jsdom v11.1.0 but fails with jsdom v11.2.0.
Issue Analytics
- State:
- Created 6 years ago
- Comments:7 (3 by maintainers)
@TimothyGu Thanks for the fix 👏 !
See https://github.com/tmpvar/jsdom/pull/1979.