amol- commented on a change in pull request #11076:
URL: https://github.com/apache/arrow/pull/11076#discussion_r702058640
##########
File path: python/pyarrow/tests/test_convert_builtin.py
##########
@@ -173,15 +191,15 @@ def test_sequence_boolean(seq):
assert arr.to_pylist() == expected
-@parametrize_with_iterable_types
+@parametrize_with_sequence_types
Review comment:
Tests that require being able to contain duplicated values (for example
this tests has `expected = [np.bool_(True), None, np.bool_(False), None]` with
`None` twice) can't work against all iterables because some iterable types
don't allow duplicates (for example `set`)
--
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]
