Split checks on PRs so acceptance tests and unit tests run separately

[SarahFrench]

Description

If a PR is adding unit tests it is unclear where in the PR's checks the unit tests are running. There is a clear check for acceptance tests, but there isn't an equivalent for unit tests:

Potential Terraform Configuration Change

We could update the existing acceptance tests check run only acceptance tests (filter tests by TestAcc prefix) and have a separate check that's specific to unit tests (e.g achieved by having TF_ACC environment variable unset).

References

• Example of a PR adding unit tests : 🛠️ Refactor metadata flattener #2345

Community Note

• Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
• If you are interested in working on this issue or have submitted a pull request, please leave a comment

[arybolovlev]

💡 We can prefix unit tests with TestUnit to filter them additionally and avoid Acceptance tests skipped unless env 'TF_ACC' set noisy messages.

[SarahFrench]

Discussion from triage: Select for unit tests using a negated regex similar to ^Test[^Ac]{3}.*$ - though this example would accidentally exclude TestAAA..., Testccc.... Another option could be Test(?!Acc) if it's compatible with go test.