Test Suite Overhaul

[FireLemons]

There are 4 things to check in our test suite

• DRY Test Code #6874
• Prune request tests that check for content and make sure there's tests in view tests instead because it takes less resources(not including content stored in the session).
• Missing Tests
• Remove Race Condition From All System Tests #6693
• Ensuring System Tests Refactored for Speed

[stefannibrasil]

@FireLemons @compwron for the missing tests, are there any critical flows you would recommend to get started with?

[compwron]

@stefannibrasil see https://gist.github.com/compwron/e1c399823b0c56c2b9cd1091abe3c243

1. EmancipationsController — most complex untested controller with branching JSON logic
2. CreateCasaAdminService / CreateAllCasaAdminService — user creation is security-critical
3. CaseContactsExportCsvService — primary data export, used for grant reporting
4. PlacementsController — full CRUD with no coverage

[stefannibrasil]

Thank you, that's really helpful. I will probably create one issue for each flow to keep the PRs small and get more contributors.