katello 4.5.1 → 4.8.0.rc2

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetailConfig.js

@@ -9,6 +9,7 @@ import {
   SecurityIcon,
   EnhancementIcon,
 } from '@patternfly/react-icons';
+import { Tooltip } from '@patternfly/react-core';
 import {
   getContentViewVersions,
   getDebPackages,
@@ -54,6 +55,8 @@ export const TableType = PropTypes.shape({
   responseSelector: PropTypes.func,
   statusSelector: PropTypes.func,
   autocompleteEndpoint: PropTypes.string,
+  autocompleteQueryParams: PropTypes.shape({}),
+  bookmarkController: PropTypes.string,
   fetchItems: PropTypes.func,
   columnHeaders:
     PropTypes.arrayOf(PropTypes.shape({
@@ -71,6 +74,8 @@ export default ({ cvId, versionId }) => [
     responseSelector: state => selectCVVersions(state, cvId),
     statusSelector: state => selectCVVersionsStatus(state, cvId),
     autocompleteEndpoint: '',
+    autocompleteQueryParams: undefined,
+    bookmarkController: '',
     fetchItems: params => getContentViewVersions(
       cvId,
       { composite_version_id: versionId, ...params, content_view_id: undefined },
@@ -90,7 +95,7 @@ export default ({ cvId, versionId }) => [
           <LongDateTime date={item.updated_at} showRelativeTimeTooltip />,
       },
     ],
-    disableSearch: true,
+    hideSearch: true,
   },
   {
     name: __('Repositories'),
@@ -98,7 +103,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.repositories?.length,
     responseSelector: state => selectRepositories(state),
     statusSelector: state => selectRepositoriesStatus(state),
-    autocompleteEndpoint: `/repositories/auto_complete_search?archived=true&content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/repositories',
+    autocompleteQueryParams: { archived: true, content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_repositories',
     fetchItems: params => getRepositories({
       content_view_version_id: versionId,
       archived: true,
@@ -108,7 +115,7 @@ export default ({ cvId, versionId }) => [
       {
         title: __('Name'),
         getProperty: item => (
-          <a href={urlBuilder(`products/${item?.product?.id}/repositories/${item?.id}`, '')}>
+          <a href={urlBuilder(`products/${item?.product?.id}/repositories/${item?.library_instance_id ?? item?.id}`, '')}>
             {item?.name}
           </a>),
       },
@@ -136,7 +143,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.rpm_count,
     responseSelector: state => selectRPMPackages(state),
     statusSelector: state => selectRPMPackagesStatus(state),
-    autocompleteEndpoint: `/packages/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/packages',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getRPMPackages({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -159,7 +168,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.package_group_count,
     responseSelector: state => selectRPMPackageGroups(state),
     statusSelector: state => selectRPMPackageGroupsStatus(state),
-    autocompleteEndpoint: `/package_groups/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/package_groups',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getPackageGroups({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       { title: __('Name'), getProperty: item => item?.name },
@@ -173,7 +184,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.file_count,
     responseSelector: state => selectFiles(state),
     statusSelector: state => selectFilesStatus(state),
-    autocompleteEndpoint: `/files/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/files',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getFiles({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -193,7 +206,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.erratum_count,
     responseSelector: state => selectErrata(state),
     statusSelector: state => selectErrataStatus(state),
-    autocompleteEndpoint: `/errata/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/errata',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getErrata({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -217,8 +232,9 @@ export default ({ cvId, versionId }) => [
             enhancements: EnhancementIcon,
           };
           const ErrataIcon = errataIcons[item?.type];
-          if (!ErrataIcon) return startCase(item?.type);
-          return <><ErrataIcon />{' '}{startCase(item?.type)}{item?.severity && ` - ${item.severity}`}</>;
+          const itemType = startCase(item?.type);
+          if (!ErrataIcon) return itemType;
+          return <><Tooltip content={itemType} ><ErrataIcon style={{ marginRight: '4px' }} /></Tooltip>{itemType}{item?.severity !== 'None' && ` - ${item.severity}`}</>;
         },
       },
       {
@@ -247,7 +263,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.module_stream_count,
     responseSelector: state => selectModuleStreams(state),
     statusSelector: state => selectModuleStreamsStatus(state),
-    autocompleteEndpoint: `/module_streams/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/module_streams',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getModuleStreams({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -270,7 +288,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.deb_count,
     responseSelector: state => selectDebPackages(state),
     statusSelector: state => selectDebPackagesStatus(state),
-    autocompleteEndpoint: `/debs/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/debs',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getDebPackages({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -291,7 +311,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item?.docker_tag_count,
     responseSelector: state => selectDockerTags(state),
     statusSelector: state => selectDockerTagsStatus(state),
-    autocompleteEndpoint: `/docker_tags/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: '/katello/api/v2/docker_tags',
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params => getDockerTags({ content_view_version_id: versionId, ...params }),
     columnHeaders: [
       {
@@ -321,7 +343,9 @@ export default ({ cvId, versionId }) => [
     getCountKey: item => item[`${singularLabel}_count`],
     responseSelector: state => selectContent(pluralLabel, state),
     statusSelector: state => selectContentStatus(pluralLabel, state),
-    autocompleteEndpoint: `/${pluralLabel}/auto_complete_search?content_view_version_id=${versionId}`,
+    autocompleteEndpoint: `/katello/api/v2/${pluralLabel}`,
+    autocompleteQueryParams: { content_view_version_id: versionId },
+    bookmarkController: 'katello_content_view_components',
     fetchItems: params =>
       getContent(pluralLabel, { content_view_version_id: versionId, ...params }),
     columnHeaders,

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetails.js

@@ -70,6 +70,7 @@ const ContentViewVersionDetails = ({ cvId, details }) => {
         <div className="grid-with-top-border">
           <Tabs
             activeKey={getCurrentActiveKey}
+            ouiaId="cv-version-details-tabs"
             onSelect={onSelect}
             isVertical
           >

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetailsHeader.js

@@ -49,6 +49,7 @@ const ContentViewVersionDetailsHeader = ({
   const [bulkDeleteOpen, setBulkDeleteOpen] = useState(false);
   const dropDownItems = [
     <DropdownItem
+      ouiaId="remove"
       key="remove"
       onClick={() => {
         setRemovingFromEnv(true);
@@ -57,6 +58,7 @@ const ContentViewVersionDetailsHeader = ({
       {__('Remove from environment')}
     </DropdownItem>,
     <DropdownItem
+      ouiaId="delete"
       key="delete"
       onClick={() =>
         setBulkDeleteOpen(true)
@@ -70,7 +72,7 @@ const ContentViewVersionDetailsHeader = ({
     <Grid className="margin-0-24">
       <GridItem sm={6} >
         <TextContent>
-          <Text component={TextVariants.h2}>{__('Version ')}{version}</Text>
+          <Text ouiaId="cv-version" component={TextVariants.h2}>{__('Version ')}{version}</Text>
         </TextContent>
       </GridItem>
       <GridItem sm={6} style={{ display: 'flex' }}>
@@ -85,6 +87,7 @@ const ContentViewVersionDetailsHeader = ({
         </Button>
         <Dropdown
           isPlain
+          ouiaId="cv-version-header-actions-dropdown"
           style={{ width: 'inherit' }}
           position={DropdownPosition.right}
           toggle={

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/ContentViewVersionDetailsTable.js

@@ -32,9 +32,11 @@ const ContentViewVersionDetailsTable = ({
     responseSelector,
     statusSelector,
     autocompleteEndpoint,
+    autocompleteQueryParams,
+    bookmarkController,
     fetchItems,
     columnHeaders,
-    disableSearch,
+    hideSearch,
     route,
   }, repositories,
 }) => {
@@ -77,7 +79,9 @@ const ContentViewVersionDetailsTable = ({
           updateSearchQuery,
           status,
           autocompleteEndpoint,
-          disableSearch,
+          autocompleteQueryParams,
+          bookmarkController,
+          hideSearch,
         }}
         ouiaId={`content-view-version-details-${kebabCase(route)}-table`}
         additionalListeners={[selected]}
@@ -92,6 +96,7 @@ const ContentViewVersionDetailsTable = ({
           <Select
             onToggle={setOpen}
             isOpen={open}
+            ouiaId="repo-type-selector"
             variant={SelectVariant.single}
             onSelect={(_e, selection) => {
               const index = selectedList
@@ -108,14 +113,14 @@ const ContentViewVersionDetailsTable = ({
         }
       >
         <Thead>
-          <Tr>
+          <Tr ouiaId="column-headers">
             {columnHeaders.map(({ title, width }) =>
               <Th width={width} key={`${title}-header`}>{title}</Th>)}
           </Tr>
         </Thead>
         <Tbody>
           {results?.map((result, index) => (
-            <Tr key={`column-${index}`}>
+            <Tr key={`column-${index}`} ouiaId={`column-${index}`}>
               {columnHeaders.map(({ getProperty }, colIndex) =>
                 <Td key={`cell-${colIndex}`}>{getProperty(result)} </Td>)}
             </Tr>

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/__tests__/ContentViewVersionDetails.test.js

@@ -2,11 +2,10 @@ import React from 'react';
 import { renderWithRedux, patientlyWaitFor } from 'react-testing-lib-wrapper';
 import { Route } from 'react-router-dom';
 import { head, last } from 'lodash';
-import nock, { nockInstance, assertNockRequest, mockSetting, mockAutocomplete } from '../../../../../../test-utils/nockWrapper';
+import { nockInstance, assertNockRequest, mockAutocomplete } from '../../../../../../test-utils/nockWrapper';
 import api from '../../../../../../services/api';
 import { cvVersionDetailsKey } from '../../../../ContentViewsConstants';
 import ContentViewVersionDetails from '../ContentViewVersionDetails';
-import { AUTOSEARCH_DELAY, AUTOSEARCH_WHILE_TYPING } from '../../../../../Settings/SettingsConstants';
 import ContentViewVersionDetailsData from './ContentViewVersionDetails.fixtures.json';
 import ContentViewVersionDetailsCounts from './ContentViewVersionDetailsCounts.fixtures.json';
 import cvDetailData from '../../../../__tests__/mockDetails.fixtures.json';
@@ -42,6 +41,28 @@ const renderOptions = {
   },
 };
 
+const autocompleteQuery = name => ((name === 'Repositories') ? {
+  archived: true,
+  organization_id: 1,
+  content_view_version_id: 73,
+  search: '',
+} : {
+  organization_id: 1,
+  content_view_version_id: 73,
+  search: '',
+});
+
+const queryParams = name => ((name === 'Repositories') ? {
+  archived: true,
+  content_view_version_id: 73,
+  per_page: 20,
+  page: 1,
+} : {
+  content_view_version_id: 73,
+  per_page: 20,
+  page: 1,
+});
+
 beforeEach(() => {
   envScope = nockInstance
     .get(environmentPathsPath)
@@ -51,7 +72,6 @@ beforeEach(() => {
 
 afterEach(() => {
   assertNockRequest(envScope);
-  nock.cleanAll();
 });
 // This is written separately, as the autocomplete/search scopes are not needed.
 test('Can show versions details - Components Tab', async (done) => {
@@ -91,7 +111,6 @@ test('Can show versions details - Components Tab', async (done) => {
   assertNockRequest(componentScope, done);
 });
 
-
 const testConfig = [
   {
     name: 'Repositories',
@@ -197,9 +216,11 @@ testConfig.forEach(({
   test(`Can show versions details - ${name} Tab`, async (done) => {
     const { version } = ContentViewVersionDetailsData;
 
-    const autocompleteScope = mockAutocomplete(nockInstance, autoCompleteUrl);
-    const searchDelayScope = mockSetting(nockInstance, AUTOSEARCH_DELAY);
-    const autoSearchScope = mockSetting(nockInstance, AUTOSEARCH_WHILE_TYPING);
+    const autocompleteScope = mockAutocomplete(
+      nockInstance,
+      autoCompleteUrl,
+      autocompleteQuery(name),
+    );
 
     const scope = nockInstance
       .get(cvVersions)
@@ -208,7 +229,7 @@ testConfig.forEach(({
 
     const tabScope = nockInstance
       .get(dataUrl)
-      .query(true)
+      .query(queryParams(name))
       .reply(200, data);
 
     const { getByText, queryByText } = renderWithRedux(
@@ -232,8 +253,6 @@ testConfig.forEach(({
     });
 
     assertNockRequest(autocompleteScope);
-    assertNockRequest(searchDelayScope);
-    assertNockRequest(autoSearchScope);
     assertNockRequest(scope);
     assertNockRequest(tabScope);
     assertNockRequest(scope, done);
@@ -248,9 +267,7 @@ test('Can change repository selector', async (done) => {
   } = testConfig[1]; // RPM Packages
 
   const { version } = ContentViewVersionDetailsData;
-  const autocompleteScope = mockAutocomplete(nockInstance, autoCompleteUrl);
-  const searchDelayScope = mockSetting(nockInstance, AUTOSEARCH_DELAY, 0);
-  const autoSearchScope = mockSetting(nockInstance, AUTOSEARCH_WHILE_TYPING);
+  const autocompleteScope = mockAutocomplete(nockInstance, autoCompleteUrl, autocompleteQuery);
 
   const scope = nockInstance
     .get(cvVersions)
@@ -298,8 +315,6 @@ test('Can change repository selector', async (done) => {
   });
 
   assertNockRequest(autocompleteScope);
-  assertNockRequest(searchDelayScope);
-  assertNockRequest(autoSearchScope);
   assertNockRequest(scope);
   assertNockRequest(tabScope);
   assertNockRequest(scope, done);

data/webpack/scenes/ContentViews/Details/Versions/VersionDetails/__tests__/ContentViewVersionDetailsEmpty.test.js

@@ -2,7 +2,7 @@ import React from 'react';
 import { renderWithRedux, patientlyWaitFor, act } from 'react-testing-lib-wrapper';
 import { Route } from 'react-router-dom';
 
-import nock, { nockInstance, assertNockRequest } from '../../../../../../test-utils/nockWrapper';
+import { nockInstance, assertNockRequest } from '../../../../../../test-utils/nockWrapper';
 import api from '../../../../../../services/api';
 import { cvVersionDetailsKey } from '../../../../ContentViewsConstants';
 import ContentViewVersionDetails from '../ContentViewVersionDetails';
@@ -39,7 +39,6 @@ beforeEach(() => {
 afterEach(() => {
   assertNockRequest(envScope);
   assertNockRequest(versionScope);
-  nock.cleanAll();
 });
 
 test('Can show versions detail header', async (done) => {

data/webpack/scenes/ContentViews/Details/Versions/__tests__/contentViewVersions.test.js

@@ -1,7 +1,7 @@
 import React from 'react';
 import { act, renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
 import { Route } from 'react-router-dom';
-import nock, { nockInstance, assertNockRequest, mockAutocomplete, mockSetting } from '../../../../../test-utils/nockWrapper';
+import { nockInstance, assertNockRequest, mockAutocomplete, mockSetting } from '../../../../../test-utils/nockWrapper';
 import api from '../../../../../services/api';
 import CONTENT_VIEWS_KEY from '../../../ContentViewsConstants';
 import ContentViewVersions from '../ContentViewVersions';
@@ -12,8 +12,10 @@ import contentViewTaskInProgressResponseData from './contentViewTaskInProgressRe
 import contentViewTaskResponseData from './contentViewTaskResponse.fixtures.json';
 import cvDetailData from '../../../../ContentViews/__tests__/mockDetails.fixtures.json';
 import environmentPathsData from '../../../Publish/__tests__/environmentPaths.fixtures.json';
+import cvIndexData from '../../../__tests__/contentViewList.fixtures.json';
 
 const cvPromotePath = api.getApiUrl('/content_view_versions/10/promote');
+const cvIndexPath = api.getApiUrl('/content_views');
 const promoteResponseData = contentViewTaskInProgressResponseData;
 
 
@@ -50,7 +52,6 @@ beforeEach(() => {
 });
 
 afterEach(() => {
-  nock.cleanAll();
   assertNockRequest(envScope);
   assertNockRequest(searchDelayScope);
   assertNockRequest(autoSearchScope);
@@ -75,10 +76,11 @@ test('Can call API and show versions on page load', async (done) => {
     expect(getByText(`Version ${firstVersion.version}`)).toBeTruthy();
   });
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope, done);
+  assertNockRequest(scope);
+  act(done);
 });
 
-test('Can link to view environment and see publish time', async () => {
+test('Can link to view environment and see publish time', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(cvVersions)
@@ -105,9 +107,10 @@ test('Can link to view environment and see publish time', async () => {
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
+  act(done);
 });
 
-test('Can show package and erratas and link to list page', async () => {
+test('Can show package and erratas and link to list page', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(cvVersions)
@@ -132,9 +135,10 @@ test('Can show package and erratas and link to list page', async () => {
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
+  act(done);
 });
 
-test('Can show additional content and link to list page', async () => {
+test('Can show additional content and link to list page', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(cvVersions)
@@ -157,9 +161,10 @@ test('Can show additional content and link to list page', async () => {
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
+  act(done);
 });
 
-test('Can load for empty versions', async () => {
+test('Can load for empty versions', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(cvVersions)
@@ -176,6 +181,7 @@ test('Can load for empty versions', async () => {
     expect(queryByText("You currently don't have any versions for this content view.")).toBeInTheDocument());
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
+  act(done);
 });
 
 test('Can call API and show versions with tasks on page load', async (done) => {
@@ -211,10 +217,66 @@ test('Can call API and show versions with tasks on page load', async (done) => {
 
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
-  assertNockRequest(taskInProgressScope, done);
+  assertNockRequest(taskInProgressScope);
   act(done);// Force the test to stop listening for tasks
 });
 
+test('Can open Promote Modal', async (done) => {
+  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
+  const cvScope = nockInstance
+    .get(cvIndexPath)
+    .query(true)
+    .reply(200, cvIndexData);
+  const scope = nockInstance
+    .get(cvVersions)
+    .query(true)
+    .reply(200, cvVersionsData);
+
+  const cvPromoteParams = {
+    id: 10, versionEnvironments: [], description: '', environment_ids: [5], force: true,
+  };
+
+  const promoteScope = nockInstance
+    .post(cvPromotePath, cvPromoteParams)
+    .reply(202, promoteResponseData);
+
+  const {
+    getByText, queryByText, getByLabelText, getAllByLabelText,
+  } = renderWithRedux(
+    withCVRoute(<ContentViewVersions cvId={5} details={cvDetailData} />),
+    renderOptions,
+  );
+
+  expect(queryByText(`Version ${firstVersion.version}`)).toBeNull();
+  await patientlyWaitFor(() => {
+    expect(getByText(`Version ${firstVersion.version}`)).toBeInTheDocument();
+  });
+  // Expand Row Action
+  expect(getAllByLabelText('Actions')[1]).toHaveAttribute('aria-expanded', 'false');
+  fireEvent.click(getAllByLabelText('Actions')[1]);
+  expect(getAllByLabelText('Actions')[1]).toHaveAttribute('aria-expanded', 'true');
+  fireEvent.click(getByText('Promote'));
+  await patientlyWaitFor(() => {
+    expect(getByText('Select a lifecycle environment from the available promotion paths to promote new version.')).toBeInTheDocument();
+    expect(getByLabelText('prod')).toBeInTheDocument();
+  });
+  // Select env prod
+  fireEvent.click(getByLabelText('prod'));
+  fireEvent.click(getByLabelText('promote_content_view'));
+  // Modal closes itself
+  await patientlyWaitFor(() => {
+    expect(queryByText('Select a lifecycle environment from the available promotion paths to promote new version.')).toBeNull();
+    expect(getByText(`Version ${firstVersion.version}`)).toBeInTheDocument();
+  });
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(scope);
+  assertNockRequest(promoteScope);
+  // Page is refreshed
+  assertNockRequest(scope);
+  assertNockRequest(cvScope);
+  act(done);
+});
+
 test('Can reload versions upon task completion', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const { results: withTaskResults } = cvVersionsTasksData;
@@ -223,6 +285,10 @@ test('Can reload versions upon task completion', async (done) => {
     .get(cvVersions)
     .query(true)
     .reply(200, cvVersionsTasksData);
+  const cvScope = nockInstance
+    .get(cvIndexPath)
+    .query(true)
+    .reply(200, cvIndexData);
 
   const taskSuccessScope = nockInstance
     .get(taskPollingUrl)
@@ -253,56 +319,63 @@ test('Can reload versions upon task completion', async (done) => {
   assertNockRequest(scope);
   assertNockRequest(taskSuccessScope);
   // Assert CV Versions API is called upon task completion
-  assertNockRequest(reloadScope, done);
+  assertNockRequest(reloadScope);
+  assertNockRequest(cvScope);
   act(done);
 });
 
-test('Can open Promote Modal', async (done) => {
+test('Shows call-to-action when there are no versions', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const scope = nockInstance
     .get(cvVersions)
     .query(true)
-    .reply(200, cvVersionsData);
-
-  const cvPromoteParams = {
-    id: 10, versionEnvironments: [], description: '', environment_ids: [5], force: true,
-  };
+    .reply(200, emptyCVVersionData);
 
-  const promoteScope = nockInstance
-    .post(cvPromotePath, cvPromoteParams)
-    .reply(202, promoteResponseData);
+  const scopeWizard = nockInstance
+    .get(environmentPathsPath)
+    .query(true)
+    .reply(200, environmentPathsData);
 
-  const {
-    getByText, queryByText, getByLabelText, getAllByLabelText,
-  } = renderWithRedux(
+  const { getByText, queryByText } = renderWithRedux(
     withCVRoute(<ContentViewVersions cvId={5} details={cvDetailData} />),
     renderOptions,
   );
 
-  expect(queryByText(`Version ${firstVersion.version}`)).toBeNull();
-  await patientlyWaitFor(() => {
-    expect(getByText(`Version ${firstVersion.version}`)).toBeInTheDocument();
-  });
-  // Expand Row Action
-  expect(getAllByLabelText('Actions')[1]).toHaveAttribute('aria-expanded', 'false');
-  fireEvent.click(getAllByLabelText('Actions')[1]);
-  expect(getAllByLabelText('Actions')[1]).toHaveAttribute('aria-expanded', 'true');
-  fireEvent.click(getByText('Promote'));
+  // Nothing will show at first, page is loading
+  expect(queryByText('Publish new version')).toBeNull();
+  // Assert that the CV version is now showing on the screen, but wait for it to appear.
   await patientlyWaitFor(() => {
-    expect(getByText('Select a lifecycle environment from the available promotion paths to promote new version.')).toBeInTheDocument();
-    expect(getByLabelText('prod')).toBeInTheDocument();
+    expect(getByText('Publish new version')).toBeInTheDocument();
   });
-  // Select env prod
-  fireEvent.click(getByLabelText('prod'));
-  fireEvent.click(getByLabelText('promote_content_view'));
-  // Modal closes itself
+  fireEvent.click(getByText('Publish new version'));
+  await patientlyWaitFor(() => expect(getByText('Publish new version - 1.0')).toBeInTheDocument);
+
+  assertNockRequest(scopeWizard);
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(scope);
+  act(done);
+});
+
+test('Hides selectAllCheckbox when there are no versions', async (done) => {
+  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
+  const scope = nockInstance
+    .get(cvVersions)
+    .query(true)
+    .reply(200, emptyCVVersionData);
+
+  const { queryByLabelText, queryByText, getByText } = renderWithRedux(
+    withCVRoute(<ContentViewVersions cvId={5} details={cvDetailData} />),
+    renderOptions,
+  );
+  // Nothing will show at first, page is loading
+  expect(queryByText('Publish new version')).toBeNull();
+  // Assert that the CV version is now showing on the screen, but wait for it to appear.
   await patientlyWaitFor(() => {
-    expect(queryByText('Select a lifecycle environment from the available promotion paths to promote new version.')).toBeNull();
+    expect(getByText('Publish new version')).toBeInTheDocument();
+    expect(queryByLabelText('Select all')).not.toBeInTheDocument();
   });
+
   assertNockRequest(autocompleteScope);
   assertNockRequest(scope);
-  assertNockRequest(promoteScope);
-  // Page is refreshed
-  assertNockRequest(scope, done);
   act(done);
 });

data/webpack/scenes/ContentViews/Publish/CVPublishForm.js

@@ -43,7 +43,9 @@ const CVPublishForm = ({
           </>}
       />
       <TextContent>
-        <Text component={TextVariants.h3}>{__('Publish new version - ')}{nextVersion || '1.0'}</Text>
+        <Text ouiaId="next-version-text" component={TextVariants.h3}>
+          {__('Publish new version - ')}{nextVersion || '1.0'}
+        </Text>
       </TextContent>
       <Form>
         <FormGroup label={__('Description')} fieldId="description">
@@ -60,6 +62,7 @@ const CVPublishForm = ({
         <FormGroup label={__('Promote')} fieldId="promote">
           <Switch
             id="promote-switch"
+            ouiaId="promote-switch"
             aria-label="promote-switch"
             isChecked={promote}
             onChange={checkPromote}
@@ -68,6 +71,7 @@ const CVPublishForm = ({
       </Form>
       {!alertDismissed && promote && forcePromote.length > 0 && (
         <Alert
+          ouiaId="force-promotion-alert"
           variant="info"
           isInline
           title={__('Force promotion')}

data/webpack/scenes/ContentViews/Publish/CVPublishReview.js

@@ -42,16 +42,16 @@ const CVPublishReview = ({
             {__('Review your currently selected changes for ')}<b>{composite ? <RegistryIcon /> : <EnterpriseIcon />} {name}.</b>
           </>}
       />
-      <TableComposable aria-label="Review Table">
+      <TableComposable ouiaId="cv-publish-review-table" aria-label="Review Table">
         <Thead>
-          <Tr>
+          <Tr ouiaId="cv-publish-review-table-headers">
             <Th>{__('Content view name')}</Th>
             <Th>{__('Version')}</Th>
             <Th>{__('Environments')}</Th>
           </Tr>
         </Thead>
         <Tbody>
-          <Tr>
+          <Tr ouiaId="cv-publish-review-table-row">
             <Td>
               <><ContentViewIcon composite={composite} description={name} /><InactiveText text={__('Newly published')} /></>
             </Td>