Skip to content

Add kw_only to VWSCredentials test dataclass#1311

Merged
adamtheturtle merged 1 commit intomainfrom
dataclass-kw-only-credentials
Mar 29, 2026
Merged

Add kw_only to VWSCredentials test dataclass#1311
adamtheturtle merged 1 commit intomainfrom
dataclass-kw-only-credentials

Conversation

@adamtheturtle
Copy link
Copy Markdown
Member

@adamtheturtle adamtheturtle commented Mar 29, 2026

Adds kw_only=True to the test VWSCredentials dataclass. The session fixture already passes email_address= and password=.

Made with Cursor


Note

Low Risk
Low risk: test-only change that only affects how VWSCredentials is instantiated, with no production logic or data handling impact.

Overview
Updates the test VWSCredentials dataclass to require keyword-only initialization by adding kw_only=True, preventing accidental positional argument use in tests.

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

Matches the pattern used across VWS-related packages; conftest already
constructs with keywords.

Made-with: Cursor
@adamtheturtle adamtheturtle merged commit f13e8be into main Mar 29, 2026
30 checks passed
@adamtheturtle adamtheturtle deleted the dataclass-kw-only-credentials branch March 29, 2026 08:02
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