Skip to content

Commit 8249829

Browse files
author
Mike Phung
committed
TST GH#40498 Use frame_or_series to consolidate new fillna tests.
1 parent 20bea4e commit 8249829

File tree

2 files changed

+14
-51
lines changed

2 files changed

+14
-51
lines changed

pandas/tests/frame/methods/test_fillna.py

Lines changed: 0 additions & 42 deletions
Original file line numberDiff line numberDiff line change
@@ -153,48 +153,6 @@ def test_fillna_tzaware_different_column(self):
153153
)
154154
tm.assert_frame_equal(result, expected)
155155

156-
def test_null_value_replacement(self):
157-
# GH$40498
158-
df = DataFrame(
159-
{
160-
"A": [1, 2, None, 4],
161-
"B": [2, None, 6, 8],
162-
}
163-
)
164-
value = np.nan
165-
result = df.fillna(value)
166-
167-
expected = DataFrame(
168-
{
169-
"A": [1, 2, np.nan, 4],
170-
"B": [2, np.nan, 6, 8],
171-
}
172-
)
173-
174-
tm.assert_frame_equal(result, expected)
175-
176-
def test_other_missing_values_not_modified(
177-
self, unique_nulls_fixture, unique_nulls_fixture2
178-
):
179-
# GH#40498
180-
df = DataFrame(
181-
{
182-
"A": [1, unique_nulls_fixture, unique_nulls_fixture2, "four"],
183-
"B": [2, unique_nulls_fixture, unique_nulls_fixture2, "eight"],
184-
}
185-
)
186-
value = {"A": {1: 0}, "B": {2: 0}}
187-
result = df.fillna(value)
188-
189-
expected = DataFrame(
190-
{
191-
"A": [1, 0, unique_nulls_fixture2, "four"],
192-
"B": [2, unique_nulls_fixture, 0, "eight"],
193-
}
194-
)
195-
196-
tm.assert_frame_equal(result, expected)
197-
198156
def test_na_actions_categorical(self):
199157

200158
cat = Categorical([1, 2, 3, np.nan], categories=[1, 2, 3])

pandas/tests/series/methods/test_fillna.py

Lines changed: 14 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -623,29 +623,34 @@ def test_fillna_numeric_inplace(self):
623623
expected = x.fillna(value=0)
624624
tm.assert_series_equal(y, expected)
625625

626-
def test_fillna_null_value_replacement(self):
626+
def test_fillna_null_value_replacement(self, frame_or_series):
627627
# GH#40498
628-
ser = Series([None])
628+
ser = Series([None, None])
629+
obj = frame_or_series(ser)
629630

630631
value = np.nan
631-
result = ser.fillna(value)
632+
result = obj.fillna(value)
632633

633-
expected = Series([np.nan])
634+
expected = Series([np.nan, np.nan])
635+
expected = frame_or_series(expected)
634636

635-
tm.assert_series_equal(result, expected)
637+
tm.assert_equal(result, expected)
636638

637639
def test_fillna_other_missing_values_not_modified(
638-
self, unique_nulls_fixture, unique_nulls_fixture2
640+
self, unique_nulls_fixture, unique_nulls_fixture2, frame_or_series
639641
):
640642
# GH#40498
641643
ser = Series([1, unique_nulls_fixture, unique_nulls_fixture2, "four"])
644+
obj = frame_or_series(ser)
642645

643-
value = {2: 0}
644-
result = ser.fillna(value)
646+
value = {2: 0} if frame_or_series is Series else {0: {2: 0}}
647+
648+
result = obj.fillna(value)
645649

646650
expected = Series([1, unique_nulls_fixture, 0, "four"])
651+
expected = frame_or_series(expected)
647652

648-
tm.assert_series_equal(result, expected)
653+
tm.assert_equal(result, expected)
649654

650655
# ---------------------------------------------------------------
651656
# CategoricalDtype

0 commit comments

Comments
 (0)