From 278d153f8f48adf2d24d5d5a4b30842e34502cb4 Mon Sep 17 00:00:00 2001 From: Reinder Kraaij <kraaij@astron.nl> Date: Tue, 21 Nov 2023 14:08:26 +0100 Subject: [PATCH] Fixes Scheduling unit unit test --- .../frontend/tmss_webapp/src/routes/Project/create.test.js | 2 +- .../src/routes/Scheduling/__snapshots__/create.test.js.snap | 2 +- .../frontend/tmss_webapp/src/routes/Scheduling/create.js | 2 +- .../tmss_webapp/src/routes/Scheduling/create.test.js | 6 ++++-- 4 files changed, 7 insertions(+), 5 deletions(-) diff --git a/SAS/TMSS/frontend/tmss_webapp/src/routes/Project/create.test.js b/SAS/TMSS/frontend/tmss_webapp/src/routes/Project/create.test.js index 4b7bc52b44c..bee5e40b6ef 100644 --- a/SAS/TMSS/frontend/tmss_webapp/src/routes/Project/create.test.js +++ b/SAS/TMSS/frontend/tmss_webapp/src/routes/Project/create.test.js @@ -406,7 +406,7 @@ it("remove default resource and added resource", async () => { await act(async () => { fireEvent.click(screen.getAllByText("Add Resources")[0].parentElement.parentElement.parentElement.children[3]); }); - await new Promise((r) => setTimeout(r, 1000)); + const addResourceInput = screen.getAllByText("LOFAR Observing Time Commissioning")[0]; fireEvent.click(addResourceInput); // After selecting New Resource diff --git a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/__snapshots__/create.test.js.snap b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/__snapshots__/create.test.js.snap index 04ff8abee57..984a0022612 100644 --- a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/__snapshots__/create.test.js.snap +++ b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/__snapshots__/create.test.js.snap @@ -3,7 +3,7 @@ exports[`Scheduling Unit create page default Renders with all standard service mocks 1`] = ` Array [ <div - className="page-header" + className="page-header SchedulingUnit-PageHeader" > <div className="title" diff --git a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.js b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.js index 574440f8e3d..3e37d2876bd 100644 --- a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.js +++ b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.js @@ -949,7 +949,7 @@ export class SchedulingUnitCreate extends Component { return ( <React.Fragment> <Toast ref={(el) => this.growl = el}/> - <PageHeader location={this.props.location} title={'Scheduling Unit - Add'} + <PageHeader location={this.props.location} className="SchedulingUnit-PageHeader" title={'Scheduling Unit - Add'} actions={[{ icon: 'fa-window-close', title: 'Click to close Scheduling Unit creation', type: 'button', actOn: 'click', props: {callback: this.checkIsDirty} diff --git a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.test.js b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.test.js index c89e313986c..222886a5ed0 100644 --- a/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.test.js +++ b/SAS/TMSS/frontend/tmss_webapp/src/routes/Scheduling/create.test.js @@ -83,12 +83,14 @@ describe('Scheduling Unit create page default', () => { cleanup(); }); - test("Renders with all standard service mocks", () => { + test("Renders with all standard service mocks", async () => { const page = renderer .create(<SchedulingUnitCreate location={{ pathname: '/schedulingunit/create' }} />) .toJSON(); - expect(page).toMatchSnapshot(); + + + await waitFor (() => expect(page).toMatchSnapshot() ); }) -- GitLab