diff options
author | Muriithi Frederick Muriuki | 2021-08-04 09:43:20 +0300 |
---|---|---|
committer | Muriithi Frederick Muriuki | 2021-08-04 09:43:20 +0300 |
commit | 7ab35adb5afb242092e25763474c747e267ded81 (patch) | |
tree | e85fd8ed4e8ad78d33b142e231689abe4ccdb6ed | |
parent | 630488edfd75c428dc18e09d9336c6f16531130d (diff) | |
download | genenetwork3-7ab35adb5afb242092e25763474c747e267ded81.tar.gz |
Update tests
Issue:
https://github.com/genenetwork/gn-gemtext-threads/blob/main/topics/gn1-migration-to-gn2/clustering.gmi
* Update the tests to deal with changes in the code.
-rw-r--r-- | tests/unit/db/test_traits.py | 36 |
1 files changed, 22 insertions, 14 deletions
diff --git a/tests/unit/db/test_traits.py b/tests/unit/db/test_traits.py index 2a52ea6..3840dd1 100644 --- a/tests/unit/db/test_traits.py +++ b/tests/unit/db/test_traits.py @@ -14,13 +14,15 @@ class TestTraitsDBFunctions(TestCase): def test_retrieve_trait_dataset_name(self): """Test that the function is called correctly.""" - for trait_type, thresh, trait_dataset_name, columns in [ + for trait_type, thresh, trait_dataset_name, columns, table in [ ["ProbeSet", 9, "testName", - "Id, Name, FullName, ShortName, DataScale"], - ["Geno", 3, "genoTraitName", "Id, Name, FullName, ShortName"], + "Id, Name, FullName, ShortName, DataScale", "ProbeSetFreeze"], + ["Geno", 3, "genoTraitName", "Id, Name, FullName, ShortName", + "GenoFreeze"], ["Publish", 6, "publishTraitName", - "Id, Name, FullName, ShortName"], - ["Temp", 4, "tempTraitName", "Id, Name, FullName, ShortName"]]: + "Id, Name, FullName, ShortName", "PublishFreeze"], + ["Temp", 4, "tempTraitName", "Id, Name, FullName, ShortName", + "TempFreeze"]]: db_mock = mock.MagicMock() with self.subTest(trait_type=trait_type): with db_mock.cursor() as cursor: @@ -33,12 +35,13 @@ class TestTraitsDBFunctions(TestCase): ("testName", "testNameFull", "testNameShort", "dataScale")) cursor.execute.assert_called_once_with( - "SELECT {cols} " - "FROM {ttype}Freeze " + "SELECT %(columns)s " + "FROM %(table)s " "WHERE public > %(threshold)s AND " "(Name = %(name)s OR FullName = %(name)s OR ShortName = %(name)s)".format( cols=columns, ttype=trait_type), - {"threshold": thresh, "name": trait_dataset_name}) + {"threshold": thresh, "name": trait_dataset_name, + "table": table, "columns": columns}) def test_retrieve_publish_trait_info(self): """Test retrieval of type `Publish` traits.""" @@ -147,11 +150,16 @@ class TestTraitsDBFunctions(TestCase): def test_retrieve_trait_info(self): """Test that information on traits is retrieved as appropriate.""" - for trait_type, trait_name, trait_dataset_id, trait_dataset_name, in [ - ["Publish", "PublishTraitName", 1, "PublishDatasetTraitName"], - ["ProbeSet", "ProbeSetTraitName", 2, "ProbeSetDatasetTraitName"], - ["Geno", "GenoTraitName", 3, "GenoDatasetTraitName"], - ["Temp", "TempTraitName", 4, "TempDatasetTraitName"]]: + for trait_type, trait_name, trait_dataset_id, trait_dataset_name, expected in [ + ["Publish", "PublishTraitName", 1, "PublishDatasetTraitName", + {"haveinfo": 0, "homologeneid": None, "type": "Publish", + "confidential": 0}], + ["ProbeSet", "ProbeSetTraitName", 2, "ProbeSetDatasetTraitName", + {"haveinfo": 0, "homologeneid": None, "type": "ProbeSet"}], + ["Geno", "GenoTraitName", 3, "GenoDatasetTraitName", + {"haveinfo": 0, "homologeneid": None, "type": "Geno"}], + ["Temp", "TempTraitName", 4, "TempDatasetTraitName", + {"haveinfo": 0, "homologeneid": None, "type": "Temp"}]]: db_mock = mock.MagicMock() with self.subTest(trait_type=trait_type): with db_mock.cursor() as cursor: @@ -160,7 +168,7 @@ class TestTraitsDBFunctions(TestCase): retrieve_trait_info( trait_type, trait_name, trait_dataset_id, trait_dataset_name, db_mock), - {}) + expected) def test_update_sample_data(self): """Test that the SQL queries when calling update_sample_data are called with |