TEST: ensure Masked tests run also without matplotlib installed#16843
TEST: ensure Masked tests run also without matplotlib installed#16843pllim merged 1 commit intoastropy:mainfrom
Conversation
|
Thank you for your contribution to Astropy! 🌌 This checklist is meant to remind the package maintainers who will review this pull request of some common things to look for.
|
|
No worries, it was not at all obvious this would cause the tests to be skipped, rather than given an error! |
|
This appears to crash on daily cron (specifically on pyinstaller) |
| pass | ||
|
|
||
|
|
||
| # For grep'ing: test requires MATPLOTLIB_LT_3_8 |
There was a problem hiding this comment.
didn't you mean "test requires not MATPLOTLIB_LT_3_8" ?
What? How is this possible. Does it evaluate it even if |
|
Yes I think this condition is evaluated: matplotlib is indeed installed on this job, but somehow I never heard of pyinstaller before so I don't know off hand how this is happening but it's oddly specific to that env |
|
It's weird that the previous version in which |
|
Argh, sorry to cause so much trouble. |
|
Hopefully #16854 is finally the correct fix? 🤞 |
This pull request is a subtle bug introduced in #16820, where an import of
MATPLOTLIB_LT_3_8in a masked test file causes the tests to be skipped altogether if matplotlib is not installed.In review, should make sure that the masked tests are all run also in the minimal dependencies case.