Skip to content
New issue

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

#

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

Already on GitHub? # to your account

Jump to bottom

Improve test files #262

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

Improve test files #262

wants to merge 4 commits into from

Conversation

CracktheDom
Copy link

@CracktheDom CracktheDom commented Oct 22, 2024 •
edited
Loading

⚠ 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 ===================================================================================================

# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Improve tests
1 participant
@CracktheDom