diff --git a/awx/ui_next/src/screens/WorkflowApproval/WorkflowApprovalDetail/WorkflowApprovalDetail.jsx b/awx/ui_next/src/screens/WorkflowApproval/WorkflowApprovalDetail/WorkflowApprovalDetail.jsx index fb20460a2d..eb9e8f6677 100644 --- a/awx/ui_next/src/screens/WorkflowApproval/WorkflowApprovalDetail/WorkflowApprovalDetail.jsx +++ b/awx/ui_next/src/screens/WorkflowApproval/WorkflowApprovalDetail/WorkflowApprovalDetail.jsx @@ -204,7 +204,8 @@ function WorkflowApprovalDetail({ i18n, workflowApproval }) { )} - {workflowApproval.summary_fields.user_capabilities && + {workflowApproval.status !== 'pending' && + workflowApproval.summary_fields.user_capabilities && workflowApproval.summary_fields.user_capabilities.delete && ( ', () => { assertDetail('Elapsed', '00:00:22'); expect(wrapper.find('Button[aria-label="Approve"]').length).toBe(1); expect(wrapper.find('Button[aria-label="Deny"]').length).toBe(1); - expect(wrapper.find('DeleteButton').length).toBe(1); }); test('should show expiration date/time', () => { @@ -227,12 +226,35 @@ describe('', () => { + ); + expect(wrapper.find('DeleteButton').length).toBe(0); + }); + + test('delete button should be hidden when job is pending', () => { + const wrapper = mountWithContexts( + @@ -245,7 +267,19 @@ describe('', () => { Promise.reject(new Error()) ); const wrapper = mountWithContexts( - + ); await waitForElement( wrapper,