Skip to content

Commit 40bcdc7

Browse files
author
Ben Kandel
committed
add test for inplace=False
1 parent 6e56ff3 commit 40bcdc7

File tree

1 file changed

+16
-15
lines changed

1 file changed

+16
-15
lines changed

pandas/tests/indexes/test_multi.py

Lines changed: 16 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -207,21 +207,22 @@ def assert_matching(actual, expected):
207207
# illegal level changing should not change levels
208208
# GH 13754
209209
original_index = self.index.copy()
210-
with assertRaisesRegexp(ValueError, "^On"):
211-
self.index.set_levels(['c'], level=0, inplace=True)
212-
assert_matching(self.index.levels, original_index.levels)
213-
214-
with assertRaisesRegexp(ValueError, "^On"):
215-
self.index.set_labels([0, 1, 2, 3, 4, 5], level=0, inplace=True)
216-
assert_matching(self.index.labels, original_index.labels)
217-
218-
with assertRaisesRegexp(TypeError, "^Levels"):
219-
self.index.set_levels('c', level=0, inplace=True)
220-
assert_matching(self.index.levels, original_index.levels)
221-
222-
with assertRaisesRegexp(TypeError, "^Labels"):
223-
self.index.set_labels(1, level=0, inplace=True)
224-
assert_matching(self.index.labels, original_index.labels)
210+
for inplace in [True, False]:
211+
with assertRaisesRegexp(ValueError, "^On"):
212+
self.index.set_levels(['c'], level=0, inplace=inplace)
213+
assert_matching(self.index.levels, original_index.levels)
214+
215+
with assertRaisesRegexp(ValueError, "^On"):
216+
self.index.set_labels([0, 1, 2, 3, 4, 5], level=0, inplace=inplace)
217+
assert_matching(self.index.labels, original_index.labels)
218+
219+
with assertRaisesRegexp(TypeError, "^Levels"):
220+
self.index.set_levels('c', level=0, inplace=inplace)
221+
assert_matching(self.index.levels, original_index.levels)
222+
223+
with assertRaisesRegexp(TypeError, "^Labels"):
224+
self.index.set_labels(1, level=0, inplace=inplace)
225+
assert_matching(self.index.labels, original_index.labels)
225226

226227
def test_set_labels(self):
227228
# side note - you probably wouldn't want to use levels and labels

0 commit comments

Comments
 (0)