fix: update test results finisher to split out old impl

[joseph-sentry]

deps on: #1177

[seer-by-sentry]

✅ Sentry found no issues in your recent changes ✅

[codecov]

Codecov Report

Attention: Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.

Project coverage is 97.72%. Comparing base (dba53c5) to head (6a224b0).
Report is 6 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
tasks/test_results_finisher.py	90.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #1182      +/-   ##
==========================================
- Coverage   97.72%   97.72%   -0.01%     
==========================================
  Files         453      453              
  Lines       37117    37129      +12     
==========================================
+ Hits        36273    36284      +11     
- Misses        844      845       +1     

Flag	Coverage Δ
integration	42.81% <95.65%> (+0.01%)	⬆️
unit	90.53% <73.91%> (-0.02%)	⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

[codecov-notifications]

Codecov Report

Attention: Patch coverage is 95.65217% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
tasks/test_results_finisher.py	90.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

[sentry]

Suspect Issues

This pull request was deployed and Sentry observed the following issues:

• ‼️ ValueError: not enough values to unpack (expected 2, got 1) app.tasks.test_results.TestResultsFinisherTask View Issue
• ‼️ RepositoryWithoutValidBotError app.tasks.test_results.TestResultsFinisherTask View Issue
• ‼️ IntegrityError: (psycopg2.errors.UniqueViolation) duplicate key value violates unique constraint "reports_testresultreporttotals_report_id_key" app.tasks.test_results.TestResultsFinisherTask View Issue

_{Did you find this useful? React with a 👍 or 👎}