Default to full aspect thumbnail

nulib · Sep 26, 2022 · 9388068 · 9388068
1 parent 076d59f
commit 9388068
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/src/handlers/get-work-thumbnail.js b/src/handlers/get-work-thumbnail.js
@@ -15,7 +15,7 @@ function getAxiosResponse(url, config) {
 
 function validateRequest(event) {
   const id = event.pathParameters.id;
-  const aspect = event?.queryStringParameters?.aspect || "square";
+  const aspect = event?.queryStringParameters?.aspect || "full";
   const sizeParam = event?.queryStringParameters?.size || 300;
   const size = Number(sizeParam);
 

diff --git a/test/integration/get-thumbnail.test.js b/test/integration/get-thumbnail.test.js
@@ -21,8 +21,8 @@ describe("Work thumbnail", () => {
       .get("/dc-v2-work/_doc/1234")
       .reply(200, helpers.testFixture("mocks/work-1234.json"));
     mock
-      .get("/iiif/2/mbk-dev/5678/square/!300,300/0/default.jpg")
-      .reply(200, helpers.testFixture("mocks/thumbnail_square.jpg"), {
+      .get("/iiif/2/mbk-dev/5678/full/!300,300/0/default.jpg")
+      .reply(200, helpers.testFixture("mocks/thumbnail_full.jpg"), {
         "Content-Type": "image/jpeg",
       });
 
@@ -36,7 +36,7 @@ describe("Work thumbnail", () => {
       .get("/dc-v2-work/_doc/1234")
       .reply(200, helpers.testFixture("mocks/work-1234.json"));
     mock
-      .get("/iiif/2/mbk-dev/5678/square/!300,300/0/default.jpg")
+      .get("/iiif/2/mbk-dev/5678/full/!300,300/0/default.jpg")
       .reply(403, "Forbidden", { "Content-Type": "text/plain" });
 
     const result = await handler(event.render());
@@ -67,8 +67,8 @@ describe("Work thumbnail", () => {
       .get("/dc-v2-work/_doc/1234")
       .reply(200, helpers.testFixture("mocks/work-1234.json"));
     mock
-      .get("/iiif/2/mbk-dev/5678/square/!200,200/0/default.jpg")
-      .reply(200, helpers.testFixture("mocks/thumbnail_square.jpg"), {
+      .get("/iiif/2/mbk-dev/5678/full/!200,200/0/default.jpg")
+      .reply(200, helpers.testFixture("mocks/thumbnail_full.jpg"), {
         "Content-Type": "image/jpeg",
       });