Skip to content
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

Improve test files #262

Open
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

CracktheDom
Copy link

@CracktheDom CracktheDom commented Oct 22, 2024

⚠ Pull Requests not made with this template will be automatically closed 🔥

Prerequisites

Why do we need this pull request?

  • Explain the why behind your PR. We can see what it does from the code. But why does it do that?

Refactored test functions in tests/test_click.py to utilize a helper function and reduce redundant code. Consolidated similar tests into a single test.

What GitHub issues does this fix?

Copy / paste of output

Please copy and paste the output of PyWhat with your new addition using an example that tests this addition below:

I temporarily removed the following options from pyproject.toml to allow execution of tests.

[tool.pytest.ini_options]
addopts = "--black --isort --flake8 --mypy"
pytest tests/test_click.py 
======================================================================================================== test session starts ========================================================================================================
platform linux -- Python 3.12.6, pytest-8.3.3, pluggy-1.5.0
rootdir: /workspace/pyWhat
configfile: pyproject.toml
plugins: anyio-4.4.0, pylama-8.4.1
collected 95 items                                                                                                                                                                                                                  

tests/test_click.py .......................................................................................ss......                                                                                                           [100%]

=================================================================================================== 93 passed, 2 skipped in 3.04s ===================================================================================================

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.

Improve tests
1 participant