fix: remap volume provider

.github/workflows/docker-img.yml

@@ -105,7 +105,7 @@ jobs:
     needs: build-docker-img
     steps:
     - uses: actions/checkout@v4
-    - uses: actions/setup-python@v4
+    - uses: actions/setup-python@v5
       with:
         python-version: '3.10'
     - name: 'Setting env var'

.helm/adhoc/configmaps-siibra-api-common.yml

@@ -6,7 +6,7 @@ data:
   MONITOR_FIRSTLVL_DIR: /siibra-api-volume
   # SIIBRA_CACHEDIR: /siibra-api-volume
   SIIBRA_API_REMAP_PROVIDERS: |-
-    https://neuroglancer.humanbrainproject.eu:https:://data-proxy.ebrains.eu/api/v1/buckets/reference-atlas-data
+    https://neuroglancer.humanbrainproject.eu::https://data-proxy.ebrains.eu/api/v1/buckets/reference-atlas-data
     https://object.cscs.ch/v1/AUTH_08c08f9f119744cbbf77e216988da3eb::https://data-proxy.ebrains.eu/api/v1/buckets/reference-atlas-data/imgsvc
 kind: ConfigMap
 metadata:

api/serialization/volumes/volume.py

@@ -27,9 +27,10 @@ def remap_provider(obj: Dict[str, Union[dict, str]]) -> None:
     for key, value in obj.items():
         if isinstance(value, str):
             for from_host, to_host in SIIBRA_API_REMAP_PROVIDERS.items():
+                value = obj[key]
                 obj[key] = value.replace(from_host, to_host)
         if isinstance(value, dict):
-            remap_provider(value)
+            remap_provider(value)   
     
 
 @serialize(Volume)

test/serialization/volumes/test_volume.py

+from unittest.mock import patch
+import pytest
+
+from api.serialization.volumes.volume import remap_provider
+from api import siibra_api_config
+
+@pytest.fixture
+def provider():
+    siibra_api_config.SIIBRA_API_REMAP_PROVIDERS["http://foo.bar"] = "http://bazz.bizz"
+    siibra_api_config.SIIBRA_API_REMAP_PROVIDERS["http://hello.world"] = "http://apple.banana"
+    yield {
+        "foo": "http://foo.bar/two/one",
+        "bar": "http://hello.world/me/you",
+        "bazz": "http://bazz.bizz/cable/two"
+    }
+    siibra_api_config.SIIBRA_API_REMAP_PROVIDERS.clear()
+
+def test_remap_provider(provider):
+    remap_provider(provider)
+    assert provider["foo"] == "http://bazz.bizz/two/one"
+    assert provider["bar"] == "http://apple.banana/me/you"
+    assert provider["bazz"] == "http://bazz.bizz/cable/two"