feat: Make calls to numtracker async

[tpoliaw]

No description provided.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.18%. Comparing base (818c29f) to head (fc9782a).

Additional details and impacted files

@@           Coverage Diff           @@
##             main    #1075   +/-   ##
=======================================
  Coverage   95.18%   95.18%           
=======================================
  Files          43       43           
  Lines        3009     3010    +1     
=======================================
+ Hits         2864     2865    +1     
  Misses        145      145           

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

[callumforrester]

@tpoliaw the fact that this isn't making any tests fail is concerning

[tpoliaw]

We're never relying on bluesky calling it in tests and the system tests don't use numtracker

[abbiemery]

Since we've moved on somewhat since this was opened: do we now use the call in system tests/is this tested, and if so does the change to make it async affect anything?

Could we either get this tested and in, or close this PR?

[tpoliaw]

Rebasing it on main still doesn't cause any tests to fail so I guess it works. This probably makes sense now that bluesky support async scan_id_sources.

Whether we still want to merge this without removing other uses of requests is a separate question.

[abbiemery]

nit: explicitly import httpx_mock

[abbiemery]

Has the bluesky change made it into a release? yes