Miscellaneous test fixes

Signed-off-by: Kevin Cormier <[email protected]>
stolostron · Nov 13, 2024 · 213bcff · 213bcff
1 parent 622bf2a
commit 213bcff
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 12 deletions.
diff --git a/frontend/src/components/TemplateEditor/controls/ControlPanelLabels.test.js b/frontend/src/components/TemplateEditor/controls/ControlPanelLabels.test.js
@@ -24,13 +24,13 @@ describe('ControlPanelLabels component', () => {
     const Component = () => {
       return <ControlPanelLabels key={'key'} control={control} controlId={'controlId'} handleChange={fn} i18n={t} />
     }
-    const { getByTestId, asFragment, container } = render(<Component />)
+    const { getByTestId, getByRole, asFragment } = render(<Component />)
     expect(asFragment()).toMatchSnapshot()
 
     userEvent.type(getByTestId('controlId'), 'label=test{enter}')
     expect(control.active).toEqual([{ key: 'label', value: 'test' }])
     userEvent.type(getByTestId('controlId'), 'label=test2{enter}')
-    container.querySelector('.pf-v5-c-button').click()
+    userEvent.click(getByRole('button', { name: 'Close label=test' }))
     expect(control.active).toEqual([])
     userEvent.type(getByTestId('controlId'), 'label=test,')
     userEvent.type(getByTestId('controlId'), 'label={esc}')

diff --git a/.../src/routes/Infrastructure/Clusters/ManagedClusters/components/NodePoolsProgress.test.tsx b/.../src/routes/Infrastructure/Clusters/ManagedClusters/components/NodePoolsProgress.test.tsx
@@ -27,18 +27,14 @@ const mockClusterImageSet0: ClusterImageSetK8sResource = {
 }
 
 const resultPending = {
-  icon: (
-    <Icon size="sm">
-      <InProgressIcon color="currentColor" />
-    </Icon>
-  ),
+  icon: <InProgressIcon />,
   text: 'Not ready',
   type: 'pending',
 }
 
 const resultOK = {
   icon: (
-    <Icon size="sm" status="success">
+    <Icon status="success">
       <CheckCircleIcon />
     </Icon>
   ),

diff --git a/...end/src/routes/Infrastructure/Clusters/ManagedClusters/components/NodePoolsTable.test.tsx b/...end/src/routes/Infrastructure/Clusters/ManagedClusters/components/NodePoolsTable.test.tsx
@@ -815,12 +815,11 @@ describe('NodePoolsTable', () => {
     expect(screen.queryAllByText('Node pool name').length).toBe(1)
     await waitForText('Cancel')
     userEvent.click(screen.getByTestId('cancel-nodepool-form'))
-    expect(screen.queryAllByTestId(/pf-dropdown-toggle-id-[0-9]*/)[0]).toBeTruthy()
-    userEvent.click(screen.queryAllByTestId(/pf-dropdown-toggle-id-[0-9]*/)[0])
+    userEvent.click(screen.getAllByLabelText('Actions')[0])
     userEvent.click(screen.getByText('Manage node pool'))
     expect(screen.queryAllByText('Manage node pool').length).toBe(1)
     userEvent.click(screen.getByTestId('cancel-nodepool-form'))
-    userEvent.click(screen.queryAllByTestId(/pf-dropdown-toggle-id-[0-9]*/)[0])
+    userEvent.click(screen.getAllByLabelText('Actions')[0])
     userEvent.click(screen.getByText('Remove node pool'))
     userEvent.click(screen.getByText('Cancel'))
   })

diff --git a/frontend/src/ui-components/AcmSelect/AcmSelect.test.tsx b/frontend/src/ui-components/AcmSelect/AcmSelect.test.tsx
@@ -57,7 +57,7 @@ describe('AcmSelect', () => {
       )
     }
     const { container } = render(<TypeaheadSelect />)
-    expect(container.querySelector<HTMLInputElement>('.pf-v5-c-select__toggle-typeahead')).toHaveAttribute(
+    expect(container.querySelector<HTMLInputElement>('.pf-v5-c-select__toggle-typeahead input')).toHaveAttribute(
       'placeholder',
       'Select one'
     )