diff --git a/awx/ui_next/src/components/FormActionGroup/FormActionGroup.test.jsx b/awx/ui_next/src/components/FormActionGroup/FormActionGroup.test.jsx index 3a810aa884..d81d9e9e59 100644 --- a/awx/ui_next/src/components/FormActionGroup/FormActionGroup.test.jsx +++ b/awx/ui_next/src/components/FormActionGroup/FormActionGroup.test.jsx @@ -10,15 +10,4 @@ describe('FormActionGroup', () => { ); expect(wrapper).toHaveLength(1); }); - - test('should display error message if given', () => { - const wrapper = mountWithContexts( - {}} - onCancel={() => {}} - errorMessage={
oh noes
} - /> - ); - expect(wrapper.find('.error').text()).toEqual('oh noes'); - }); }); diff --git a/awx/ui_next/src/components/FormField/FormSubmitError.test.jsx b/awx/ui_next/src/components/FormField/FormSubmitError.test.jsx index 59a7b7d283..a52211d667 100644 --- a/awx/ui_next/src/components/FormField/FormSubmitError.test.jsx +++ b/awx/ui_next/src/components/FormField/FormSubmitError.test.jsx @@ -48,7 +48,7 @@ describe('', () => { ); }); wrapper.update(); - expect(wrapper.text()).toEqual('There was an error'); + expect(wrapper.find('Alert').prop('title')).toEqual('There was an error'); expect(global.console.error).toHaveBeenCalledWith(error); global.console = realConsole; }); diff --git a/awx/ui_next/src/screens/Project/ProjectAdd/ProjectAdd.test.jsx b/awx/ui_next/src/screens/Project/ProjectAdd/ProjectAdd.test.jsx index e24249a739..b4048ed55c 100644 --- a/awx/ui_next/src/screens/Project/ProjectAdd/ProjectAdd.test.jsx +++ b/awx/ui_next/src/screens/Project/ProjectAdd/ProjectAdd.test.jsx @@ -106,7 +106,8 @@ describe('', () => { project_local_paths: ['foobar', 'qux'], project_base_dir: 'dir/foo/bar', }; - ProjectsAPI.create.mockImplementation(() => Promise.reject(new Error())); + const error = new Error('oops'); + ProjectsAPI.create.mockImplementation(() => Promise.reject(error)); await act(async () => { wrapper = mountWithContexts(, { context: { config }, @@ -121,7 +122,7 @@ describe('', () => { }); wrapper.update(); expect(ProjectsAPI.create).toHaveBeenCalledTimes(1); - expect(wrapper.find('ProjectAdd .formSubmitError').length).toBe(1); + expect(wrapper.find('ProjectForm').prop('submitError')).toEqual(error); }); test('CardBody cancel button should navigate to projects list', async () => { diff --git a/awx/ui_next/src/screens/Project/ProjectEdit/ProjectEdit.test.jsx b/awx/ui_next/src/screens/Project/ProjectEdit/ProjectEdit.test.jsx index 6f8ca42945..ceb0e2c5b2 100644 --- a/awx/ui_next/src/screens/Project/ProjectEdit/ProjectEdit.test.jsx +++ b/awx/ui_next/src/screens/Project/ProjectEdit/ProjectEdit.test.jsx @@ -120,7 +120,10 @@ describe('', () => { project_local_paths: [], project_base_dir: 'foo/bar', }; - ProjectsAPI.update.mockImplementation(() => Promise.reject(new Error())); + const error = new Error('oops'); + const realConsoleError = global.console.error; + global.console.error = jest.fn(); + ProjectsAPI.update.mockImplementation(() => Promise.reject(error)); await act(async () => { wrapper = mountWithContexts( , @@ -135,7 +138,8 @@ describe('', () => { }); wrapper.update(); expect(ProjectsAPI.update).toHaveBeenCalledTimes(1); - expect(wrapper.find('ProjectEdit .formSubmitError').length).toBe(1); + expect(wrapper.find('ProjectForm').prop('submitError')).toEqual(error); + global.console.error = realConsoleError; }); test('CardBody cancel button should navigate to project details', async () => {