Add pytest dependency

[q-posev]

ctest proceeds to test the Python bindings via the pytest, which is not a built-in Python package. Thus, ctest fails if pytest is not installed. The source of failure is only visible when ctest -V is invoked.

It might be a good idea to document an optional pytest dependency and/or install it automatically (as pybind11).

Description

Add pytest to the list of dependencies.

Motivation and Context

ctest proceeds to test the Python bindings via the pytest, which is not a built-in Python package. Thus, ctest fails if pytest is not installed.

How Has This Been Tested?

No testing required. No side effects on the source code.

Types of changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Questions

• [ ] Should pytest be installed automatically (if not yet present) when XCFUN_PYTHON_INTERFACE is ON ?

Status

• [X] Ready to go

[robertodr]

Thanks for clarifying the documentation. pytest is included in requirements.txt and environment.yml, but you're right that it should have been spelled out that it's required.

[q-posev]

@robertodr thanks for clarifying! Indeed, I should have looked into the requirements.txt.

In this case, perhaps it would be helpful to have a note in the docs reminding the user to use requirements.txt or environment.yml file.

[robertodr]

Yeah, it would be nice if you could add that as well 😄

And to answer the question:

Should pytest be installed automatically (if not yet present) when XCFUN_PYTHON_INTERFACE is ON ?

I would say "no" since then we'd have to track how Python packages are installed (virtualenv+pip? conda? conda-env+pip? something else?) and I think that's overkill.

[bast]

Sorry for silence:

• I agree that better no auto-install to give users more control over their environment
• Can I help with the documentation to unblock the pull request?

[q-posev]

Sorry for my late reply. I push the necessary changes now.

[robertodr]

The windows test runners are somehow stuck... Merging regardless