-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: Fix warnings & skip -> xfail #46370
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
Changes from all commits
18dc4e3
ca4293c
6997b49
29c3dcf
a5f7706
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -26,7 +26,7 @@ | |
|
||
def split_array(arr): | ||
if arr.dtype.storage != "pyarrow": | ||
pytest.skip("chunked array n/a") | ||
pytest.skip("only applicable for pyarrow chunked array n/a") | ||
|
||
def _split_array(arr): | ||
import pyarrow as pa | ||
|
@@ -156,13 +156,9 @@ def test_reduce_series_numeric(self, data, all_numeric_reductions, skipna): | |
|
||
|
||
class TestMethods(base.BaseMethodsTests): | ||
@pytest.mark.skip(reason="returns nullable") | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. yah really ought to pull the trigger on #44692 and just get internal consistency here There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Added that issue ref to this xfail |
||
def test_value_counts(self, all_data, dropna): | ||
return super().test_value_counts(all_data, dropna) | ||
|
||
@pytest.mark.skip(reason="returns nullable") | ||
@pytest.mark.xfail(reason="returns nullable: GH 44692") | ||
def test_value_counts_with_normalize(self, data): | ||
pass | ||
super().test_value_counts_with_normalize(data) | ||
|
||
|
||
class TestCasting(base.BaseCastingTests): | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -56,6 +56,7 @@ markers = [ | |
"arm_slow: mark a test as slow for arm64 architecture", | ||
"arraymanager: mark a test to run with ArrayManager enabled", | ||
] | ||
asyncio_mode = "strict" | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. where do we use asyncio? There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. We have some tab completion tests that use this
|
||
|
||
[tool.mypy] | ||
# Import discovery | ||
|
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.
this is a never-expected-to-change thing, i.e. shouldnt be an xfail?
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.
I assume so since I think that would require a pyarrow API to get memory usage (didn't see one quickly skimming)