diff options
author | Frederick Muriuki Muriithi | 2022-03-08 08:07:38 +0300 |
---|---|---|
committer | Frederick Muriuki Muriithi | 2022-03-08 08:07:38 +0300 |
commit | e30f9a9b746f7dcdc8707e66cc65e580e6f83f47 (patch) | |
tree | 2edbbe2ff8957daf22ec60a562a633bdfa064723 /tests/performance | |
parent | eae345ed252c01e541d64c7e5b60b488d84268c6 (diff) | |
download | genenetwork3-e30f9a9b746f7dcdc8707e66cc65e580e6f83f47.tar.gz |
Fix tests, and issues caught by tests
Fix some issues caught by tests due to changes introducing the hand-off of the partial correlations computations to an external process Fix some issues due to the changes that introduce context managers for database connections Update some tests to take the above two changes into consideration
Diffstat (limited to 'tests/performance')
-rw-r--r-- | tests/performance/perf_query.py | 14 |
1 files changed, 6 insertions, 8 deletions
diff --git a/tests/performance/perf_query.py b/tests/performance/perf_query.py index c22dcf5..e534e9b 100644 --- a/tests/performance/perf_query.py +++ b/tests/performance/perf_query.py @@ -28,15 +28,13 @@ def timer(func): def query_executor(query: str, fetch_all: bool = True): """function to execute a query""" - conn, _ = database_connector() + with database_connector() as conn: + with conn.cursor() as cursor: + cursor.execute(query) - with conn: - cursor = conn.cursor() - cursor.execute(query) - - if fetch_all: - return cursor.fetchall() - return cursor.fetchone() + if fetch_all: + return cursor.fetchall() + return cursor.fetchone() def fetch_probeset_query(dataset_name: str): |