Reverb Chu

Results 12 comments of Reverb Chu

AFAIK, `pytest` will only register the fixtures that are available in the `locals()` of the current module, or from `locals()` of the magic `conftest.py`. If we don't explicitly import the...

> When you have a fixture that is local to the test module, not global for the whole test session. Also, you may override the global fixtures with some values...

@iakum this is implemented and released with v1.1.0 Can you help to verify if it's working as expected on your end?

Thanks for reporting the issue @crown-bdee! Apparently pytest is not happy when I run the collection on `setup.py`. I reproduced the error: ``` setup.py:13: in setup( ../_venv_/pylint-pytest-Q6jowuRR/lib/python3.8/site-packages/setuptools/__init__.py:145: in setup return...

@crown-bdee can you provide more info? More specifically the output from `pytest --collect-only --fixtures src/my_package/tests/test_my_module.py` It could probably relate to how the folders/modules were structured.

> Out of interest, @reverbc did that reproduce the error on your end? #22 should fix this issue when the `raise`/`sys.exit(1)` occurs in an unrelated module

Hi, thanks for trying out this plugin! Unfortunately I'm not able to reproduce the issue with given code snippets :/ If you can help to provide more code example on...

I'm sorry but I still can't reproduce the issue on my end to dig deeper. The way this plugin works for `unused-argument`, is first to collect all the fixtures (https://github.com/reverbc/pylint-pytest/blob/pylint-pytest-1.0.0/pylint_pytest/checkers/fixture.py#L73-L78),...

@OlgaPaw I'm not sure if your provided example is relevant to the original problem. 1. This plugin does not suppress `unused-variable` warnings. See https://github.com/reverbc/pylint-pytest#suppressed-pylint-warnings for the list of warnings that...

> > e2e_test.py:7:20: W0613: Unused argument 'virtual_site' (unused-argument) > > This sounds reasonable. If you're not going to use the fixture's return value, just apply it via `@pytest.mark.usefixtures('virtual_site')`. > >...