Use temp_file directly instead of converting to str

Author: kianelbo

pandas/tests/io/parser/common/test_chunksize.py

@@ -302,20 +302,19 @@ def test_read_csv_memory_growth_chunksize(temp_file, all_parsers):
     # as we iteratively process all chunks.
     parser = all_parsers
 
-    path = str(temp_file)
-    with open(path, "w", encoding="utf-8") as f:
+    with open(temp_file, "w", encoding="utf-8") as f:
         for i in range(1000):
             f.write(str(i) + "\n")
 
     if parser.engine == "pyarrow":
         msg = "The 'chunksize' option is not supported with the 'pyarrow' engine"
         with pytest.raises(ValueError, match=msg):
-            with parser.read_csv(path, chunksize=20) as result:
+            with parser.read_csv(temp_file, chunksize=20) as result:
                 for _ in result:
                     pass
         return
 
-    with parser.read_csv(path, chunksize=20) as result:
+    with parser.read_csv(temp_file, chunksize=20) as result:
         for _ in result:
             pass
 

pandas/tests/io/parser/common/test_iterator.py

@@ -146,11 +146,10 @@ def test_iteration_open_handle(temp_file, all_parsers):
     parser = all_parsers
     kwargs = {"header": None}
 
-    path = str(temp_file)
-    with open(path, "w", encoding="utf-8") as f:
+    with open(temp_file, "w", encoding="utf-8") as f:
         f.write("AAA\nBBB\nCCC\nDDD\nEEE\nFFF\nGGG")
 
-    with open(path, encoding="utf-8") as f:
+    with open(temp_file, encoding="utf-8") as f:
         for line in f:
             if "CCC" in line:
                 break

pandas/tests/io/parser/test_index_col.py

@@ -209,9 +209,8 @@ def test_no_multi_index_level_names_empty(temp_file, all_parsers):
         index=midx,
         columns=["x", "y", "z"],
     )
-    path = str(temp_file)
-    expected.to_csv(path)
-    result = parser.read_csv(path, index_col=[0, 1, 2])
+    expected.to_csv(temp_file)
+    result = parser.read_csv(temp_file, index_col=[0, 1, 2])
     tm.assert_frame_equal(result, expected)
 
 
@@ -252,11 +251,10 @@ def test_index_col_large_csv(temp_file, all_parsers, monkeypatch):
         }
     )
 
-    path = str(temp_file)
-    df.to_csv(path, index=False)
+    df.to_csv(temp_file, index=False)
     with monkeypatch.context() as m:
         m.setattr("pandas.core.algorithms._MINIMUM_COMP_ARR_LEN", ARR_LEN)
-        result = parser.read_csv(path, index_col=[0])
+        result = parser.read_csv(temp_file, index_col=[0])
 
     tm.assert_frame_equal(result, df.set_index("a"))
 

pandas/tests/io/parser/test_python_parser_only.py

@@ -173,11 +173,10 @@ def test_decompression_regex_sep(
     module = pytest.importorskip(compression)
     klass = getattr(module, klass)
 
-    path = str(temp_file)
-    with klass(path, mode="wb") as tmp:
+    with klass(temp_file, mode="wb") as tmp:
         tmp.write(data)
 
-    result = parser.read_csv(path, sep="::", compression=compression)
+    result = parser.read_csv(temp_file, sep="::", compression=compression)
     tm.assert_frame_equal(result, expected)
 
 

pandas/tests/io/pytables/test_round_trip.py

@@ -28,11 +28,9 @@
 
 
 def test_conv_read_write(temp_file):
-    path = str(temp_file)
-
     def roundtrip(key, obj, **kwargs):
-        obj.to_hdf(path, key=key, **kwargs)
-        return read_hdf(path, key)
+        obj.to_hdf(temp_file, key=key, **kwargs)
+        return read_hdf(temp_file, key)
 
     o = Series(
         np.arange(10, dtype=np.float64), index=date_range("2020-01-01", periods=10)
@@ -51,8 +49,8 @@ def roundtrip(key, obj, **kwargs):
 
     # table
     df = DataFrame({"A": range(5), "B": range(5)})
-    df.to_hdf(path, key="table", append=True)
-    result = read_hdf(path, "table", where=["index>2"])
+    df.to_hdf(temp_file, key="table", append=True)
+    result = read_hdf(temp_file, "table", where=["index>2"])
     tm.assert_frame_equal(df[df.index > 2], result)
 
 

pandas/tests/io/test_sql.py

@@ -750,8 +750,7 @@ def postgresql_psycopg2_conn_types(postgresql_psycopg2_engine_types):
 @pytest.fixture
 def sqlite_str(temp_file):
     pytest.importorskip("sqlalchemy")
-    name = str(temp_file)
-    return f"sqlite:///{name}"
+    return f"sqlite:///{temp_file}"
 
 
 @pytest.fixture
@@ -822,8 +821,7 @@ def sqlite_adbc_conn(temp_file):
     pytest.importorskip("adbc_driver_sqlite")
     from adbc_driver_sqlite import dbapi
 
-    name = str(temp_file)
-    uri = f"file:{name}"
+    uri = f"file:{temp_file}"
     with dbapi.connect(uri) as conn:
         yield conn
         for view in get_all_views(conn):
@@ -2585,11 +2583,10 @@ def test_sql_open_close(temp_file, test_frame3):
     # Test if the IO in the database still work if the connection closed
     # between the writing and reading (as in many real situations).
 
-    name = str(temp_file)
-    with contextlib.closing(sqlite3.connect(name)) as conn:
+    with contextlib.closing(sqlite3.connect(temp_file)) as conn:
         assert sql.to_sql(test_frame3, "test_frame3_legacy", conn, index=False) == 4
 
-    with contextlib.closing(sqlite3.connect(name)) as conn:
+    with contextlib.closing(sqlite3.connect(temp_file)) as conn:
         result = sql.read_sql_query("SELECT * FROM test_frame3_legacy;", conn)
 
     tm.assert_frame_equal(test_frame3, result)