Reduce deep copies in the test suite

Author: mroeschke

pandas/conftest.py

@@ -720,7 +720,7 @@ def index(request):
         - ...
     """
     # copy to avoid mutation, e.g. setting .name
-    return indices_dict[request.param].copy()
+    return indices_dict[request.param].copy(deep=False)
 
 
 @pytest.fixture(
@@ -733,7 +733,7 @@ def index_flat(request):
     index fixture, but excluding MultiIndex cases.
     """
     key = request.param
-    return indices_dict[key].copy()
+    return indices_dict[key].copy(deep=False)
 
 
 @pytest.fixture(
@@ -756,18 +756,15 @@ def index_with_missing(request):
 
     MultiIndex is excluded because isna() is not defined for MultiIndex.
     """
-
-    # GH 35538. Use deep copy to avoid illusive bug on np-dev
-    # GHA pipeline that writes into indices_dict despite copy
-    ind = indices_dict[request.param].copy(deep=True)
-    vals = ind.values.copy()
+    ind = indices_dict[request.param]
     if request.param in ["tuples", "mi-with-dt64tz-level", "multi"]:
         # For setting missing values in the top level of MultiIndex
         vals = ind.tolist()
         vals[0] = (None,) + vals[0][1:]
         vals[-1] = (None,) + vals[-1][1:]
         return MultiIndex.from_tuples(vals)
     else:
+        vals = ind.values.copy()
         vals[0] = None
         vals[-1] = None
         return type(ind)(vals)
@@ -848,7 +845,7 @@ def index_or_series_obj(request):
     Fixture for tests on indexes, series and series with a narrow dtype
     copy to avoid mutation, e.g. setting .name
     """
-    return _index_or_series_objs[request.param].copy(deep=True)
+    return _index_or_series_objs[request.param].copy(deep=False)
 
 
 _typ_objects_series = {
@@ -871,7 +868,7 @@ def index_or_series_memory_obj(request):
     series with empty objects type
     copy to avoid mutation, e.g. setting .name
     """
-    return _index_or_series_memory_objs[request.param].copy(deep=True)
+    return _index_or_series_memory_objs[request.param].copy(deep=False)
 
 
 # ----------------------------------------------------------------

pandas/tests/arrays/interval/test_interval.py

@@ -115,9 +115,7 @@ def test_shift_datetime(self):
 class TestSetitem:
     def test_set_na(self, left_right_dtypes):
         left, right = left_right_dtypes
-        left = left.copy(deep=True)
-        right = right.copy(deep=True)
-        result = IntervalArray.from_arrays(left, right)
+        result = IntervalArray.from_arrays(left, right, copy=True)
 
         if result.dtype.subtype.kind not in ["m", "M"]:
             msg = "'value' should be an interval type, got <.*NaTType'> instead."
@@ -168,8 +166,6 @@ def test_setitem_mismatched_closed(self):
 class TestReductions:
     def test_min_max_invalid_axis(self, left_right_dtypes):
         left, right = left_right_dtypes
-        left = left.copy(deep=True)
-        right = right.copy(deep=True)
         arr = IntervalArray.from_arrays(left, right)
 
         msg = "`axis` must be fewer than the number of dimensions"
@@ -188,8 +184,6 @@ def test_min_max_invalid_axis(self, left_right_dtypes):
     def test_min_max(self, left_right_dtypes, index_or_series_or_array):
         # GH#44746
         left, right = left_right_dtypes
-        left = left.copy(deep=True)
-        right = right.copy(deep=True)
         arr = IntervalArray.from_arrays(left, right)
 
         # The expected results below are only valid if monotonic

pandas/tests/copy_view/test_methods.py

@@ -560,7 +560,7 @@ def test_to_frame():
     ],
     ids=["shallow-copy", "reset_index", "rename", "select_dtypes"],
 )
-def test_chained_methods(request, method, idx):
+def test_chained_methods(method, idx):
     df = DataFrame({"a": [1, 2, 3], "b": [4, 5, 6], "c": [0.1, 0.2, 0.3]})
     df_orig = df.copy()