foreman_remote_execution 16.6.6 → 16.7.0

data/webpack/JobInvocationDetail/__tests__/TableToolbarActions.test.js

@@ -2,6 +2,7 @@ import '@testing-library/jest-dom/extend-expect';
 import { fireEvent, render, screen, waitFor } from '@testing-library/react';
 import axios from 'axios';
 import { foremanUrl } from 'foremanReact/common/helpers';
+import { usePermissions } from 'foremanReact/common/hooks/Permissions/permissionHooks';
 import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
 import React from 'react';
 import { Provider } from 'react-redux';
@@ -13,6 +14,7 @@ import { PopupAlert } from '../OpenAllInvocationsModal';
 
 jest.mock('axios');
 jest.mock('foremanReact/common/hooks/API/APIHooks');
+jest.mock('foremanReact/common/hooks/Permissions/permissionHooks');
 jest.mock('../JobInvocationSelectors');
 
 jest.mock('../JobInvocationConstants', () => ({
@@ -26,8 +28,8 @@ jest.mock('../JobInvocationConstants', () => ({
 selectors.selectItems.mockImplementation(() => ({
   targeting: { search_query: 'name~*' },
 }));
-selectors.selectHasPermission.mockImplementation(() => () => true);
 selectors.selectTaskCancelable.mockImplementation(() => true);
+usePermissions.mockReturnValue(true);
 
 const mockStore = configureStore([]);
 const store = mockStore({});

data/webpack/JobInvocationDetail/__tests__/foremanTestHelpers.js

@@ -0,0 +1,30 @@
+import React from 'react';
+import PropTypes from 'prop-types';
+import { getForemanContext } from 'foremanReact/Root/Context/ForemanContext';
+
+export const createForemanContextWrapper = (
+  permissions = ['view_job_invocations', 'create_job_invocations']
+) => {
+  const foremanContextValue = {
+    context: {
+      metadata: {
+        permissions: new Set(permissions),
+        UISettings: { perPage: 20 },
+      },
+    },
+    setContext: jest.fn(),
+  };
+  const ForemanContext = getForemanContext(foremanContextValue);
+
+  const ForemanContextWrapper = ({ children }) => (
+    <ForemanContext.Provider value={foremanContextValue}>
+      {children}
+    </ForemanContext.Provider>
+  );
+
+  ForemanContextWrapper.propTypes = {
+    children: PropTypes.node.isRequired,
+  };
+
+  return ForemanContextWrapper;
+};

data/webpack/JobInvocationDetail/index.js

@@ -5,29 +5,33 @@ import {
   PageSectionVariants,
   Skeleton,
 } from '@patternfly/react-core';
-import React, { useEffect, useState } from 'react';
+import React, { useEffect, useMemo, useState } from 'react';
 import { translate as __, documentLocale } from 'foremanReact/common/I18n';
 import { useDispatch, useSelector } from 'react-redux';
 import PageLayout from 'foremanReact/routes/common/PageLayout/PageLayout';
 import PropTypes from 'prop-types';
 import SkeletonLoader from 'foremanReact/components/common/SkeletonLoader';
 import { stopInterval } from 'foremanReact/redux/middlewares/IntervalMiddleware';
-import { useAPI } from 'foremanReact/common/hooks/API/APIHooks';
+import { STATUS as API_STATUS } from 'foremanReact/constants';
+import {
+  selectAPIErrorMessage,
+  selectAPIHttpStatus,
+  selectAPIStatus,
+} from 'foremanReact/redux/API/APISelectors';
 
 import { JobAdditionInfo } from './JobAdditionInfo';
 import JobInvocationHostTable from './JobInvocationHostTable';
 import JobInvocationOverview from './JobInvocationOverview';
 import JobInvocationSystemStatusChart from './JobInvocationSystemStatusChart';
+import JobInvocationEmptyState from './JobInvocationEmptyState';
 import JobInvocationToolbarButtons from './JobInvocationToolbarButtons';
 import { getJobInvocation, getTask } from './JobInvocationActions';
 import './JobInvocationDetail.scss';
 import {
-  CURRENT_PERMISSIONS,
   DATE_OPTIONS,
   JOB_INVOCATION_KEY,
   STATUS,
   STATUS_UPPERCASE,
-  currentPermissionsUrl,
 } from './JobInvocationConstants';
 import { selectItems } from './JobInvocationSelectors';
 
@@ -51,9 +55,15 @@ const JobInvocationDetailPage = ({
     statusLabel === STATUS.SUCCEEDED ||
     statusLabel === STATUS.CANCELLED;
   const autoRefresh = task?.state === STATUS.PENDING || false;
-  useAPI('get', currentPermissionsUrl, {
-    key: CURRENT_PERMISSIONS,
-  });
+  const jobInvocationApiStatus = useSelector(state =>
+    selectAPIStatus(state, JOB_INVOCATION_KEY)
+  );
+  const jobInvocationErrorMessage = useSelector(state =>
+    selectAPIErrorMessage(state, JOB_INVOCATION_KEY)
+  );
+  const jobInvocationHttpStatus = useSelector(state =>
+    selectAPIHttpStatus(state, JOB_INVOCATION_KEY)
+  );
   const [selectedFilter, setSelectedFilter] = useState('');
 
   const handleFilterChange = newFilter => {
@@ -72,7 +82,7 @@ const JobInvocationDetailPage = ({
   }
 
   useEffect(() => {
-    dispatch(getJobInvocation(`/api/job_invocations/${id}?host_status=true`));
+    dispatch(getJobInvocation(`/api/job_invocations/${id}`));
     if (finished && !autoRefresh) {
       dispatch(stopInterval(JOB_INVOCATION_KEY));
     }
@@ -88,6 +98,25 @@ const JobInvocationDetailPage = ({
     }
   }, [dispatch, taskId]);
 
+  const apiFailed = jobInvocationApiStatus === API_STATUS.ERROR;
+
+  const backendErrorMessage = useMemo(() => {
+    if (jobInvocationApiStatus === API_STATUS.ERROR) {
+      return jobInvocationErrorMessage || null;
+    }
+    return null;
+  }, [jobInvocationApiStatus, jobInvocationErrorMessage]);
+
+  if (apiFailed) {
+    return (
+      <JobInvocationEmptyState
+        jobInvocationId={id}
+        httpStatus={jobInvocationHttpStatus}
+        errorMessage={backendErrorMessage}
+      />
+    );
+  }
+
   const pageStatus =
     items.id === undefined
       ? STATUS_UPPERCASE.PENDING
@@ -123,7 +152,11 @@ const JobInvocationDetailPage = ({
       <PageLayout
         header={description}
         breadcrumbOptions={breadcrumbOptions}
-        toolbarButtons={<JobInvocationToolbarButtons jobId={id} data={items} />}
+        toolbarButtons={
+          items.id !== undefined && (
+            <JobInvocationToolbarButtons jobId={id} data={items} />
+          )
+        }
         searchable={false}
       >
         <Flex className="job-invocation-detail-flex">

data/webpack/__mocks__/foremanReact/Root/Context/ForemanContext/index.js

@@ -1,6 +1,21 @@
+import React from 'react';
+
+const getForemanContext = contextData => {
+  window.tfm_forced_singletons = window.tfm_forced_singletons || {};
+
+  if (!window.tfm_forced_singletons.Context) {
+    window.tfm_forced_singletons.Context = React.createContext(contextData);
+  }
+
+  return window.tfm_forced_singletons.Context;
+};
+
+export { getForemanContext };
 export const useForemanOrganization = () => ({ id: 1 });
 export const useForemanLocation = () => ({ id: 2 });
 export const useForemanVersion = () => '3.7';
 export const useForemanHostsPageUrl = () => '/hosts';
 export const useForemanHostDetailsPageUrl = () => '/hosts/';
 export const useForemanSettings = () => ({ perPage: 20 });
+export const useForemanPermissions = () =>
+  new Set(['view_job_invocations', 'create_job_invocations']);

data/webpack/__mocks__/foremanReact/common/hooks/Permissions/permissionHooks.js

@@ -0,0 +1 @@
+export const usePermissions = jest.fn(() => true);

data/webpack/__mocks__/foremanReact/redux/API/APISelectors.js

@@ -19,3 +19,8 @@ export const selectAPIErrorMessage = (state, key) => {
   const error = selectAPIError(state, key);
   return error && error.message;
 };
+
+export const selectAPIHttpStatus = (state, key) => {
+  const error = selectAPIError(state, key);
+  return error?.response?.status;
+};

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification
 name: foreman_remote_execution
 version: !ruby/object:Gem::Version
-  version: 16.6.6
+  version: 16.7.0
 platform: ruby
 authors:
 - Foreman Remote Execution team
 bindir: bin
 cert_chain: []
-date: 2026-10-02 00:00:00.000000000 Z
+date: 2026-07-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: deface
@@ -30,9 +30,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 8.3.0
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '13'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -40,9 +37,6 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: 8.3.0
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '13'
 description: A plugin bringing remote execution to the Foreman, completing the config
   management functionality with remote management functionality.
 email:
@@ -388,6 +382,7 @@ files:
 - webpack/JobInvocationDetail/JobInvocationActions.js
 - webpack/JobInvocationDetail/JobInvocationConstants.js
 - webpack/JobInvocationDetail/JobInvocationDetail.scss
+- webpack/JobInvocationDetail/JobInvocationEmptyState.js
 - webpack/JobInvocationDetail/JobInvocationHostTable.js
 - webpack/JobInvocationDetail/JobInvocationOverview.js
 - webpack/JobInvocationDetail/JobInvocationSelectors.js
@@ -401,11 +396,13 @@ files:
 - webpack/JobInvocationDetail/TemplateInvocationComponents/TemplateActionButtons.js
 - webpack/JobInvocationDetail/TemplateInvocationComponents/index.scss
 - webpack/JobInvocationDetail/TemplateInvocationPage.js
+- webpack/JobInvocationDetail/__tests__/JobInvocationDetailEmptyState.test.js
 - webpack/JobInvocationDetail/__tests__/MainInformation.test.js
 - webpack/JobInvocationDetail/__tests__/OutputCodeBlock.test.js
 - webpack/JobInvocationDetail/__tests__/TableToolbarActions.test.js
 - webpack/JobInvocationDetail/__tests__/TemplateInvocation.test.js
 - webpack/JobInvocationDetail/__tests__/fixtures.js
+- webpack/JobInvocationDetail/__tests__/foremanTestHelpers.js
 - webpack/JobInvocationDetail/index.js
 - webpack/JobWizard/Footer.js
 - webpack/JobWizard/JobWizard.js
@@ -475,6 +472,7 @@ files:
 - webpack/__mocks__/foremanReact/common/globalIdHelpers.js
 - webpack/__mocks__/foremanReact/common/helpers.js
 - webpack/__mocks__/foremanReact/common/hooks/API/APIHooks.js
+- webpack/__mocks__/foremanReact/common/hooks/Permissions/permissionHooks.js
 - webpack/__mocks__/foremanReact/components/AutoComplete/AutoCompleteActions.js
 - webpack/__mocks__/foremanReact/components/AutoComplete/AutoCompleteConstants.js
 - webpack/__mocks__/foremanReact/components/BreadcrumbBar/index.js
@@ -566,7 +564,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 4.0.20
+rubygems_version: 4.0.10
 specification_version: 4
 summary: A plugin bringing remote execution to the Foreman, completing the config
   management functionality with remote management functionality.