feat: rename license name to license title

BREAKING CHANGE: Renames a field in the /datasets response model

Author: tillywoodfield

oc4ids_datastore_api/models.py

@@ -10,7 +10,7 @@ class DatasetSQLModel(SQLModel, table=True):
     source_url: str
     publisher_name: str
     license_url: str | None
-    license_name: str | None
+    license_title: str | None
     json_url: str | None
     csv_url: str | None
     xlsx_url: str | None

oc4ids_datastore_api/schemas.py

@@ -9,7 +9,7 @@ class Publisher(BaseModel):
 
 class License(BaseModel):
     url: str | None
-    name: str | None
+    title: str | None
 
 
 class Download(BaseModel):

oc4ids_datastore_api/services.py

@@ -16,7 +16,10 @@ def _transform_dataset(dataset: DatasetSQLModel) -> Dataset:
         loaded_at=dataset.updated_at,
         source_url=dataset.source_url,
         publisher=Publisher(name=dataset.publisher_name),
-        license=License(url=dataset.license_url, name=dataset.license_name),
+        license=License(
+            url=dataset.license_url,
+            title=dataset.license_title,
+        ),
         downloads=downloads,
     )
 

tests/test_services.py

@@ -17,7 +17,7 @@ def test_get_all_datasets(mocker: MockerFixture) -> None:
         source_url="https://test-dataset.json",
         publisher_name="test_publisher",
         license_url="https://license.com",
-        license_name="License",
+        license_title="License",
         json_url="https://downloads/test_dataset.json",
         csv_url="https://downloads/test_dataset.csv",
         xlsx_url="https://downloads/test_dataset.xlsx",
@@ -31,7 +31,7 @@ def test_get_all_datasets(mocker: MockerFixture) -> None:
         loaded_at=now,
         source_url="https://test-dataset.json",
         publisher=Publisher(name="test_publisher"),
-        license=License(name="License", url="https://license.com"),
+        license=License(title="License", url="https://license.com"),
         downloads=[
             Download(format="json", url="https://downloads/test_dataset.json"),
             Download(format="csv", url="https://downloads/test_dataset.csv"),
@@ -51,7 +51,7 @@ def test_get_all_datasets_missing_download_formats(mocker: MockerFixture) -> Non
         source_url="https://test-dataset.json",
         publisher_name="test_publisher",
         license_url="https://license.com",
-        license_name="License",
+        license_title="License",
         json_url="https://downloads/test_dataset.json",
         csv_url=None,
         xlsx_url=None,
@@ -65,7 +65,7 @@ def test_get_all_datasets_missing_download_formats(mocker: MockerFixture) -> Non
         loaded_at=now,
         source_url="https://test-dataset.json",
         publisher=Publisher(name="test_publisher"),
-        license=License(name="License", url="https://license.com"),
+        license=License(title="License", url="https://license.com"),
         downloads=[
             Download(format="json", url="https://downloads/test_dataset.json"),
         ],