diff --git a/awx/ui_next/src/screens/ActivityStream/buildActivityDescription.jsx b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.jsx similarity index 99% rename from awx/ui_next/src/screens/ActivityStream/buildActivityDescription.jsx rename to awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.jsx index 95d613a44a..d933e0c259 100644 --- a/awx/ui_next/src/screens/ActivityStream/buildActivityDescription.jsx +++ b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.jsx @@ -1,6 +1,7 @@ import React from 'react'; import { Link } from 'react-router-dom'; import { t } from '@lingui/macro'; +import { withI18n } from '@lingui/react'; const buildAnchor = (obj, resource, activity) => { let url; @@ -150,7 +151,7 @@ const buildLabeledLink = (label, link) => { ); }; -export default (activity, i18n) => { +function ActivityStreamDescription({ i18n, activity }) { const labeledLinks = []; // Activity stream objects will outlive the resources they reference // in that case, summary_fields will not be available - show generic error text instead @@ -578,4 +579,6 @@ export default (activity, i18n) => { )} ); -}; +} + +export default withI18n()(ActivityStreamDescription); diff --git a/awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.test.jsx b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.test.jsx new file mode 100644 index 0000000000..9f3a2982d0 --- /dev/null +++ b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDescription.test.jsx @@ -0,0 +1,12 @@ +import React from 'react'; +import { mountWithContexts } from '../../../testUtils/enzymeHelpers'; +import ActivityStreamDescription from './ActivityStreamDescription'; + +describe('ActivityStreamDescription', () => { + test('initially renders succesfully', () => { + const description = mountWithContexts( + + ); + expect(description.find('span').length).toBe(1); + }); +}); diff --git a/awx/ui_next/src/screens/ActivityStream/StreamDetailButton.jsx b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.jsx similarity index 82% rename from awx/ui_next/src/screens/ActivityStream/StreamDetailButton.jsx rename to awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.jsx index b2e8dc368b..22559831b2 100644 --- a/awx/ui_next/src/screens/ActivityStream/StreamDetailButton.jsx +++ b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.jsx @@ -9,7 +9,7 @@ import { formatDateString } from '../../util/dates'; import { DetailList, Detail } from '../../components/DetailList'; import { VariablesDetail } from '../../components/CodeMirrorInput'; -function StreamDetailButton({ i18n, streamItem, user, description }) { +function ActivityStreamDetailButton({ i18n, streamItem, user, description }) { const [isOpen, setIsOpen] = useState(false); const setting = streamItem?.summary_fields?.setting; @@ -50,15 +50,17 @@ function StreamDetailButton({ i18n, streamItem, user, description }) { value={setting && setting[0]?.name} /> - + {streamItem?.changes && ( + + )} ); } -export default withI18n()(StreamDetailButton); +export default withI18n()(ActivityStreamDetailButton); diff --git a/awx/ui_next/src/screens/ActivityStream/StreamDetailButton.test.jsx b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.test.jsx similarity index 73% rename from awx/ui_next/src/screens/ActivityStream/StreamDetailButton.test.jsx rename to awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.test.jsx index 04c112c9e0..40dc104117 100644 --- a/awx/ui_next/src/screens/ActivityStream/StreamDetailButton.test.jsx +++ b/awx/ui_next/src/screens/ActivityStream/ActivityStreamDetailButton.test.jsx @@ -2,14 +2,14 @@ import React from 'react'; import { Link } from 'react-router-dom'; import { mountWithContexts } from '../../../testUtils/enzymeHelpers'; -import StreamDetailButton from './StreamDetailButton'; +import ActivityStreamDetailButton from './ActivityStreamDetailButton'; jest.mock('../../api/models/ActivityStream'); -describe('', () => { +describe('', () => { test('initially renders succesfully', () => { mountWithContexts( - ; return ( @@ -48,7 +48,7 @@ function ActivityStreamListItem({ streamItem, i18n }) { - { - test('initially renders succesfully', () => { - const description = mount(buildDescription({}, {})); - expect(description.find('span').length).toBe(1); - }); -}); diff --git a/awx/ui_next/src/screens/Inventory/Inventories.jsx b/awx/ui_next/src/screens/Inventory/Inventories.jsx index e137032ea4..17ee02b3be 100644 --- a/awx/ui_next/src/screens/Inventory/Inventories.jsx +++ b/awx/ui_next/src/screens/Inventory/Inventories.jsx @@ -31,7 +31,10 @@ function Inventories({ i18n }) { if (passedInventory && passedInventory.name !== inventory?.name) { setInventory(passedInventory); } - if (passedNestedObject && passedNestedObject.name !== nestedObject?.name) { + if ( + passedNestedObject && + passedNestedObject.name !== nestedObject?.name + ) { setNestedGroup(passedNestedObject); } if (passedSchedule && passedSchedule.name !== schedule?.name) {