fix: conda python interpreter path on Windows

[yeyuan98]

Description

PythonScript._is_python_env was not working properly as of v7.8.2 on Windows thereby preventing python scripts from executing with python interpreters from rule-based conda environments.

The bug should be resulting from inconsistencies when conda creates Python environments on Windows versus Unix-like systems. On Windows, the python interpreter will not exist as <conda_env_root>/bin/python but as <conda_env_root>/python.exe. Please refer to Anaconda documentation here.

Added a new test case test_conda_python_script in tests folder and added the test to tests/tests.py. However, it seems that nose is unmaintained and I could not run the nosetests in my local computer.

QC

• The PR contains a test case for the changes or the changes are already covered by an existing test case.
• The documentation (docs/) is updated to reflect the changes or this is not necessary (e.g. if the change does neither modify the language nor the behavior or functionalities of Snakemake).

[johanneskoester]

Just use pytests, we no longer use nose. Maybe it needs to be removed somewhere from the docs?

[johanneskoester]

Mhm, this gigantic change in this file is fishy (all the lines below, not this single one)... What is going on here?

[johanneskoester]

Maybe windows linebreaks?

[yeyuan98]

Thanks a lot for your explanation.. Sorry, new git user here. I think the problem is I pushed from a Windows machine and did not set hard line break settings. Git forces CRLF and replace LF on Windows by default.

Fixed the line break problem. Hopefully it will work!

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[yeyuan98]

The previous commit got an issue with the black formatting guideline. Reformatted tests.py with black.