diff options
| author | Munyoki Kilyungi | 2023-09-05 14:39:38 +0300 |
|---|---|---|
| committer | BonfaceKilz | 2023-09-05 14:57:51 +0300 |
| commit | 259aec828fa3908eed62db9530db7d9efd72f136 (patch) | |
| tree | 4308d35db61b6044b0d5da7b9c969f6c886cbf56 | |
| parent | b1916a0c0f2656847f07a174f8e8f8a863f4d282 (diff) | |
| download | genenetwork3-259aec828fa3908eed62db9530db7d9efd72f136.tar.gz | |
Remove un-necessary string concatenation
Signed-off-by: Munyoki Kilyungi <me@bonfacemunyoki.com>
| -rw-r--r-- | gn3/db/case_attributes.py | 2 | ||||
| -rw-r--r-- | tests/integration/test_gemma.py | 2 | ||||
| -rw-r--r-- | tests/unit/db/test_case_attributes.py | 6 |
3 files changed, 5 insertions, 5 deletions
diff --git a/gn3/db/case_attributes.py b/gn3/db/case_attributes.py index 1d82f00..bb15248 100644 --- a/gn3/db/case_attributes.py +++ b/gn3/db/case_attributes.py @@ -90,7 +90,7 @@ def approve_case_attribute(conn: Any, case_attr_audit_id: int) -> int: elif diff_data.get("Deletion"): data = diff_data.get("Deletion") cursor.execute( - "DELETE FROM CaseAttribute " "WHERE Id = %s", + "DELETE FROM CaseAttribute WHERE Id = %s", (data.get("id"),), ) # Modification diff --git a/tests/integration/test_gemma.py b/tests/integration/test_gemma.py index d8c07f4..79ddaca 100644 --- a/tests/integration/test_gemma.py +++ b/tests/integration/test_gemma.py @@ -382,7 +382,7 @@ class GemmaAPITest(unittest.TestCase): "snps": "snpfile.txt", } mock_hash.return_value = "hash" - response = self.app.post(("/api/gemma/k-gwa-compute/" "my-token")) + response = self.app.post(("/api/gemma/k-gwa-compute/my-token")) mock_hash.assert_called_with([ ('/tmp/cache/QmQPeNsJPyVWPFDVHb77w8G42Fvo15z4bG2X8D2GhfbSXc/' 'genotype.txt'), '/tmp/my-token/phenofile.txt', diff --git a/tests/unit/db/test_case_attributes.py b/tests/unit/db/test_case_attributes.py index 175249b..97a0703 100644 --- a/tests/unit/db/test_case_attributes.py +++ b/tests/unit/db/test_case_attributes.py @@ -140,7 +140,7 @@ def test_approve_deleting_case_attribute(mocker: MockFixture) -> None: "WHERE id = %s", (3,), ), - mocker.call("DELETE FROM CaseAttribute " "WHERE Id = %s", ("12",)), + mocker.call("DELETE FROM CaseAttribute WHERE Id = %s", ("12",)), mocker.call( "UPDATE caseattributes_audit SET " "status = 'approved' WHERE id = %s", @@ -183,14 +183,14 @@ def test_approve_modifying_case_attribute(mocker: MockFixture) -> None: (3,), ), mocker.call( - "UPDATE CaseAttribute SET " "Description = %s WHERE Id = %s", + "UPDATE CaseAttribute SET Description = %s WHERE Id = %s", ( "Test", "12", ), ), mocker.call( - "UPDATE CaseAttribute SET " "Name = %s WHERE Id = %s", + "UPDATE CaseAttribute SET Name = %s WHERE Id = %s", ( "Height (A)", "12", |