fix: add attachment name in download endpoint (#147)

Signed-off-by: Joris Mancini <joris.mancini_externe@rte-france.com>

Author: TheMaskedTurtle

src/main/java/com/powsybl/caseserver/Utils.java

@@ -7,6 +7,7 @@
 package com.powsybl.caseserver;
 
 import com.powsybl.commons.datasource.DataSourceUtil;
+import org.springframework.http.ContentDisposition;
 import org.springframework.http.HttpHeaders;
 
 import java.util.List;
@@ -61,6 +62,9 @@ public static HttpHeaders buildHeaders(String name, Boolean isUploadedAsPlainFil
         if (Boolean.TRUE.equals(isUploadedAsPlainFile)) {
             headers.add(HttpHeaders.CONTENT_ENCODING, GZIP_ENCODING);
         }
+        headers.setContentDisposition(
+                ContentDisposition.attachment().filename(name).build()
+        );
         return headers;
     }
 }

src/test/java/com/powsybl/caseserver/service/CaseControllerTest.java

@@ -50,13 +50,11 @@
 
 import static com.powsybl.caseserver.Utils.ZIP_EXTENSION;
 import static com.powsybl.caseserver.service.CaseService.DELIMITER;
-import static org.hamcrest.Matchers.hasSize;
-import static org.hamcrest.Matchers.startsWith;
+import static org.hamcrest.Matchers.*;
 import static org.junit.Assert.assertEquals;
 import static org.junit.jupiter.api.Assertions.*;
 import static org.springframework.test.web.servlet.request.MockMvcRequestBuilders.*;
-import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.content;
-import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.status;
+import static org.springframework.test.web.servlet.result.MockMvcResultMatchers.*;
 
 /**
  * @author Abdelsalem Hedhili <abdelsalem.hedhili at rte-france.com>
@@ -272,6 +270,7 @@ void test() throws Exception {
             byte[] expectedGzippedBytes = byteArrayOutputStream.toByteArray();
             mvc.perform(get(GET_CASE_URL, firstCaseUuid))
                     .andExpect(status().isOk())
+                    .andExpect(header().string("Content-Disposition", "attachment; filename=\"" + TEST_CASE + "\""))
                     .andExpect(content().bytes(expectedGzippedBytes))
                     .andReturn();
         }
@@ -280,6 +279,7 @@ void test() throws Exception {
         mvc.perform(get(GET_CASE_URL, gzipCaseUuid))
                 .andExpect(status().isOk())
                 .andExpect(content().bytes(getClass().getResourceAsStream("/" + TEST_GZIP_CASE).readAllBytes()))
+                .andExpect(header().string("Content-Disposition", "attachment; filename=\"" + TEST_GZIP_CASE + "\""))
                 .andReturn();
         assertNotNull(outputDestination.receive(1000, caseImportDestination));