Merge pull request #5116 from jakemcdermott/remove-jdetails-close

Remove close button from job details

Reviewed-by: https://github.com/apps/softwarefactory-project-zuul
This commit is contained in:
softwarefactory-project-zuul[bot]
2019-10-25 16:43:21 +00:00
committed by GitHub
2 changed files with 0 additions and 15 deletions

View File

@@ -279,14 +279,6 @@ function JobDetail({ job, i18n, history }) {
> >
{i18n._(t`Delete`)} {i18n._(t`Delete`)}
</Button> </Button>
<Button
variant="secondary"
aria-label="close"
component={Link}
to="/jobs"
>
{i18n._(t`Close`)}
</Button>
</ActionButtonWrapper> </ActionButtonWrapper>
{isDeleteModalOpen && ( {isDeleteModalOpen && (
<AlertModal <AlertModal

View File

@@ -13,13 +13,6 @@ describe('<JobDetail />', () => {
mountWithContexts(<JobDetail job={mockJobData} />); mountWithContexts(<JobDetail job={mockJobData} />);
}); });
test('should display a Close button', () => {
const wrapper = mountWithContexts(<JobDetail job={mockJobData} />);
expect(wrapper.find('Button[aria-label="close"]').length).toBe(1);
wrapper.unmount();
});
test('should display details', () => { test('should display details', () => {
const wrapper = mountWithContexts(<JobDetail job={mockJobData} />); const wrapper = mountWithContexts(<JobDetail job={mockJobData} />);