[chore] [scraperhelper] add more unit tests

[axw]

Description

Add tests to the internal/controller package, so we have baseline coverage before making the changes in
#15116.

Link to tracking issue

N/A

Testing

Yes

Documentation

No

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 91.24%. Comparing base (307e3ab) to head (fae92cf).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #15125   +/-   ##
=======================================
  Coverage   91.24%   91.24%           
=======================================
  Files         699      699           
  Lines       44913    44913           
=======================================
+ Hits        40979    40980    +1     
+ Misses       2786     2785    -1     
  Partials     1148     1148           

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

[codspeed-hq]

Merging this PR will degrade performance by 31.92%

⚠️ Unknown Walltime execution environment detected

Using the Walltime instrument on standard Hosted Runners will lead to inconsistent data.

For the most accurate results, we recommend using CodSpeed Macro Runners: bare-metal machines fine-tuned for performance measurement consistency.

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

❌ 2 regressed benchmarks
✅ 5 untouched benchmarks
⏩ 76 skipped benchmarks¹

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

	Benchmark	BASE	HEAD	Efficiency
❌	BenchmarkHTTPProtoLogsSequential	3.7 ms	5.1 ms	-26.32%
❌	BenchmarkGRPCLogsSequential	4.3 ms	6.3 ms	-31.92%

---

_{Comparing axw:scraperherlper-controller-tests (fae92cf) with main (307e3ab)}

Footnotes

1. 76 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports. ↩