jorisvandenbossche commented on a change in pull request #12522:
URL: https://github.com/apache/arrow/pull/12522#discussion_r819369771
##########
File path: python/pyarrow/tests/strategies.py
##########
@@ -224,6 +231,9 @@ def _pymap(draw, key_type, value_type, size, nullable=True):
@st.composite
def arrays(draw, type, size=None, nullable=True):
+ pytest.importorskip("pytz")
Review comment:
> 5 tests are still being skipped and I wasn't able to check the code
change from the `test_arrays`
For seeing the reason something is skipped, you can pass `-r S` to pytest:
```
$ pytest python/pyarrow/tests/test_array.py --enable-hypothesis -r S
..
====================================== short test summary info
==============================
SKIPPED [1] python/pyarrow/tests/conftest.py:221: nopandas NOT enabled
SKIPPED [1] python/pyarrow/tests/strategies.py:236: could not import
'ZoneInfo': No module named 'ZoneInfo'
SKIPPED [2] python/pyarrow/tests/conftest.py:221: large_memory NOT enabled
====================================== 213 passed, 4 skipped in 0.87s
=========================
```
So one test is skipped because of pandas being present, and two tests are
skipped by default because they are "large memory" tests. One because
`ZoneInfo` could not be imported, but so that was actually because of a bug
here (the module name is lowercase, see the inline comment below).
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
