Use * for keyword-only arguments in pytest fixtures

[adamtheturtle]

Pytest fixture functions should use * for keyword-only arguments when they have arguments. This change adds * to all affected fixtures.

Made with Cursor

---

Note

Low Risk
Low risk: test-only change that only tightens fixture signatures to require keyword-only injection, with no production logic impact.

Overview
Updates tests/conftest.py fixtures to make their dependencies keyword-only by adding * to vws_client and cloud_reco_client parameters, aligning with pytest fixture style and preventing accidental positional usage.

^{Written by Cursor Bugbot for commit b5c351f. This will update automatically on new commits. Configure here.}

[cursor]

Cursor Bugbot has reviewed your changes and found 1 potential issue.

^{Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.}

[cursor]

Missing * in fixture_vumark_target fixture signature

Low Severity

The PR intends to add * for keyword-only arguments to all fixtures with parameters, but fixture_vumark_target was missed. It still uses vumark_database as a positional argument while the other two fixtures with arguments (vws_client and cloud_reco_client) were updated with *. This creates an inconsistency in the codebase.