fix: rewriting of test result reports

[joseph-sentry]

the rewrite_readable and writing of the storage file should've been outside the loop, having them inside the loop was:

1. doing extra useless work by calling rewrite_readable a bunch of times
2. causing rate limit errors in GCS due to trying to write to the same object so often, resulting in the task failing

[sentry-io]

🔍 Existing Issues For Review

Your pull request is modifying functions with the following pre-existing issues:

📄 File: tasks/test_results_processor.py

Function	Unhandled Issue
process_individual_arg	JSONDecodeError: Expecting value: line 1 column 1 (char 0) app.tasks.test_results.TestResult... Event Count: 20
process_individual_arg	FileNotInStorageError: File test_results/v1/raw/2024-10-29/4788895BC85C5F4AF1085A3459D47062/3de9021bfd9be17de97453a687b0... ... Event Count: 1

---

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

[codecov-notifications]

Codecov Report

All modified and coverable lines are covered by tests ✅

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   98.03%   98.03%           
=======================================
  Files         444      444           
  Lines       36083    36083           
=======================================
  Hits        35373    35373           
  Misses        710      710           

Flag	Coverage Δ
integration	98.03% <100.00%> (ø)
unit	98.03% <100.00%> (ø)

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

Components	Coverage Δ
NonTestCode	95.98% <100.00%> (ø)
OutsideTasks	98.06% <ø> (ø)

Files with missing lines	Coverage Δ
tasks/test_results_processor.py	97.18% <100.00%> (ø)

[codecov-qa]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.03%. Comparing base (9c70365) to head (2ed7131).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   98.03%   98.03%           
=======================================
  Files         444      444           
  Lines       36083    36083           
=======================================
  Hits        35373    35373           
  Misses        710      710           

Flag	Coverage Δ
integration	98.03% <100.00%> (ø)
unit	98.03% <100.00%> (ø)

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

Components	Coverage Δ
NonTestCode	95.98% <100.00%> (ø)
OutsideTasks	98.06% <ø> (ø)

Files with missing lines	Coverage Δ
tasks/test_results_processor.py	97.18% <100.00%> (ø)

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.03%. Comparing base (9c70365) to head (2ed7131).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Additional details and impacted files

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   98.03%   98.03%           
=======================================
  Files         444      444           
  Lines       36083    36083           
=======================================
  Hits        35373    35373           
  Misses        710      710           

Flag	Coverage Δ
integration	98.03% <100.00%> (ø)
unit	98.03% <100.00%> (ø)

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

Components	Coverage Δ
NonTestCode	95.98% <100.00%> (ø)
OutsideTasks	98.06% <ø> (ø)

Files with missing lines	Coverage Δ
tasks/test_results_processor.py	97.18% <100.00%> (ø)

[codecov-public-qa]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 98.03%. Comparing base (9c70365) to head (2ed7131).

✅ All tests successful. No failed tests found.

@@           Coverage Diff           @@
##             main     #844   +/-   ##
=======================================
  Coverage   98.03%   98.03%           
=======================================
  Files         444      444           
  Lines       36083    36083           
=======================================
  Hits        35373    35373           
  Misses        710      710           

Flag	Coverage Δ
integration	98.03% <100.00%> (ø)
unit	98.03% <100.00%> (ø)

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

Components	Coverage Δ
NonTestCode	95.98% <100.00%> (ø)
OutsideTasks	98.06% <ø> (ø)

Files	Coverage Δ
tasks/test_results_processor.py	97.18% <100.00%> (ø)

[matt-codecov]

should this also be moved out of the loop?