Fixes #38191: Skip reassignment for multiCV in remove

Author: pavanshekar

app/views/katello/api/v2/content_view_versions/base.json.rabl

@@ -73,9 +73,21 @@ child :sorted_organization_readable_environments => :environments do
     ::Host.authorized('view_hosts').in_content_view_environment(:content_view => version.content_view, :lifecycle_environment => env).count
   end
 
+  node :multi_env_host_count do |env|
+    hosts = ::Host.authorized('view_hosts')
+      .in_content_view_environments(content_views: [version.content_view], lifecycle_environments: [env])
+    hosts.count { |host| host.content_facet.multi_content_view_environment? }
+  end
+
   node :activation_key_count do |env|
     Katello::ActivationKey.with_content_views(version.content_view).with_environments(env).count
   end
+
+  node :multi_env_ak_count do |env|
+    keys = Katello::ActivationKey.with_content_views(version.content_view)
+      .with_environments(env)
+    keys.count { |key| key.multi_content_view_environment? }
+  end
 end
 
 child :archived_repos => :repositories do

webpack/scenes/ContentViews/Details/Versions/Delete/RemoveSteps/CVEnvironmentSelectionForm.js

@@ -25,9 +25,15 @@ const CVEnvironmentSelectionForm = () => {
 
   // Based on env selected for removal, decide if we need to reassign hosts and activation keys.
   useDeepCompareEffect(() => {
-    const selectedEnv = versionEnvironments.filter(env => selectedEnvSet.has(env.id));
-    setAffectedActivationKeys(!!(selectedEnv.filter(env => env.activation_key_count > 0).length));
-    setAffectedHosts(!!(selectedEnv.filter(env => env.host_count > 0).length));
+    const selectedEnvironments = versionEnvironments.filter(env => selectedEnvSet.has(env.id));
+
+    const needsHostReassignment = selectedEnvironments.some(env =>
+      (env.host_count || 0) > (env.multi_env_host_count || 0));
+    setAffectedHosts(needsHostReassignment);
+
+    const needsAKReassignment = selectedEnvironments.some(env =>
+      (env.activation_key_count || 0) > (env.multi_env_ak_count || 0));
+    setAffectedActivationKeys(needsAKReassignment);
   }, [setAffectedActivationKeys, setAffectedHosts,
     versionEnvironments, selectedEnvSet, selectedEnvSet.size]);
 

webpack/scenes/ContentViews/Details/Versions/Delete/RemoveSteps/CVVersionRemoveReview.js

@@ -1,24 +1,24 @@
 import React, { useContext, useState } from 'react';
 import { useSelector } from 'react-redux';
-import { Alert, Flex, FlexItem, Label, AlertActionCloseButton } from '@patternfly/react-core';
+import { Alert, Flex, FlexItem, Label, AlertActionCloseButton, ExpandableSection } from '@patternfly/react-core';
 import { ExclamationTriangleIcon } from '@patternfly/react-icons';
 import { FormattedMessage } from 'react-intl';
 import { translate as __ } from 'foremanReact/common/I18n';
-import { selectCVActivationKeys, selectCVHosts, selectCVVersions } from '../../../ContentViewDetailSelectors';
+import { selectCVVersions } from '../../../ContentViewDetailSelectors';
 import DeleteContext from '../DeleteContext';
 import WizardHeader from '../../../../components/WizardHeader';
+import AffectedHosts from '../affectedHosts';
+import AffectedActivationKeys from '../affectedActivationKeys';
 
 const CVVersionRemoveReview = () => {
   const [alertDismissed, setAlertDismissed] = useState(false);
+  const [showHosts, setShowHosts] = useState(false);
+  const [showAKs, setShowAKs] = useState(false);
   const {
-    cvId, versionIdToRemove, versionNameToRemove, selectedEnvSet,
+    cvId, versionEnvironments, versionIdToRemove, versionNameToRemove, selectedEnvSet,
     selectedEnvForAK, selectedCVNameForAK, selectedCVNameForHosts,
-    selectedEnvForHost, affectedActivationKeys, affectedHosts, deleteFlow, removeDeletionFlow,
+    selectedEnvForHost, deleteFlow, removeDeletionFlow,
   } = useContext(DeleteContext);
-  const activationKeysResponse = useSelector(state => selectCVActivationKeys(state, cvId));
-  const hostsResponse = useSelector(state => selectCVHosts(state, cvId));
-  const { results: hostResponse = [] } = hostsResponse || {};
-  const { results: akResponse = [] } = activationKeysResponse || {};
   const cvVersions = useSelector(state => selectCVVersions(state, cvId));
   const versionDeleteInfo = __(`Version ${versionNameToRemove} will be deleted from all environments. It will no longer be available for promotion.`);
   const removalNotice = __(`Version ${versionNameToRemove} will be removed from the environments listed below, and will remain available for later promotion. ` +
@@ -29,16 +29,19 @@ const CVVersionRemoveReview = () => {
     .flatMap(cv => cv.content_view_environments || [])
     .filter(env => selectedEnvSet.has(env.environment_id));
 
-  const multiCVHosts = hostResponse?.filter(host =>
-    host.content_facet_attributes?.multi_content_view_environment) || [];
-  const multiCVHostsCount = multiCVHosts.length;
+  const selectedEnvs = versionEnvironments.filter(env => selectedEnvSet.has(env.id));
 
-  const singleCVHostsCount = (hostResponse?.length || 0) - multiCVHostsCount;
+  const hostCount = selectedEnvs.reduce((sum, env) =>
+    sum + (env.host_count || 0), 0);
+  const multiCVHostsCount = selectedEnvs.reduce((sum, env) =>
+    sum + (env.multi_env_host_count || 0), 0);
+  const singleCVHostsCount = hostCount - multiCVHostsCount;
 
-  const multiCVActivationKeys = akResponse.filter(key => key.multi_content_view_environment);
-  const multiCVActivationKeysCount = multiCVActivationKeys.length;
-
-  const singleCVActivationKeysCount = akResponse.length - multiCVActivationKeysCount;
+  const akCount = selectedEnvs.reduce((sum, env) =>
+    sum + (env.activation_key_count || 0), 0);
+  const multiCVActivationKeysCount = selectedEnvs.reduce((sum, env) =>
+    sum + (env.multi_env_ak_count || 0), 0);
+  const singleCVActivationKeysCount = akCount - multiCVActivationKeysCount;
 
   return (
     <>
@@ -66,7 +69,7 @@ const CVVersionRemoveReview = () => {
               <FlexItem key={name}><Label color="purple" href={`/lifecycle_environments/${id}`}>{name}</Label></FlexItem>)}
           </Flex>
         </>}
-      {affectedHosts &&
+      {hostCount > 0 &&
         <>
           <h3>{__('Content hosts')}</h3>
           {singleCVHostsCount > 0 && (
@@ -121,8 +124,22 @@ const CVVersionRemoveReview = () => {
               </FlexItem>
             </Flex>
           )}
+          <ExpandableSection
+            toggleText={showHosts ? 'Hide hosts' : 'Show hosts'}
+            onToggle={() => setShowHosts(prev => !prev)}
+            isExpanded={showHosts}
+          >
+            <AffectedHosts
+              {...{
+                cvId,
+                versionEnvironments,
+                selectedEnvSet,
+              }}
+              deleteCV={false}
+            />
+          </ExpandableSection>
         </>}
-      {affectedActivationKeys &&
+      {akCount > 0 &&
         <>
           <h3>{__('Activation keys')}</h3>
           {singleCVActivationKeysCount > 0 && (
@@ -177,6 +194,20 @@ const CVVersionRemoveReview = () => {
               </FlexItem>
             </Flex>
           )}
+          <ExpandableSection
+            toggleText={showAKs ? 'Hide activation keys' : 'Show activation keys'}
+            onToggle={() => setShowAKs(prev => !prev)}
+            isExpanded={showAKs}
+          >
+            <AffectedActivationKeys
+              {...{
+                cvId,
+                versionEnvironments,
+                selectedEnvSet,
+              }}
+              deleteCV={false}
+            />
+          </ExpandableSection>
         </>}
     </>
   );

webpack/scenes/ContentViews/Details/Versions/Delete/__tests__/cvVersionRemove.test.js

@@ -1,6 +1,6 @@
 import React from 'react';
 import { renderWithRedux, patientlyWaitFor, fireEvent } from 'react-testing-lib-wrapper';
-import { nockInstance, assertNockRequest, mockAutocomplete, mockForemanAutocomplete } from '../../../../../../test-utils/nockWrapper';
+import { nockInstance, assertNockRequest, mockAutocomplete } from '../../../../../../test-utils/nockWrapper';
 import api, { foremanApi } from '../../../../../../services/api';
 import CONTENT_VIEWS_KEY from '../../../../ContentViewsConstants';
 import ContentViewVersions from '../../ContentViewVersions';
@@ -123,7 +123,11 @@ test('Can open Remove wizard and remove version from simple environment', async
 test('Can open Remove wizard and remove version from environment with hosts', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const hostAutocompleteUrl = '/hosts/auto_complete_search';
-  const hostAutocompleteScope = mockForemanAutocomplete(nockInstance, hostAutocompleteUrl);
+  const hostAutocompleteScope = nockInstance
+    .get(foremanApi.getApiUrl(hostAutocompleteUrl))
+    .query(true)
+    .times(2)
+    .reply(200, []);
 
   const scope = nockInstance
     .get(cvVersions)
@@ -138,6 +142,7 @@ test('Can open Remove wizard and remove version from environment with hosts', as
   const hostScope = nockInstance
     .get(hostURL)
     .query(true)
+    .times(2)
     .reply(200, affectedHostData);
 
   const cVDropDownOptionsScope = nockInstance
@@ -211,7 +216,11 @@ test('Can open Remove wizard and remove version from environment with hosts', as
 test('Can open Remove wizard and remove version from environment with activation keys', async (done) => {
   const autocompleteScope = mockAutocomplete(nockInstance, autocompleteUrl);
   const akAutocompleteUrl = '/activation_keys/auto_complete_search';
-  const akAutocompleteScope = mockAutocomplete(nockInstance, akAutocompleteUrl);
+  const akAutocompleteScope = nockInstance
+    .get(api.getApiUrl(akAutocompleteUrl))
+    .query(true)
+    .times(2)
+    .reply(200, []);
 
   const scope = nockInstance
     .get(cvVersions)
@@ -226,6 +235,7 @@ test('Can open Remove wizard and remove version from environment with activation
   const activationKeysScope = nockInstance
     .get(activationKeyURL)
     .query(true)
+    .times(2)
     .reply(200, affectedActivationKeysData);
 
   const cVDropDownOptionsScope = nockInstance