Run geocoding tests separately

.github/workflows/tests.yml

@@ -86,6 +86,8 @@ jobs:
         run: gunicorn local_intelligence_hub.asgi:application -k uvicorn.workers.UvicornWorker --bind 0.0.0.0:8000 > server.log 2>&1 &
       - name: Run django tests
         run: cat .env && coverage run --source=. --branch manage.py test || (cat server.log && exit 1)
+      - name: Run geocoding tests in isolation
+        run: RUN_GEOCODING_TESTS=1 manage.py test hub.tests.test_external_data_source_parsers || (cat server.log && exit 1)
       - name: Generate coverage xml
         run: coverage xml
       - name: Upload coverage.xml

hub/tests/test_external_data_source_parsers.py

@@ -1,4 +1,5 @@
 import json
+import os
 import subprocess
 from datetime import datetime, timezone
 from unittest import skipIf
@@ -12,7 +13,7 @@
 from hub.validation import validate_and_format_phone_number
 from utils import mapit_types
 
-ignore_geocoding_tests = False
+ignore_geocoding_tests = os.getenv("RUN_GEOCODING_TESTS") != "1"
 
 
 class TestDateFieldParer(TestCase):
@@ -107,7 +108,7 @@ def test_valid_phone_number_for_usa(self):
         self.assertEqual(result, "+14155552671")
 
 
-# @skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
+@skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
 class TestMultiLevelGeocoding(TestCase):
     fixture = geocoding_cases
 
@@ -301,7 +302,7 @@ def test_skipping(self):
                 raise
 
 
-# @skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
+@skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
 class TestComplexAddressGeocoding(TestCase):
     @classmethod
     def setUpTestData(cls):
@@ -409,7 +410,7 @@ def test_skipping(self):
                 raise
 
 
-# @skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
+@skipIf(ignore_geocoding_tests, "It messes up data for other tests.")
 class TestCoordinateGeocoding(TestCase):
     @classmethod
     def setUpTestData(cls):