Wizard: Show Users under Base Settings in image mode

Users are configured in Base Settings when in image mode
(cockpit-image-builder), but the Review step always displayed them
under Advanced Settings. Show Users under Image Overview in image
mode, and keep them under Advanced Settings in package mode.

Author: mgold1234

src/Components/CreateImageWizard/steps/Review/components/AdvancedSettings/index.tsx

@@ -2,6 +2,9 @@ import React from 'react';
 
 import { Card, CardBody } from '@patternfly/react-core';
 
+import { useAppSelector } from '@/store/hooks';
+import { selectIsImageMode } from '@/store/slices';
+
 import {
   Filesystem,
   Firewall,
@@ -33,6 +36,7 @@ const AdvancedSettingsOverview = ({
   oscapKernelArgs = [],
   oscapServices,
 }: AdvancedSettingsOverviewProps) => {
+  const isImageMode = useAppSelector(selectIsImageMode);
   return (
     <Card>
       <ReviewCardHeader
@@ -54,7 +58,7 @@ const AdvancedSettingsOverview = ({
             oscapServices={oscapServices}
           />
           <Firewall shouldHide={restrictions.firewall.shouldHide} />
-          <Users shouldHide={restrictions.users.shouldHide} />
+          {!isImageMode && <Users shouldHide={restrictions.users.shouldHide} />}
           <Firstboot shouldHide={restrictions.firstBoot.shouldHide} />
         </ReviewList>
       </CardBody>

src/Components/CreateImageWizard/steps/Review/components/AdvancedSettings/tests/AdvancedSettings.test.tsx

@@ -901,6 +901,19 @@ echo 'Hello there, General Kenobi!'`;
       expect(screen.getByText('guest')).toBeInTheDocument();
       expect(screen.getAllByText('*****')).toHaveLength(3);
     });
+
+    test('does not render users section in image mode', () => {
+      renderWithRedux(
+        <AdvancedSettingsOverview restrictions={createDefaultRestrictions()} />,
+        {
+          imageTypes: ['guest-image'],
+          blueprintMode: 'image',
+          users: [adminUser],
+        },
+      );
+
+      expect(screen.queryByText('Users')).not.toBeInTheDocument();
+    });
   });
 
   describe('Firewall', () => {

src/Components/CreateImageWizard/steps/Review/components/ImageOverview/index.tsx

@@ -4,6 +4,7 @@ import { Card, CardBody } from '@patternfly/react-core';
 
 import { ON_PREM_RELEASES, RELEASES } from '@/constants';
 import { useTargetEnvironmentCategories } from '@/Hooks';
+import { useCustomizationRestrictions } from '@/store/api/distributions';
 import { useAppSelector } from '@/store/hooks';
 import {
   selectArchitecture,
@@ -18,6 +19,7 @@ import {
 
 import { MiscFormats, PrivateClouds, PublicClouds } from './components';
 
+import { Users } from '../AdvancedSettings/components';
 import { ReviewCardHeader, ReviewGroup, ReviewList } from '../shared';
 
 const ImageOverview = () => {
@@ -29,8 +31,12 @@ const ImageOverview = () => {
   const distribution = useAppSelector(selectDistribution);
   const arch = useAppSelector(selectArchitecture);
 
+  const environments = useAppSelector(selectImageTypes);
   const { publicClouds, privateClouds, miscFormats } =
-    useTargetEnvironmentCategories(useAppSelector(selectImageTypes));
+    useTargetEnvironmentCategories(environments);
+  const { restrictions } = useCustomizationRestrictions({
+    selectedImageTypes: environments,
+  });
 
   const releases = isOnPremise ? ON_PREM_RELEASES : RELEASES;
 
@@ -70,6 +76,7 @@ const ImageOverview = () => {
           <PrivateClouds environments={privateClouds} />
           <PublicClouds environments={publicClouds} />
           <MiscFormats environments={miscFormats} />
+          {isImageMode && <Users shouldHide={restrictions.users.shouldHide} />}
         </ReviewList>
       </CardBody>
     </Card>

src/Components/CreateImageWizard/steps/Review/components/ImageOverview/tests/ImageOverview.test.tsx

@@ -5,6 +5,7 @@ import { screen } from '@testing-library/react';
 import { RHEL_10, X86_64 } from '@/constants';
 import { renderWithRedux } from '@/test/testUtils';
 
+import { adminUser } from '../../AdvancedSettings/tests/mocks';
 import ImageOverview from '../index';
 
 describe('ImageOverview', () => {
@@ -230,4 +231,27 @@ describe('ImageOverview', () => {
       ).not.toBeInTheDocument();
     });
   });
+
+  describe('Users', () => {
+    test('displays users section in image mode', () => {
+      renderWithRedux(<ImageOverview />, {
+        imageTypes: ['guest-image'],
+        blueprintMode: 'image',
+        users: [adminUser],
+      });
+
+      expect(screen.getByText('Users')).toBeInTheDocument();
+      expect(screen.getByText('admin')).toBeInTheDocument();
+    });
+
+    test('does not display users section in package mode', () => {
+      renderWithRedux(<ImageOverview />, {
+        imageTypes: ['guest-image'],
+        blueprintMode: 'package',
+        users: [adminUser],
+      });
+
+      expect(screen.queryByText('Users')).not.toBeInTheDocument();
+    });
+  });
 });