Merge pull request #1458 from cmu-delphi/krivard/patch-validator

Patch validator for chng subdivision mapping change

Author: krivard

_delphi_utils_python/delphi_utils/validator/datafetcher.py

@@ -111,8 +111,10 @@ def get_geo_signal_combos(data_source):
     Cross references based on combinations reported available by COVIDcast metadata.
     """
     # Maps data_source name with what's in the API, lists used in case of multiple names
+    # pylint: disable=fixme
+    # TODO: Extract this mapping from meta response instead of hard-coding
+    # https://github.com/cmu-delphi/covidcast-indicators/issues/1457
     source_signal_mappings = {
-        'chng': ['chng-cli', 'chng-covid'],
         'indicator-combination': ['indicator-combination-cases-deaths'],
         'quidel': ['quidel-covid-ag'],
         'safegraph': ['safegraph-weekly']

_delphi_utils_python/tests/validator/test_datafetcher.py

@@ -21,6 +21,9 @@ def test_make_date_filter(self):
         assert not date_filter(FILENAME_REGEX.match("20200620_a_b.csv"))
         assert not date_filter(FILENAME_REGEX.match("202006_a_b.csv"))
 
+    # pylint: disable=fixme
+    # TODO: mock out the advanced meta endpoint /covidcast/meta as well
+    # https://github.com/cmu-delphi/covidcast-indicators/issues/1456
     @mock.patch("covidcast.metadata")
     def test_get_geo_signal_combos(self, mock_metadata):
         """Test that the geo signal combos are correctly pulled from the covidcast metadata."""