test(*): remove all t.Parallel() calls in the tests

[barakmich]

Description

Testing

References

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 73.84%. Comparing base (cf317c7) to head (e637f92).

❌ Your project check has failed because the head coverage (73.84%) is below the target coverage (75.00%). You can increase the head coverage or adjust the target coverage.

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #2955      +/-   ##
==========================================
- Coverage   74.77%   73.84%   -0.93%     
==========================================
  Files         494      494              
  Lines       60623    60623              
==========================================
- Hits        45323    44758     -565     
- Misses      12136    12695     +559     
- Partials     3164     3170       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[jzelinskie]

There have been various PRs over the years adding/removing t.Parallel everywhere.
It'd be great if we could codify it somewhere that you shouldn't add this unless you've really tested it makes things faster. Maybe an approach similar to replacing panic with our linter + MustBugf()?

[tstirrat15]

I added it as a note in CONTRIBUTING.md. I'm disinclined to enforce anything programmatically because I think it's likely that there are going to be exceptions to our current rule.