katello 4.16.1 → 4.17.0.rc1

data/webpack/scenes/ContentViews/Details/Filters/__tests__/cvModuleStreamFilter.test.js

@@ -1,5 +1,5 @@
 import React from 'react';
-import { renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
+import { renderWithRedux, patientlyWaitFor, fireEvent, act } from 'react-testing-lib-wrapper';
 import { Route } from 'react-router-dom';
 
 import ContentViewFilterDetails from '../ContentViewFilterDetails';
@@ -36,6 +36,68 @@ const renderOptions = {
 
 const withCVRoute = component => <Route path="/content_views/:id([0-9]+)#/filters/:filterId([0-9]+)">{component}</Route>;
 
+test('Can filter by added/not added rules', async (done) => {
+  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
+  const { rules } = cvFilterDetails;
+  const { name } = rules[0];
+
+  const cvFilterScope = nockInstance
+    .get(cvFilterDetailsPath)
+    .query(true)
+    .reply(200, cvFilterDetails);
+
+  const cvFiltersScope = nockInstance
+    .get(cvFiltersPath)
+    .query(true)
+    .reply(200, cvFilterFixtures);
+
+  const moduleStreamsScope = nockInstance
+    .get(moduleStreamsPath)
+    .query(true)
+    .times(3) // For first call (All), Added, and Not Added
+    .reply(200, allModuleStreams);
+
+  const {
+    getByText, queryByText, getByTestId, getByLabelText,
+  } = renderWithRedux(withCVRoute(<ContentViewFilterDetails
+    cvId={1}
+    details={details}
+  />), renderOptions);
+
+  // Nothing will show at first, page is loading
+  expect(queryByText(name)).toBeNull();
+
+  await patientlyWaitFor(() => {
+    expect(getByText(name)).toBeInTheDocument();
+    expect(getByTestId('allAddedNotAdded')).toBeInTheDocument();
+    act(() => {
+      fireEvent.click(getByTestId('allAddedNotAdded')?.childNodes[0]?.childNodes[0]);
+    });
+  });
+
+  await patientlyWaitFor(() => {
+    expect(getByLabelText(ADDED)).toBeInTheDocument();
+    getByLabelText(ADDED).click();
+  });
+
+  await patientlyWaitFor(() => {
+    expect(getByText(name)).toBeInTheDocument();
+    expect(getByTestId('allAddedNotAdded')).toBeInTheDocument();
+    fireEvent.click(getByTestId('allAddedNotAdded')?.childNodes[0]?.childNodes[0]);
+  });
+
+  await patientlyWaitFor(() => {
+    expect(getByLabelText(NOT_ADDED)).toBeInTheDocument();
+    fireEvent.click(getByLabelText(NOT_ADDED));
+  });
+
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(cvFilterScope);
+  assertNockRequest(cvFiltersScope);
+  assertNockRequest(moduleStreamsScope);
+  act(done);
+});
+
 test('Can enable and disable add filter button', async (done) => {
   const { name: cvFilterName } = cvFilterDetails;
   const cvFilterScope = nockInstance
@@ -75,6 +137,7 @@ test('Can enable and disable add filter button', async (done) => {
   assertNockRequest(cvFilterScope);
   assertNockRequest(cvFiltersScope);
   assertNockRequest(moduleStreamsScope, done);
+  act(done);
 });
 
 test('Can add a filter rule', async (done) => {
@@ -119,11 +182,14 @@ test('Can add a filter rule', async (done) => {
 
   await patientlyWaitFor(() => {
     expect(getByText(name)).toBeInTheDocument();
-    expect(getAllByLabelText('Actions')[3]).toHaveAttribute('aria-expanded', 'false');
+    expect(getAllByLabelText('Kebab toggle')[1]).toHaveAttribute('aria-expanded', 'false');
   });
-  fireEvent.click(getAllByLabelText('Actions')[3]);
-  expect(getAllByLabelText('Actions')[3]).toHaveAttribute('aria-expanded', 'true');
+  await act(async () => {
+    fireEvent.click(getAllByLabelText('Kebab toggle')[1]);
+  });
+  expect(getAllByLabelText('Kebab toggle')[1]).toHaveAttribute('aria-expanded', 'true');
   await patientlyWaitFor(() => expect(getByText('Add')).toBeInTheDocument());
+
   fireEvent.click(getByText('Add'));
 
   assertNockRequest(autocompleteScope);
@@ -132,6 +198,7 @@ test('Can add a filter rule', async (done) => {
   assertNockRequest(cvFiltersRuleScope);
   assertNockRequest(cvRequestCallbackScope);
   assertNockRequest(moduleStreamsScope, done);
+  act(done);
 });
 
 test('Can remove a filter rule', async (done) => {
@@ -175,10 +242,10 @@ test('Can remove a filter rule', async (done) => {
 
   await patientlyWaitFor(() => {
     expect(getByText(name)).toBeInTheDocument();
-    expect(getAllByLabelText('Actions')[2]).toHaveAttribute('aria-expanded', 'false');
+    expect(getAllByLabelText('Kebab toggle')[2]).toHaveAttribute('aria-expanded', 'false');
   });
-  fireEvent.click(getAllByLabelText('Actions')[2]);
-  expect(getAllByLabelText('Actions')[2]).toHaveAttribute('aria-expanded', 'true');
+  fireEvent.click(getAllByLabelText('Kebab toggle')[2]);
+  expect(getAllByLabelText('Kebab toggle')[2]).toHaveAttribute('aria-expanded', 'true');
   await patientlyWaitFor(() => expect(getByText('Remove')).toBeInTheDocument());
   fireEvent.click(getByText('Remove'));
 
@@ -190,6 +257,7 @@ test('Can remove a filter rule', async (done) => {
   assertNockRequest(cvRequestCallbackScope);
 
   assertNockRequest(moduleStreamsScope, done);
+  act(done);
 });
 
 test('Can bulk remove filter rules', async (done) => {
@@ -252,6 +320,7 @@ test('Can bulk remove filter rules', async (done) => {
   assertNockRequest(cvRequestCallbackScope);
 
   assertNockRequest(moduleStreamsScope, done);
+  act(done);
 });
 
 test('Can bulk add filter rules', async (done) => {
@@ -310,64 +379,5 @@ test('Can bulk add filter rules', async (done) => {
   assertNockRequest(cvRequestCallbackScope);
 
   assertNockRequest(moduleStreamsScope, done);
-});
-
-
-test('Can filter by added/not added rules', async (done) => {
-  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
-  const { rules } = cvFilterDetails;
-  const { name } = rules[0];
-
-  const cvFilterScope = nockInstance
-    .get(cvFilterDetailsPath)
-    .query(true)
-    .reply(200, cvFilterDetails);
-
-  const cvFiltersScope = nockInstance
-    .get(cvFiltersPath)
-    .query(true)
-    .reply(200, cvFilterFixtures);
-
-  const moduleStreamsScope = nockInstance
-    .get(moduleStreamsPath)
-    .query(true)
-    .times(3) // For first call (All), Added, and Not Added
-    .reply(200, allModuleStreams);
-
-  const {
-    getByText, queryByText, getByTestId, getByLabelText,
-  } = renderWithRedux(withCVRoute(<ContentViewFilterDetails
-    cvId={1}
-    details={details}
-  />), renderOptions);
-
-  // Nothing will show at first, page is loading
-  expect(queryByText(name)).toBeNull();
-
-  await patientlyWaitFor(() => {
-    expect(getByText(name)).toBeInTheDocument();
-    expect(getByTestId('allAddedNotAdded')).toBeInTheDocument();
-    fireEvent.click(getByTestId('allAddedNotAdded')?.childNodes[0]?.childNodes[0]);
-  });
-
-  await patientlyWaitFor(() => {
-    expect(getByLabelText(ADDED)).toBeInTheDocument();
-    getByLabelText(ADDED).click();
-  });
-
-  await patientlyWaitFor(() => {
-    expect(getByText(name)).toBeInTheDocument();
-    expect(getByTestId('allAddedNotAdded')).toBeInTheDocument();
-    fireEvent.click(getByTestId('allAddedNotAdded')?.childNodes[0]?.childNodes[0]);
-  });
-
-  await patientlyWaitFor(() => {
-    expect(getByLabelText(NOT_ADDED)).toBeInTheDocument();
-    getByLabelText(NOT_ADDED).click();
-  });
-
-  assertNockRequest(autocompleteScope);
-  assertNockRequest(cvFilterScope);
-  assertNockRequest(cvFiltersScope);
-  assertNockRequest(moduleStreamsScope, done);
+  act(done);
 });

data/webpack/scenes/ContentViews/Details/Filters/__tests__/cvModuleStreamFilterDetails.fixtures.json

@@ -7,6 +7,7 @@
    "updated_at": "2021-02-23 10:09:24 -0500",
    "content_view": {
       "composite": false,
+      "rolling": false,
       "component_ids": [],
       "default": false,
       "version_count": 0,

data/webpack/scenes/ContentViews/Details/Filters/__tests__/cvPackageFilterDetail.fixtures.json

@@ -8,6 +8,7 @@
   "updated_at": "2021-03-31 12:59:12 -0400",
   "content_view": {
     "composite": false,
+    "rolling": false,
     "component_ids": [],
     "default": false,
     "version_count": 4,

data/webpack/scenes/ContentViews/Details/Histories/ContentViewHistories.js

@@ -49,9 +49,9 @@ const ContentViewHistories = ({ cvId }) => {
     const taskType = task ? task.label : taskTypes[action];
 
     if (taskType === taskTypes.removal) {
-      return <>{__('Deleted from ')} <Label isTruncated key="1" color="blue" href={`/lifecycle_environments/${environment?.id}`}>{environment?.name ?? __('all environments')}</Label></>;
+      return <>{__('Deleted from ')} <Label key="1" color="blue" href={`/lifecycle_environments/${environment?.id}`}>{environment?.name ?? __('all environments')}</Label></>;
     } else if (action === 'promotion' || taskType === taskTypes.promotion) {
-      return <>{__('Promoted to ')}<Label isTruncated key="2" color="blue" href={`/lifecycle_environments/${environment?.id}`}>{environment?.name}</Label></>;
+      return <>{__('Promoted to ')}<Label key="2" color="blue" href={`/lifecycle_environments/${environment?.id}`}>{environment?.name}</Label></>;
     } else if (taskType === taskTypes.publish) {
       return __('Published new version');
     } else if (taskType === taskTypes.export) {

data/webpack/scenes/ContentViews/Details/Histories/__tests__/contentViewHistory.test.js

@@ -35,7 +35,8 @@ test('Can call API and show history on page load', async (done) => {
   // Assert that the repo name is now showing on the screen, but wait for it to appear.
   await patientlyWaitFor(() => expect(getByText(firstHistory.description)).toBeInTheDocument());
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });
 
 test('Can link to view environment', async () => {
@@ -79,5 +80,6 @@ test('Can handle no History being present', async (done) => {
   expect(queryByText(firstHistory.description)).toBeNull();
   await patientlyWaitFor(() => expect(queryByText('History will appear here when the content view is published or promoted.')).toBeInTheDocument());
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });

data/webpack/scenes/ContentViews/Details/Promote/ContentViewVersionPromote.js

@@ -133,7 +133,7 @@ const ContentViewVersionPromote = ({
               aria-label="input_description"
               name="description"
               value={description}
-              onChange={value => setDescription(value)}
+              onChange={(_event, value) => setDescription(value)}
             />
           </FormGroup>
           {!alertDismissed && forcePromote.length > 0 && (

data/webpack/scenes/ContentViews/Details/Repositories/ContentViewRepositories.js

@@ -25,13 +25,15 @@ import {
   ActionListItem,
   Bullseye,
   Button,
-  Dropdown,
-  DropdownItem,
-  KebabToggle,
   Split,
   SplitItem,
   Checkbox,
 } from '@patternfly/react-core';
+import {
+  Dropdown,
+  DropdownItem,
+  KebabToggle,
+} from '@patternfly/react-core/deprecated';
 import {
   TableVariant,
   Thead,
@@ -373,7 +375,7 @@ const ContentViewRepositories = ({ cvId, details }) => {
     >
       <Thead>
         <Tr key="version-header" ouiaId="version-header">
-          {hasPermission(permissions, 'edit_content_views') && <Th key="select-all" />}
+          {hasPermission(permissions, 'edit_content_views') && <Th key="select-all" aria-label="Select all table header" />}
           {columnHeaders.map((title, index) => {
             if (index === 0) {
               return <Th modifier="fitContent" key={`col-header-${title}`}>{title}</Th>;
@@ -402,7 +404,7 @@ const ContentViewRepositories = ({ cvId, details }) => {
                     id={id}
                     ouiaId={`repository-checkbox-${id}`}
                     isChecked={isSelected(id)}
-                    onChange={selected =>
+                    onChange={(_event, selected) =>
                       selectOne(selected, id, repo)
                     }
                   />

data/webpack/scenes/ContentViews/Details/Repositories/__tests__/contentViewAddRemove.test.js

@@ -61,7 +61,8 @@ test('Can enable and disable add repositories button', async (done) => {
   });
   assertNockRequest(autocompleteScope);
   assertNockRequest(noReposScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
 });
 
 test('Can add repositories', async (done) => {
@@ -107,7 +108,8 @@ test('Can add repositories', async (done) => {
   assertNockRequest(cvDetailScope);
   assertNockRequest(autocompleteScope);
   assertNockRequest(noReposScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
   act(done);
 });
 
@@ -149,6 +151,7 @@ test('Can remove repositories', async (done) => {
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(cvDetailScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  done();
   act(done);
 });

data/webpack/scenes/ContentViews/Details/Repositories/__tests__/contentViewRepoAdd.fixture.json

@@ -1,6 +1,7 @@
 {
   "content_host_count": 0,
   "composite": false,
+  "rolling": false,
   "component_ids": [],
   "default": false,
   "version_count": 0,

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/ActionSummary.js

@@ -5,6 +5,7 @@ import {
   FlexItem,
   Label,
   Text,
+  Icon,
 } from '@patternfly/react-core';
 import { ExclamationTriangleIcon } from '@patternfly/react-icons';
 import {
@@ -19,14 +20,14 @@ const ActionSummary = ({ title, text, selectedEnv: { name, id } }) => (
     {text &&
       <Flex>
         <FlexItem style={{ marginRight: '8px' }}>
-          <ExclamationTriangleIcon color={warningColor.value} />
+          <Icon color={warningColor.value}><ExclamationTriangleIcon /></Icon>
         </FlexItem>
         <FlexItem style={{ marginRight: '8px' }}>
           <Text ouiaId="action-summary-text">{text}</Text>
         </FlexItem>
         {name && id &&
           <FlexItem>
-            <Label isTruncated color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>
+            <Label color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>
           </FlexItem>
         }
       </Flex>

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/BulkDeleteModal.js

@@ -4,7 +4,9 @@ import { translate as __ } from 'foremanReact/common/I18n';
 import { PropTypes } from 'prop-types';
 import { FormattedMessage } from 'react-intl';
 
-import { Wizard } from '@patternfly/react-core';
+import {
+  Wizard,
+} from '@patternfly/react-core/deprecated';
 
 import BulkDeleteContextWrapper, {
   BulkDeleteContext,

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/Steps/ConfirmBulkDelete.js

@@ -64,7 +64,7 @@ export default () => {
                   versionOrVersions: affectedVersions.length > 1 ? __('Versions ') : __('Version '),
                   envLabel: (() => {
                     const { id, name } = first(environments);
-                    return <Label isTruncated color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>;
+                    return <Label color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>;
                   })(),
                 }}
                 defaultMessage={pluralEnvironments ?
@@ -77,7 +77,7 @@ export default () => {
             <Flex>
               {environments.map(({ name, id }) => (
                 <FlexItem key={id}>
-                  <Label isTruncated color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>
+                  <Label color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label>
                 </FlexItem>))
               }
             </Flex>}

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/Steps/ReassignActivationKeys.js

@@ -17,9 +17,11 @@ import {
   ExpandableSection,
   Popover,
   PopoverPosition,
+} from '@patternfly/react-core';
+import {
   SelectDirection,
   SelectOption,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { OutlinedQuestionCircleIcon } from '@patternfly/react-icons';
 
 import EnvironmentPaths

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/Steps/ReassignHosts.js

@@ -17,9 +17,11 @@ import {
   ExpandableSection,
   Popover,
   PopoverPosition,
+} from '@patternfly/react-core';
+import {
   SelectDirection,
   SelectOption,
-} from '@patternfly/react-core';
+} from '@patternfly/react-core/deprecated';
 import { OutlinedQuestionCircleIcon } from '@patternfly/react-icons';
 
 import EnvironmentPaths

data/webpack/scenes/ContentViews/Details/Versions/BulkDelete/Steps/ReviewEnvironments.js

@@ -5,7 +5,7 @@ import { FormattedMessage } from 'react-intl';
 
 import { Label } from '@patternfly/react-core';
 import {
-  TableComposable,
+  Table /* data-codemods */,
   TableVariant,
   Tbody,
   Td,
@@ -64,7 +64,7 @@ export default () => {
               __('{versionOrVersions} {versionList} will be removed from the listed environment and will no longer be available for promotion.')}
           />}
       />
-      <TableComposable ouiaId="bulk-delete-env-table" variant={TableVariant.compact}>
+      <Table ouiaId="bulk-delete-env-table" variant={TableVariant.compact}>
         <Thead>
           <Tr ouiaId="bulk-delete-env-header">
             {columnHeaders.map(col =>
@@ -84,7 +84,7 @@ export default () => {
               <Td>
                 <Label
                   /* TODO: Add "isCompact" to this on update of patternfly */
-                  isTruncated
+
                   color="purple"
                 >
                   {name}
@@ -97,7 +97,7 @@ export default () => {
           )))
           }
         </Tbody>
-      </TableComposable>
+      </Table>
     </>
   );
 };

data/webpack/scenes/ContentViews/Details/Versions/Compare/CVVersionCompare.scss

@@ -1,13 +1,13 @@
 .content-view-header-content {
   padding-top: 24px;
 
-  .pf-c-content {
+  .pf-v5-c-content {
     dl {
       margin: 0;
     }
   }
 
-  .pf-l-flex {
+  .pf-v5-l-flex {
     padding-top: 24px;
   }
 }
@@ -20,31 +20,31 @@
   .compare-table-container {
     flex-grow: 4;
   }
-  .pf-c-tabs.pf-m-vertical {
+  .pf-v5-c-tabs.pf-m-vertical {
     max-width: 230px;
     border-right: 1px solid #d2d2d2;
     height: unset;
 
-    .pf-c-tabs__item {
+    .pf-v5-c-tabs__item {
       margin-top: 0;
 
       button {
         align-items: center;
 
-        .pf-c-label__content {
+        .pf-v5-c-label__content {
           &::before {
             border: none;
           }
         }
       }
 
-      &.pf-m-current .pf-c-label__content {
+      &.pf-m-current .pf-v5-c-label__content {
         font-weight: bold;
       }
     }
   }
 
-  .pf-l-grid.pf-m-gutter {
+  .pf-v5-l-grid.pf-m-gutter {
     display: initial;
     width: 100%;
   }
@@ -63,16 +63,16 @@
 
   th:nth-last-child(3) {
     margin-left: 2rem;
-    border-right:  var(--pf-global--BorderWidth--sm);
-    border-right-color: var(--pf-c-table--BorderColor);
+    border-right:  var(--pf-v5-global--BorderWidth--sm);
+    border-right-color: var(--pf-v5-c-table--BorderColor);
     border-right-style: inset;
   }
 }
 
 .cvv-compare-bordered-table-rows {
   td:nth-last-child(3) {
-    border-right:  var(--pf-global--BorderWidth--sm);
-    border-right-color: var(--pf-c-table--BorderColor);
+    border-right:  var(--pf-v5-global--BorderWidth--sm);
+    border-right-color: var(--pf-v5-c-table--BorderColor);
     border-right-style: inset;
   }
 

data/webpack/scenes/ContentViews/Details/Versions/Compare/CVVersionCompareHeader.js

@@ -3,9 +3,19 @@ import PropTypes from 'prop-types';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { useSelector } from 'react-redux';
 import {
-  Grid, GridItem, TextContent, Text, TextVariants,
-  Select, SelectOption, SelectVariant, Flex, FlexItem,
+  Grid,
+  GridItem,
+  TextContent,
+  Text,
+  TextVariants,
+  Flex,
+  FlexItem,
 } from '@patternfly/react-core';
+import {
+  Select,
+  SelectOption,
+  SelectVariant,
+} from '@patternfly/react-core/deprecated';
 import { selectCVDetails } from '../../ContentViewDetailSelectors';
 import { HelpToolTip } from '../../../Create/ContentViewFormComponents';
 
@@ -97,7 +107,7 @@ const CVVersionCompareHeader = ({
                       placeholderText={__('Select an option')}
                       aria-label="Select version one"
                       ouiaId="select-version-one"
-                      onToggle={setIsOpenSelectVersionOne}
+                      onToggle={(_event, val) => setIsOpenSelectVersionOne(val)}
                       onSelect={onSelectVersionOne}
                       selections={versionOne}
                       isOpen={isOpenSelectVersionOne}
@@ -124,7 +134,7 @@ const CVVersionCompareHeader = ({
                       placeholderText="Select an option"
                       aria-label="Select version two"
                       ouiaId="select-version-two"
-                      onToggle={setIsOpenSelectVersionTwo}
+                      onToggle={(_event, val) => setIsOpenSelectVersionTwo(val)}
                       onSelect={onSelectVersionTwo}
                       selections={versionTwo}
                       isOpen={isOpenSelectVersionTwo}
@@ -154,7 +164,7 @@ const CVVersionCompareHeader = ({
                   placeholderText="Select an option"
                   aria-label="Select view by"
                   ouiaId="select-view-by"
-                  onToggle={setIsOpenSelectViewBy}
+                  onToggle={(_event, val) => setIsOpenSelectViewBy(val)}
                   onSelect={onSelectViewBy}
                   selections={selectedViewBy}
                   isOpen={isOpenSelectViewBy}

data/webpack/scenes/ContentViews/Details/Versions/ContentViewVersionEnvironments.js

@@ -15,7 +15,7 @@ const ContentViewVersionEnvironments = ({ environments }) => {
     <React.Fragment key={env.id}>
       <Flex style={{ margin: '4px 0' }} >
         <FlexItem>
-          <Label isTruncated color="purple" href={`/lifecycle_environments/${env.id}`}>{env.name}</Label>
+          <Label color="purple" href={`/lifecycle_environments/${env.id}`}>{env.name}</Label>
         </FlexItem>
         <FlexItem>
           <InactiveText text={` ${env.publish_date} ago`} />

data/webpack/scenes/ContentViews/Details/Versions/ContentViewVersions.js

@@ -1,7 +1,17 @@
 import React, { useState, useEffect, useCallback } from 'react';
 import { useDispatch, useSelector } from 'react-redux';
 import { TableVariant, TableText, Tbody, Thead, Td, Tr, Th } from '@patternfly/react-table';
-import { Checkbox, Dropdown, DropdownItem, Grid, KebabToggle, GridItem, Button } from '@patternfly/react-core';
+import {
+  Checkbox,
+  Grid,
+  GridItem,
+  Button,
+} from '@patternfly/react-core';
+import {
+  Dropdown,
+  DropdownItem,
+  KebabToggle,
+} from '@patternfly/react-core/deprecated';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { urlBuilder } from 'foremanReact/common/urlHelpers';
 import { STATUS } from 'foremanReact/constants';
@@ -151,7 +161,7 @@ const ContentViewVersions = ({ cvId, details }) => {
         id={versionId}
         aria-label={`Select version ${versionId}`}
         isChecked={isSelected(versionId)}
-        onChange={selected => selectOne(selected, versionId)}
+        onChange={(_event, selected) => selectOne(selected, versionId)}
       />,
       <>
         <Link to={`/versions/${versionId}`}>{__('Version ')}{version}</Link>
@@ -303,7 +313,7 @@ const ContentViewVersions = ({ cvId, details }) => {
               </GridItem>
               <GridItem md={4} sm={12}>
                 <Dropdown
-                  toggle={<KebabToggle aria-label="bulk_actions" onToggle={setKebabOpen} />}
+                  toggle={<KebabToggle aria-label="bulk_actions" onToggle={(_event, val) => setKebabOpen(val)} />}
                   isOpen={kebabOpen}
                   ouiaId="cv-versions-bulk-actions"
                   isPlain

data/webpack/scenes/ContentViews/Details/Versions/Delete/RemoveCVVersionWizard.js

@@ -1,7 +1,9 @@
 import React, { useEffect, useState } from 'react';
 import { useDispatch } from 'react-redux';
 import PropTypes from 'prop-types';
-import { Wizard } from '@patternfly/react-core';
+import {
+  Wizard,
+} from '@patternfly/react-core/deprecated';
 import { translate as __ } from 'foremanReact/common/I18n';
 import { useSet } from 'foremanReact/components/PF4/TableIndexPage/Table/TableHooks';
 import getEnvironmentPaths from '../../../components/EnvironmentPaths/EnvironmentPathActions';