pr comments

Author: Tom McCormick

tests/io/test_pyarrow.py

@@ -2640,32 +2640,26 @@ def test_retry_strategy_not_found() -> None:
         io.new_input("s3://bucket/path/to/file")
 
 
-def test_parse_location_environment_defaults() -> None:
-    """Test that parse_location uses environment variables for defaults."""
+def test_parse_location_defaults() -> None:
+    """Test that parse_location uses defaults."""
 
     from pyiceberg.io.pyarrow import PyArrowFileIO
 
-    # Test with default environment (no env vars set)
     scheme, netloc, path = PyArrowFileIO.parse_location("/foo/bar")
     assert scheme == "file"
     assert netloc == ""
     assert path == "/foo/bar"
 
-    # Test with properties set
-    properties = {}
-    properties["DEFAULT_SCHEME"] = "scheme"
-    properties["DEFAULT_NETLOC"] = "netloc:8000"
-
-    scheme, netloc, path = PyArrowFileIO.parse_location("/foo/bar", properties=properties)
+    scheme, netloc, path = PyArrowFileIO.parse_location(
+        "/foo/bar", properties={"DEFAULT_SCHEME": "scheme", "DEFAULT_NETLOC": "netloc:8000"}
+    )
     assert scheme == "scheme"
     assert netloc == "netloc:8000"
     assert path == "netloc:8000/foo/bar"
 
-    # Set properties
-    properties["DEFAULT_SCHEME"] = "hdfs"
-    properties["DEFAULT_NETLOC"] = "netloc:8000"
-
-    scheme, netloc, path = PyArrowFileIO.parse_location("/foo/bar", properties=properties)
+    scheme, netloc, path = PyArrowFileIO.parse_location(
+        "/foo/bar", properties={"DEFAULT_SCHEME": "hdfs", "DEFAULT_NETLOC": "netloc:8000"}
+    )
     assert scheme == "hdfs"
     assert netloc == "netloc:8000"
     assert path == "/foo/bar"