Fokko commented on code in PR #6521:
URL: https://github.com/apache/iceberg/pull/6521#discussion_r1063500822


##########
python/tests/io/test_pyarrow.py:
##########
@@ -195,62 +195,6 @@ def test_raise_on_opening_a_local_file_not_found() -> None:
         assert "[Errno 2] Failed to open local file" in str(exc_info.value)
 
 
-def test_raise_on_opening_a_local_file_no_permission() -> None:

Review Comment:
   I removed these tests since they don't work inside of a plain docker 
container because, in a docker container, everything runs under root unless 
specified otherwise. I don't feel like creating a special Docker image for 
this. These tests just assert the `PermissionError` that comes from Python 
itself.



-- 
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]


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to