BUG: fix .loc.__setitem__ not raising when using too many indexers

pandas/core/indexing.py

@@ -911,7 +911,7 @@ def _getitem_nested_tuple(self, tup: tuple):
                 #  we are only getting non-hashable tuples, in particular ones
                 #  that themselves contain a slice entry
                 # See test_loc_series_getitem_too_many_dimensions
-                raise ValueError("Too many indices")
+                raise IndexingError("Too many indexers")
 
             # this is a series with a multi-index specified a tuple of
             # selectors
@@ -1231,6 +1231,14 @@ def _convert_to_indexer(self, key, axis: int):
         is_int_index = labels.is_integer()
         is_int_positional = is_integer(key) and not is_int_index
 
+        if (
+            isinstance(key, tuple)
+            and not isinstance(labels, MultiIndex)
+            and self.ndim < 2
+            and len(key) > 1
+        ):
+            raise IndexingError("Too many indexers")
+
         if is_scalar(key) or (isinstance(labels, MultiIndex) and is_hashable(key)):
             # Otherwise get_loc will raise InvalidIndexError
 
@@ -1262,7 +1270,7 @@ def _convert_to_indexer(self, key, axis: int):
         if is_nested_tuple(key, labels):
             if self.ndim == 1 and any(isinstance(k, tuple) for k in key):
                 # GH#35349 Raise if tuple in tuple for series
-                raise ValueError("Too many indices")
+                raise IndexingError("Too many indexers")
             return labels.get_locs(key)
 
         elif is_list_like_indexer(key):

pandas/tests/indexing/test_iloc.py

@@ -1189,6 +1189,18 @@ def test_iloc_getitem_setitem_fancy_exceptions(self, float_frame):
             # GH#32257 we let numpy do validation, get their exception
             float_frame.iloc[:, :, :] = 1
 
+    @pytest.mark.parametrize(
+        "ser, keys",
+        [(Series([10]), (0, 0)), (Series([1, 2, 3], index=list("abc")), (0, 1))],
+    )
+    def test_iloc_setitem_indexer_length(self, ser, keys):
+        # GH#13831
+        with pytest.raises(IndexError, match="too many indices for array"):
+            ser.iloc[keys] = 1000
+
+        with pytest.raises(IndexingError, match="Too many indexers"):
+            ser.iloc[keys]
+
     # TODO(ArrayManager) "split" path doesn't properly implement DataFrame indexer
     @td.skip_array_manager_not_yet_implemented
     def test_iloc_frame_indexer(self):

pandas/tests/indexing/test_loc.py

@@ -2685,6 +2685,18 @@ def test_loc_with_period_index_indexer():
     tm.assert_frame_equal(df, df.loc[list(idx)])
 
 
+def test_loc_setitem_multiindex_timestamp():
+    # GH#13831
+    vals = np.random.randn(8, 6)
+    idx = date_range("1/1/2000", periods=8)
+    cols = ["A", "B", "C", "D", "E", "F"]
+    df_mt = DataFrame(vals, index=idx, columns=cols)
+    df_mt.loc[df_mt.index[1], ("A", "B")] = np.nan
+    vals[1][0:2] = np.nan
+    res = DataFrame(vals, index=idx, columns=cols)
+    tm.assert_frame_equal(res, df_mt)
+
+
 def test_loc_getitem_multiindex_tuple_level():
     # GH#27591
     lev1 = ["a", "b", "c"]
@@ -2715,6 +2727,20 @@ def test_loc_getitem_multiindex_tuple_level():
     assert result2 == 6
 
 
+@pytest.mark.parametrize(
+    "ser, keys",
+    [(Series([10]), (0, 0)), (Series([1, 2, 3], index=list("abc")), (0, 1))],
+)
+def test_loc_setitem_indexer_length(ser, keys):
+    # GH#13831
+    msg = "Too many indexers"
+    with pytest.raises(IndexingError, match=msg):
+        ser.loc[keys] = 1000
+
+    with pytest.raises(IndexingError, match=msg):
+        ser.loc[keys]
+
+
 class TestLocSeries:
     @pytest.mark.parametrize("val,expected", [(2 ** 63 - 1, 3), (2 ** 63, 4)])
     def test_loc_uint64(self, val, expected):
@@ -2889,11 +2915,11 @@ def test_loc_series_getitem_too_many_dimensions(self, indexer):
             index=MultiIndex.from_tuples([("A", "0"), ("A", "1"), ("B", "0")]),
             data=[21, 22, 23],
         )
-        msg = "Too many indices"
-        with pytest.raises(ValueError, match=msg):
+        msg = "Too many indexers"
+        with pytest.raises(IndexingError, match=msg):
             ser.loc[indexer, :]
 
-        with pytest.raises(ValueError, match=msg):
+        with pytest.raises(IndexingError, match=msg):
             ser.loc[indexer, :] = 1
 
     def test_loc_setitem(self, string_series):