refactor: use the fake test client in more api tests

[zimeg]

Summary

This PR refactors tests to use the NewFakeClient more often for less duplication and perhaps faster test runs?

Notes

• Recent passing unit tests on main: 46 seconds ⏳

Requirements

• I've read and understood the Contributing Guidelines and have done my best effort to follow them.
• I've read and agree to the Code of Conduct.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 64.59%. Comparing base (13f00d3) to head (2f59f1e).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #254      +/-   ##
==========================================
+ Coverage   64.58%   64.59%   +0.01%     
==========================================
  Files         212      212              
  Lines       17505    17505              
==========================================
+ Hits        11305    11308       +3     
+ Misses       5134     5133       -1     
+ Partials     1066     1064       -2     

☔ 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.

[zimeg]

👻 Spooked but initial tests ran not as fast: 1 minute 8 seconds

Edit: I don't think this should motivate the change - was curious!

[mwbrooks]

🦸🏻 Heroic test refactor @zimeg! Thanks for bringing in our table test standard format as well!

🧠 In the future, we may want to rename FakeClient to MockClient (or ClientMock).

[mwbrooks]

praise: 👌🏻

[zimeg]

@mwbrooks Thanks so much for a kind review too! I'm optimistic that these changes will make ongoing contribution a bit more clear in pattern so let's merge this now.

In the future, we may want to rename FakeClient to MockClient (or ClientMock).

Oh I thought about this too but found it unclear if a "fake" client was meant to be different than a "mock" client... So open to this follow up if it makes more sense to you! 💡