Skip to content

Commit b4c1715

Browse files
committed
Use select and fetch_all_rows
1 parent 96fea87 commit b4c1715

File tree

1 file changed

+9
-12
lines changed

1 file changed

+9
-12
lines changed

tests/database/filters/test_analysis_filters.py

Lines changed: 9 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,6 @@
11
"""Module to test the analysis filters."""
22

3+
from sqlalchemy.future import select
34
from sqlalchemy.orm import Query
45

56
from genotype_api.database.filters.analysis_filter import (
@@ -17,10 +18,9 @@ async def test_filter_analyses_by_id(base_store: Store, test_analysis, helpers):
1718
# GIVEN an analysis
1819

1920
# WHEN filtering analyses by id
20-
query: Query = base_store._get_query(Analysis)
21+
query: Query = select(Analysis)
2122
filtered_query = filter_analyses_by_id(analysis_id=test_analysis.id, analyses=query)
22-
result = await base_store.session.execute(filtered_query)
23-
analyses: list[Analysis] = result.scalars().all()
23+
analyses: list[Analysis] = await base_store.fetch_all_rows(filtered_query)
2424

2525
# THEN assert the analysis is returned
2626
assert analyses
@@ -33,10 +33,9 @@ async def test_filter_analyses_by_type(
3333
# GIVEN an analysis
3434

3535
# WHEN filtering analyses by type
36-
query: Query = base_store._get_query(Analysis)
36+
query: Query = select(Analysis)
3737
filtered_query = filter_analyses_by_type(type=test_analysis.type, analyses=query)
38-
result = await base_store.session.execute(filtered_query)
39-
analyses: list[Analysis] = result.scalars().all()
38+
analyses: list[Analysis] = await base_store.fetch_all_rows(filtered_query)
4039

4140
# THEN assert the analysis is returned
4241
assert analyses
@@ -49,10 +48,9 @@ async def test_filter_analyses_by_plate_id(
4948
# GIVEN an analysis
5049

5150
# WHEN filtering analyses by plate id
52-
query: Query = base_store._get_query(Analysis)
51+
query: Query = select(Analysis)
5352
filtered_query = filter_analyses_by_plate_id(plate_id=test_analysis.plate_id, analyses=query)
54-
result = await base_store.session.execute(filtered_query)
55-
analyses: list[Analysis] = result.scalars().all()
53+
analyses: list[Analysis] = await base_store.fetch_all_rows(filtered_query)
5654

5755
# THEN assert the analysis is returned
5856
assert analyses
@@ -63,10 +61,9 @@ async def test_filter_analyses_by_sample_id(base_store: Store, test_analysis, he
6361
# GIVEN an analysis
6462

6563
# WHEN filtering analyses by sample id
66-
query: Query = base_store._get_query(Analysis)
64+
query: Query = select(Analysis)
6765
filtered_query = filter_analyses_by_sample_id(sample_id=test_analysis.sample_id, analyses=query)
68-
result = await base_store.session.execute(filtered_query)
69-
analyses: list[Analysis] = result.scalars().all()
66+
analyses: list[Analysis] = await base_store.fetch_all_rows(filtered_query)
7067

7168
# THEN assert the analysis is returned
7269
assert analyses

0 commit comments

Comments
 (0)