gave a method a better name

inventory_provider/tasks/worker.py

@@ -560,7 +560,7 @@ def refresh_finalizer(self, pending_task_ids_json):
 
         _wait_for_tasks(task_ids, update_callback=_update)
         _build_subnet_db(update_callback=_update)
-        _build_interface_services(update_callback=_update)
+        _build_service_category_interface_list(update_callback=_update)
 
     except (jsonschema.ValidationError,
             json.JSONDecodeError,
@@ -574,7 +574,7 @@ def refresh_finalizer(self, pending_task_ids_json):
     logger.debug('<<< refresh_finalizer')
 
 
-def _build_interface_services(update_callback=lambda s: None):
+def _build_service_category_interface_list(update_callback=lambda s: None):
     logger.debug('>>> _build_interface_services')
 
     r = get_next_redis(InventoryTask.config)

test/test_general_poller_routes.py

@@ -34,7 +34,7 @@ INTERFACE_LIST_SCHEMA = {
 
 @pytest.mark.parametrize('category', ['mdvpn', 'lhcone', 'MDVpn', 'LHCONE'])
 def test_service_category(client, mocked_worker_module, category):
-    worker._build_interface_services()
+    worker._build_service_category_interface_list()
     rv = client.get(
         f'/poller/services/{category}',
         headers=DEFAULT_REQUEST_HEADERS)
@@ -47,7 +47,7 @@ def test_service_category(client, mocked_worker_module, category):
 
 @pytest.mark.parametrize('category', ['mdvpn ', ' mdvpn', 'mdvpn1', 'aaa'])
 def test_service_category_not_found(client, mocked_worker_module, category):
-    worker._build_interface_services()
+    worker._build_service_category_interface_list()
     rv = client.get(
         f'/poller/services/{category}',
         headers=DEFAULT_REQUEST_HEADERS)

test/test_worker_utils.py

@@ -40,7 +40,7 @@ def test_build_interface_services(mocked_worker_module):
     }
 
     db = backend_db()  # also forces initialization
-    worker._build_interface_services()
+    worker._build_service_category_interface_list()
 
     seen_types = set()
     for k, v in db.items():