Fix overlapping act() calls error on SubmissionComments.test.js

[scotthallock]

Overview

When running the test suites, SubmissionComments.test.js logs two identical console errors:

Changes

• In the it('should allow user to edit comment and discard changes when edit button is pressed, but not edit more than 1 comment in a chain', ... ) test, remove the unneccessary nested waitFor() calls.
• The findByText() gets an await keyword in front of it because it is async
• The test suite no longer logs errors on the console:

Notes

• We can be certain all 4 tests are running because of expects.assertions(4)
• Related conversation about overlapping act calls, and needing to await the findBy* queries.

[vercel]

@scotthallock is attempting to deploy a commit to the c0d3-prod Team on Vercel.

A member of the Team first needs to authorize it.

[codecov]

Codecov Report

Merging #2887 (18e8778) into master (60d8d48) will not change coverage.
The diff coverage is n/a.

❗ Current head 18e8778 differs from pull request most recent head 8601a30. Consider uploading reports for the commit 8601a30 to get more accurate results

@@            Coverage Diff            @@
##            master     #2887   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          190       190           
  Lines         3576      3576           
  Branches       970       970           
=========================================
  Hits          3576      3576           

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
c0d3-app	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Apr 5, 2023 11:55pm

[flacial]

Nice one fixing these concerning console screams!