BXC-4375 add new tests

UNC-Libraries · Dec 12, 2023 · 282f643 · 282f643
1 parent e09ecad
commit 282f643
Showing 1 changed file with 39 additions and 4 deletions.
diff --git a/...-app/src/test/java/edu/unc/lib/boxc/web/services/rest/ImageServerProxyControllerTest.java b/...-app/src/test/java/edu/unc/lib/boxc/web/services/rest/ImageServerProxyControllerTest.java
@@ -3,24 +3,29 @@
 import com.github.tomakehurst.wiremock.client.WireMock;
 import com.github.tomakehurst.wiremock.junit5.WireMockTest;
 import edu.unc.lib.boxc.auth.api.services.AccessControlService;
+import edu.unc.lib.boxc.web.services.processing.ImageServerProxyService;
 import edu.unc.lib.boxc.web.services.rest.modify.AbstractAPIIT;
 import edu.unc.lib.boxc.web.services.utils.ImageServerUtil;
 import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.Assertions;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
+import org.mockito.Mock;
 import org.springframework.beans.factory.annotation.Autowired;
 import org.springframework.http.HttpStatus;
 import org.springframework.test.context.ContextConfiguration;
 import org.springframework.test.context.junit.jupiter.SpringExtension;
 import org.springframework.test.context.web.WebAppConfiguration;
 import org.springframework.test.web.servlet.MvcResult;
 
+import java.io.IOException;
+
 import static com.github.tomakehurst.wiremock.client.WireMock.aResponse;
 import static com.github.tomakehurst.wiremock.client.WireMock.stubFor;
 import static com.github.tomakehurst.wiremock.client.WireMock.urlMatching;
 import static org.mockito.ArgumentMatchers.any;
+import static org.mockito.Mockito.doThrow;
 import static org.mockito.Mockito.when;
 import static org.mockito.MockitoAnnotations.openMocks;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.get;
@@ -37,6 +42,8 @@
 public class ImageServerProxyControllerTest extends AbstractAPIIT {
     @Autowired
     private AccessControlService accessControlService;
+    @Mock
+    private ImageServerProxyService imageServerProxyService;
     private AutoCloseable closeable;
 
     @BeforeEach
@@ -50,7 +57,7 @@ void closeService() throws Exception {
     }
 
     @Test
-    void getRegionTestNoAccess() throws Exception {
+    void testGetRegionNoAccess() throws Exception {
         var pid = makePid();
         var pidString = pid.getId();
         when(accessControlService.hasAccess(any(), any(), any())).thenReturn(false);
@@ -61,7 +68,7 @@ void getRegionTestNoAccess() throws Exception {
     }
 
     @Test
-    void getRegionTest() throws Exception {
+    void testGetRegionSuccess() throws Exception {
         var pid = makePid();
         var pidString = pid.getId();
         var formattedBasePath = "/iiif/v3/" + ImageServerUtil.getImageServerEncodedId(pidString);
@@ -81,9 +88,21 @@ void getRegionTest() throws Exception {
         Assertions.assertEquals(response.getContentAsString(), filename);
     }
 
+    @Test
+    void testGetRegionIOException() throws Exception {
+        var pid = makePid();
+        var pidString = pid.getId();
+        when(accessControlService.hasAccess(any(), any(), any())).thenReturn(true);
+        doThrow(new IOException()).when(imageServerProxyService)
+                .streamJP2(pidString, "full", "max", "0", "default", "jpg");
+
+        mvc.perform(get("/iiif/v3/" + pidString + "/full/max/0/default.jpg"))
+                .andExpect(status().isInternalServerError())
+                .andReturn();
+    }
 
     @Test
-    void getMetadataTestNoAccess() throws Exception {
+    void testGetMetadataNoAccess() throws Exception {
         var pid = makePid();
         var pidString = pid.getId();
         when(accessControlService.hasAccess(any(), any(), any())).thenReturn(false);
@@ -94,7 +113,7 @@ void getMetadataTestNoAccess() throws Exception {
     }
 
     @Test
-    void getMetadataTest() throws Exception {
+    void testGetMetadataSuccess() throws Exception {
         var pid = makePid();
         var pidString = pid.getId();
         var formattedBasePath = "/iiif/v3/" + ImageServerUtil.getImageServerEncodedId(pidString);
@@ -114,4 +133,20 @@ void getMetadataTest() throws Exception {
         var response = result.getResponse();
         Assertions.assertEquals(response.getContentAsString(), json);
     }
+
+    @Test
+    void testGetMetadataIOException() throws Exception {
+        var pid = makePid();
+        var pidString = pid.getId();
+        var formattedBasePath = "/iiif/v3/" + ImageServerUtil.getImageServerEncodedId(pidString);
+        when(accessControlService.hasAccess(any(), any(), any())).thenReturn(true);
+        doThrow(new IOException()).when(imageServerProxyService).getMetadata(pidString);
+
+        stubFor(WireMock.get(urlMatching(formattedBasePath + "/info.json"))
+                .willReturn(aResponse().withHeader("Content-Type", "application/json")));
+
+        mvc.perform(get("/iiif/v3/" + pidString + "/info.json"))
+                .andExpect(status().isInternalServerError())
+                .andReturn();
+    }
 }