🤖 format everything with pre-commit by stirlingbot (#4075)

Auto-generated by [create-pull-request][1] with **stirlingbot** [1]: https://github.com/peter-evans/create-pull-request Signed-off-by: stirlingbot[bot] <stirlingbot[bot]@users.noreply.github.com> Co-authored-by: stirlingbot[bot] <195170888+stirlingbot[bot]@users.noreply.github.com>
2026-07-15 11:00:47 +02:00 · 2025-08-02 23:18:48 +01:00
parent 62779d99d1
commit 56c79eb63c
31 changed files with 1097 additions and 870 deletions
@@ -36,14 +36,11 @@ import stirling.software.common.service.CustomPDFDocumentFactory;
@ExtendWith(MockitoExtension.class)
 class EditTableOfContentsControllerTest {

-    @Mock
-    private CustomPDFDocumentFactory pdfDocumentFactory;
+    @Mock private CustomPDFDocumentFactory pdfDocumentFactory;

-    @Mock
-    private ObjectMapper objectMapper;
+    @Mock private ObjectMapper objectMapper;

-    @InjectMocks
-    private EditTableOfContentsController editTableOfContentsController;
+    @InjectMocks private EditTableOfContentsController editTableOfContentsController;

    private MockMultipartFile mockFile;
    private PDDocument mockDocument;
@@ -56,7 +53,9 @@ class EditTableOfContentsControllerTest {

    @BeforeEach
    void setUp() {
-        mockFile = new MockMultipartFile("file", "test.pdf", "application/pdf", "PDF content".getBytes());
+        mockFile =
+                new MockMultipartFile(
+                        "file", "test.pdf", "application/pdf", "PDF content".getBytes());
        mockDocument = mock(PDDocument.class);
        mockCatalog = mock(PDDocumentCatalog.class);
        mockPages = mock(PDPageTree.class);
@@ -149,7 +148,8 @@ class EditTableOfContentsControllerTest {
        assertEquals(1, parentBookmark.get("pageNumber"));

        @SuppressWarnings("unchecked")
-        List<Map<String, Object>> children = (List<Map<String, Object>>) parentBookmark.get("children");
+        List<Map<String, Object>> children =
+                (List<Map<String, Object>>) parentBookmark.get("children");
        assertEquals(1, children.size());

        Map<String, Object> childBookmark = children.get(0);
@@ -202,17 +202,21 @@ class EditTableOfContentsControllerTest {
        bookmarks.add(bookmark);

        when(pdfDocumentFactory.load(mockFile)).thenReturn(mockDocument);
-        when(objectMapper.readValue(eq(request.getBookmarkData()), any(TypeReference.class))).thenReturn(bookmarks);
+        when(objectMapper.readValue(eq(request.getBookmarkData()), any(TypeReference.class)))
+                .thenReturn(bookmarks);
        when(mockDocument.getDocumentCatalog()).thenReturn(mockCatalog);
        when(mockDocument.getNumberOfPages()).thenReturn(5);
        when(mockDocument.getPage(0)).thenReturn(mockPage1);

        // Mock saving behavior
-        doAnswer(invocation -> {
-            ByteArrayOutputStream baos = invocation.getArgument(0);
-            baos.write("mocked pdf content".getBytes());
-            return null;
-        }).when(mockDocument).save(any(ByteArrayOutputStream.class));
+        doAnswer(
+                        invocation -> {
+                            ByteArrayOutputStream baos = invocation.getArgument(0);
+                            baos.write("mocked pdf content".getBytes());
+                            return null;
+                        })
+                .when(mockDocument)
+                .save(any(ByteArrayOutputStream.class));

        // When
        ResponseEntity<byte[]> result = editTableOfContentsController.editTableOfContents(request);
@@ -221,7 +225,8 @@ class EditTableOfContentsControllerTest {
        assertNotNull(result);
        assertNotNull(result.getBody());

-        ArgumentCaptor<PDDocumentOutline> outlineCaptor = ArgumentCaptor.forClass(PDDocumentOutline.class);
+        ArgumentCaptor<PDDocumentOutline> outlineCaptor =
+                ArgumentCaptor.forClass(PDDocumentOutline.class);
        verify(mockCatalog).setDocumentOutline(outlineCaptor.capture());

        PDDocumentOutline capturedOutline = outlineCaptor.getValue();
@@ -236,7 +241,8 @@ class EditTableOfContentsControllerTest {
        EditTableOfContentsRequest request = new EditTableOfContentsRequest();
        request.setFileInput(mockFile);

-        String bookmarkJson = "[{\"title\":\"Chapter 1\",\"pageNumber\":1,\"children\":[{\"title\":\"Section 1.1\",\"pageNumber\":2,\"children\":[]}]}]";
+        String bookmarkJson =
+                "[{\"title\":\"Chapter 1\",\"pageNumber\":1,\"children\":[{\"title\":\"Section 1.1\",\"pageNumber\":2,\"children\":[]}]}]";
        request.setBookmarkData(bookmarkJson);

        List<BookmarkItem> bookmarks = new ArrayList<>();
@@ -255,17 +261,21 @@ class EditTableOfContentsControllerTest {
        bookmarks.add(parentBookmark);

        when(pdfDocumentFactory.load(mockFile)).thenReturn(mockDocument);
-        when(objectMapper.readValue(eq(bookmarkJson), any(TypeReference.class))).thenReturn(bookmarks);
+        when(objectMapper.readValue(eq(bookmarkJson), any(TypeReference.class)))
+                .thenReturn(bookmarks);
        when(mockDocument.getDocumentCatalog()).thenReturn(mockCatalog);
        when(mockDocument.getNumberOfPages()).thenReturn(5);
        when(mockDocument.getPage(0)).thenReturn(mockPage1);
        when(mockDocument.getPage(1)).thenReturn(mockPage2);

-        doAnswer(invocation -> {
-            ByteArrayOutputStream baos = invocation.getArgument(0);
-            baos.write("mocked pdf content".getBytes());
-            return null;
-        }).when(mockDocument).save(any(ByteArrayOutputStream.class));
+        doAnswer(
+                        invocation -> {
+                            ByteArrayOutputStream baos = invocation.getArgument(0);
+                            baos.write("mocked pdf content".getBytes());
+                            return null;
+                        })
+                .when(mockDocument)
+                .save(any(ByteArrayOutputStream.class));

        // When
        ResponseEntity<byte[]> result = editTableOfContentsController.editTableOfContents(request);
@@ -281,7 +291,8 @@ class EditTableOfContentsControllerTest {
        // Given
        EditTableOfContentsRequest request = new EditTableOfContentsRequest();
        request.setFileInput(mockFile);
-        request.setBookmarkData("[{\"title\":\"Chapter 1\",\"pageNumber\":-5,\"children\":[]},{\"title\":\"Chapter 2\",\"pageNumber\":100,\"children\":[]}]");
+        request.setBookmarkData(
+                "[{\"title\":\"Chapter 1\",\"pageNumber\":-5,\"children\":[]},{\"title\":\"Chapter 2\",\"pageNumber\":100,\"children\":[]}]");

        List<BookmarkItem> bookmarks = new ArrayList<>();

@@ -299,17 +310,21 @@ class EditTableOfContentsControllerTest {
        bookmarks.add(bookmark2);

        when(pdfDocumentFactory.load(mockFile)).thenReturn(mockDocument);
-        when(objectMapper.readValue(eq(request.getBookmarkData()), any(TypeReference.class))).thenReturn(bookmarks);
+        when(objectMapper.readValue(eq(request.getBookmarkData()), any(TypeReference.class)))
+                .thenReturn(bookmarks);
        when(mockDocument.getDocumentCatalog()).thenReturn(mockCatalog);
        when(mockDocument.getNumberOfPages()).thenReturn(5);
        when(mockDocument.getPage(0)).thenReturn(mockPage1); // For negative page number
        when(mockDocument.getPage(4)).thenReturn(mockPage2); // For page number exceeding bounds

-        doAnswer(invocation -> {
-            ByteArrayOutputStream baos = invocation.getArgument(0);
-            baos.write("mocked pdf content".getBytes());
-            return null;
-        }).when(mockDocument).save(any(ByteArrayOutputStream.class));
+        doAnswer(
+                        invocation -> {
+                            ByteArrayOutputStream baos = invocation.getArgument(0);
+                            baos.write("mocked pdf content".getBytes());
+                            return null;
+                        })
+                .when(mockDocument)
+                .save(any(ByteArrayOutputStream.class));

        // When
        ResponseEntity<byte[]> result = editTableOfContentsController.editTableOfContents(request);
@@ -332,16 +347,20 @@ class EditTableOfContentsControllerTest {
        when(mockDocument.getPage(2)).thenReturn(mockPage1); // 0-indexed

        // When
-        Method createOutlineItemMethod = EditTableOfContentsController.class.getDeclaredMethod("createOutlineItem", PDDocument.class, BookmarkItem.class);
+        Method createOutlineItemMethod =
+                EditTableOfContentsController.class.getDeclaredMethod(
+                        "createOutlineItem", PDDocument.class, BookmarkItem.class);
        createOutlineItemMethod.setAccessible(true);
-        PDOutlineItem result = (PDOutlineItem) createOutlineItemMethod.invoke(editTableOfContentsController, mockDocument, bookmark);
+        PDOutlineItem result =
+                (PDOutlineItem)
+                        createOutlineItemMethod.invoke(
+                                editTableOfContentsController, mockDocument, bookmark);

        // Then
        assertNotNull(result);
        verify(mockDocument).getPage(2);
    }

-
    @Test
    void testBookmarkItem_GettersAndSetters() {
        // Given
@@ -365,18 +384,24 @@ class EditTableOfContentsControllerTest {
        EditTableOfContentsRequest request = new EditTableOfContentsRequest();
        request.setFileInput(mockFile);

-        when(pdfDocumentFactory.load(mockFile)).thenThrow(new RuntimeException("Failed to load PDF"));
+        when(pdfDocumentFactory.load(mockFile))
+                .thenThrow(new RuntimeException("Failed to load PDF"));

        // When & Then
-        assertThrows(RuntimeException.class, () -> editTableOfContentsController.editTableOfContents(request));
+        assertThrows(
+                RuntimeException.class,
+                () -> editTableOfContentsController.editTableOfContents(request));
    }

    @Test
    void testExtractBookmarks_IOExceptionDuringLoad_ThrowsException() throws Exception {
        // Given
-        when(pdfDocumentFactory.load(mockFile)).thenThrow(new RuntimeException("Failed to load PDF"));
+        when(pdfDocumentFactory.load(mockFile))
+                .thenThrow(new RuntimeException("Failed to load PDF"));

        // When & Then
-        assertThrows(RuntimeException.class, () -> editTableOfContentsController.extractBookmarks(mockFile));
+        assertThrows(
+                RuntimeException.class,
+                () -> editTableOfContentsController.extractBookmarks(mockFile));
    }
 }
@@ -29,11 +29,9 @@ import stirling.software.common.service.CustomPDFDocumentFactory;
@ExtendWith(MockitoExtension.class)
 class MergeControllerTest {

-    @Mock
-    private CustomPDFDocumentFactory pdfDocumentFactory;
+    @Mock private CustomPDFDocumentFactory pdfDocumentFactory;

-    @InjectMocks
-    private MergeController mergeController;
+    @InjectMocks private MergeController mergeController;

    private MockMultipartFile mockFile1;
    private MockMultipartFile mockFile2;
@@ -47,9 +45,15 @@ class MergeControllerTest {

    @BeforeEach
    void setUp() {
-        mockFile1 = new MockMultipartFile("file1", "document1.pdf", "application/pdf", "PDF content 1".getBytes());
-        mockFile2 = new MockMultipartFile("file2", "document2.pdf", "application/pdf", "PDF content 2".getBytes());
-        mockFile3 = new MockMultipartFile("file3", "chapter3.pdf", "application/pdf", "PDF content 3".getBytes());
+        mockFile1 =
+                new MockMultipartFile(
+                        "file1", "document1.pdf", "application/pdf", "PDF content 1".getBytes());
+        mockFile2 =
+                new MockMultipartFile(
+                        "file2", "document2.pdf", "application/pdf", "PDF content 2".getBytes());
+        mockFile3 =
+                new MockMultipartFile(
+                        "file3", "chapter3.pdf", "application/pdf", "PDF content 3".getBytes());

        mockDocument = mock(PDDocument.class);
        mockMergedDocument = mock(PDDocument.class);
@@ -85,12 +89,15 @@ class MergeControllerTest {
        when(doc3.getNumberOfPages()).thenReturn(2);

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);
        addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files);

        // Then
-        ArgumentCaptor<PDDocumentOutline> outlineCaptor = ArgumentCaptor.forClass(PDDocumentOutline.class);
+        ArgumentCaptor<PDDocumentOutline> outlineCaptor =
+                ArgumentCaptor.forClass(PDDocumentOutline.class);
        verify(mockCatalog).setDocumentOutline(outlineCaptor.capture());

        PDDocumentOutline capturedOutline = outlineCaptor.getValue();
@@ -121,7 +128,9 @@ class MergeControllerTest {
        when(doc1.getNumberOfPages()).thenReturn(3);

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);
        addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files);

@@ -138,7 +147,9 @@ class MergeControllerTest {
        when(mockMergedDocument.getDocumentCatalog()).thenReturn(mockCatalog);

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);
        addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files);

@@ -155,7 +166,8 @@ class MergeControllerTest {

        when(mockMergedDocument.getDocumentCatalog()).thenReturn(mockCatalog);
        when(mockMergedDocument.getNumberOfPages()).thenReturn(4);
-        when(mockMergedDocument.getPage(anyInt())).thenReturn(mockPage1); // Use anyInt() to avoid stubbing conflicts
+        when(mockMergedDocument.getPage(anyInt()))
+                .thenReturn(mockPage1); // Use anyInt() to avoid stubbing conflicts

        // First document loads successfully
        PDDocument doc1 = mock(PDDocument.class);
@@ -163,16 +175,18 @@ class MergeControllerTest {
        when(doc1.getNumberOfPages()).thenReturn(2);

        // Second document throws IOException
-        when(pdfDocumentFactory.load(mockFile2)).thenThrow(new IOException("Failed to load document"));
+        when(pdfDocumentFactory.load(mockFile2))
+                .thenThrow(new IOException("Failed to load document"));

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);

        // Should not throw exception
-        assertDoesNotThrow(() ->
-            addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files)
-        );
+        assertDoesNotThrow(
+                () -> addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files));

        // Then
        verify(mockCatalog).setDocumentOutline(any(PDDocumentOutline.class));
@@ -184,7 +198,9 @@ class MergeControllerTest {
    @Test
    void testAddTableOfContents_FilenameWithoutExtension_UsesFullName() throws Exception {
        // Given
-        MockMultipartFile fileWithoutExtension = new MockMultipartFile("file", "document_no_ext", "application/pdf", "PDF content".getBytes());
+        MockMultipartFile fileWithoutExtension =
+                new MockMultipartFile(
+                        "file", "document_no_ext", "application/pdf", "PDF content".getBytes());
        MultipartFile[] files = {fileWithoutExtension};

        when(mockMergedDocument.getDocumentCatalog()).thenReturn(mockCatalog);
@@ -196,7 +212,9 @@ class MergeControllerTest {
        when(doc.getNumberOfPages()).thenReturn(1);

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);
        addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files);

@@ -218,13 +236,14 @@ class MergeControllerTest {
        when(doc1.getNumberOfPages()).thenReturn(3);

        // When
-        Method addTableOfContentsMethod = MergeController.class.getDeclaredMethod("addTableOfContents", PDDocument.class, MultipartFile[].class);
+        Method addTableOfContentsMethod =
+                MergeController.class.getDeclaredMethod(
+                        "addTableOfContents", PDDocument.class, MultipartFile[].class);
        addTableOfContentsMethod.setAccessible(true);

        // Should not throw exception
-        assertDoesNotThrow(() ->
-            addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files)
-        );
+        assertDoesNotThrow(
+                () -> addTableOfContentsMethod.invoke(mergeController, mockMergedDocument, files));

        // Then
        verify(mockCatalog).setDocumentOutline(any(PDDocumentOutline.class));
@@ -275,5 +294,4 @@ class MergeControllerTest {
        assertEquals(mockMergedDocument, result);
        verify(mockMergedDocument, never()).addPage(any(PDPage.class));
    }
-
 }
@@ -4,8 +4,6 @@ import static org.junit.jupiter.api.Assertions.*;
 import static org.mockito.ArgumentMatchers.*;
 import static org.mockito.Mockito.*;

-import org.mockito.MockedStatic;
-
 import java.io.IOException;
 import java.util.List;

@@ -15,6 +13,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.extension.ExtendWith;
 import org.mockito.InjectMocks;
 import org.mockito.Mock;
+import org.mockito.MockedStatic;
 import org.mockito.junit.jupiter.MockitoExtension;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
@@ -29,14 +28,11 @@ import stirling.software.common.util.WebResponseUtils;
@ExtendWith(MockitoExtension.class)
 class AttachmentControllerTest {

-    @Mock
-    private CustomPDFDocumentFactory pdfDocumentFactory;
+    @Mock private CustomPDFDocumentFactory pdfDocumentFactory;

-    @Mock
-    private AttachmentServiceInterface pdfAttachmentService;
+    @Mock private AttachmentServiceInterface pdfAttachmentService;

-    @InjectMocks
-    private AttachmentController attachmentController;
+    @InjectMocks private AttachmentController attachmentController;

    private MockMultipartFile pdfFile;
    private MockMultipartFile attachment1;
@@ -47,9 +43,15 @@ class AttachmentControllerTest {

    @BeforeEach
    void setUp() {
-        pdfFile = new MockMultipartFile("fileInput", "test.pdf", "application/pdf", "PDF content".getBytes());
-        attachment1 = new MockMultipartFile("attachment1", "file1.txt", "text/plain", "File 1 content".getBytes());
-        attachment2 = new MockMultipartFile("attachment2", "file2.jpg", "image/jpeg", "Image content".getBytes());
+        pdfFile =
+                new MockMultipartFile(
+                        "fileInput", "test.pdf", "application/pdf", "PDF content".getBytes());
+        attachment1 =
+                new MockMultipartFile(
+                        "attachment1", "file1.txt", "text/plain", "File 1 content".getBytes());
+        attachment2 =
+                new MockMultipartFile(
+                        "attachment2", "file2.jpg", "image/jpeg", "Image content".getBytes());
        request = new AddAttachmentRequest();
        mockDocument = mock(PDDocument.class);
        modifiedMockDocument = mock(PDDocument.class);
@@ -60,13 +62,21 @@ class AttachmentControllerTest {
        List<MultipartFile> attachments = List.of(attachment1, attachment2);
        request.setAttachments(attachments);
        request.setFileInput(pdfFile);
-        ResponseEntity<byte[]> expectedResponse = ResponseEntity.ok("modified PDF content".getBytes());
+        ResponseEntity<byte[]> expectedResponse =
+                ResponseEntity.ok("modified PDF content".getBytes());

        when(pdfDocumentFactory.load(pdfFile, false)).thenReturn(mockDocument);
-        when(pdfAttachmentService.addAttachment(mockDocument, attachments)).thenReturn(modifiedMockDocument);
+        when(pdfAttachmentService.addAttachment(mockDocument, attachments))
+                .thenReturn(modifiedMockDocument);

-        try (MockedStatic<WebResponseUtils> mockedWebResponseUtils = mockStatic(WebResponseUtils.class)) {
-            mockedWebResponseUtils.when(() -> WebResponseUtils.pdfDocToWebResponse(eq(modifiedMockDocument), eq("test_with_attachments.pdf")))
+        try (MockedStatic<WebResponseUtils> mockedWebResponseUtils =
+                mockStatic(WebResponseUtils.class)) {
+            mockedWebResponseUtils
+                    .when(
+                            () ->
+                                    WebResponseUtils.pdfDocToWebResponse(
+                                            eq(modifiedMockDocument),
+                                            eq("test_with_attachments.pdf")))
                    .thenReturn(expectedResponse);

            ResponseEntity<byte[]> response = attachmentController.addAttachments(request);
@@ -84,13 +94,21 @@ class AttachmentControllerTest {
        List<MultipartFile> attachments = List.of(attachment1);
        request.setAttachments(attachments);
        request.setFileInput(pdfFile);
-        ResponseEntity<byte[]> expectedResponse = ResponseEntity.ok("modified PDF content".getBytes());
+        ResponseEntity<byte[]> expectedResponse =
+                ResponseEntity.ok("modified PDF content".getBytes());

        when(pdfDocumentFactory.load(pdfFile, false)).thenReturn(mockDocument);
-        when(pdfAttachmentService.addAttachment(mockDocument, attachments)).thenReturn(modifiedMockDocument);
+        when(pdfAttachmentService.addAttachment(mockDocument, attachments))
+                .thenReturn(modifiedMockDocument);

-        try (MockedStatic<WebResponseUtils> mockedWebResponseUtils = mockStatic(WebResponseUtils.class)) {
-            mockedWebResponseUtils.when(() -> WebResponseUtils.pdfDocToWebResponse(eq(modifiedMockDocument), eq("test_with_attachments.pdf")))
+        try (MockedStatic<WebResponseUtils> mockedWebResponseUtils =
+                mockStatic(WebResponseUtils.class)) {
+            mockedWebResponseUtils
+                    .when(
+                            () ->
+                                    WebResponseUtils.pdfDocToWebResponse(
+                                            eq(modifiedMockDocument),
+                                            eq("test_with_attachments.pdf")))
                    .thenReturn(expectedResponse);

            ResponseEntity<byte[]> response = attachmentController.addAttachments(request);
@@ -20,11 +20,11 @@ import org.springframework.http.ResponseEntity;

 import jakarta.servlet.ServletContext;

-import stirling.software.common.service.UserServiceInterface;
 import stirling.software.SPDF.model.PipelineConfig;
 import stirling.software.SPDF.model.PipelineOperation;
 import stirling.software.SPDF.model.PipelineResult;
 import stirling.software.SPDF.service.ApiDocService;
+import stirling.software.common.service.UserServiceInterface;

@ExtendWith(MockitoExtension.class)
 class PipelineProcessorTest {
@@ -1,15 +1,17 @@
 package stirling.software.SPDF.service;

+import static org.junit.jupiter.api.Assertions.*;
+import static org.mockito.Mockito.mock;
+import static org.mockito.Mockito.when;
+
 import java.io.ByteArrayInputStream;
 import java.io.IOException;
 import java.util.List;
+
 import org.apache.pdfbox.pdmodel.PDDocument;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 import org.springframework.web.multipart.MultipartFile;
-import static org.junit.jupiter.api.Assertions.*;
-import static org.mockito.Mockito.mock;
-import static org.mockito.Mockito.when;

 class AttachmentServiceTest {

@@ -27,8 +29,8 @@ class AttachmentServiceTest {
            var attachments = List.of(mock(MultipartFile.class));

            when(attachments.get(0).getOriginalFilename()).thenReturn("test.txt");
-            when(attachments.get(0).getInputStream()).thenReturn(
-                new ByteArrayInputStream("Test content".getBytes()));
+            when(attachments.get(0).getInputStream())
+                    .thenReturn(new ByteArrayInputStream("Test content".getBytes()));
            when(attachments.get(0).getSize()).thenReturn(12L);
            when(attachments.get(0).getContentType()).thenReturn("text/plain");

@@ -49,14 +51,14 @@ class AttachmentServiceTest {
            var attachments = List.of(attachment1, attachment2);

            when(attachment1.getOriginalFilename()).thenReturn("document.pdf");
-            when(attachment1.getInputStream()).thenReturn(
-                new ByteArrayInputStream("PDF content".getBytes()));
+            when(attachment1.getInputStream())
+                    .thenReturn(new ByteArrayInputStream("PDF content".getBytes()));
            when(attachment1.getSize()).thenReturn(15L);
            when(attachment1.getContentType()).thenReturn("application/pdf");

            when(attachment2.getOriginalFilename()).thenReturn("image.jpg");
-            when(attachment2.getInputStream()).thenReturn(
-                new ByteArrayInputStream("Image content".getBytes()));
+            when(attachment2.getInputStream())
+                    .thenReturn(new ByteArrayInputStream("Image content".getBytes()));
            when(attachment2.getSize()).thenReturn(20L);
            when(attachment2.getContentType()).thenReturn("image/jpeg");

@@ -74,8 +76,8 @@ class AttachmentServiceTest {
            var attachments = List.of(mock(MultipartFile.class));

            when(attachments.get(0).getOriginalFilename()).thenReturn("image.jpg");
-            when(attachments.get(0).getInputStream()).thenReturn(
-                new ByteArrayInputStream("Image content".getBytes()));
+            when(attachments.get(0).getInputStream())
+                    .thenReturn(new ByteArrayInputStream("Image content".getBytes()));
            when(attachments.get(0).getSize()).thenReturn(25L);
            when(attachments.get(0).getContentType()).thenReturn("");

@@ -17,8 +17,8 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.io.TempDir;
 import org.mockito.MockedStatic;

-import stirling.software.common.configuration.InstallationPathConfig;
 import stirling.software.SPDF.model.SignatureFile;
+import stirling.software.common.configuration.InstallationPathConfig;

 class SignatureServiceTest {

@@ -12,36 +12,28 @@ import org.mockito.Mock;
 import org.mockito.MockitoAnnotations;
 import org.springframework.http.HttpStatus;
 import org.springframework.http.ResponseEntity;
-import org.springframework.mock.web.MockHttpServletRequest;
 import org.springframework.mock.web.MockHttpSession;
+
 import jakarta.servlet.http.HttpServletRequest;
-import jakarta.servlet.http.HttpSession;

 import stirling.software.common.model.job.JobResult;
-import stirling.software.common.model.job.JobStats;
-import stirling.software.common.model.job.ResultFile;
 import stirling.software.common.service.FileStorage;
 import stirling.software.common.service.JobQueue;
 import stirling.software.common.service.TaskManager;

 class JobControllerTest {

-    @Mock
-    private TaskManager taskManager;
+    @Mock private TaskManager taskManager;

-    @Mock
-    private FileStorage fileStorage;
+    @Mock private FileStorage fileStorage;

-    @Mock
-    private JobQueue jobQueue;
+    @Mock private JobQueue jobQueue;

-    @Mock
-    private HttpServletRequest request;
+    @Mock private HttpServletRequest request;

    private MockHttpSession session;

-    @InjectMocks
-    private JobController controller;
+    @InjectMocks private JobController controller;

    @BeforeEach
    void setUp() {
@@ -139,7 +131,8 @@ class JobControllerTest {

        JobResult mockResult = new JobResult();
        mockResult.setJobId(jobId);
-        mockResult.completeWithSingleFile(fileId, originalFileName, contentType, fileContent.length);
+        mockResult.completeWithSingleFile(
+                fileId, originalFileName, contentType, fileContent.length);

        when(taskManager.getJobResult(jobId)).thenReturn(mockResult);
        when(fileStorage.retrieveBytes(fileId)).thenReturn(fileContent);
@@ -150,7 +143,8 @@ class JobControllerTest {
        // Assert
        assertEquals(HttpStatus.OK, response.getStatusCode());
        assertEquals(contentType, response.getHeaders().getFirst("Content-Type"));
-        assertTrue(response.getHeaders().getFirst("Content-Disposition").contains(originalFileName));
+        assertTrue(
+                response.getHeaders().getFirst("Content-Disposition").contains(originalFileName));
        assertEquals(fileContent, response.getBody());
    }

@@ -229,45 +223,45 @@ class JobControllerTest {
        assertTrue(response.getBody().toString().contains("Error retrieving file"));
    }

-	/*
-	 * @Test void testGetJobStats() { // Arrange JobStats mockStats =
-	 * JobStats.builder() .totalJobs(10) .activeJobs(3) .completedJobs(7) .build();
-	 *
-	 * when(taskManager.getJobStats()).thenReturn(mockStats);
-	 *
-	 * // Act ResponseEntity<?> response = controller.getJobStats();
-	 *
-	 * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
-	 * assertEquals(mockStats, response.getBody()); }
-	 *
-	 * @Test void testCleanupOldJobs() { // Arrange when(taskManager.getJobStats())
-	 * .thenReturn(JobStats.builder().totalJobs(10).build())
-	 * .thenReturn(JobStats.builder().totalJobs(7).build());
-	 *
-	 * // Act ResponseEntity<?> response = controller.cleanupOldJobs();
-	 *
-	 * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
-	 *
-	 * @SuppressWarnings("unchecked") Map<String, Object> responseBody =
-	 * (Map<String, Object>) response.getBody(); assertEquals("Cleanup complete",
-	 * responseBody.get("message")); assertEquals(3,
-	 * responseBody.get("removedJobs")); assertEquals(7,
-	 * responseBody.get("remainingJobs"));
-	 *
-	 * verify(taskManager).cleanupOldJobs(); }
-	 *
-	 * @Test void testGetQueueStats() { // Arrange Map<String, Object>
-	 * mockQueueStats = Map.of( "queuedJobs", 5, "queueCapacity", 10,
-	 * "resourceStatus", "OK" );
-	 *
-	 * when(jobQueue.getQueueStats()).thenReturn(mockQueueStats);
-	 *
-	 * // Act ResponseEntity<?> response = controller.getQueueStats();
-	 *
-	 * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
-	 * assertEquals(mockQueueStats, response.getBody());
-	 * verify(jobQueue).getQueueStats(); }
-	 */
+    /*
+     * @Test void testGetJobStats() { // Arrange JobStats mockStats =
+     * JobStats.builder() .totalJobs(10) .activeJobs(3) .completedJobs(7) .build();
+     *
+     * when(taskManager.getJobStats()).thenReturn(mockStats);
+     *
+     * // Act ResponseEntity<?> response = controller.getJobStats();
+     *
+     * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
+     * assertEquals(mockStats, response.getBody()); }
+     *
+     * @Test void testCleanupOldJobs() { // Arrange when(taskManager.getJobStats())
+     * .thenReturn(JobStats.builder().totalJobs(10).build())
+     * .thenReturn(JobStats.builder().totalJobs(7).build());
+     *
+     * // Act ResponseEntity<?> response = controller.cleanupOldJobs();
+     *
+     * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
+     *
+     * @SuppressWarnings("unchecked") Map<String, Object> responseBody =
+     * (Map<String, Object>) response.getBody(); assertEquals("Cleanup complete",
+     * responseBody.get("message")); assertEquals(3,
+     * responseBody.get("removedJobs")); assertEquals(7,
+     * responseBody.get("remainingJobs"));
+     *
+     * verify(taskManager).cleanupOldJobs(); }
+     *
+     * @Test void testGetQueueStats() { // Arrange Map<String, Object>
+     * mockQueueStats = Map.of( "queuedJobs", 5, "queueCapacity", 10,
+     * "resourceStatus", "OK" );
+     *
+     * when(jobQueue.getQueueStats()).thenReturn(mockQueueStats);
+     *
+     * // Act ResponseEntity<?> response = controller.getQueueStats();
+     *
+     * // Assert assertEquals(HttpStatus.OK, response.getStatusCode());
+     * assertEquals(mockQueueStats, response.getBody());
+     * verify(jobQueue).getQueueStats(); }
+     */
    @Test
    void testCancelJob_InQueue() {
        // Arrange