Fixes #39389 - all-host schedule job bulk search

Co-authored-by: OpenAI Codex <codex@openai.com>

Author: sbernhard

webpack/assets/javascripts/react_app/components/HostsIndex/index.js

@@ -80,6 +80,16 @@ export const ForemanHostsIndexActionsBarContext = forceSingleton(
   () => createContext({})
 );
 
+export const getScheduleJobSearch = ({
+  selectedCount,
+  areAllRowsSelected,
+  selectedHostsSearch,
+}) => {
+  if (selectedCount === 0) return null;
+  if (areAllRowsSelected && selectedHostsSearch === '') return 'name ~ *';
+  return selectedHostsSearch;
+};
+
 const HostsIndex = () => {
   const [menuOpen, setMenuOpen] = useState(false);
   const [allColumns, setAllColumns] = useState(
@@ -188,6 +198,11 @@ const HostsIndex = () => {
     selectedResults,
   } = selectAllOptions;
   const selectAllHostsMode = areAllRowsSelected() && searchQuery === '';
+  const scheduleJobSearch = getScheduleJobSearch({
+    selectedCount,
+    areAllRowsSelected: areAllRowsSelected(),
+    selectedHostsSearch: selectedCount > 0 ? fetchBulkParams() : null,
+  });
 
   const selectionToolbar = (
     <ToolbarItem key="selectAll">
@@ -448,7 +463,7 @@ const HostsIndex = () => {
           <SplitItem>
             <Slot
               id="_all-hosts-schedule-a-job"
-              hostSearch={selectedCount ? fetchBulkParams() : null}
+              hostSearch={scheduleJobSearch}
               hostResponse={response}
               selectedCount={selectedCount}
             />

webpack/assets/javascripts/react_app/components/HostsIndex/index.test.js

@@ -5,7 +5,7 @@ import * as ReactRedux from 'react-redux';
 import configureMockStore from 'redux-mock-store';
 import thunk from 'redux-thunk';
 import '@testing-library/jest-dom';
-import HostsIndex from './index';
+import HostsIndex, { getScheduleJobSearch } from './index';
 
 const mockStore = configureMockStore([thunk]);
 
@@ -155,4 +155,34 @@ describe('HostsIndex', () => {
       per_page: 20, // From API response, not from params state (which has 10)
     });
   });
+
+  test('returns an explicit all-hosts search for empty select-all queries', () => {
+    expect(
+      getScheduleJobSearch({
+        selectedCount: 100,
+        areAllRowsSelected: true,
+        selectedHostsSearch: '',
+      })
+    ).toBe('name ~ *');
+  });
+
+  test('does not append the all-hosts fallback to a non-empty select-all search', () => {
+    expect(
+      getScheduleJobSearch({
+        selectedCount: 1,
+        areAllRowsSelected: true,
+        selectedHostsSearch: 'name = "centos9-katello-devel.example.com"',
+      })
+    ).toBe('name = "centos9-katello-devel.example.com"');
+  });
+
+  test('does not use the all-hosts fallback when select-all is not active', () => {
+    expect(
+      getScheduleJobSearch({
+        selectedCount: 1,
+        areAllRowsSelected: false,
+        selectedHostsSearch: '',
+      })
+    ).toBe('');
+  });
 });