Remove redundant act calls in unit tests once a clear RHF workaround or fix exists

[brandonlenz]

Describe the bug

As noted in this discussion, RHF causes extra re-renders which are not captured in the implicit User Event act. In order to avoid large amounts of console output during tests, an extra act wrapper is required:

To Reproduce

Steps to reproduce the behavior:

1. On any unit test involving user events and form updates (e.g. YesNoQuestion.test.tsx)
2. Update the unit tests to simply await user.doSomething
3. Run the tests
4. See the console warnings

Expected behavior

User events should not need a wrapping act block.
Even waiting for a checked element to be checked before proceeding is not enough.

Code Snippet

Current workaround:

await act(() => user.click(submitButton))

or

await act(() => user.click(yesAnswer))

Correct syntax

await user.click(submitButton)

or

await user.click(yesAnswer)

[brandonlenz]

Moved to 'blocked' status as I don't think there's currently a better workaround for this