Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

pytest: Simplify usage. #19

Open
wxtim opened this issue Jan 4, 2021 · 2 comments
Open

pytest: Simplify usage. #19

wxtim opened this issue Jan 4, 2021 · 2 comments
Milestone

Comments

@wxtim
Copy link
Member

wxtim commented Jan 4, 2021

Describe the bug
At commit 1fca257d4d5 it was found necessary to run pytest separately for the doctests and tests folder.

Steps to reproduce the bug
run pytest --doctest-modules.

Expected behavior
Doctests from cylc/ and tests from tests/ should both be run.

Pull requests welcome!
This is an Open Source project - please consider contributing a bug fix
yourself (please read CONTRIBUTING.md before starting any work though).

@wxtim wxtim added the bug Something isn't working label Jan 4, 2021
@wxtim wxtim added this to the later milestone Feb 8, 2021
@wxtim
Copy link
Member Author

wxtim commented Aug 18, 2021

I've replicated the problem without running the entire battery using

pytest cylc/rose/entry_points.py tests/test_functional_post_install.py --dist=no -n0 --pdb

Looks like somewhere code is importing rose without the leading metomi, but it's not clear why.

@MetRonnie
Copy link
Member

@oliver-sanders oliver-sanders removed the bug Something isn't working label Apr 27, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants