Skip to content

Commit 4ffaf89

Browse files
refactor test to move fixture into the test where it's used
1 parent 913bb7b commit 4ffaf89

File tree

1 file changed

+27
-31
lines changed

1 file changed

+27
-31
lines changed

tests/unit/test_schemas.py

Lines changed: 27 additions & 31 deletions
Original file line numberDiff line numberDiff line change
@@ -41,36 +41,6 @@ def response_summary() -> ResponseSummary:
4141
return ResponseSummary(exists=False, numTotalResults=0)
4242

4343

44-
@pytest.fixture(scope="module")
45-
def responses_with_invalid_resultset_ids(valid_handover_id: str) -> list[ResponseField]:
46-
return [
47-
ResponseField(
48-
resultSets=[
49-
ResultSet(
50-
resultset_id=f"invalid_handover_id {Zygosity.HOMOZYGOUS}",
51-
resultsCount=0,
52-
)
53-
]
54-
),
55-
ResponseField(
56-
resultSets=[
57-
ResultSet(
58-
resultset_id=f"{valid_handover_id} invalid_zygosity",
59-
resultsCount=0,
60-
)
61-
]
62-
),
63-
ResponseField(
64-
resultSets=[
65-
ResultSet(
66-
resultset_id=f"{Zygosity.HOMOZYGOUS}-{valid_handover_id}", # incorrect order/formatting
67-
resultsCount=0,
68-
)
69-
]
70-
),
71-
]
72-
73-
7444
def test_valid_resultset_id(
7545
valid_handover_id: str,
7646
beacon_handovers: list[BeaconHandover],
@@ -100,9 +70,35 @@ def test_valid_resultset_id(
10070

10171
def test_invalid_resultset_ids(
10272
response_summary: ResponseSummary,
103-
responses_with_invalid_resultset_ids: list[ResponseField],
10473
beacon_handovers: list[BeaconHandover],
10574
):
75+
responses_with_invalid_resultset_ids: list[ResponseField] = [
76+
ResponseField(
77+
resultSets=[
78+
ResultSet(
79+
resultset_id=f"invalid_handover_id {Zygosity.HOMOZYGOUS}",
80+
resultsCount=0,
81+
)
82+
]
83+
),
84+
ResponseField(
85+
resultSets=[
86+
ResultSet(
87+
resultset_id=f"{valid_handover_id} invalid_zygosity",
88+
resultsCount=0,
89+
)
90+
]
91+
),
92+
ResponseField(
93+
resultSets=[
94+
ResultSet(
95+
resultset_id=f"{Zygosity.HOMOZYGOUS}-{valid_handover_id}", # incorrect order/formatting
96+
resultsCount=0,
97+
)
98+
]
99+
),
100+
]
101+
106102
for response in responses_with_invalid_resultset_ids:
107103
with pytest.raises(
108104
ValueError,

0 commit comments

Comments
 (0)