Skip to content

Use keyword-only parameters in tests#2986

Merged
adamtheturtle merged 1 commit intomainfrom
keyword-only-test-params
Feb 22, 2026
Merged

Use keyword-only parameters in tests#2986
adamtheturtle merged 1 commit intomainfrom
keyword-only-test-params

Conversation

@adamtheturtle
Copy link
Member

@adamtheturtle adamtheturtle commented Feb 21, 2026

Summary

  • Enforce keyword-only arguments (using *) in test functions that accept multiple parameters
  • This improves call-site clarity and prevents accidental positional argument errors

Test plan

  • Verify tests still pass

Note

Low Risk
Mechanical test-signature change only; risk is limited to potential pytest collection/parametrization edge cases if any test relies on positional arguments.

Overview
This PR changes many pytest test functions (and one CI linter test) to enforce keyword-only parameters by adding * to function signatures.

No runtime behavior is intentionally changed; the update is a broad signature/style adjustment across the mock VWS test suite to make argument passing explicit and reduce accidental positional ordering mistakes.

Written by Cursor Bugbot for commit 4e311e6. This will update automatically on new commits. Configure here.

Enforce keyword-only arguments in test functions that accept
multiple parameters, improving call-site clarity.
@adamtheturtle adamtheturtle enabled auto-merge (squash) February 22, 2026 00:04
@adamtheturtle adamtheturtle merged commit cc510cd into main Feb 22, 2026
106 checks passed
@adamtheturtle adamtheturtle deleted the keyword-only-test-params branch February 22, 2026 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant