PDFBOX-5660: Sonar fix

git-svn-id: https://svn.apache.org/repos/asf/pdfbox/trunk@1935320 13f79535-47bb-0310-9956-ffa450edef68

Author: THausherr

pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java

@@ -34,7 +34,6 @@
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertThrows;
 import static org.junit.jupiter.api.Assertions.assertTrue;
-import static org.junit.jupiter.api.Assertions.fail;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
@@ -205,15 +204,8 @@ void testDeleteGoodFile() throws IOException
         }
 
         Loader.loadPDF(f).close();
-        
-        try
-        {
-            Files.delete(f.toPath());
-        }
-        catch (IOException ex)
-        {
-            fail("delete good file failed after successful load() and close()");
-        }
+        assertDoesNotThrow(() -> Files.delete(f.toPath()), 
+                "delete good file failed after successful load() and close()");
     }
 
     /**
@@ -237,7 +229,7 @@ void testSaveArabicLocale() throws IOException
             doc.save(baos);
         }
 
-        Loader.loadPDF(baos.toByteArray()).close();
+        assertDoesNotThrow(() -> Loader.loadPDF(baos.toByteArray()).close());
 
         Locale.setDefault(defaultLocale);
     }

pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocumentCatalog.java

@@ -26,6 +26,7 @@
 import org.apache.pdfbox.io.RandomAccessReadBuffer;
 import org.apache.pdfbox.pdmodel.graphics.color.PDOutputIntent;
 
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 import static org.junit.jupiter.api.Assertions.assertNull;
 import static org.junit.jupiter.api.Assertions.assertTrue;
@@ -90,7 +91,7 @@ void retrievePageLabelsOnMalformedPdf() throws IOException
         {
             PDDocumentCatalog cat = doc.getDocumentCatalog();
             // getLabelsByPageIndices() should not throw an exception
-            cat.getPageLabels().getLabelsByPageIndices();
+            assertDoesNotThrow(() -> cat.getPageLabels().getLabelsByPageIndices());
         }
     }
 

pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDPage.java

@@ -24,6 +24,7 @@
 import org.apache.pdfbox.pdmodel.interactive.annotation.PDAnnotationWidget;
 import org.apache.pdfbox.pdmodel.interactive.form.PDAcroForm;
 import org.apache.pdfbox.pdmodel.interactive.form.PDTextField;
+import static org.junit.jupiter.api.Assertions.assertDoesNotThrow;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
 import org.junit.jupiter.api.Test;
@@ -51,7 +52,7 @@ void testAddingPageAfterCreatingAnnotation() throws IOException
             acroForm.getFields().add(textField);
 
             // Adding page AFTER creating form fields causes a StackOverflowError
-            document.addPage(page);
+            assertDoesNotThrow(() -> document.addPage(page));
 
             document.save(OutputStream.nullOutputStream());
             document.close();