[ArrayManager] TST: run (+fix/skip) pandas/tests/series/indexing tests

.github/workflows/ci.yml

@@ -165,9 +165,7 @@ jobs:
         pytest pandas/tests/groupby/
         pytest pandas/tests/resample/
         pytest pandas/tests/reshape/merge
-
-        pytest pandas/tests/series/methods
-        pytest pandas/tests/series/test_*
+        pytest pandas/tests/series/
 
         # indexing subset (temporary since other tests don't pass yet)
         pytest pandas/tests/frame/indexing/test_indexing.py::TestDataFrameIndexing::test_setitem_boolean

pandas/core/internals/array_manager.py

@@ -1153,7 +1153,13 @@ def __init__(
     def _verify_integrity(self) -> None:
         (n_rows,) = self.shape
         assert len(self.arrays) == 1
-        assert len(self.arrays[0]) == n_rows
+        arr = self.arrays[0]
+        assert len(arr) == n_rows
+        if not arr.ndim == 1:
+            raise ValueError(
+                "Passed array should be 1-dimensional, got array with "
+                f"{arr.ndim} dimensions instead."
+            )
 
     @staticmethod
     def _normalize_axis(axis):

pandas/core/internals/blocks.py

@@ -1745,7 +1745,6 @@ def shift(self, periods: int, axis: int = 0, fill_value: Any = None) -> List[Blo
         return [self.make_block_same_class(new_values)]
 
     def where(self, other, cond, errors="raise", axis: int = 0) -> List[Block]:
-
         cond = extract_bool_array(cond)
         assert not isinstance(other, (ABCIndex, ABCSeries, ABCDataFrame))
 

pandas/tests/series/indexing/test_getitem.py

@@ -234,10 +234,12 @@ def test_getitem_partial_str_slice_high_reso_with_timedeltaindex(self):
         result = ser["1 days, 10:11:12.001001"]
         assert result == ser.iloc[1001]
 
-    def test_getitem_slice_2d(self, datetime_series):
+    def test_getitem_slice_2d(self, datetime_series, using_array_manager):
         # GH#30588 multi-dimensional indexing deprecated
 
-        with tm.assert_produces_warning(FutureWarning):
+        with tm.assert_produces_warning(
+            FutureWarning, check_stacklevel=not using_array_manager
+        ):
             # GH#30867 Don't want to support this long-term, but
             # for now ensure that the warning from Index
             # doesn't comes through via Series.__getitem__.
@@ -518,9 +520,11 @@ def test_getitem_generator(string_series):
         Series(date_range("2012-01-01", periods=2, tz="CET")),
     ],
 )
-def test_getitem_ndim_deprecated(series):
+def test_getitem_ndim_deprecated(series, using_array_manager):
     with tm.assert_produces_warning(
-        FutureWarning, match="Support for multi-dimensional indexing"
+        FutureWarning,
+        match="Support for multi-dimensional indexing",
+        check_stacklevel=not using_array_manager,
     ):
         result = series[:, None]
 

pandas/tests/series/indexing/test_setitem.py

@@ -299,15 +299,16 @@ def test_setitem_invalidates_datetime_index_freq(self):
         assert dti[1] == ts
         assert dti.freq == "D"
 
-    def test_dt64tz_setitem_does_not_mutate_dti(self):
+    def test_dt64tz_setitem_does_not_mutate_dti(self, using_array_manager):
         # GH#21907, GH#24096
         dti = date_range("2016-01-01", periods=10, tz="US/Pacific")
         ts = dti[0]
         ser = Series(dti)
         assert ser._values is not dti
         assert ser._values._data.base is not dti._data._data.base
-        assert ser._mgr.blocks[0].values is not dti
-        assert ser._mgr.blocks[0].values._data.base is not dti._data._data.base
+        if not using_array_manager:
+            assert ser._mgr.blocks[0].values is not dti
+            assert ser._mgr.blocks[0].values._data.base is not dti._data._data.base
 
         ser[::3] = NaT
         assert ser[0] is NaT

pandas/tests/series/indexing/test_where.py

@@ -1,6 +1,8 @@
 import numpy as np
 import pytest
 
+import pandas.util._test_decorators as td
+
 from pandas.core.dtypes.common import is_integer
 
 import pandas as pd
@@ -471,6 +473,9 @@ def test_where_categorical(klass):
     tm.assert_equal(exp, res)
 
 
+# TODO(ArrayManager) DataFrame.values not yet correctly returning datetime array
+# for categorical with datetime categories
+@td.skip_array_manager_not_yet_implemented
 def test_where_datetimelike_categorical(tz_naive_fixture):
     # GH#37682
     tz = tz_naive_fixture