Skip to content

Commit 225d449

Browse files
committed
.sql files instead of python sql
1 parent 8fab355 commit 225d449

File tree

7 files changed

+206
-40
lines changed

7 files changed

+206
-40
lines changed

expected_results.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
{
2-
"query1": {
2+
"active_sites_positive": {
33
"expected": [
44
{"study": "1368-0015", "site": "DEU1", "active_sites": 3},
55
{"study": "1368-0004", "site": "DEU2", "active_sites": 2},
66
{"study": "1368-0017", "site": "DEU4", "active_sites": 2},
77
{"study": "1368-0018", "site": "DEU5", "active_sites": 3}
88
]
99
},
10-
"query2": {
10+
"randomized_patients_positive": {
1111
"expected": [
1212
{"study": "1368-0015", "site": "DEU1", "randomized_patients": 40, "trial_status": "recruitment"},
1313
{"study": "1368-0004", "site": "DEU2", "randomized_patients": 35, "trial_status": "ongoing"},
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
SELECT study, site, active_sites FROM clinical_trials WHERE active_sites > 0;
Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
SELECT study, site, randomized_patients, trial_status FROM clinical_trials WHERE randomized_patients >= 0;

sql_queries/sql_queries.py

Lines changed: 0 additions & 5 deletions
This file was deleted.

tests/pytest_html_report.html

Lines changed: 189 additions & 25 deletions
Large diffs are not rendered by default.

tests/test_sql_logic.py

Lines changed: 13 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -1,12 +1,10 @@
1-
# tests/test_queries.py
1+
# tests/test_sql_logic.py
22
import sys
33
import os
44
sys.path.insert(0, os.path.abspath(os.path.join(os.path.dirname(__file__), '..')))
5-
65
import pytest
76
import json
8-
from sql_queries.sql_queries import query1, query2
9-
from validation import validate_entry, common_rules
7+
from validation_rules import validate_entry, common_rules
108

119
def fetch_results_as_dict(results):
1210
return [dict(row) for row in results]
@@ -18,9 +16,14 @@ def load_expected_results():
1816

1917
expected_results = load_expected_results()
2018

19+
def load_sql_query(file_name):
20+
sql_path = os.path.join(os.path.dirname(__file__), '..', 'sql_queries', file_name)
21+
with open(sql_path, 'r') as file:
22+
return file.read()
23+
2124
def test_query1(db_connection):
22-
query_name = "query1"
23-
query_sql = query1()
25+
query_name = "active_sites_positive"
26+
query_sql = load_sql_query("active_sites_positive.sql")
2427
results = db_connection.execute(query_sql)
2528
results_dict = fetch_results_as_dict(results)
2629

@@ -32,8 +35,8 @@ def test_query1(db_connection):
3235
assert filtered_results == expected, f"Error in {query_name}: {filtered_results} != {expected}"
3336

3437
def test_query2(db_connection):
35-
query_name = "query2"
36-
query_sql = query2()
38+
query_name = "randomized_patients_positive"
39+
query_sql = load_sql_query("randomized_patients_positive.sql")
3740
results = db_connection.execute(query_sql)
3841
results_dict = fetch_results_as_dict(results)
3942

@@ -43,3 +46,5 @@ def test_query2(db_connection):
4346
# Comparar con resultados esperados
4447
expected = expected_results[query_name]["expected"]
4548
assert filtered_results == expected, f"Error in {query_name}: {filtered_results} != {expected}"
49+
50+
File renamed without changes.

0 commit comments

Comments
 (0)