paleolimbot commented on code in PR #444:
URL: https://github.com/apache/arrow-nanoarrow/pull/444#discussion_r1584911692
##########
python/tests/test_c_array.py:
##########
@@ -81,7 +69,7 @@ def _export_to_c(self, *args):
# Make sure that this heuristic won't result in trying to import
# something else that has an _export_to_c method
- with pytest.raises(TypeError, match="Can't convert object of type
DataType"):
+ with pytest.raises(TypeError, match="Can't resolve ArrayBuilder"):
Review Comment:
Good call! I made sure this wrapped at the top level:
```
import nanoarrow as na
na.Array(None)
#> TypeError: An error occurred whilst converting NoneType to
nanoarrow.c_array_stream or nanoarrow.c_array:
#> Can't convert object of type NoneType to nanoarrow.c_array:
#> Can't resolve ArrayBuilder for object of type NoneType
```
--
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]
