diff --git a/__tests__/pages/Jobs/Jobs.test.jsx b/__tests__/pages/Jobs/Jobs.test.jsx index 5fd09bbc57..8e25ce6220 100644 --- a/__tests__/pages/Jobs/Jobs.test.jsx +++ b/__tests__/pages/Jobs/Jobs.test.jsx @@ -10,7 +10,7 @@ describe('', () => { ); }); - test('should display correct breadcrumb heading', () => { + test('should display a breadcrumb heading', () => { const history = createMemoryHistory({ initialEntries: ['/jobs'], }); @@ -30,7 +30,7 @@ describe('', () => { } } ); - expect(wrapper.find('BreadcrumbHeading').text()).toEqual('Jobs'); + expect(wrapper.find('BreadcrumbHeading').length).toBe(1); wrapper.unmount(); }); }); diff --git a/__tests__/pages/Jobs/screens/Job/JobDetail.test.jsx b/__tests__/pages/Jobs/screens/Job/JobDetail.test.jsx index e490999c50..de475516ad 100644 --- a/__tests__/pages/Jobs/screens/Job/JobDetail.test.jsx +++ b/__tests__/pages/Jobs/screens/Job/JobDetail.test.jsx @@ -12,4 +12,13 @@ describe('', () => { ); }); + + test('should display a Close button', () => { + const wrapper = mountWithContexts( + + ); + + expect(wrapper.find('Button[aria-label="close"]').length).toBe(1); + wrapper.unmount(); + }); }); diff --git a/src/pages/Jobs/JobDetail/JobDetail.jsx b/src/pages/Jobs/JobDetail/JobDetail.jsx index ff1fa26a95..eab33c9f89 100644 --- a/src/pages/Jobs/JobDetail/JobDetail.jsx +++ b/src/pages/Jobs/JobDetail/JobDetail.jsx @@ -1,7 +1,14 @@ import React, { Component } from 'react'; -import { CardBody } from '@patternfly/react-core'; - +import { Link, withRouter } from 'react-router-dom'; +import { withI18n } from '@lingui/react'; +import { t } from '@lingui/macro'; +import { CardBody, Button } from '@patternfly/react-core'; +import styled from 'styled-components'; +const ActionButtonWrapper = styled.div` + display: flex; + justify-content: flex-end; +`; class JobDetail extends Component { constructor (props) { super(props); @@ -9,15 +16,28 @@ class JobDetail extends Component { render () { const { - job + job, + i18n } = this.props; + return ( {job.name} + + + + ); } } -export default JobDetail; +export default withI18n()(withRouter(JobDetail));