fix: project url

alfredotoselli · alfredotoselli · commit 14b71d97fe7e · 2025-06-04T11:42:11.000+02:00
diff --git a/cads_catalogue_api_service/schema_org.py b/cads_catalogue_api_service/schema_org.py
@@ -140,7 +140,9 @@ def schema_org_json_ld(
         isPartOf=[
             {
                 "@type": "DataCatalog",
-                "@id": os.getenv(f"{site.upper()}_PROJECT_URL", None) if site else None,
+                "@id": f"{os.getenv(f'{site.upper()}_PROJECT_URL', None)}/datasets"
+                if site
+                else None,
                 "name": "ECMWF Data Store",
             }
         ],
diff --git a/tests/test_40_schema_org.py b/tests/test_40_schema_org.py
@@ -105,7 +105,7 @@ def test_schema_org_jsonId(monkeypatch) -> None:
         "cads_catalogue_api_service.schema_org.query_collection",
         static_collection_query,
     )
-    monkeypatch.setenv("CDS_PROJECT_URL", "https://cds.climate.copernicus.eu/datasets")
+    monkeypatch.setenv("CDS_PROJECT_URL", "https://cds.climate.copernicus.eu")
 
     response = client.get(
         "/collections/era5-something/schema.org",

Original file line number	Diff line number	Diff line change
`@@ -140,7 +140,9 @@ def schema_org_json_ld(`
`140`	`140`	`isPartOf=[`
`141`	`141`	`{`
`142`	`142`	`"@type": "DataCatalog",`
`143`		`- "@id": os.getenv(f"{site.upper()}_PROJECT_URL", None) if site else None,`
	`143`	`+ "@id": f"{os.getenv(f'{site.upper()}_PROJECT_URL', None)}/datasets"`
	`144`	`+ if site`
	`145`	`+ else None,`
`144`	`146`	`"name": "ECMWF Data Store",`
`145`	`147`	`}`
`146`	`148`	`],`
Original file line number	Diff line number	Diff line change
`@@ -105,7 +105,7 @@ def test_schema_org_jsonId(monkeypatch) -> None:`
`105`	`105`	`"cads_catalogue_api_service.schema_org.query_collection",`
`106`	`106`	`static_collection_query,`
`107`	`107`	`)`
`108`		`- monkeypatch.setenv("CDS_PROJECT_URL", "https://cds.climate.copernicus.eu/datasets")`
	`108`	`+ monkeypatch.setenv("CDS_PROJECT_URL", "https://cds.climate.copernicus.eu")`
`109`	`109`
`110`	`110`	`response = client.get(`
`111`	`111`	`"/collections/era5-something/schema.org",`