Skip to content

Enhance pytest parameterization validation and AST node conversion#234

Merged
dphuang2 merged 1 commit intomainfrom
validate-invalid-completion-params-pytest-mark
Sep 29, 2025
Merged

Enhance pytest parameterization validation and AST node conversion#234
dphuang2 merged 1 commit intomainfrom
validate-invalid-completion-params-pytest-mark

Conversation

@dphuang2
Copy link
Copy Markdown
Collaborator

  • Added validation to ensure that argvalues in pytest.mark.parametrize is a list or tuple, not a dict.
  • Introduced utility functions to convert AST nodes, including dictionaries, lists, and tuples, to their string representations for better error messaging.
  • Updated the _is_pytest_parametrize_with_completion_params function to include argvalues validation.

- Added validation to ensure that argvalues in pytest.mark.parametrize is a list or tuple, not a dict.
- Introduced utility functions to convert AST nodes, including dictionaries, lists, and tuples, to their string representations for better error messaging.
- Updated the _is_pytest_parametrize_with_completion_params function to include argvalues validation.
@dphuang2 dphuang2 merged commit 6fe0baa into main Sep 29, 2025
6 of 7 checks passed
@dphuang2 dphuang2 deleted the validate-invalid-completion-params-pytest-mark branch September 29, 2025 20:14
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