Skip to content

Commit af964ca

Browse files
authored
TST verify return none inplace in tests/indexing (#35230)
1 parent 591da6b commit af964ca

File tree

5 files changed

+20
-10
lines changed

5 files changed

+20
-10
lines changed

pandas/tests/frame/test_constructors.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -1208,8 +1208,10 @@ def test_constructor_single_row(self):
12081208
data = [OrderedDict([["a", 1.5], ["b", 3], ["c", 4], ["d", 6]])]
12091209

12101210
result = DataFrame(data)
1211-
expected = DataFrame.from_dict(dict(zip([0], data)), orient="index")
1212-
tm.assert_frame_equal(result, expected.reindex(result.index))
1211+
expected = DataFrame.from_dict(dict(zip([0], data)), orient="index").reindex(
1212+
result.index
1213+
)
1214+
tm.assert_frame_equal(result, expected)
12131215

12141216
def test_constructor_ordered_dict_preserve_order(self):
12151217
# see gh-13304

pandas/tests/indexing/multiindex/test_indexing_slow.py

+6-3
Original file line numberDiff line numberDiff line change
@@ -34,12 +34,15 @@ def validate(mi, df, key):
3434
right = df[mask].copy()
3535

3636
if i + 1 != len(key): # partial key
37-
right.drop(cols[: i + 1], axis=1, inplace=True)
38-
right.set_index(cols[i + 1 : -1], inplace=True)
37+
return_value = right.drop(cols[: i + 1], axis=1, inplace=True)
38+
assert return_value is None
39+
return_value = right.set_index(cols[i + 1 : -1], inplace=True)
40+
assert return_value is None
3941
tm.assert_frame_equal(mi.loc[key[: i + 1]], right)
4042

4143
else: # full key
42-
right.set_index(cols[:-1], inplace=True)
44+
return_value = right.set_index(cols[:-1], inplace=True)
45+
assert return_value is None
4346
if len(right) == 1: # single hit
4447
right = Series(
4548
right["jolia"].values, name=right.index[0], index=["jolia"]

pandas/tests/indexing/multiindex/test_ix.py

+2-1
Original file line numberDiff line numberDiff line change
@@ -35,7 +35,8 @@ def test_loc_general(self):
3535
tm.assert_frame_equal(df.loc[key], df.iloc[2:])
3636

3737
# this is ok
38-
df.sort_index(inplace=True)
38+
return_value = df.sort_index(inplace=True)
39+
assert return_value is None
3940
res = df.loc[key]
4041

4142
# col has float dtype, result should be Float64Index

pandas/tests/indexing/multiindex/test_sorted.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -43,8 +43,10 @@ def test_frame_getitem_not_sorted2(self, key):
4343
df2 = df.set_index(["col1", "col2"])
4444
df2_original = df2.copy()
4545

46-
df2.index.set_levels(["b", "d", "a"], level="col1", inplace=True)
47-
df2.index.set_codes([0, 1, 0, 2], level="col1", inplace=True)
46+
return_value = df2.index.set_levels(["b", "d", "a"], level="col1", inplace=True)
47+
assert return_value is None
48+
return_value = df2.index.set_codes([0, 1, 0, 2], level="col1", inplace=True)
49+
assert return_value is None
4850
assert not df2.index.is_lexsorted()
4951
assert not df2.index.is_monotonic
5052

pandas/tests/indexing/multiindex/test_xs.py

+4-2
Original file line numberDiff line numberDiff line change
@@ -237,9 +237,11 @@ def test_series_getitem_multiindex_xs_by_label():
237237
[("a", "one"), ("a", "two"), ("b", "one"), ("b", "two")]
238238
)
239239
s = Series([1, 2, 3, 4], index=idx)
240-
s.index.set_names(["L1", "L2"], inplace=True)
240+
return_value = s.index.set_names(["L1", "L2"], inplace=True)
241+
assert return_value is None
241242
expected = Series([1, 3], index=["a", "b"])
242-
expected.index.set_names(["L1"], inplace=True)
243+
return_value = expected.index.set_names(["L1"], inplace=True)
244+
assert return_value is None
243245

244246
result = s.xs("one", level="L2")
245247
tm.assert_series_equal(result, expected)

0 commit comments

Comments
 (0)