Add notes on testing, remove --runInBand

[dcloud]

Description of change

How to test

Issue(s)

• Backend tests too brittle. HHS/Head-Start-TTADP#263

Checklist

• Meets issue criteria
• Code tested
• Meets accessibility standards (WCAG 2.1 Levels A, AA)
• Documentation updated
  • API methods
  • Boundary and Data Flow Diagrams
  • Architectural Decision Records written for major infrastructure decisions with the Nygard template
  • OSCAL templates completed when security controls are implemented or modified

[dcloud]

This is a light PR with some notes and minor changes, since the cause of backend tests failures was already addressed.

I also have a branch with some complex changes that I would like to run by the team.

[jasalisbury]

🎉 Looks good! Just one question about the example Promise.all usage!

[jasalisbury]

In this example wouldn't await someAsyncFun(); wait for the resulting someAsyncFunc promise to resolve? This would mean a is not a promise but the resolved value of the promise? Should this be let a = someAsyncFunc()?

[dcloud]

Oh yeah, yup. I was going to write a before/after example, and then did neither. Good catch

[jasalisbury]

👍

[kryswisnaskas]

👍

[dcloud]

After looking into some issues with @dcmcand, we added some changes that mitigate errors arising from tests deleting more database records than they create. I also added a section on that into the docs/testing.md.

cc @kryswisnaskas