Skip to content

Commit fade31e

Browse files
TomAugspurgertm9k1
authored andcommitted
TST: Fix test assertion (pandas-dev#23357)
1 parent 3e7021b commit fade31e

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

pandas/tests/extension/base/setitem.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -27,18 +27,20 @@ def test_setitem_sequence(self, data, box_in_series):
2727
@pytest.mark.parametrize('as_array', [True, False])
2828
def test_setitem_sequence_mismatched_length_raises(self, data, as_array):
2929
ser = pd.Series(data)
30+
original = ser.copy()
3031
value = [data[0]]
3132
if as_array:
3233
value = data._from_sequence(value)
3334

3435
xpr = 'cannot set using a {} indexer with a different length'
3536
with tm.assert_raises_regex(ValueError, xpr.format('list-like')):
3637
ser[[0, 1]] = value
37-
assert ser._values[[0, 1]] == value
38+
# Ensure no modifications made before the exception
39+
self.assert_series_equal(ser, original)
3840

3941
with tm.assert_raises_regex(ValueError, xpr.format('slice')):
4042
ser[slice(3)] = value
41-
assert ser._values[slice(3)] == value
43+
self.assert_series_equal(ser, original)
4244

4345
def test_setitem_empty_indxer(self, data, box_in_series):
4446
if box_in_series:

0 commit comments

Comments
 (0)