From 2c608a98837ddcd92700c82a92d27bdfddd52c4a Mon Sep 17 00:00:00 2001
From: Ethan Freestone <ethan.freestone@k-int.co.uk>
Date: Tue, 19 Nov 2024 15:22:30 +0000
Subject: [PATCH] test: DocumentFilterForm test tweaks

---
 lib/DocumentFilter/DocumentFilterForm.test.js | 29 +++++++++----------
 1 file changed, 14 insertions(+), 15 deletions(-)

diff --git a/lib/DocumentFilter/DocumentFilterForm.test.js b/lib/DocumentFilter/DocumentFilterForm.test.js
index 608a5fad..adaf0d0d 100644
--- a/lib/DocumentFilter/DocumentFilterForm.test.js
+++ b/lib/DocumentFilter/DocumentFilterForm.test.js
@@ -69,8 +69,8 @@ describe('DocumentFilterForm', () => {
 
     test('filter form is not open', async () => {
       const { queryByText } = renderComponent;
-      await waitFor(async () => {
-        await expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
+      await waitFor(() => {
+        expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
       });
     });
 
@@ -83,8 +83,8 @@ describe('DocumentFilterForm', () => {
 
       test('filter form is open', async () => {
         const { queryByText } = renderComponent;
-        await waitFor(async () => {
-          await expect(queryByText('DocumentFilterFieldArray')).toBeInTheDocument();
+        await waitFor(() => {
+          expect(queryByText('DocumentFilterFieldArray')).toBeInTheDocument();
         });
       });
     });
@@ -128,12 +128,11 @@ describe('DocumentFilterForm', () => {
         });
       });
 
-      it('should open the document filter builder modal when the edit button is clicked', async () => {
+      it('should render the document filter builder modal', async () => {
         const { getByText } = renderComponent;
-        await waitFor(async () => {
-          await Button('Edit document filters').click();
+        await waitFor(() => {
+          expect(getByText('Document filter builder')).toBeInTheDocument();
         });
-        expect(getByText('Document filter builder')).toBeInTheDocument();
       });
 
       describe('making form dirty for submittal', () => {
@@ -150,7 +149,7 @@ describe('DocumentFilterForm', () => {
           });
         });
 
-        it('should display the cancel button when the filter form is open', async () => {
+        it('should display the cancel button', async () => {
           await waitFor(async () => {
             await Button('ui-test-implementor.cancel').exists();
           });
@@ -165,8 +164,8 @@ describe('DocumentFilterForm', () => {
 
           it('should close the filter form', async () => {
             const { queryByText } = renderComponent;
-            await waitFor(async () => {
-              await expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
+            await waitFor(() => {
+              expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
             });
           });
         });
@@ -181,15 +180,15 @@ describe('DocumentFilterForm', () => {
           });
 
           test(`submit handler acted as expected for ${filtersInfo}`, async () => {
-            await waitFor(async () => {
-              await expect(filterHandlers.state.mock.calls[0][0]).toEqual(expectedSubmit);
+            await waitFor(() => {
+              expect(filterHandlers.state.mock.calls[0][0]).toEqual(expectedSubmit);
             });
           });
 
           test('filter form is not open', async () => {
             const { queryByText } = renderComponent;
-            await waitFor(async () => {
-              await expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
+            await waitFor(() => {
+              expect(queryByText('DocumentFilterFieldArray')).not.toBeInTheDocument();
             });
           });
         });