From 8e902a74db0005582127264a214a8aab06c85d1b Mon Sep 17 00:00:00 2001 From: hawkespoulterva Date: Mon, 21 Oct 2024 14:33:07 -0400 Subject: [PATCH] VirtualHearingFields RTL refactor --- .../details/VirtualHearingFields.test.js | 29 ++++++++----------- .../VirtualHearingFields.test.js.snap | 7 +++++ 2 files changed, 19 insertions(+), 17 deletions(-) diff --git a/client/test/app/hearings/components/details/VirtualHearingFields.test.js b/client/test/app/hearings/components/details/VirtualHearingFields.test.js index 71c866362e9..fd995de1b8e 100644 --- a/client/test/app/hearings/components/details/VirtualHearingFields.test.js +++ b/client/test/app/hearings/components/details/VirtualHearingFields.test.js @@ -56,7 +56,6 @@ describe('VirtualHearingFields', () => { , { wrapper: Wrapper, @@ -89,8 +88,6 @@ describe('VirtualHearingFields', () => { } ); - const hearingMeetingType = amaHearing.judge.meetingType; - // Assertions const virtualHearingForm = screen.getByRole('heading', { name: /Hearing Links/i }); expect(virtualHearingForm).toBeInTheDocument(); @@ -104,10 +101,11 @@ describe('VirtualHearingFields', () => { // Test HearingLinks presence const guestLink = screen.getByText(/Guest Link:/); expect(guestLink).toBeInTheDocument(); - expect(hearingMeetingType).toBeTruthy(); - expect(hearingMeetingType).toStrictEqual('pexip' || 'webex'); - expect(virtualHearingForm).toMatchSnapshot(); + const links = screen.getAllByRole('button'); + expect(links).toHaveLength(1); + + expect(asFragment()).toMatchSnapshot(); }); test('Renders webex conference when conference provider is webex', () => { @@ -117,7 +115,7 @@ describe('VirtualHearingFields', () => { }; // Run the test - const virtualHearingForm = mount( + const { asFragment } = customRender( { conferenceProvider: 'webex' }} />, - { - wrappingComponent: hearingDetailsWrapper(anyUser, webexHearing), - wrappingComponentProps: { store: detailsStore } + wrapper: Wrapper, + wrapperProps: { user: anyUser, hearing: webexHearing, store: detailsStore } } ); // Assertions - expect(virtualHearingForm.text().includes('Webex Hearing')).toBeTruthy(); - - expect(virtualHearingForm).toMatchSnapshot(); + expect(screen.getByText('Webex Hearing')).toBeInTheDocument(); + expect(asFragment()).toMatchSnapshot(); }); test('Renders pexip conference when conference provider is pexip', () => { @@ -146,7 +142,7 @@ describe('VirtualHearingFields', () => { }; // Run the test - const virtualHearingForm = mount( + const { asFragment } = customRender( { conferenceProvider: 'pexip' }} />, - { - wrappingComponent: hearingDetailsWrapper(anyUser, webexHearing), - wrappingComponentProps: { store: detailsStore } + wrapper: Wrapper, + wrapperProps: { user: anyUser, hearing: webexHearing, store: detailsStore } } ); diff --git a/client/test/app/hearings/components/details/__snapshots__/VirtualHearingFields.test.js.snap b/client/test/app/hearings/components/details/__snapshots__/VirtualHearingFields.test.js.snap index d4fa9eb9f32..1c922a41c8c 100644 --- a/client/test/app/hearings/components/details/__snapshots__/VirtualHearingFields.test.js.snap +++ b/client/test/app/hearings/components/details/__snapshots__/VirtualHearingFields.test.js.snap @@ -302,6 +302,13 @@ exports[`VirtualHearingFields Renders pexip conference when conference provider
+
+ + Pexip Hearing + +