PyTest not working in dev

MolarVerse / PQAnalysis

PQAnalysis is a API/CLI python package for the analysis of MD simulations

https://molarverse.github.io/PQAnalysis/

MIT License

4 stars 2 forks source link

PyTest not working in dev #21

Closed galjos closed 2 months ago

galjos commented 2 months ago

Describe the bug pytest does not start in dev, since it could not recognize arguments.

To Reproduce Steps to reproduce the behavior:

Clean environment with Python 3.12.3: python -m venv venv
Activate env: source venv/bin/activate
Install project: pip install .
Try testing: pytest (Maybe add as depedency, for easy use)

$ pytest
ERROR: usage: pytest [options] [file_or_dir] [file_or_dir] [...]
pytest: error: unrecognized arguments: --cov=PQAnalysis --cov-report=html
  inifile: /home/jog/projects/PQAnalysis/pytest.ini
  rootdir: /home/jog/projects/PQAnalysis

Expected behavior Should result in test execution.

97gamjak commented 2 months ago

Did you try pip install .[test]?

The test dependencies are only optional dependencies. In the next release to main this will be included in the developer guide in the documentation.

On Thu, 25 Apr 2024, 15:09 Josef M. Gallmetzer, @.***> wrote:

Assigned #21 https://github.com/MolarVerse/PQAnalysis/issues/21 to @97gamjak https://github.com/97gamjak.

— Reply to this email directly, view it on GitHub https://github.com/MolarVerse/PQAnalysis/issues/21#event-12608264751, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASNGWAUH6QCFJN26XAVYWELY7D6AFAVCNFSM6AAAAABGY3QFG2VHI2DSMVQWIX3LMV45UABCJFZXG5LFIV3GK3TUJZXXI2LGNFRWC5DJN5XDWMJSGYYDQMRWGQ3TKMI . You are receiving this because you were assigned.Message ID: @.***>

galjos commented 2 months ago

pip install '.[test]' did work, but still same issue

97gamjak commented 2 months ago

Ok, I'll check it out.

On Thu, 25 Apr 2024, 16:41 Josef M. Gallmetzer, @.***> wrote:

pip install '.[test]' did work, but still same issue

— Reply to this email directly, view it on GitHub https://github.com/MolarVerse/PQAnalysis/issues/21#issuecomment-2077381644, or unsubscribe https://github.com/notifications/unsubscribe-auth/ASNGWARLHE4ATAJTMCUIJBLY7EIZZAVCNFSM6AAAAABGY3QFG2VHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMZDANZXGM4DCNRUGQ . You are receiving this because you were mentioned.Message ID: @.***>

97gamjak commented 2 months ago

@galjos I reproduced now exactly each step as given in the description and it works on my side.

Could you try to write python -m pytest instead

galjos commented 2 months ago

Works