Skip to content

Add with_case_tags decorator #361

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Draft
wants to merge 10 commits into
base: main
Choose a base branch
from

Conversation

michele-riva
Copy link
Contributor

@michele-riva michele-riva commented May 8, 2025

As proposed in #351.

According to suggestions by @smarie here the necessary tasks:

  • Port the decorator code to pytest-cases. Currently in case_funcs.py.
  • Update the changelog.
  • Add appropriate documentation.
  • Write tests for full coverage.

Document addition of with_case_tags class decorator.
Can be imported as

```python
from pytest_cases import with_case_tags
```
@michele-riva
Copy link
Contributor Author

Note: python 3.7 tests are failing because of a SyntaxError in typing_extensions. The / symbol was introduced in py3.8 to delimit positional-only arguments. Perhaps this is because of a misconfiguration (wrong version of typing_extensions installed, incompatible with python 3.7)?

Co-authored-by: Sylvain Marié <[email protected]>
michele-riva and others added 2 commits May 9, 2025 16:05
@michele-riva
Copy link
Contributor Author

Thanks @smarie for the review of the current status. I'll get to the unresolved points above soon, and add the (till now completely) missing tests.

@michele-riva
Copy link
Contributor Author

Do you perhaps have any insights into the test failures in py37 I mentioned above? I'm not a nox expert, so I may need a second pair of eyes.

michele-riva and others added 2 commits May 11, 2025 04:57
@smarie
Copy link
Owner

smarie commented May 14, 2025

The failing tests are due to a problem in typing extension with python 3.7. Apparently they forgot to declare that the latest release of typing extensions was not compliant with 3.7.
Anyway since this is very obsolete now I will open an issue and a PR to remove the old versions.

Here : #362

Stay tuned :)

As suggested by @smarie in smarie#361 (comment)

Co-Authored-By: Sylvain Marié <[email protected]>
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.

2 participants