-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: Skip flaky, crashing test in Windows #45786
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
TST: Skip flaky, crashing test in Windows #45786
Conversation
pandas/tests/test_sorting.py
Outdated
@@ -424,6 +426,11 @@ def test_codes(self, verify, codes, exp_codes, na_sentinel): | |||
tm.assert_numpy_array_equal(result, expected) | |||
tm.assert_numpy_array_equal(result_codes, expected_codes) | |||
|
|||
@pytest.mark.skipif( | |||
is_platform_windows(), |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
should do the os.environ("CI") check?
individually the skips you've been adding over the last couple weeks all make sense, but added up they make me nervous that we're losing non-trivial coverage
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah seems reasonable (could make an easy skip for this too)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yup make sense (I'll follow up by making a pandas.compat.is_ci_environment
function)
lgtm merge on green (and backport) |
@meeseeksdev backport 1.4.x |
Something went wrong ... Please have a look at my logs. |
Co-authored-by: Matthew Roeschke <[email protected]>
* TST: Skip flaky, crashing test in Windows * Add CI check
* TST: Skip flaky, crashing test in Windows * Add CI check
Discovered while working on #45478
Logs: https://dev.azure.com/pandas-dev/e01c9671-576d-40f3-a1a1-7bef78604087/_apis/build/builds/73534/logs/54