From 279fe53837b65aeccb084862e4f38d0fd1a9f06c Mon Sep 17 00:00:00 2001 From: Jake McDermott Date: Mon, 6 Apr 2020 12:10:44 -0400 Subject: [PATCH] Remove state checks from user list test Don't check for loading in UserList test --- .../screens/User/UserList/UserList.test.jsx | 35 +++---------------- 1 file changed, 4 insertions(+), 31 deletions(-) diff --git a/awx/ui_next/src/screens/User/UserList/UserList.test.jsx b/awx/ui_next/src/screens/User/UserList/UserList.test.jsx index a727cdae81..7433c63208 100644 --- a/awx/ui_next/src/screens/User/UserList/UserList.test.jsx +++ b/awx/ui_next/src/screens/User/UserList/UserList.test.jsx @@ -121,17 +121,8 @@ describe('UsersList with full permissions', () => { }); test('Users are retrieved from the api and the components finishes loading', async () => { - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === true - ); + await waitForElement(wrapper, 'ContentLoading', el => el.length === 0); expect(loadUsers).toHaveBeenCalled(); - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === false - ); }); test('Selects one team when row is checked', async () => { @@ -255,17 +246,7 @@ describe('UsersList with full permissions', () => { }); test('Add button shown for users with ability to POST', async () => { - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === true - ); - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === false - ); - expect(wrapper.find('ToolbarAddButton').length).toBe(1); + await waitForElement(wrapper, 'ToolbarAddButton', el => el.length === 1); }); }); @@ -280,16 +261,8 @@ describe('UsersList without full permissions', () => { }); wrapper = mountWithContexts(); - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === true - ); - await waitForElement( - wrapper, - 'UsersList', - el => el.state('hasContentLoading') === false - ); + await waitForElement(wrapper, 'ContentLoading', el => el.length === 1); + await waitForElement(wrapper, 'ContentLoading', el => el.length === 0); expect(wrapper.find('ToolbarAddButton').length).toBe(0); }); });