Fixes #39494 - Fix create report button is enabled when state is 'running'

webpack/JobInvocationDetail/JobInvocationToolbarButtons.js

@@ -19,7 +19,6 @@ import {
   enableRecurringLogic,
 } from './JobInvocationActions';
 import {
-  STATUS,
   GET_REPORT_TEMPLATES,
   GET_REPORT_TEMPLATE_INPUTS,
 } from './JobInvocationConstants';
@@ -43,6 +42,11 @@ const JobInvocationToolbarButtons = ({ jobId, data }) => {
   const [isActionOpen, setIsActionOpen] = useState(false);
   const [reportTemplateJobId, setReportTemplateJobId] = useState(undefined);
   const [templateInputId, setTemplateInputId] = useState(undefined);
+  const isCreateReportDisabled =
+    !useHasPermission('generate_report_templates') ||
+    task?.state === 'running' ||
+    reportTemplateJobId === undefined ||
+    templateInputId === undefined;
   const queryParams = new URLSearchParams({
     [`report_template_report[input_values][${templateInputId}][value]`]: jobId,
   });
@@ -210,11 +214,7 @@ const JobInvocationToolbarButtons = ({ jobId, data }) => {
               `/templates/report_templates/${reportTemplateJobId}/generate?${queryParams.toString()}`
             )}
             variant="secondary"
-            isDisabled={
-              !useHasPermission('generate_report_templates') ||
-              task?.state === STATUS.PENDING ||
-              templateInputId === undefined
-            }
+            isDisabled={isCreateReportDisabled}
           >
             {__(`Create report`)}
           </Button>

webpack/JobInvocationDetail/__tests__/MainInformation.test.js

@@ -198,6 +198,41 @@ describe('JobInvocationDetailPage', () => {
     ).toEqual(`/legacy/job_invocations/${jobId}`);
   });
 
+  it('disables Create report when the job task state is running', async () => {
+    const jobId = jobInvocationData.id;
+    const store = mockStore({
+      ...initialState,
+      JOB_INVOCATION_KEY: {
+        response: {
+          ...jobInvocationData,
+          task: { ...jobInvocationData.task, state: 'running' },
+        },
+      },
+      CURRENT_PERMISSIONS: {
+        response: {
+          results: [{ name: 'generate_report_templates' }],
+        },
+        status: 'RESOLVED',
+      },
+    });
+
+    render(
+      <Provider store={store}>
+        <JobInvocationDetailPage
+          match={{ params: { id: `${jobId}` } }}
+          {...props}
+        />
+      </Provider>
+    );
+
+    const createReportButton = screen.getByRole('link', {
+      name: 'Create report',
+    });
+
+    expect(createReportButton).toHaveAttribute('aria-disabled', 'true');
+    expect(createReportButton).toHaveClass('pf-m-disabled');
+  });
+
   it('shows scheduled date', async () => {
     const store = mockStore(initialStateScheduled);
     render(