katello 4.5.0.rc1 → 4.5.0.rc2

data/webpack/components/extensions/HostDetails/Tabs/__tests__/packageInstallModal.test.js

@@ -4,9 +4,17 @@ import { nockInstance, assertNockRequest, mockForemanAutocomplete, mockSetting }
 import katelloApi, { foremanApi } from '../../../../../services/api';
 import mockPackagesData from './yumInstallablePackages.fixtures.json';
 import PackageInstallModal from '../PackagesTab/PackageInstallModal';
-import { HOST_YUM_INSTALLABLE_PACKAGES_KEY, PACKAGE_SEARCH_QUERY } from '../PackagesTab/YumInstallablePackagesConstants';
+import { HOST_YUM_INSTALLABLE_PACKAGES_KEY } from '../PackagesTab/YumInstallablePackagesConstants';
 import { REX_FEATURES } from '../RemoteExecutionConstants';
 
+jest.mock('../../hostDetailsHelpers', () => ({
+  ...jest.requireActual('../../hostDetailsHelpers'),
+  userPermissionsFromHostDetails: () => ({
+    create_job_invocations: true,
+    edit_hosts: true,
+  }),
+}));
+
 const contentFacetAttributes = {
   id: 11,
   uuid: 'e5761ea3-4117-4ecf-83d0-b694f99b389e',
@@ -32,7 +40,6 @@ const renderOptions = (facetAttributes = contentFacetAttributes) => ({
 
 const hostYumInstallablePackages = katelloApi.getApiUrl('/packages');
 const hostPackages = foremanApi.getApiUrl('/hosts/1/packages/install');
-const jobInvocations = foremanApi.getApiUrl('/job_invocations');
 const autocompleteUrl = '/hosts/1/packages/auto_complete_search';
 const fakeTask = { id: '21c0f9e4-b27b-49aa-8774-6be66126043b' };
 
@@ -77,6 +84,7 @@ test('Can call API for installable packages and show on screen on page load', as
        hostId={1}
        hostName="test-host"
        showKatelloAgent={false}
+       triggerPackageInstall={jest.fn()}
      />, renderOptions());
 
   // Assert that the packages are now showing on the screen, but wait for them to appear.
@@ -109,6 +117,7 @@ test('Can handle no installable packages being present', async (done) => {
       hostId={1}
       hostName="test-host"
       showKatelloAgent={false}
+      triggerPackageInstall={jest.fn()}
     />, renderOptions());
 
   // Assert that there are not any packages showing on the screen.
@@ -134,6 +143,7 @@ test('Does not show katello-agent option when disabled', async (done) => {
     hostId={1}
     hostName="test-host"
     showKatelloAgent={false}
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   // Assert that the packages are now showing on the screen, but wait for them to appear.
@@ -172,6 +182,7 @@ test('Shows the katello-agent option when enabled', async (done) => {
     hostId={1}
     hostName="test-host"
     showKatelloAgent
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   // Assert that the packages are now showing on the screen, but wait for them to appear.
@@ -209,6 +220,7 @@ test('Can install packages via katello-agent', async (done) => {
     hostId={1}
     hostName="test-host"
     showKatelloAgent
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   await patientlyWaitFor(() => expect(getAllByText(firstPackages.name)[0]).toBeInTheDocument());
@@ -236,17 +248,7 @@ test('Can install a package via remote execution', async (done) => {
     .get(hostYumInstallablePackages)
     .query(defaultQuery)
     .reply(200, mockPackagesData);
-  const installScope = nockInstance
-    .post(jobInvocations, {
-      job_invocation: {
-        inputs: {
-          [PACKAGE_SEARCH_QUERY]: `id ^ (${firstPackages.id},${secondPackages.id})`,
-        },
-        search_query: 'name ^ (test-host)',
-        feature: REX_FEATURES.KATELLO_PACKAGE_INSTALL_BY_SEARCH,
-      },
-    })
-    .reply(201);
+  const triggerPackageInstall = jest.fn();
 
   const {
     getAllByText, getByText, getByRole,
@@ -256,6 +258,7 @@ test('Can install a package via remote execution', async (done) => {
     hostId={1}
     hostName="test-host"
     showKatelloAgent
+    triggerPackageInstall={triggerPackageInstall}
   />, renderOptions());
 
   await patientlyWaitFor(() => expect(getAllByText(firstPackages.name)[0]).toBeInTheDocument());
@@ -271,9 +274,9 @@ test('Can install a package via remote execution', async (done) => {
   const rexOption = getByText('Install via remote execution');
   fireEvent.click(rexOption);
 
+  expect(triggerPackageInstall).toHaveBeenCalled();
   assertNockRequest(autocompleteScope);
-  assertNockRequest(scope);
-  assertNockRequest(installScope, done);
+  assertNockRequest(scope, done);
 });
 
 test('Can install a package via customized remote execution', async (done) => {
@@ -290,6 +293,7 @@ test('Can install a package via customized remote execution', async (done) => {
     closeModal={jest.fn()}
     hostId={1}
     hostName="test-host"
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   await patientlyWaitFor(() => expect(getAllByText(firstPackages.name)[0]).toBeInTheDocument());
@@ -326,6 +330,7 @@ test('Uses package_install_by_search_query template when in select all mode', as
     closeModal={jest.fn()}
     hostId={1}
     hostName="test-host"
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   await patientlyWaitFor(() => expect(getAllByText(firstPackages.name)[0]).toBeInTheDocument());
@@ -364,6 +369,7 @@ test('Disables the katello-agent option when in select all mode', async (done) =
     hostId={1}
     hostName="test-host"
     showKatelloAgent
+    triggerPackageInstall={jest.fn()}
   />, renderOptions());
 
   await patientlyWaitFor(() => expect(getAllByText(firstPackages.name)[0]).toBeInTheDocument());

data/webpack/components/extensions/HostDetails/Tabs/__tests__/packagesTab.test.js

@@ -9,6 +9,14 @@ import { REX_FEATURES } from '../RemoteExecutionConstants';
 import * as hooks from '../../../../Table/TableHooks';
 import mockBookmarkData from './bookmarks.fixtures.json';
 
+jest.mock('../../hostDetailsHelpers', () => ({
+  ...jest.requireActual('../../hostDetailsHelpers'),
+  userPermissionsFromHostDetails: () => ({
+    create_job_invocations: true,
+    edit_hosts: true,
+  }),
+}));
+
 const contentFacetAttributes = {
   id: 11,
   uuid: 'e5761ea3-4117-4ecf-83d0-b694f99b389e',

data/webpack/components/extensions/HostDetails/Tabs/__tests__/repositorySets.fixtures.json

@@ -41,6 +41,7 @@
       "type": "yum",
       "gpgUrl": null,
       "contentUrl": "/custom/ParthaProduct/empty_repo",
+      "osRestricted": null,
       "override": "default",
       "overrides": [],
       "enabled_content_override": null
@@ -75,6 +76,7 @@
       "type": "yum",
       "gpgUrl": null,
       "contentUrl": "/custom/ParthaProduct/partha_multi-errata",
+      "osRestricted": "rhel-7",
       "override": "0",
       "overrides": [
         {
@@ -107,6 +109,7 @@
       "type": "yum",
       "gpgUrl": null,
       "contentUrl": "/custom/Pull_Provider/yggdrasil",
+      "osRestricted": null,
       "override": "1",
       "overrides": [
         {
@@ -117,4 +120,4 @@
       "enabled_content_override": true
     }
   ]
-}
+}

data/webpack/components/extensions/HostDetails/Tabs/__tests__/repositorySetsTab.test.js

@@ -8,6 +8,16 @@ import mockRepoSetData from './repositorySets.fixtures.json';
 import mockBookmarkData from './bookmarks.fixtures.json';
 import mockContentOverride from './contentOverrides.fixtures.json';
 
+jest.mock('../../hostDetailsHelpers', () => ({
+  ...jest.requireActual('../../hostDetailsHelpers'),
+  userPermissionsFromHostDetails: () => ({
+    view_hosts: true,
+    view_activation_keys: true,
+    view_products: true,
+    edit_hosts: true,
+  }),
+}));
+
 const contentFacetAttributes = {
   id: 11,
   uuid: 'e5761ea3-4117-4ecf-83d0-b694f99b389e',
@@ -378,3 +388,19 @@ test('Can filter by status', async (done) => {
   assertNockRequest(autoSearchScope);
   assertNockRequest(scope2, done); // Pass jest callback to confirm test is done
 });
+
+test('Can display osRestricted as a label', async (done) => {
+  const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
+  const scope = nockInstance
+    .get(hostRepositorySets)
+    .query(defaultQuery)
+    .reply(200, mockRepoSetData);
+
+  const { getByText } = renderWithRedux(<RepositorySetsTab />, renderOptions());
+
+  await patientlyWaitFor(() => expect(getByText(secondRepoSet.contentUrl)).toBeInTheDocument());
+  expect(secondRepoSet.osRestricted).not.toBeNull();
+  expect(getByText(secondRepoSet.osRestricted)).toBeInTheDocument();
+  assertNockRequest(autocompleteScope);
+  assertNockRequest(scope, done); // Pass jest callback to confirm test is done
+});

data/webpack/components/extensions/HostDetails/Tabs/__tests__/tracesTab.test.js

@@ -14,6 +14,12 @@ import mockBookmarkData from './bookmarks.fixtures.json';
 const hostName = 'client.example.com';
 const tracesBookmarks = foremanApi.getApiUrl('/bookmarks?search=controller%3Dkatello_host_tracers');
 
+jest.mock('../../hostDetailsHelpers', () => ({
+  ...jest.requireActual('../../hostDetailsHelpers'),
+  userPermissionsFromHostDetails: () => ({
+    create_job_invocations: true,
+  }),
+}));
 
 const tracerInstalledResponse = {
   id: 1,
@@ -117,7 +123,6 @@ describe('With tracer installed', () => {
     const scope = nockInstance
       .get(hostTraces)
       .query(true)
-      .times(2)
       .reply(200, mockTraceData);
     const resolveTracesScope = nockInstance
       .post(jobInvocations)
@@ -140,7 +145,7 @@ describe('With tracer installed', () => {
     const restartAppButton = getByText('Restart app');
     // wait 50ms so that the button is enabled
     await waitFor(() => {
-      expect(getByText('Restart app')).not.toBeDisabled();
+      expect(restartAppButton.parentElement).not.toHaveClass('pf-m-disabled');
       restartAppButton.click();
     });
 
@@ -157,7 +162,6 @@ describe('With tracer installed', () => {
     const scope = nockInstance
       .get(hostTraces)
       .query(true)
-      .times(2)
       .reply(200, mockTraceData);
     const resolveTracesScope = nockInstance
       .post(jobInvocations)
@@ -195,7 +199,6 @@ describe('With tracer installed', () => {
     const scope = nockInstance
       .get(hostTraces)
       .query(true)
-      .times(2)
       .reply(200, mockTraceData);
     const jobInvocationBody = ({ job_invocation: { inputs } }) =>
       inputs[TRACES_SEARCH_QUERY] === `id !^ (${firstTrace.id},${thirdTrace.id})`;

data/webpack/components/extensions/HostDetails/hostDetailsHelpers.js

@@ -25,4 +25,22 @@ export const hostIsNotRegistered = ({ hostDetails }) => {
 
 export const hostIsRegistered = ({ hostDetails }) => !hostIsNotRegistered({ hostDetails });
 
+export const userPermissionsFromHostDetails = ({ hostDetails }) => {
+  const {
+    permissions: hostPermissions,
+    content_facet_attributes: cfAttributes = {},
+  } = hostDetails;
+  return { ...hostPermissions, ...cfAttributes?.permissions };
+};
+
+// requiredPermissions is an array
+// userPermissions is an object, e.g. { view_hosts: true }
+export const hasRequiredPermissions = (requiredPermissions = [], userPermissions) => {
+  const permittedActions = Object.keys(userPermissions).filter(key => userPermissions[key]);
+  return requiredPermissions.every(permission => permittedActions.includes(permission));
+};
+
+export const missingRequiredPermissions = (requiredPermissions = [], userPermissions) =>
+  !hasRequiredPermissions(requiredPermissions, userPermissions);
+
 export default defaultRemoteActionMethod;

data/webpack/global_index.js

@@ -11,7 +11,6 @@ import ErrataOverviewCard from './components/extensions/HostDetails/Cards/Errata
 import InstalledProductsCard from './components/extensions/HostDetails/DetailsTabCards/InstalledProductsCard';
 import RegistrationCard from './components/extensions/HostDetails/DetailsTabCards/RegistrationCard';
 
-import RepositorySetsTab from './components/extensions/HostDetails/Tabs/RepositorySetsTab/RepositorySetsTab';
 import TracesTab from './components/extensions/HostDetails/Tabs/TracesTab/TracesTab.js';
 import extendReducer from './components/extensions/reducers';
 import rootReducer from './redux/reducers';
@@ -19,6 +18,7 @@ import HostCollectionsCard from './components/extensions/HostDetails/Cards/HostC
 import { hostIsNotRegistered } from './components/extensions/HostDetails/hostDetailsHelpers';
 import SystemPropertiesCardExtensions from './components/extensions/HostDetails/DetailsTabCards/SystemPropertiesCardExtensions';
 import HostActionsBar from './components/extensions/HostDetails/ActionsBar';
+import RecentCommunicationCardExtensions from './components/extensions/HostDetails/DetailsTabCards/RecentCommunicationCardExtensions';
 
 registerReducer('katelloExtends', extendReducer);
 registerReducer('katello', rootReducer);
@@ -29,7 +29,6 @@ addGlobalFill('registrationAdvanced', '[katello]RegistrationCommands', <Registra
 // Host details page tabs
 addGlobalFill('host-details-page-tabs', 'Content', <ContentTab key="content" />, 900, { title: __('Content'), hideTab: hostIsNotRegistered });
 addGlobalFill('host-details-page-tabs', 'Traces', <TracesTab key="traces" />, 800, { title: __('Traces'), hideTab: hostIsNotRegistered });
-addGlobalFill('host-details-page-tabs', 'Repository sets', <RepositorySetsTab key="repository-sets" />, 700, { title: __('Repository sets'), hideTab: hostIsNotRegistered });
 
 // Overview tab cards
 addGlobalFill(
@@ -45,6 +44,7 @@ addGlobalFill(
   700,
 );
 addGlobalFill('host-overview-cards', 'Installable errata', <ErrataOverviewCard key="errata-overview" />, 1900);
+addGlobalFill('recent-communication-card-item', 'Recent communication', <RecentCommunicationCardExtensions key="recent-communication" />, 3000);
 
 // Details tab cards & card extensions
 addGlobalFill('host-tab-details-cards', 'Installed products', <InstalledProductsCard key="installed-products" />, 100);

data/webpack/redux/actions/RedHatRepositories/helpers.js

@@ -23,9 +23,9 @@ const recommendedRepositoriesRHEL = [
 ];
 
 const recommendedRepositoriesSatTools = [
+  'satellite-client-6-for-rhel-9-x86_64-rpms',
   'satellite-client-6-for-rhel-8-x86_64-rpms',
   'rhel-7-server-satellite-client-6-rpms',
-  'rhel-7-server-satellite-maintenance-6.11-rpms',
   'rhel-6-server-els-satellite-client-6-rpms',
 ];
 
@@ -35,6 +35,10 @@ const recommendedRepositoriesMisc = [
   'satellite-capsule-6.11-for-rhel-8-x86_64-rpms',
   'rhel-7-server-ansible-2.9-rpms',
   'ansible-2-for-rhel-8-x86_64-rpms',
+  'rhel-7-server-satellite-maintenance-6.11-rpms',
+  'rhel-7-server-satellite-utils-6.11-rpms',
+  'satellite-maintenance-6.11-for-rhel-8-x86_64-rpms',
+  'satellite-utils-6.11-for-rhel-8-x86_64-rpms',
 ];
 
 const recommendedRepositorySetLables = recommendedRepositoriesRHEL

data/webpack/scenes/AlternateContentSources/ACSActions.js

@@ -1,8 +1,9 @@
 import { API_OPERATIONS, APIActions, get, post } from 'foremanReact/redux/API';
 import { translate as __ } from 'foremanReact/common/I18n';
 import api, { orgId } from '../../services/api';
-import ACS_KEY, { CREATE_ACS_KEY, DELETE_ACS_KEY } from './ACSConstants';
+import ACS_KEY, { acsRefreshKey, CREATE_ACS_KEY, DELETE_ACS_KEY } from './ACSConstants';
 import { getResponseErrorMsgs } from '../../utils/helpers';
+import { renderTaskStartedToast } from '../Tasks/helpers';
 
 const acsSuccessToast = (response) => {
   const { data: { name } } = response;
@@ -47,6 +48,17 @@ export const deleteACS = (acsId, handleSuccess) => APIActions.delete({
   errorToast: error => __(`Something went wrong while deleting this alternate content source! ${getResponseErrorMsgs(error.response)}`),
 });
 
+export const refreshACS = (acsId, handleSuccess) => post({
+  type: API_OPERATIONS.POST,
+  key: acsRefreshKey(acsId),
+  url: api.getApiUrl(`/alternate_content_sources/${acsId}/refresh`),
+  params: { id: acsId },
+  handleSuccess: (response) => {
+    if (handleSuccess) handleSuccess();
+    return renderTaskStartedToast(response.data);
+  },
+  errorToast: error => __(`Something went wrong while refreshing this alternate content source! ${getResponseErrorMsgs(error.response)}`),
+});
 export default getAlternateContentSources;
 
 // acs = Katello::AlternateContentSource.new

data/webpack/scenes/AlternateContentSources/ACSConstants.js

@@ -1,4 +1,18 @@
+import { translate as __ } from 'foremanReact/common/I18n';
+
 const ACS_KEY = 'ACS';
 export const CREATE_ACS_KEY = 'ACS_CREATE';
 export const DELETE_ACS_KEY = 'ACS_DELETE';
+export const SMART_PROXY_KEY = 'SMART_PROXY';
+export const SSL_CERTS = 'SSL_CERTS';
+export const acsRefreshKey = acsId => `${ACS_KEY}_REFRESH_${acsId}`;
+
+export const YUM = __('Yum');
+export const FILE = __('File');
+
+export const ACS_TYPE_TRANSLATIONS_ENUM = {
+  [YUM]: 'yum',
+  [FILE]: 'file',
+};
+
 export default ACS_KEY;

data/webpack/scenes/AlternateContentSources/ACSSelectors.js

@@ -4,7 +4,7 @@ import {
   selectAPIResponse,
 } from 'foremanReact/redux/API/APISelectors';
 import { STATUS } from 'foremanReact/constants';
-import ACS_KEY from './ACSConstants';
+import ACS_KEY, { CREATE_ACS_KEY } from './ACSConstants';
 
 export const selectAlternateContentSources = (state, index = '') => selectAPIResponse(state, ACS_KEY + index) || {};
 
@@ -13,3 +13,12 @@ export const selectAlternateContentSourcesStatus = (state, index = '') =>
 
 export const selectAlternateContentSourcesError = (state, index = '') =>
   selectAPIError(state, ACS_KEY + index);
+
+export const selectCreateACS = state =>
+  selectAPIResponse(state, CREATE_ACS_KEY) || {};
+
+export const selectCreateACSStatus = state =>
+  selectAPIStatus(state, CREATE_ACS_KEY) || STATUS.PENDING;
+
+export const selectCreateACSError = state =>
+  selectAPIError(state, CREATE_ACS_KEY);

data/webpack/scenes/AlternateContentSources/Create/ACSCreateContext.js

@@ -0,0 +1,4 @@
+import { createContext } from 'react';
+
+const ACSCreateContext = createContext({});
+export default ACSCreateContext;

data/webpack/scenes/AlternateContentSources/Create/ACSCreateWizard.js

@@ -0,0 +1,160 @@
+import React, { useEffect, useState } from 'react';
+import { useDispatch } from 'react-redux';
+import PropTypes from 'prop-types';
+import { Wizard } from '@patternfly/react-core';
+import { translate as __ } from 'foremanReact/common/I18n';
+import ACSCreateContext from './ACSCreateContext';
+import SelectSource from './Steps/SelectSource';
+import NameACS from './Steps/NameACS';
+import AcsUrlPaths from './Steps/AcsUrlPaths';
+import ACSCredentials from './Steps/ACSCredentials';
+import ACSSmartProxies from './Steps/ACSSmartProxies';
+import ACSReview from './Steps/ACSReview';
+import ACSCreateFinish from './Steps/ACSCreateFinish';
+import { getContentCredentials } from '../../ContentCredentials/ContentCredentialActions';
+import { getSmartProxies } from '../../SmartProxy/SmartProxyContentActions';
+import { CONTENT_CREDENTIAL_CERT_TYPE } from '../../ContentCredentials/ContentCredentialConstants';
+
+const ACSCreateWizard = ({ show, setIsOpen }) => {
+  const [acsType, setAcsType] = useState(null);
+  const [contentType, setContentType] = useState('yum');
+  const [name, setName] = useState('');
+  const [description, setDescription] = useState('');
+  const [smartProxies, setSmartProxies] = useState([]);
+  const [url, setUrl] = useState('');
+  const [subpaths, setSubpaths] = useState('');
+  const [verifySSL, setVerifySSL] = useState(false);
+  const [authentication, setAuthentication] = useState('');
+  const [sslCert, setSslCert] = useState('');
+  const [sslKey, setSslKey] = useState('');
+  const [sslCertName, setSslCertName] = useState('');
+  const [sslKeyName, setSslKeyName] = useState('');
+  const [username, setUsername] = useState('');
+  const [password, setPassword] = useState('');
+  const [caCert, setCACert] = useState('');
+  const [caCertName, setCACertName] = useState('');
+  const [currentStep, setCurrentStep] = useState(1);
+  const dispatch = useDispatch();
+
+  useEffect(
+    () => {
+      dispatch(getContentCredentials({ content_type: CONTENT_CREDENTIAL_CERT_TYPE }));
+      dispatch(getSmartProxies());
+    },
+    [dispatch],
+  );
+
+  const steps = [
+    {
+      id: 1,
+      name: __('Select source type'),
+      component: <SelectSource />,
+      enableNext: acsType && contentType,
+    },
+    {
+      id: 2,
+      name: __('Name source'),
+      component: <NameACS />,
+      enableNext: name !== '',
+    },
+    {
+      id: 3,
+      name: __('Select smart proxy'),
+      component: <ACSSmartProxies />,
+      enableNext: smartProxies.length,
+    },
+    {
+      id: 4,
+      name: __('URL and paths'),
+      component: <AcsUrlPaths />,
+      enableNext: url !== '' && subpaths !== '',
+    },
+    {
+      id: 5,
+      name: __('Credentials'),
+      component: <ACSCredentials />,
+    },
+    {
+      id: 6,
+      name: __('Review details'),
+      component: <ACSReview />,
+      nextButtonText: __('Add'),
+    },
+    {
+      id: 7,
+      name: __('Create ACS'),
+      component: <ACSCreateFinish />,
+      isFinishedStep: true,
+    },
+
+  ];
+
+  return (
+    <ACSCreateContext.Provider value={{
+      show,
+      setIsOpen,
+      currentStep,
+      setCurrentStep,
+      acsType,
+      setAcsType,
+      contentType,
+      setContentType,
+      name,
+      setName,
+      description,
+      setDescription,
+      smartProxies,
+      setSmartProxies,
+      url,
+      setUrl,
+      subpaths,
+      setSubpaths,
+      verifySSL,
+      setVerifySSL,
+      authentication,
+      setAuthentication,
+      sslCert,
+      setSslCert,
+      sslKey,
+      setSslKey,
+      sslCertName,
+      setSslCertName,
+      sslKeyName,
+      setSslKeyName,
+      username,
+      setUsername,
+      password,
+      setPassword,
+      caCert,
+      setCACert,
+      caCertName,
+      setCACertName,
+    }}
+    >
+      <Wizard
+        title={__('Add an alternate content source')}
+        steps={steps}
+        startAtStep={currentStep}
+        onGoToStep={({ id }) => setCurrentStep(id)}
+        onNext={({ id }) => setCurrentStep(id)}
+        onBack={({ id }) => setCurrentStep(id)}
+        onClose={() => {
+          setIsOpen(false);
+        }}
+        isOpen={show}
+      />
+    </ACSCreateContext.Provider>
+  );
+};
+
+ACSCreateWizard.propTypes = {
+  show: PropTypes.bool,
+  setIsOpen: PropTypes.func,
+};
+
+ACSCreateWizard.defaultProps = {
+  show: false,
+  setIsOpen: null,
+};
+
+export default ACSCreateWizard;

data/webpack/scenes/AlternateContentSources/Create/Steps/ACSCreateFinish.js

@@ -0,0 +1,79 @@
+import React, { useContext, useState } from 'react';
+import { useDispatch, useSelector } from 'react-redux';
+import useDeepCompareEffect from 'use-deep-compare-effect';
+import { translate as __ } from 'foremanReact/common/I18n';
+import { STATUS } from 'foremanReact/constants';
+import ACSCreateContext from '../ACSCreateContext';
+import { selectCreateACS, selectCreateACSError, selectCreateACSStatus } from '../../ACSSelectors';
+import getAlternateContentSources, { createACS } from '../../ACSActions';
+import Loading from '../../../../components/Loading';
+
+const ACSCreateFinish = () => {
+  const {
+    currentStep,
+    setIsOpen,
+    acsType,
+    contentType,
+    name,
+    description,
+    smartProxies,
+    url,
+    subpaths,
+    verifySSL,
+    authentication,
+    sslCert,
+    sslKey,
+    username,
+    password,
+    caCert,
+  } = useContext(ACSCreateContext);
+  const dispatch = useDispatch();
+  const response = useSelector(state => selectCreateACS(state));
+  const status = useSelector(state => selectCreateACSStatus(state));
+  const error = useSelector(state => selectCreateACSError(state));
+  const [createACSDispatched, setCreateACSDispatched] = useState(false);
+  const [saving, setSaving] = useState(true);
+
+  useDeepCompareEffect(() => {
+    if (currentStep === 7 && !createACSDispatched) {
+      setCreateACSDispatched(true);
+      let params = {
+        name,
+        description,
+        base_url: url,
+        subpaths: subpaths.split(','),
+        smart_proxy_names: smartProxies,
+        content_type: contentType,
+        alternate_content_source_type: acsType,
+        verify_ssl: verifySSL,
+        ssl_ca_cert_id: caCert,
+      };
+      if (authentication === 'content_credentials') {
+        params = { ssl_client_cert_id: sslCert, ssl_client_key_id: sslKey, ...params };
+      }
+      if (authentication === 'manual') {
+        params = { upstream_username: username, upstream_password: password, ...params };
+      }
+      dispatch(createACS(params));
+    }
+  }, [dispatch, createACSDispatched, setCreateACSDispatched,
+    acsType, authentication, name, description, url, subpaths,
+    smartProxies, contentType, verifySSL, caCert, sslCert, sslKey,
+    username, password, currentStep]);
+
+  useDeepCompareEffect(() => {
+    const { id } = response;
+    if (id && status === STATUS.RESOLVED && saving) {
+      setSaving(false);
+      dispatch(getAlternateContentSources());
+      setIsOpen(false);
+    } else if (status === STATUS.ERROR) {
+      setSaving(false);
+      setIsOpen(false);
+    }
+  }, [response, status, error, saving, dispatch, setIsOpen]);
+
+  return <Loading loadingText={__('Saving alternate content source...')} />;
+};
+
+export default ACSCreateFinish;