From 4ec37e0995cafdd8d5165d20e2c015a970e899e4 Mon Sep 17 00:00:00 2001 From: gitdallas Date: Wed, 17 May 2023 12:42:26 -0500 Subject: [PATCH 1/6] fix(Multiple): update old component classnames --- .../AboutModal/AboutModalBoxContent.tsx | 2 +- .../AboutModalBoxContent.test.tsx.snap | 2 +- .../Accordion/__tests__/Accordion.test.tsx | 2 +- .../__tests__/AccordionContent.test.tsx | 2 +- .../AccordionExpandedContentBody.test.tsx | 2 +- .../__tests__/AccordionToggle.test.tsx | 6 +- .../components/ActionList/ActionListItem.tsx | 2 +- .../ActionList/__tests__/ActionList.test.tsx | 2 +- .../__tests__/ActionListGroup.test.tsx | 2 +- .../__tests__/ActionListItem.test.tsx | 4 +- .../ActionListItem.test.tsx.snap | 2 +- .../components/Alert/__tests__/Alert.test.tsx | 12 +-- .../Alert/__tests__/AlertIcon.test.tsx | 2 +- .../AlertToggleExpandButton.test.tsx | 2 +- .../Alert/examples/AlertDynamicLiveRegion.tsx | 2 +- .../Alert/examples/AlertGroupAsync.tsx | 2 +- .../examples/AlertGroupMultipleDynamic.tsx | 2 +- .../examples/AlertGroupSingularDynamic.tsx | 2 +- .../AlertGroupSingularDynamicOverflow.tsx | 2 +- .../Alert/examples/AlertGroupToast.tsx | 2 +- .../AlertGroupToastOverflowCapture.tsx | 2 +- .../Backdrop/__tests__/Backdrop.test.tsx | 4 +- .../components/Badge/__tests__/Badge.test.tsx | 2 +- .../Banner/__tests__/Banner.test.tsx | 2 +- .../react-core/src/components/Brand/Brand.tsx | 2 +- .../components/CodeBlock/CodeBlockAction.tsx | 2 +- .../__snapshots__/CodeBlock.test.tsx.snap | 4 +- .../DescriptionListDescription.tsx | 2 +- .../DescriptionList.test.tsx.snap | 2 +- .../DualListSelectorControl.tsx | 2 +- .../DualListSelectorControlsWrapper.tsx | 6 +- .../DualListSelectorListWrapper.tsx | 14 +-- .../DualListSelector/DualListSelectorPane.tsx | 2 +- .../DualListSelector.test.tsx.snap | 72 +++++++-------- .../EmptyState/EmptyStateHeader.tsx | 4 +- .../__snapshots__/EmptyState.test.tsx.snap | 22 ++--- .../src/components/Form/FormAlert.tsx | 4 +- .../__snapshots__/FormAlert.test.tsx.snap | 2 +- .../components/Hint/__tests__/Hint.test.tsx | 4 +- .../Hint/__tests__/HintBody.test.tsx | 2 +- .../Hint/__tests__/HintFooter.test.tsx | 2 +- .../Hint/__tests__/HintTitle.test.tsx | 2 +- .../src/components/JumpLinks/JumpLinks.tsx | 2 +- .../__snapshots__/JumpLinks.test.tsx.snap | 10 +-- .../react-core/src/components/Menu/Menu.tsx | 34 ++++--- .../src/components/Menu/MenuGroup.tsx | 2 +- .../src/components/Menu/MenuItem.tsx | 4 +- .../src/components/Menu/MenuSearchInput.tsx | 2 +- .../MultipleFileUploadStatus.tsx | 2 +- .../MultipleFileUploadStatus.test.tsx.snap | 8 +- .../react-core/src/components/Nav/NavItem.tsx | 6 +- .../NotificationDrawerList.tsx | 2 +- .../NotificationDrawerList.test.tsx.snap | 4 +- .../Page/__tests__/PageSidebarBody.test.tsx | 2 +- .../components/Panel/__tests__/Panel.test.tsx | 4 +- .../Panel/__tests__/PanelFooter.test.tsx | 4 +- .../Panel/__tests__/PanelHeader.test.tsx | 4 +- .../Panel/__tests__/PanelMain.test.tsx | 4 +- .../Panel/__tests__/PanelMainBody.test.tsx | 4 +- .../__tests__/ProgressHelperText.test.tsx | 2 +- .../__tests__/ProgressStep.test.tsx | 8 +- .../__tests__/ProgressStepper.test.tsx | 4 +- .../Sidebar/__tests__/Sidebar.test.tsx | 4 +- .../Sidebar/__tests__/SidebarContent.test.tsx | 4 +- .../Sidebar/__tests__/SidebarPanel.test.tsx | 4 +- .../src/components/Tabs/TabContent.tsx | 4 +- .../react-core/src/components/Tabs/Tabs.tsx | 2 +- .../__snapshots__/TabContent.test.tsx.snap | 2 +- .../Tabs/__tests__/OverflowTab.test.tsx | 6 +- .../__snapshots__/Tabs.test.tsx.snap | 90 +++++++++---------- .../Text/__tests__/TextContent.test.tsx | 2 +- .../__tests__/TextInputGroup.test.tsx | 2 +- .../__tests__/TextInputGroupMain.test.tsx | 8 +- .../TextInputGroupUtilities.test.tsx | 2 +- .../src/components/Timestamp/Timestamp.tsx | 2 +- .../Timestamp/__tests__/Timestamp.test.tsx | 6 +- .../components/Title/__tests__/Title.test.tsx | 4 +- .../src/components/TreeView/TreeViewList.tsx | 2 +- .../__snapshots__/TreeView.test.tsx.snap | 40 ++++----- .../Truncate/__tests__/Truncate.test.tsx | 4 +- 80 files changed, 257 insertions(+), 259 deletions(-) diff --git a/packages/react-core/src/components/AboutModal/AboutModalBoxContent.tsx b/packages/react-core/src/components/AboutModal/AboutModalBoxContent.tsx index 15faa389c35..0c37db076ca 100644 --- a/packages/react-core/src/components/AboutModal/AboutModalBoxContent.tsx +++ b/packages/react-core/src/components/AboutModal/AboutModalBoxContent.tsx @@ -19,7 +19,7 @@ export const AboutModalBoxContent: React.FunctionComponent (
-
+
{hasNoContentContainer ? children :
{children}
}

{trademark}

diff --git a/packages/react-core/src/components/AboutModal/__tests__/__snapshots__/AboutModalBoxContent.test.tsx.snap b/packages/react-core/src/components/AboutModal/__tests__/__snapshots__/AboutModalBoxContent.test.tsx.snap index 722caf78b74..8bd73134f9a 100644 --- a/packages/react-core/src/components/AboutModal/__tests__/__snapshots__/AboutModalBoxContent.test.tsx.snap +++ b/packages/react-core/src/components/AboutModal/__tests__/__snapshots__/AboutModalBoxContent.test.tsx.snap @@ -7,7 +7,7 @@ exports[`AboutModalBoxContent Test 1`] = ` id="id" >
{ expect(screen.getByText('Test')).toHaveAccessibleName('Label'); }); -test('Renders with class name pf-c-accordion', () => { +test('Renders with class name pf-v5-c-accordion', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-accordion'); diff --git a/packages/react-core/src/components/Accordion/__tests__/AccordionContent.test.tsx b/packages/react-core/src/components/Accordion/__tests__/AccordionContent.test.tsx index 9ef10489638..74a2b1a33a9 100644 --- a/packages/react-core/src/components/Accordion/__tests__/AccordionContent.test.tsx +++ b/packages/react-core/src/components/Accordion/__tests__/AccordionContent.test.tsx @@ -85,7 +85,7 @@ test('Renders with inherited element props spread to the component', () => { expect(screen.getByRole('heading')).toHaveAccessibleName('Label'); }); -test('Renders with class name pf-c-accordion__expandable-content', () => { +test('Renders with class name pf-v5-c-accordion__expandable-content', () => { render( Test diff --git a/packages/react-core/src/components/Accordion/__tests__/AccordionExpandedContentBody.test.tsx b/packages/react-core/src/components/Accordion/__tests__/AccordionExpandedContentBody.test.tsx index 161a31ae0cd..de1481577d7 100644 --- a/packages/react-core/src/components/Accordion/__tests__/AccordionExpandedContentBody.test.tsx +++ b/packages/react-core/src/components/Accordion/__tests__/AccordionExpandedContentBody.test.tsx @@ -19,7 +19,7 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class name pf-c-accordion__expandable-content-body', () => { +test('Renders with class name pf-v5-c-accordion__expandable-content-body', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-accordion__expandable-content-body'); diff --git a/packages/react-core/src/components/Accordion/__tests__/AccordionToggle.test.tsx b/packages/react-core/src/components/Accordion/__tests__/AccordionToggle.test.tsx index 4c27f4e2782..2beb7c10bce 100644 --- a/packages/react-core/src/components/Accordion/__tests__/AccordionToggle.test.tsx +++ b/packages/react-core/src/components/Accordion/__tests__/AccordionToggle.test.tsx @@ -42,7 +42,7 @@ test('Renders with inherited element props spread to the component', () => { expect(screen.getByRole('button')).toHaveAccessibleName('Label'); }); -test('Renders the accordion toggle with class pf-c-accordion__toggle', () => { +test('Renders the accordion toggle with class pf-v5-c-accordion__toggle', () => { render( @@ -66,7 +66,7 @@ test('Renders the accordion toggle with custom class names provided via prop', ( expect(screen.getByRole('button')).toHaveClass('test-class'); }); -test('Renders with children inside class pf-c-accordion__toggle-text', () => { +test('Renders with children inside class pf-v5-c-accordion__toggle-text', () => { render( @@ -78,7 +78,7 @@ test('Renders with children inside class pf-c-accordion__toggle-text', () => { expect(screen.getByText('Test')).toHaveClass('pf-v5-c-accordion__toggle-text'); }); -test('Renders with the toggle icon inside class pf-c-accordion__toggle-icon', () => { +test('Renders with the toggle icon inside class pf-v5-c-accordion__toggle-icon', () => { render( diff --git a/packages/react-core/src/components/ActionList/ActionListItem.tsx b/packages/react-core/src/components/ActionList/ActionListItem.tsx index ac796de70fc..da24cc5d6cb 100644 --- a/packages/react-core/src/components/ActionList/ActionListItem.tsx +++ b/packages/react-core/src/components/ActionList/ActionListItem.tsx @@ -13,7 +13,7 @@ export const ActionListItem: React.FunctionComponent = ({ className = '', ...props }: ActionListItemProps) => ( -
+
{children}
); diff --git a/packages/react-core/src/components/ActionList/__tests__/ActionList.test.tsx b/packages/react-core/src/components/ActionList/__tests__/ActionList.test.tsx index d0413b686ac..553a7c88c08 100644 --- a/packages/react-core/src/components/ActionList/__tests__/ActionList.test.tsx +++ b/packages/react-core/src/components/ActionList/__tests__/ActionList.test.tsx @@ -13,7 +13,7 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class pf-c-action-list', () => { +test('Renders with class pf-v5-c-action-list', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-action-list'); diff --git a/packages/react-core/src/components/ActionList/__tests__/ActionListGroup.test.tsx b/packages/react-core/src/components/ActionList/__tests__/ActionListGroup.test.tsx index af6fc98b990..c7307824391 100644 --- a/packages/react-core/src/components/ActionList/__tests__/ActionListGroup.test.tsx +++ b/packages/react-core/src/components/ActionList/__tests__/ActionListGroup.test.tsx @@ -13,7 +13,7 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class pf-c-action-list__group', () => { +test('Renders with class pf-v5-c-action-list__group', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-action-list__group'); diff --git a/packages/react-core/src/components/ActionList/__tests__/ActionListItem.test.tsx b/packages/react-core/src/components/ActionList/__tests__/ActionListItem.test.tsx index 7c11ac5859f..57bfd9e961a 100644 --- a/packages/react-core/src/components/ActionList/__tests__/ActionListItem.test.tsx +++ b/packages/react-core/src/components/ActionList/__tests__/ActionListItem.test.tsx @@ -13,10 +13,10 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class pf-c-action-list__item', () => { +test('Renders with class pf-v5-c-action-list__item', () => { render(Test); - expect(screen.getByText('Test')).toHaveClass('pf-c-action-list__item'); + expect(screen.getByText('Test')).toHaveClass('pf-v5-c-action-list__item'); }); test('Renders with custom class names provided via prop', () => { diff --git a/packages/react-core/src/components/ActionList/__tests__/__snapshots__/ActionListItem.test.tsx.snap b/packages/react-core/src/components/ActionList/__tests__/__snapshots__/ActionListItem.test.tsx.snap index 4ac9e257f7a..ff9e5225792 100644 --- a/packages/react-core/src/components/ActionList/__tests__/__snapshots__/ActionListItem.test.tsx.snap +++ b/packages/react-core/src/components/ActionList/__tests__/__snapshots__/ActionListItem.test.tsx.snap @@ -3,7 +3,7 @@ exports[`Matches the snapshot 1`] = `
test
diff --git a/packages/react-core/src/components/Alert/__tests__/Alert.test.tsx b/packages/react-core/src/components/Alert/__tests__/Alert.test.tsx index e420035e7e3..b036aeda9e6 100644 --- a/packages/react-core/src/components/Alert/__tests__/Alert.test.tsx +++ b/packages/react-core/src/components/Alert/__tests__/Alert.test.tsx @@ -34,7 +34,7 @@ test('Renders without children', () => { expect(screen.getByTestId('container').firstChild).toBeVisible(); }); -test('Renders with class pf-c-alert on the containing div', () => { +test('Renders with class pf-v5-c-alert on the containing div', () => { render( Some alert @@ -43,7 +43,7 @@ test('Renders with class pf-c-alert on the containing div', () => { expect(screen.getByTestId('Alert-test-id')).toHaveClass('pf-v5-c-alert'); }); -test('Renders with class pf-c-alert__title on the div containing the title', () => { +test('Renders with class pf-v5-c-alert__title on the div containing the title', () => { render(Some alert); expect(screen.getByRole('heading', { name: 'Custom alert: Some title' })).toHaveClass('pf-v5-c-alert__title'); }); @@ -199,7 +199,7 @@ test('Renders the element passed via the actionClose prop', () => { expect(screen.getByRole('button', { name: 'Action close' })).toBeVisible(); }); -test('Renders the actionClose element inside pf-c-alert__action', () => { +test('Renders the actionClose element inside pf-v5-c-alert__action', () => { render( Some alert @@ -239,7 +239,7 @@ test('Renders the element passed via the actionLinks prop', () => { expect(screen.getByRole('button', { name: 'Action link' })).toBeVisible(); }); -test('Renders the actionLinks element inside pf-c-alert__action-group', () => { +test('Renders the actionLinks element inside pf-v5-c-alert__action-group', () => { render( Some alert @@ -255,7 +255,7 @@ test('Renders children', () => { expect(screen.getByText('Some alert')).toBeVisible(); }); -test('Renders children inside pf-c-alert__description', () => { +test('Renders children inside pf-v5-c-alert__description', () => { render(Some alert); expect(screen.getByText('Some alert')).toHaveClass('pf-v5-c-alert__description'); @@ -609,7 +609,7 @@ test('Renders with class pf-m-expandable when isExpandable = true', () => { expect(screen.getByTestId('Alert-test-id')).toHaveClass('pf-m-expandable'); }); -test('Renders AlertToggleExpandButton inside pf-c-alert__toggle', () => { +test('Renders AlertToggleExpandButton inside pf-v5-c-alert__toggle', () => { render( Some alert diff --git a/packages/react-core/src/components/Alert/__tests__/AlertIcon.test.tsx b/packages/react-core/src/components/Alert/__tests__/AlertIcon.test.tsx index 3dccd8217dc..b7b34b09f46 100644 --- a/packages/react-core/src/components/Alert/__tests__/AlertIcon.test.tsx +++ b/packages/react-core/src/components/Alert/__tests__/AlertIcon.test.tsx @@ -65,7 +65,7 @@ test('Renders with the passed custom icon when one is passed rather than the ico expect(screen.getByText('Custom icon')).toBeVisible(); }); -test('Renders the icon inside class pf-c-alert__icon', () => { +test('Renders the icon inside class pf-v5-c-alert__icon', () => { render(); expect(screen.getByText('Bell icon mock')).toHaveClass('pf-v5-c-alert__icon'); diff --git a/packages/react-core/src/components/Alert/__tests__/AlertToggleExpandButton.test.tsx b/packages/react-core/src/components/Alert/__tests__/AlertToggleExpandButton.test.tsx index 100533e2c52..1ed71db4f80 100644 --- a/packages/react-core/src/components/Alert/__tests__/AlertToggleExpandButton.test.tsx +++ b/packages/react-core/src/components/Alert/__tests__/AlertToggleExpandButton.test.tsx @@ -108,7 +108,7 @@ test('Renders a Button with variant: ButtonVariant.plain', () => { expect(screen.getByText('variant: plain')).toBeVisible(); }); -test('Renders with the toggle icon inside class pf-c-alert__toggle-icon', () => { +test('Renders with the toggle icon inside class pf-v5-c-alert__toggle-icon', () => { render( diff --git a/packages/react-core/src/components/Alert/examples/AlertDynamicLiveRegion.tsx b/packages/react-core/src/components/Alert/examples/AlertDynamicLiveRegion.tsx index dd2c00c92f0..5438f14cf82 100644 --- a/packages/react-core/src/components/Alert/examples/AlertDynamicLiveRegion.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertDynamicLiveRegion.tsx @@ -10,7 +10,7 @@ interface AlertInfo { export const DynamicLiveRegionAlert: React.FunctionComponent = () => { const [alerts, setAlerts] = React.useState([]); const getUniqueId: () => number = () => new Date().getTime(); - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const addAlert = (alertInfo: AlertInfo) => { setAlerts(prevAlertInfo => [...prevAlertInfo, alertInfo]); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupAsync.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupAsync.tsx index 30f944c2fff..b302808bc1f 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupAsync.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupAsync.tsx @@ -14,7 +14,7 @@ export const AlertGroupAsync: React.FunctionComponent = () => { const [alerts, setAlerts] = React.useState[]>([]); const [isRunning, setIsRunning] = React.useState(false); - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => new Date().getTime(); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupMultipleDynamic.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupMultipleDynamic.tsx index 8e0058ac254..ba5c9060b09 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupMultipleDynamic.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupMultipleDynamic.tsx @@ -20,7 +20,7 @@ export const AlertGroupMultipleDynamic: React.FunctionComponent = () => { setAlerts(prevAlerts => [...prevAlerts.filter(alert => alert.key !== key)]); }; - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => String.fromCharCode(65 + Math.floor(Math.random() * 26)) + Date.now(); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamic.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamic.tsx index cbb80001cbb..d2d339d8286 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamic.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamic.tsx @@ -20,7 +20,7 @@ export const AlertGroupSingularDynamic: React.FunctionComponent = () => { setAlerts(prevAlerts => [...prevAlerts.filter(alert => alert.key !== key)]); }; - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => new Date().getTime(); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamicOverflow.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamicOverflow.tsx index a7597b23dbe..ebf127e8e82 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamicOverflow.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupSingularDynamicOverflow.tsx @@ -34,7 +34,7 @@ export const AlertGroupSingularDynamicOverflow: React.FunctionComponent = () => setOverflowMessage(getOverflowMessage(newAlerts.length)); }; - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => new Date().getTime(); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupToast.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupToast.tsx index 4df6acbf915..abf012abb88 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupToast.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupToast.tsx @@ -20,7 +20,7 @@ export const AlertGroupToast: React.FunctionComponent = () => { setAlerts(prevAlerts => [...prevAlerts.filter(alert => alert.key !== key)]); }; - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => new Date().getTime(); diff --git a/packages/react-core/src/components/Alert/examples/AlertGroupToastOverflowCapture.tsx b/packages/react-core/src/components/Alert/examples/AlertGroupToastOverflowCapture.tsx index 7e1beb8826c..cadaa98b8b2 100644 --- a/packages/react-core/src/components/Alert/examples/AlertGroupToastOverflowCapture.tsx +++ b/packages/react-core/src/components/Alert/examples/AlertGroupToastOverflowCapture.tsx @@ -34,7 +34,7 @@ export const AlertGroupToastOverflowCapture: React.FunctionComponent = () => { setOverflowMessage(getOverflowMessage(newAlerts.length)); }; - const btnClasses = ['pf-c-button', 'pf-m-secondary'].join(' '); + const btnClasses = ['pf-v5-c-button', 'pf-m-secondary'].join(' '); const getUniqueId = () => new Date().getTime(); diff --git a/packages/react-core/src/components/Backdrop/__tests__/Backdrop.test.tsx b/packages/react-core/src/components/Backdrop/__tests__/Backdrop.test.tsx index e213736e850..c4bb8386cf7 100644 --- a/packages/react-core/src/components/Backdrop/__tests__/Backdrop.test.tsx +++ b/packages/react-core/src/components/Backdrop/__tests__/Backdrop.test.tsx @@ -16,12 +16,12 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with the pf-c-backdrop', () => { +test('Renders with the pf-v5-c-backdrop', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-backdrop'); }); -test('Renders with only the class pf-c-backdrop by default', () => { +test('Renders with only the class pf-v5-c-backdrop by default', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-backdrop', { exact: true }); }); diff --git a/packages/react-core/src/components/Badge/__tests__/Badge.test.tsx b/packages/react-core/src/components/Badge/__tests__/Badge.test.tsx index 0d25a62f736..0fdeb96dd03 100644 --- a/packages/react-core/src/components/Badge/__tests__/Badge.test.tsx +++ b/packages/react-core/src/components/Badge/__tests__/Badge.test.tsx @@ -16,7 +16,7 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class name pf-c-badge', () => { +test('Renders with class name pf-v5-c-badge', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-badge'); }); diff --git a/packages/react-core/src/components/Banner/__tests__/Banner.test.tsx b/packages/react-core/src/components/Banner/__tests__/Banner.test.tsx index 403a850b18b..83d0efc3ce5 100644 --- a/packages/react-core/src/components/Banner/__tests__/Banner.test.tsx +++ b/packages/react-core/src/components/Banner/__tests__/Banner.test.tsx @@ -16,7 +16,7 @@ test('Renders children', () => { expect(screen.getByText('Test')).toBeVisible(); }); -test('Renders with class name pf-c-banner', () => { +test('Renders with class name pf-v5-c-banner', () => { render(Test); expect(screen.getByText('Test')).toHaveClass('pf-v5-c-banner'); }); diff --git a/packages/react-core/src/components/Brand/Brand.tsx b/packages/react-core/src/components/Brand/Brand.tsx index 3b8a5752beb..dfed3aa33c9 100644 --- a/packages/react-core/src/components/Brand/Brand.tsx +++ b/packages/react-core/src/components/Brand/Brand.tsx @@ -56,7 +56,7 @@ export const Brand: React.FunctionComponent = ({ } return ( - /** the brand component currently contains no styling the 'pf-c-brand' string will be used for the className */ + /** the brand component currently contains no styling the 'pf-v5-c-brand' string will be used for the className */ children !== undefined ? ( = ({ className, ...props }: CodeBlockActionProps) => ( -
+
{children}
); diff --git a/packages/react-core/src/components/CodeBlock/__tests__/__snapshots__/CodeBlock.test.tsx.snap b/packages/react-core/src/components/CodeBlock/__tests__/__snapshots__/CodeBlock.test.tsx.snap index 4ef2d0dca23..35d5269d68f 100644 --- a/packages/react-core/src/components/CodeBlock/__tests__/__snapshots__/CodeBlock.test.tsx.snap +++ b/packages/react-core/src/components/CodeBlock/__tests__/__snapshots__/CodeBlock.test.tsx.snap @@ -26,7 +26,7 @@ exports[`CodeBlock with components renders successfully 1`] = ` class="pf-v5-c-code-block__actions" >
button
@@ -53,7 +53,7 @@ exports[`CodeBlock with components renders successfully 1`] = ` exports[`CodeBlockAction renders successfully 1`] = `
action
diff --git a/packages/react-core/src/components/DescriptionList/DescriptionListDescription.tsx b/packages/react-core/src/components/DescriptionList/DescriptionListDescription.tsx index 1bee65c930c..205c0ea3268 100644 --- a/packages/react-core/src/components/DescriptionList/DescriptionListDescription.tsx +++ b/packages/react-core/src/components/DescriptionList/DescriptionListDescription.tsx @@ -15,7 +15,7 @@ export const DescriptionListDescription: React.FunctionComponent (
-
{children}
+
{children}
); DescriptionListDescription.displayName = 'DescriptionListDescription'; diff --git a/packages/react-core/src/components/DescriptionList/__tests__/__snapshots__/DescriptionList.test.tsx.snap b/packages/react-core/src/components/DescriptionList/__tests__/__snapshots__/DescriptionList.test.tsx.snap index 62e1bc126b5..b9bde9f6974 100644 --- a/packages/react-core/src/components/DescriptionList/__tests__/__snapshots__/DescriptionList.test.tsx.snap +++ b/packages/react-core/src/components/DescriptionList/__tests__/__snapshots__/DescriptionList.test.tsx.snap @@ -72,7 +72,7 @@ exports[`Description List Description 1`] = ` class="pf-v5-c-description-list__description custom-description-list-description" >
test
diff --git a/packages/react-core/src/components/DualListSelector/DualListSelectorControl.tsx b/packages/react-core/src/components/DualListSelector/DualListSelectorControl.tsx index c6ccaf748e3..39bfe859e39 100644 --- a/packages/react-core/src/components/DualListSelector/DualListSelectorControl.tsx +++ b/packages/react-core/src/components/DualListSelector/DualListSelectorControl.tsx @@ -40,7 +40,7 @@ export const DualListSelectorControlBase: React.FunctionComponent +
  • ({ ) })); -test('renders with class pf-c-truncate', () => { +test('renders with class pf-v5-c-truncate', () => { render(); const test = screen.getByLabelText('test-id'); @@ -44,7 +44,7 @@ test('renders truncate with content', () => { expect(test).toHaveTextContent('Test'); }); -test('only renders pf-c-truncate__start with default position', () => { +test('only renders pf-v5-c-truncate__start with default position', () => { render(); const start = screen.getByText('Testing truncate content'); From b0e731537afbf838a29e170d13835a0955a4c76b Mon Sep 17 00:00:00 2001 From: gitdallas Date: Wed, 17 May 2023 13:01:06 -0500 Subject: [PATCH 2/6] update cypress test selectors --- .../integration/duallistselectorbasic.spec.ts | 44 +++++++++---------- .../integration/duallistselectortree.spec.ts | 12 ++--- .../duallistselectorwithactions.spec.ts | 20 ++++----- .../cypress/integration/menu.spec.ts | 8 ++-- .../notificationdrawergroups.spec.ts | 8 ++-- .../notificationdrawerlightweight.spec.ts | 8 ++-- .../cypress/integration/select.spec.ts | 2 +- 7 files changed, 51 insertions(+), 51 deletions(-) diff --git a/packages/react-integration/cypress/integration/duallistselectorbasic.spec.ts b/packages/react-integration/cypress/integration/duallistselectorbasic.spec.ts index 4014dcb2647..b355e76db11 100644 --- a/packages/react-integration/cypress/integration/duallistselectorbasic.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectorbasic.spec.ts @@ -16,16 +16,16 @@ describe('Dual List Selector BasicDemo Test', () => { 'Available search input' ); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Add all'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Add all'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(0) .should('have.attr', 'aria-label', 'Add selected') .and('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(3) .should('have.attr', 'aria-label', 'Remove selected') .and('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(2) .should('have.attr', 'aria-label', 'Remove all') .and('have.attr', 'disabled'); @@ -47,7 +47,7 @@ describe('Dual List Selector BasicDemo Test', () => { it('Verify selecting options', () => { cy.get('.pf-v5-c-dual-list-selector__list-item .pf-m-selected').should('not.exist'); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(0).click(); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).and('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).and('not.have.attr', 'disabled'); cy.get('.pf-v5-c-dual-list-selector__list-item .pf-m-selected').should('exist'); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(1).click(); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('2 of 4 items selected'); @@ -58,20 +58,20 @@ describe('Dual List Selector BasicDemo Test', () => { }); it('Verify selecting and choosing options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(1).click(); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(1).should('not.have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(2).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(1).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(2).should('not.have.attr', 'disabled'); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('1 of 3 items selected'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('0 of 1 items selected'); - cy.get('.pf-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-v5-c-tooltip').should('exist'); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 2); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 2); @@ -81,22 +81,22 @@ describe('Dual List Selector BasicDemo Test', () => { }); it('Verify removing all options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-tooltip').should('exist'); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); it('Verify choosing all options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-tooltip').should('exist'); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 4); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(2).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(2).should('not.have.attr', 'disabled'); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('0 of 0 items selected'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('0 of 4 items selected'); @@ -112,14 +112,14 @@ describe('Dual List Selector BasicDemo Test', () => { cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}'); - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(1).should('not.have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(2).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(1).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(3).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(2).should('have.attr', 'disabled'); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('0 of 4 items selected'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('0 of 0 items selected'); diff --git a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts index 3179a716727..67b27e5432e 100644 --- a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts @@ -26,11 +26,11 @@ describe('Dual List Selector TreeDemo Test', () => { }); it('Verify checkbox selects an option', () => { - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); cy.get('.pf-v5-c-dual-list-selector__list-item .pf-v5-c-dual-list-selector__item-check').last().click(); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); cy.get('.pf-v5-c-dual-list-selector__list-item .pf-v5-c-dual-list-selector__item-check').eq(1).click(); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(0).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(2).find('li').should('have.length', 2); }); @@ -38,7 +38,7 @@ describe('Dual List Selector TreeDemo Test', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__tools-filter input').eq(0).type('Option 1'); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); - cy.get('.pf-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(0) @@ -55,14 +55,14 @@ describe('Dual List Selector TreeDemo Test', () => { it('Verify remove all filtered options works', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 2); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}'); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 2); - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); diff --git a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts index 2c1f26983bf..02fe832939a 100644 --- a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts @@ -26,16 +26,16 @@ describe('Dual List Selector With Actions Demo Test', () => { 'dual-list-selector-demo-available-pane-status' ); - cy.get('.pf-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') .eq(0) .should('have.attr', 'aria-label', 'Demo add selected') .and('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') .eq(3) .should('have.attr', 'aria-label', 'Demo remove selected') .and('have.attr', 'disabled'); - cy.get('.pf-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') .eq(2) .should('have.attr', 'aria-label', 'Demo remove all') .and('have.attr', 'disabled'); @@ -61,11 +61,11 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify selecting and choosing options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(1).click(); - cy.get('.pf-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('2 available'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('2 chosen'); @@ -75,7 +75,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify removing all options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('4 available'); @@ -83,7 +83,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify choosing all options', () => { - cy.get('.pf-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 4); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('0 available'); @@ -106,7 +106,7 @@ describe('Dual List Selector With Actions Demo Test', () => { cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}'); - cy.get('.pf-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); @@ -119,7 +119,7 @@ describe('Dual List Selector With Actions Demo Test', () => { xit('Verify adding all filtered options', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); - cy.get('.pf-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') diff --git a/packages/react-integration/cypress/integration/menu.spec.ts b/packages/react-integration/cypress/integration/menu.spec.ts index f096273f5dd..596bf68c11e 100644 --- a/packages/react-integration/cypress/integration/menu.spec.ts +++ b/packages/react-integration/cypress/integration/menu.spec.ts @@ -46,10 +46,10 @@ describe('Menu Test', () => { }); it('Verify Menu with Titled Groups', () => { - cy.get('#group-1.pf-c-menu__group > h1').should('contain', 'Group 1'); - cy.get('#group-2.pf-c-menu__group > h1').should('contain', 'Group 2'); - cy.get('#group-3.pf-c-menu__group > div > h1').should('contain', 'Group 3'); - cy.get('#group-4.pf-c-menu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); + cy.get('#group-1.pf-v5-c5-c-menu__group > h1').should('contain', 'Group 1'); + cy.get('#group-2.pf-v5-c5-c-menu__group > h1').should('contain', 'Group 2'); + cy.get('#group-3.pf-v5-c5-c-menu__group > div > h1').should('contain', 'Group 3'); + cy.get('#group-4.pf-v5-c5-c-menu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); }); it('Verify Menu with Description', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts index b411537e5da..7ded248dfd4 100644 --- a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts @@ -20,14 +20,14 @@ describe('Notification Drawer Groups Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-c-notification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-c-notification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-c5-c-notification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-c-notification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items are hoverable', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts index 73ce4ecb716..08b7c4a0b32 100644 --- a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts @@ -16,14 +16,14 @@ describe('Notification Drawer Lightweight Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-c-notification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-c-notification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-c5-c-notification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-c-notification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-c5-c-notification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items in lightweight drawer are all in unread state', () => { diff --git a/packages/react-integration/cypress/integration/select.spec.ts b/packages/react-integration/cypress/integration/select.spec.ts index 5188b262e41..2ab996bf03a 100644 --- a/packages/react-integration/cypress/integration/select.spec.ts +++ b/packages/react-integration/cypress/integration/select.spec.ts @@ -44,7 +44,7 @@ describe('Select Test', () => { it.skip('Verify Description Select', () => { cy.get('#single-select-with-descriptions').click(); - cy.get('.pf-c-select__menu-footer').should('exist'); + cy.get('.pf-v5-c5-c-select__menu-footer').should('exist'); cy.get('#Miss > .pf-v5-c-select__menu-item > .pf-v5-c-select__menu-item-description').click(); cy.get('#single-select-with-descriptions').contains('Miss').should('exist'); }); From a22df9e79ddb3fdac2b226712e2d116fc13d6564 Mon Sep 17 00:00:00 2001 From: gitdallas Date: Wed, 17 May 2023 13:16:00 -0500 Subject: [PATCH 3/6] fix cypress selectors --- .../integration/duallistselectortree.spec.ts | 10 +++++----- .../duallistselectorwithactions.spec.ts | 20 +++++++++---------- .../cypress/integration/menu.spec.ts | 8 ++++---- .../notificationdrawergroups.spec.ts | 8 ++++---- .../notificationdrawerlightweight.spec.ts | 8 ++++---- .../cypress/integration/select.spec.ts | 2 +- 6 files changed, 28 insertions(+), 28 deletions(-) diff --git a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts index 67b27e5432e..fa1ad679546 100644 --- a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts @@ -28,9 +28,9 @@ describe('Dual List Selector TreeDemo Test', () => { it('Verify checkbox selects an option', () => { cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('have.attr', 'disabled'); cy.get('.pf-v5-c-dual-list-selector__list-item .pf-v5-c-dual-list-selector__item-check').last().click(); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).should('not.have.attr', 'disabled'); cy.get('.pf-v5-c-dual-list-selector__list-item .pf-v5-c-dual-list-selector__item-check').eq(1).click(); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(0).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(2).find('li').should('have.length', 2); }); @@ -38,7 +38,7 @@ describe('Dual List Selector TreeDemo Test', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__tools-filter input').eq(0).type('Option 1'); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(0) @@ -55,14 +55,14 @@ describe('Dual List Selector TreeDemo Test', () => { it('Verify remove all filtered options works', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 2); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}'); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 2); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); diff --git a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts index 02fe832939a..d1086bd4b4a 100644 --- a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts @@ -26,16 +26,16 @@ describe('Dual List Selector With Actions Demo Test', () => { 'dual-list-selector-demo-available-pane-status' ); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-cal-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); + cy.get('.pf-v5-cal-list-selector__controls-item button') .eq(0) .should('have.attr', 'aria-label', 'Demo add selected') .and('have.attr', 'disabled'); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-cal-list-selector__controls-item button') .eq(3) .should('have.attr', 'aria-label', 'Demo remove selected') .and('have.attr', 'disabled'); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item button') + cy.get('.pf-v5-cal-list-selector__controls-item button') .eq(2) .should('have.attr', 'aria-label', 'Demo remove all') .and('have.attr', 'disabled'); @@ -61,11 +61,11 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify selecting and choosing options', () => { - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(1).click(); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(0).click(); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('2 available'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('2 chosen'); @@ -75,7 +75,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify removing all options', () => { - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('4 available'); @@ -83,7 +83,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify choosing all options', () => { - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 4); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('0 available'); @@ -106,7 +106,7 @@ describe('Dual List Selector With Actions Demo Test', () => { cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}'); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); @@ -119,7 +119,7 @@ describe('Dual List Selector With Actions Demo Test', () => { xit('Verify adding all filtered options', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); - cy.get('.pf-v5-c5-c-dual-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-cal-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') diff --git a/packages/react-integration/cypress/integration/menu.spec.ts b/packages/react-integration/cypress/integration/menu.spec.ts index 596bf68c11e..92d83bd213a 100644 --- a/packages/react-integration/cypress/integration/menu.spec.ts +++ b/packages/react-integration/cypress/integration/menu.spec.ts @@ -46,10 +46,10 @@ describe('Menu Test', () => { }); it('Verify Menu with Titled Groups', () => { - cy.get('#group-1.pf-v5-c5-c-menu__group > h1').should('contain', 'Group 1'); - cy.get('#group-2.pf-v5-c5-c-menu__group > h1').should('contain', 'Group 2'); - cy.get('#group-3.pf-v5-c5-c-menu__group > div > h1').should('contain', 'Group 3'); - cy.get('#group-4.pf-v5-c5-c-menu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); + cy.get('#group-1.pf-v5-cnu__group > h1').should('contain', 'Group 1'); + cy.get('#group-2.pf-v5-cnu__group > h1').should('contain', 'Group 2'); + cy.get('#group-3.pf-v5-cnu__group > div > h1').should('contain', 'Group 3'); + cy.get('#group-4.pf-v5-cnu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); }); it('Verify Menu with Description', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts index 7ded248dfd4..9336e2b861b 100644 --- a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts @@ -20,14 +20,14 @@ describe('Notification Drawer Groups Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-v5-c5-c-notification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-v5-c5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-v5-c5-c-notification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-ctification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-v5-c5-c-notification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items are hoverable', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts index 08b7c4a0b32..00be0623297 100644 --- a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts @@ -16,14 +16,14 @@ describe('Notification Drawer Lightweight Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-v5-c5-c-notification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-v5-c5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-v5-c5-c-notification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-ctification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-v5-c5-c-notification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-ctification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items in lightweight drawer are all in unread state', () => { diff --git a/packages/react-integration/cypress/integration/select.spec.ts b/packages/react-integration/cypress/integration/select.spec.ts index 2ab996bf03a..14386d873b7 100644 --- a/packages/react-integration/cypress/integration/select.spec.ts +++ b/packages/react-integration/cypress/integration/select.spec.ts @@ -44,7 +44,7 @@ describe('Select Test', () => { it.skip('Verify Description Select', () => { cy.get('#single-select-with-descriptions').click(); - cy.get('.pf-v5-c5-c-select__menu-footer').should('exist'); + cy.get('.pf-v5-clect__menu-footer').should('exist'); cy.get('#Miss > .pf-v5-c-select__menu-item > .pf-v5-c-select__menu-item-description').click(); cy.get('#single-select-with-descriptions').contains('Miss').should('exist'); }); From 8cafb3486e25ffc70a83da32f97d4d92e1d51528 Mon Sep 17 00:00:00 2001 From: gitdallas Date: Wed, 17 May 2023 13:45:11 -0500 Subject: [PATCH 4/6] fix cypress selectors --- .../integration/duallistselectortree.spec.ts | 4 ++-- .../duallistselectorwithactions.spec.ts | 20 +++++++++---------- .../cypress/integration/menu.spec.ts | 8 ++++---- .../notificationdrawergroups.spec.ts | 8 ++++---- .../notificationdrawerlightweight.spec.ts | 8 ++++---- .../cypress/integration/select.spec.ts | 2 +- 6 files changed, 25 insertions(+), 25 deletions(-) diff --git a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts index fa1ad679546..802e9fb5d2c 100644 --- a/packages/react-integration/cypress/integration/duallistselectortree.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectortree.spec.ts @@ -55,14 +55,14 @@ describe('Dual List Selector TreeDemo Test', () => { it('Verify remove all filtered options works', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); - cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 2); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}{backspace}'); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 2); - cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); diff --git a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts index d1086bd4b4a..d6e751df9df 100644 --- a/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts +++ b/packages/react-integration/cypress/integration/duallistselectorwithactions.spec.ts @@ -26,16 +26,16 @@ describe('Dual List Selector With Actions Demo Test', () => { 'dual-list-selector-demo-available-pane-status' ); - cy.get('.pf-v5-cal-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); - cy.get('.pf-v5-cal-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button').eq(1).should('have.attr', 'aria-label', 'Demo add all'); + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(0) .should('have.attr', 'aria-label', 'Demo add selected') .and('have.attr', 'disabled'); - cy.get('.pf-v5-cal-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(3) .should('have.attr', 'aria-label', 'Demo remove selected') .and('have.attr', 'disabled'); - cy.get('.pf-v5-cal-list-selector__controls-item button') + cy.get('.pf-v5-c-dual-list-selector__controls-item button') .eq(2) .should('have.attr', 'aria-label', 'Demo remove all') .and('have.attr', 'disabled'); @@ -61,11 +61,11 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify selecting and choosing options', () => { - cy.get('.pf-v5-cal-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 3); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__list-item').eq(1).click(); - cy.get('.pf-v5-cal-list-selector__controls-item').eq(0).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(0).click(); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('2 available'); cy.get('.pf-m-chosen .pf-v5-c-dual-list-selector__status-text').contains('2 chosen'); @@ -75,7 +75,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify removing all options', () => { - cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('4 available'); @@ -83,7 +83,7 @@ describe('Dual List Selector With Actions Demo Test', () => { }); it('Verify choosing all options', () => { - cy.get('.pf-v5-cal-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 4); cy.get('.pf-m-available .pf-v5-c-dual-list-selector__status-text').contains('0 available'); @@ -106,7 +106,7 @@ describe('Dual List Selector With Actions Demo Test', () => { cy.get('.pf-v5-c-dual-list-selector__tools-filter input') .eq(1) .type('{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}{Backspace}'); - cy.get('.pf-v5-cal-list-selector__controls-item').eq(2).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(2).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 4); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 0); }); @@ -119,7 +119,7 @@ describe('Dual List Selector With Actions Demo Test', () => { xit('Verify adding all filtered options', () => { cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 1); - cy.get('.pf-v5-cal-list-selector__controls-item').eq(1).click(); + cy.get('.pf-v5-c-dual-list-selector__controls-item').eq(1).click(); cy.get('.pf-v5-c-dual-list-selector__list').eq(0).find('li').should('have.length', 0); cy.get('.pf-v5-c-dual-list-selector__list').eq(1).find('li').should('have.length', 1); cy.get('.pf-v5-c-dual-list-selector__tools-filter input') diff --git a/packages/react-integration/cypress/integration/menu.spec.ts b/packages/react-integration/cypress/integration/menu.spec.ts index 92d83bd213a..4eaeffe1b84 100644 --- a/packages/react-integration/cypress/integration/menu.spec.ts +++ b/packages/react-integration/cypress/integration/menu.spec.ts @@ -46,10 +46,10 @@ describe('Menu Test', () => { }); it('Verify Menu with Titled Groups', () => { - cy.get('#group-1.pf-v5-cnu__group > h1').should('contain', 'Group 1'); - cy.get('#group-2.pf-v5-cnu__group > h1').should('contain', 'Group 2'); - cy.get('#group-3.pf-v5-cnu__group > div > h1').should('contain', 'Group 3'); - cy.get('#group-4.pf-v5-cnu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); + cy.get('#group-1.pf-v5-c-menu__group > h1').should('contain', 'Group 1'); + cy.get('#group-2.pf-v5-c-menu__group > h1').should('contain', 'Group 2'); + cy.get('#group-3.pf-v5-c-menu__group > div > h1').should('contain', 'Group 3'); + cy.get('#group-4.pf-v5-c-menu__group > div > h1.my-awesome-style').should('contain', 'Group 4'); }); it('Verify Menu with Description', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts index 9336e2b861b..ccc58e83c46 100644 --- a/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawergroups.spec.ts @@ -20,14 +20,14 @@ describe('Notification Drawer Groups Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-v5-ctification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-v5-ctification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-v5-ctification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-c-notification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-v5-ctification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items are hoverable', () => { diff --git a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts index 00be0623297..c29d04b126f 100644 --- a/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts +++ b/packages/react-integration/cypress/integration/notificationdrawerlightweight.spec.ts @@ -16,14 +16,14 @@ describe('Notification Drawer Lightweight Demo Test', () => { }); it('Verify first and last list items are hidden', () => { - cy.get('.pf-v5-ctification-drawer__list').first().should('have.attr', 'hidden'); - cy.get('.pf-v5-ctification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); - cy.get('.pf-v5-ctification-drawer__list').last().should('have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').first().should('have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').eq(1).should('not.have.attr', 'hidden', false); + cy.get('.pf-v5-c-notification-drawer__list').last().should('have.attr', 'hidden'); }); it('Verify first item is expanded after click', () => { cy.get('.pf-v5-c-notification-drawer__group').first().click(); - cy.get('.pf-v5-ctification-drawer__list').should('not.have.attr', 'hidden'); + cy.get('.pf-v5-c-notification-drawer__list').should('not.have.attr', 'hidden'); }); it('Verify list items in lightweight drawer are all in unread state', () => { diff --git a/packages/react-integration/cypress/integration/select.spec.ts b/packages/react-integration/cypress/integration/select.spec.ts index 14386d873b7..6c01d88ce4a 100644 --- a/packages/react-integration/cypress/integration/select.spec.ts +++ b/packages/react-integration/cypress/integration/select.spec.ts @@ -44,7 +44,7 @@ describe('Select Test', () => { it.skip('Verify Description Select', () => { cy.get('#single-select-with-descriptions').click(); - cy.get('.pf-v5-clect__menu-footer').should('exist'); + cy.get('.pf-v5-c-select__menu-footer').should('exist'); cy.get('#Miss > .pf-v5-c-select__menu-item > .pf-v5-c-select__menu-item-description').click(); cy.get('#single-select-with-descriptions').contains('Miss').should('exist'); }); From 931fdaffc230832375ed7742f5360d380cc0c8f5 Mon Sep 17 00:00:00 2001 From: gitdallas Date: Wed, 17 May 2023 16:28:36 -0500 Subject: [PATCH 5/6] update some code-editor/deprecated/table/integration directories --- .../src/components/CodeEditor/CodeEditor.tsx | 4 ++-- .../__test__/__snapshots__/CodeEditor.test.tsx.snap | 2 +- .../ApplicationLauncher/ApplicationLauncherText.tsx | 2 +- .../__snapshots__/ApplicationLauncherText.test.tsx.snap | 2 +- .../src/deprecated/components/Select/SelectMenu.tsx | 2 +- .../src/components/demos/AlertGroupDemo/AlertGroupDemo.tsx | 2 +- packages/react-table/src/components/Table/Table.tsx | 2 +- packages/react-table/src/components/Table/Td.tsx | 2 +- .../src/components/Table/utils/transformers.test.tsx | 2 +- 9 files changed, 10 insertions(+), 10 deletions(-) diff --git a/packages/react-code-editor/src/components/CodeEditor/CodeEditor.tsx b/packages/react-code-editor/src/components/CodeEditor/CodeEditor.tsx index 3c27ea2a36a..7abc5263d7d 100644 --- a/packages/react-code-editor/src/components/CodeEditor/CodeEditor.tsx +++ b/packages/react-code-editor/src/components/CodeEditor/CodeEditor.tsx @@ -605,7 +605,7 @@ export class CodeEditor extends React.Component{headerMainContent}
} {!!shortcutsPopoverProps.bodyContent && ( -
+