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

tests: add timeout to pytests #70

Closed
ap-- opened this issue Oct 18, 2019 · 1 comment
Closed

tests: add timeout to pytests #70

ap-- opened this issue Oct 18, 2019 · 1 comment

Comments

@ap--
Copy link
Owner

ap-- commented Oct 18, 2019

Some pytests might get stuck on reading data from the spectrometer.
This is especially a problem for pyseabreeze tests that might have an incorrect implementation for some spectrometers.

TODO: https://bitbucket.org/pytest-dev/pytest-timeout/src/default/

This is related to #69

@ap--
Copy link
Owner Author

ap-- commented Aug 10, 2020

I tried timeouts and it's very complicated to get this right together with the hardware backed tests.
I'll consider patching the timeout value for reading data from the spectrometer specifically for tests in the future.

(The issue is, that some tests could stall the test suite due to having to wait for the usb timeout, which is long)

@ap-- ap-- closed this as completed Aug 10, 2020
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

1 participant