links: updated thumbnails and tests

invenio_rdm_records/services/config.py

@@ -156,7 +156,7 @@ def lock_edit_published_files(service, identity, record=None, draft=None):
 
 def record_thumbnail_sizes():
     """Return configured sizes for thumbnails."""
-    return current_app.config.get("APP_RDM_RECORD_THUMBNAIL_SIZES",[])
+    return current_app.config.get("APP_RDM_RECORD_THUMBNAIL_SIZES", [])
 
 
 def record_thumbnail(size, record=None, **kwargs):
@@ -180,7 +180,7 @@ def record_thumbnail(size, record=None, **kwargs):
             None,
         )
         if file_key:
-            return f"{file_key}/full/{size},/0/default.png"
+            return f"{file_key}/full/^{size},/0/default.png"
 
     raise NotFound("Thumbnail not found")
 

tests/resources/test_serialized_links.py

@@ -64,6 +64,7 @@ def test_draft_links(client, draft_json, minimal_record, headers):
         "access_links": f"https://127.0.0.1:5000/api/records/{pid_value}/access/links",  # noqa
         "files": f"https://127.0.0.1:5000/api/records/{pid_value}/draft/files",
         "media_files": f"https://127.0.0.1:5000/api/records/{pid_value}/draft/media-files",
+        "thumbnails": {},
         "archive": f"https://127.0.0.1:5000/api/records/{pid_value}/draft/files-archive",  # noqa
         "archive_media": f"https://127.0.0.1:5000/api/records/{pid_value}/draft/media-files-archive",  # noqa
         "reserve_doi": f"https://127.0.0.1:5000/api/records/{pid_value}/draft/pids/doi",  # noqa
@@ -78,6 +79,7 @@ def test_draft_links(client, draft_json, minimal_record, headers):
         "access_users": f"https://127.0.0.1:5000/api/records/{pid_value}/access/users",
         "access_groups": f"https://127.0.0.1:5000/api/records/{pid_value}/access/groups",
     }
+
     assert expected_links == created_draft_links == read_draft_links
 
 
@@ -100,6 +102,7 @@ def test_record_links(client, published_json, headers):
         "draft": f"https://127.0.0.1:5000/api/records/{pid_value}/draft",
         "files": f"https://127.0.0.1:5000/api/records/{pid_value}/files",
         "media_files": f"https://127.0.0.1:5000/api/records/{pid_value}/media-files",
+        "thumbnails": {},
         "archive": f"https://127.0.0.1:5000/api/records/{pid_value}/files-archive",
         "archive_media": f"https://127.0.0.1:5000/api/records/{pid_value}/media-files-archive",
         "parent": f"https://127.0.0.1:5000/api/records/{parent_pid_value}",
@@ -122,6 +125,7 @@ def test_record_links(client, published_json, headers):
         "access_users": f"https://127.0.0.1:5000/api/records/{pid_value}/access/users",
         "access_groups": f"https://127.0.0.1:5000/api/records/{pid_value}/access/groups",
     }
+
     assert expected_links == published_record_links == read_record_links