Skip to content

Commit ed27192

Browse files
committed
Fix tests
1 parent f0c1309 commit ed27192

1 file changed

Lines changed: 8 additions & 6 deletions

File tree

tests/test_baseconverter.py

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -74,8 +74,12 @@ def test_nested_roundtrip_tuple(cls_and_vals):
7474

7575
@settings(suppress_health_check=[HealthCheck.filter_too_much, HealthCheck.too_slow])
7676
@given(
77-
simple_typed_classes(defaults="never", newtypes=False, allow_nan=False),
78-
simple_typed_classes(defaults="never", newtypes=False, allow_nan=False),
77+
simple_typed_classes(
78+
defaults="never", newtypes=False, allow_nan=False, min_attrs=1
79+
),
80+
simple_typed_classes(
81+
defaults="never", newtypes=False, allow_nan=False, min_attrs=1
82+
),
7983
unstructure_strats,
8084
)
8185
def test_union_field_roundtrip(cl_and_vals_a, cl_and_vals_b, strat):
@@ -85,11 +89,9 @@ def test_union_field_roundtrip(cl_and_vals_a, cl_and_vals_b, strat):
8589
converter = BaseConverter(unstruct_strat=strat)
8690
cl_a, vals_a, kwargs_a = cl_and_vals_a
8791
assume(strat is UnstructureStrategy.AS_DICT or not kwargs_a)
88-
cl_b, vals_b, _ = cl_and_vals_b
92+
cl_b, _, _ = cl_and_vals_b
8993
a_field_names = {a.name for a in fields(cl_a)}
9094
b_field_names = {a.name for a in fields(cl_b)}
91-
assume(a_field_names)
92-
assume(b_field_names)
9395

9496
common_names = a_field_names & b_field_names
9597
assume(len(a_field_names) > len(common_names))
@@ -104,7 +106,7 @@ class C:
104106
assert inst == converter.structure(converter.unstructure(inst), C)
105107
else:
106108
# Our disambiguation functions only support dictionaries for now.
107-
with pytest.raises(ValueError):
109+
with pytest.raises(StructureHandlerNotFoundError):
108110
converter.structure(converter.unstructure(inst), C)
109111

110112
def handler(obj, _):

0 commit comments

Comments
 (0)