Forward get_flag_names for the ReadOnlyReport

[Swatinem]

The api_report_service.ReadOnlyReport overrides the flags property, but it looks like the usage of .flags would be better served by get_flag_names instead.

But that method previously did not exist on that Report class, so lets add it.

[codspeed-hq]

CodSpeed Performance Report

Merging #556 will create unknown performance changes

_{Comparing swatinem/readonly-flagnames (6daa195) with main (e9567e6)}

Summary

⚠️ No benchmarks were detected in both the base of the PR and the PR.\

[codecov]

Codecov Report

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

Project coverage is 88.72%. Comparing base (e9567e6) to head (6daa195).
Report is 3 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
shared/reports/readonly.py	50.00%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #556      +/-   ##
==========================================
- Coverage   88.73%   88.72%   -0.01%     
==========================================
  Files         459      459              
  Lines       13130    13132       +2     
  Branches     1510     1510              
==========================================
+ Hits        11651    11652       +1     
- Misses       1163     1164       +1     
  Partials      316      316              

Flag	Coverage Δ
shared-docker-uploader	88.72% <66.66%> (-0.01%)	⬇️

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 66.66667% with 1 line in your changes missing coverage. Please review.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
shared/reports/readonly.py	50.00%	1 Missing ⚠️

📢 Thoughts on this report? Let us know!

🚀 New features to boost your workflow:

• ❄ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[joseph-sentry]

should we be caching the result of get_flag_names internally in the LazyRustReport or even in the implementation of the inner report in resource.py?

[Swatinem]

I think it might make sense as a followup potentially. Though I guess this is already rather lightweight already, as it it just iterating through the sessions. Though its quite possible that reports have a shitton fo sessions because of perpetually carryforwarding old shit thats never being cleaned up or other kinds of reasons.