Skip to content

Commit fca462e

Browse files
committed
Adjust where the nulls should be, generalize some tests
1 parent c338620 commit fca462e

File tree

12 files changed

+46
-37
lines changed

12 files changed

+46
-37
lines changed

pandas/tests/arrays/boolean/test_arithmetic.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
def data():
1212
"""Fixture returning boolean array with valid and missing values."""
1313
return pd.array(
14-
[True, False] + [np.nan] + [True, False] * 2 + [np.nan] + [True, False],
14+
[True, False] * 2 + [np.nan] + [True, False] + [np.nan] + [True, False],
1515
dtype="boolean",
1616
)
1717

pandas/tests/arrays/boolean/test_comparison.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -11,7 +11,7 @@
1111
def data():
1212
"""Fixture returning boolean array with valid and missing data"""
1313
return pd.array(
14-
[True, False] + [np.nan] + [True, False] * 2 + [np.nan] + [True, False],
14+
[True, False] * 2 + [np.nan] + [True, False] + [np.nan] + [True, False],
1515
dtype="boolean",
1616
)
1717

pandas/tests/arrays/boolean/test_reduction.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@
88
def data():
99
"""Fixture returning boolean array, with valid and missing values."""
1010
return pd.array(
11-
[True, False] + [np.nan] + [True, False] * 2 + [np.nan] + [True, False],
11+
[True, False] * 2 + [np.nan] + [True, False] + [np.nan] + [True, False],
1212
dtype="boolean",
1313
)
1414

pandas/tests/arrays/floating/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@ def dtype(request):
1717
def data(dtype):
1818
"""Fixture returning 'data' array according to parametrized float 'dtype'"""
1919
return pd.array(
20-
[0.1, 0.2] + [pd.NA] + [1.0, 1.1, 1.2, 1.3] + [pd.NA] + [9.9, 10.0],
20+
[0.1, 0.2, 0.3, 0.4] + [pd.NA] + [1.0, 1.1] + [pd.NA] + [9.9, 10.0],
2121
dtype=dtype,
2222
)
2323

pandas/tests/arrays/integer/conftest.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ def data(dtype):
3939
Used to test dtype conversion with and without missing values.
4040
"""
4141
return pd.array(
42-
[0, 1] + [pd.NA] + [10, 11, 12, 13] + [pd.NA] + [99, 100],
42+
[0, 1, 2, 3] + [pd.NA] + [10, 11] + [pd.NA] + [99, 100],
4343
dtype=dtype,
4444
)
4545

pandas/tests/extension/base/casting.py

Lines changed: 9 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -57,9 +57,17 @@ def test_astype_str(self, data):
5757
)
5858
def test_astype_string(self, data, nullable_string_dtype):
5959
# GH-33465, GH#45326 as of 2.0 we decode bytes instead of calling str(obj)
60+
def as_str(x):
61+
if isinstance(x, bytes):
62+
return x.decode()
63+
elif x is data.dtype.na_value:
64+
return x
65+
else:
66+
return str(x)
67+
6068
result = pd.Series(data[:5]).astype(nullable_string_dtype)
6169
expected = pd.Series(
62-
[str(x) if not isinstance(x, bytes) else x.decode() for x in data[:5]],
70+
[as_str(x) for x in data[:5]],
6371
dtype=nullable_string_dtype,
6472
)
6573
tm.assert_series_equal(result, expected)

pandas/tests/extension/base/printing.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ def test_array_repr(self, data, size):
1313
if size == "small":
1414
data = data[:5]
1515
else:
16-
data = type(data)._concat_same_type([data] * 5)
16+
data = type(data)._concat_same_type([data] * 20)
1717

1818
result = repr(data)
1919
assert type(data).__name__ in result

pandas/tests/extension/base/reshaping.py

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -303,7 +303,9 @@ def test_stack(self, data, columns, future_stack):
303303
)
304304
@pytest.mark.parametrize("obj", ["series", "frame"])
305305
def test_unstack(self, data, index, obj):
306-
data = data[: len(index)]
306+
final_length = min(len(index), len(data))
307+
index = index[:final_length]
308+
data = data[:final_length]
307309
if obj == "series":
308310
ser = pd.Series(data, index=index)
309311
else:

pandas/tests/extension/base/setitem.py

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -136,13 +136,13 @@ def test_setitem_iloc_scalar_mixed(self, data):
136136

137137
def test_setitem_iloc_scalar_single(self, data):
138138
df = pd.DataFrame({"B": data})
139-
df.iloc[10, 0] = data[1]
140-
assert df.loc[10, "B"] == data[1]
139+
df.iloc[9, 0] = data[1]
140+
assert df.loc[9, "B"] == data[1]
141141

142142
def test_setitem_iloc_scalar_multiple_homogoneous(self, data):
143143
df = pd.DataFrame({"A": data, "B": data})
144-
df.iloc[10, 1] = data[1]
145-
assert df.loc[10, "B"] == data[1]
144+
df.iloc[9, 1] = data[1]
145+
assert df.loc[9, "B"] == data[1]
146146

147147
@pytest.mark.parametrize(
148148
"mask",
@@ -281,9 +281,9 @@ def test_setitem_mask_broadcast(self, data, setter):
281281
else: # __setitem__
282282
target = ser
283283

284-
target[mask] = data[10]
285-
assert ser[0] == data[10]
286-
assert ser[1] == data[10]
284+
target[mask] = data[9]
285+
assert ser[0] == data[9]
286+
assert ser[1] == data[9]
287287

288288
def test_setitem_expand_columns(self, data):
289289
df = pd.DataFrame({"A": data})

pandas/tests/extension/json/test_json.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -190,9 +190,8 @@ def test_ffill_limit_area(
190190
)
191191

192192
def test_value_counts(self, all_data, dropna, request):
193-
if len(all_data) == 100 or dropna:
194-
mark = pytest.mark.xfail(reason="unhashable")
195-
request.applymarker(mark)
193+
if len(all_data) == 10 or dropna:
194+
request.applymarker(unhashable)
196195
super().test_value_counts(all_data, dropna)
197196

198197
@unhashable

0 commit comments

Comments
 (0)