Bug #34802
Tabbable latest version 5.3.1 is not compatible with jest dom/ JSDom without changes to PF4.
Difficulty:
Triaged:
No
Bugzilla link:
Pull request:
Description
From my debugging and googling, the latest version of Tabbable that is 5.3.0 (and onwards) needs some changes to the Modal/Wizard components in Patternfly4 to be able to support tests.
Specifically this: https://github.com/focus-trap/tabbable#testing-in-jsdom
Modal currently has the prop disableFocusTrap that can be set to false and that lets us work around the issue without ever activating the trap. I spoke to folks on PF4 slack and they lock the version of FocusTrap but Focustrap brings in the latest 5.x.x tabbable causing the test failures.
There's an open PF4 issue around this: https://github.com/patternfly/patternfly-react/issues/7288
Associated revisions
History
#1
Updated by The Foreman Bot 11 months ago
- Assignee set to Samir Jha
- Status changed from New to Ready For Testing
- Pull request https://github.com/theforeman/foreman/pull/9195 added
#2
Updated by The Foreman Bot 11 months ago
- Pull request https://github.com/theforeman/foreman-js/pull/398 added
#3
Updated by The Foreman Bot 11 months ago
- Fixed in Releases 3.3.0 added
#4
Updated by Anonymous 11 months ago
- Status changed from Ready For Testing to Closed
Applied in changeset foreman|86b63fd66254f5565000bac885dae0e947835952.
Fixes #34802 - Pin tabbable version