Bug 2078983 - Tabbable latest version 5.3.1 is not compatible with jest dom/ JSDom without changes to PF4.
Summary: Tabbable latest version 5.3.1 is not compatible with jest dom/ JSDom without ...
Alias: None
Product: Red Hat Satellite
Classification: Red Hat
Component: Provisioning
Version: 6.11.0
Hardware: Unspecified
OS: Unspecified
medium vote
Target Milestone: 6.11.0
Assignee: satellite6-bugs
QA Contact: Akhil Jha
Depends On:
TreeView+ depends on / blocked
Reported: 2022-04-26 16:02 UTC by Samir Jha
Modified: 2022-07-05 14:35 UTC (History)
3 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Last Closed: 2022-07-05 14:35:13 UTC
Target Upstream Version:

Attachments (Terms of Use)

System ID Private Priority Status Summary Last Updated
Foreman Issue Tracker 34802 0 Normal Closed Tabbable latest version 5.3.1 is not compatible with jest dom/ JSDom without changes to PF4. 2022-04-26 16:02:49 UTC
Red Hat Product Errata RHSA-2022:5498 0 None None None 2022-07-05 14:35:34 UTC

Description Samir Jha 2022-04-26 16:02:48 UTC
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

Comment 1 Samir Jha 2022-04-26 16:02:52 UTC
Created from redmine issue https://projects.theforeman.org/issues/34802

Comment 2 Samir Jha 2022-04-26 16:02:54 UTC
Upstream bug assigned to None

Comment 3 Samir Jha 2022-04-26 16:06:13 UTC
This will need to be cherry-picked to 6.11 to resolve the pipeline errors seen here: https://master-jenkins-csb-satellite-delivery.apps.ocp4.prod.psi.redhat.com/blue/organizations/jenkins/sat6-unit-test-katello/detail/sat6-unit-test-katello/722/pipeline

Comment 4 Bryan Kearney 2022-04-26 20:05:12 UTC
Moving this bug to POST for triage into Satellite since the upstream issue https://projects.theforeman.org/issues/34802 has been resolved.

Comment 9 errata-xmlrpc 2022-07-05 14:35:13 UTC
Since the problem described in this bug report should be
resolved in a recent advisory, it has been closed with a
resolution of ERRATA.

For information on the advisory (Moderate: Satellite 6.11 Release), and where to find the updated
files, follow the link below.

If the solution does not work for you, open a new bug report.


Note You need to log in before you can comment on or make changes to this bug.