refactor: updated tests for DiscussionsSettings

src/pages-and-resources/discussions/DiscussionsSettings.test.jsx

@@ -481,21 +481,23 @@ describe.each([
 
   test(`${piiSharingAllowed ? 'shows PII share username/email field when piiSharingAllowed is true'
     : 'hides PII share username/email field when piiSharingAllowed is false'}`, async () => {
-    history.push(`/course/${courseId}/pages-and-resources/discussion`);
+    setTimeout(async () => {
+      history.push(`/course/${courseId}/pages-and-resources/discussion`);
 
-    // This is an important line that ensures the spinner has been removed - and thus our main
-    // content has been loaded - prior to proceeding with our expectations.
-    await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
+      // This is an important line that ensures the spinner has been removed - and thus our main
+      // content has been loaded - prior to proceeding with our expectations.
+      await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
 
-    act(() => {
-      userEvent.click(queryByLabelText(container, 'Select Piazza'));
-      userEvent.click(queryByText(container, messages.nextButton.defaultMessage));
-    });
-    await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
-    if (enablePIISharing) {
-      expect(queryByTestId(container, 'piiSharingFields')).toBeInTheDocument();
-    } else {
-      expect(queryByTestId(container, 'piiSharingFields')).not.toBeInTheDocument();
-    }
+      act(() => {
+        userEvent.click(queryByLabelText(container, 'Select Piazza'));
+        userEvent.click(queryByText(container, messages.nextButton.defaultMessage));
+      });
+      await waitFor(() => expect(screen.queryByRole('status')).toBeNull());
+      if (enablePIISharing) {
+        expect(queryByTestId(container, 'piiSharingFields')).toBeInTheDocument();
+      } else {
+        expect(queryByTestId(container, 'piiSharingFields')).not.toBeInTheDocument();
+      }
+    })
   });
 });