Skip to content

Improve test files#262

Open
CracktheDom wants to merge 4 commits intobee-san:mainfrom
CracktheDom:improve-test-files
Open

Improve test files#262
CracktheDom wants to merge 4 commits intobee-san:mainfrom
CracktheDom:improve-test-files

Conversation

@CracktheDom
Copy link
Copy Markdown

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in 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