✨ Refactor test API service access: simplify mock patching for ServicesRepository

Author: pcrespov

services/catalog/tests/unit/with_dbs/test_api_rest_services_ports.py

@@ -54,8 +54,8 @@ async def mocked_check_service_read_access(
     assert user_groups_ids
 
     mocker.patch.object(
-        simcore_service_catalog.api._dependencies.services,
-        "ServicesRepository.get_service",
+        simcore_service_catalog.api._dependencies.services.ServicesRepository,
+        "get_service",
         autospec=True,
         return_value=True,
     )

services/catalog/tests/unit/with_dbs/test_core_background_task__sync.py

@@ -9,6 +9,7 @@
 from typing import Any
 
 import pytest
+import simcore_service_catalog.service.access_rights
 from fastapi import FastAPI, HTTPException, status
 from pytest_mock import MockerFixture
 from respx.router import MockRouter
@@ -58,8 +59,9 @@ async def test_registry_sync_task(
 
     if director_fails:
         # Emulates issue https://github.com/ITISFoundation/osparc-simcore/issues/6318
-        mocker.patch(
-            "simcore_service_catalog.services.access_rights._is_old_service",
+        mocker.patch.object(
+            simcore_service_catalog.service.access_rights,
+            "_is_old_service",
             side_effect=HTTPException(
                 status_code=status.HTTP_404_NOT_FOUND, detail="fake director error"
             ),