svn commit: r1935280 - in pdfbox/branches/2.0: pdfbox/src/test/java/org/apache/pdfbox/cos preflight/src/test/java/org/apache/pdfbox/preflight/utils

[tilman]

Author: tilman
Date: Sun Jun 14 09:17:50 2026
New Revision: 1935280

Log:
PDFBOX-5660: Sonar fix

Modified:
   
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSUpdateInfo.java
   
pdfbox/branches/2.0/preflight/src/test/java/org/apache/pdfbox/preflight/utils/TestCOSUtils.java

Modified: 
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSUpdateInfo.java
==============================================================================
--- 
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSUpdateInfo.java
       Sun Jun 14 09:03:08 2026        (r1935279)
+++ 
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSUpdateInfo.java
       Sun Jun 14 09:17:50 2026        (r1935280)
@@ -19,7 +19,6 @@ package org.apache.pdfbox.cos;
 
 import static org.junit.Assert.assertTrue;
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.fail;
 
 import java.io.IOException;
 
@@ -33,9 +32,11 @@ public class TestCOSUpdateInfo
     
     /**
      * Tests isNeedToBeUpdate() and setNeedToBeUpdate() - tests the 
getter/setter methods.
+     * 
+     * @throws IOException
      */
     @Test
-    public void testIsSetNeedToBeUpdate()
+    public void testIsSetNeedToBeUpdate() throws IOException
     {
         // COSDictionary
         COSUpdateInfo testCOSDictionary = new COSDictionary();
@@ -46,18 +47,11 @@ public class TestCOSUpdateInfo
 
         // COSObject
         COSUpdateInfo testCOSObject;
-        try
-        {
-            testCOSObject = new COSObject(null);
-            testCOSObject.setNeedToBeUpdated(true);
-            assertTrue(testCOSObject.isNeedToBeUpdated());
-            testCOSObject.setNeedToBeUpdated(false);
-            assertFalse(testCOSObject.isNeedToBeUpdated());
-        }
-        catch (IOException e)
-        {
-            fail(e.getMessage());
-        }
+        testCOSObject = new COSObject(null);
+        testCOSObject.setNeedToBeUpdated(true);
+        assertTrue(testCOSObject.isNeedToBeUpdated());
+        testCOSObject.setNeedToBeUpdated(false);
+        assertFalse(testCOSObject.isNeedToBeUpdated());
     }
 
 }

Modified: 
pdfbox/branches/2.0/preflight/src/test/java/org/apache/pdfbox/preflight/utils/TestCOSUtils.java
==============================================================================
--- 
pdfbox/branches/2.0/preflight/src/test/java/org/apache/pdfbox/preflight/utils/TestCOSUtils.java
     Sun Jun 14 09:03:08 2026        (r1935279)
+++ 
pdfbox/branches/2.0/preflight/src/test/java/org/apache/pdfbox/preflight/utils/TestCOSUtils.java
     Sun Jun 14 09:17:50 2026        (r1935280)
@@ -22,7 +22,6 @@
 package org.apache.pdfbox.preflight.utils;
 
 import static org.junit.Assert.assertFalse;
-import static org.junit.Assert.fail;
 
 import java.io.File;
 import java.io.IOException;
@@ -148,16 +147,9 @@ public class TestCOSUtils
     }
 
     @Test
-    public void testCloseCOSDocumentIO()
+    public void testCloseCOSDocumentIO() throws IOException
     {
-        try
-        {
-            COSUtils.closeDocumentQuietly(new IOCOSDocument());
-        }
-        catch (IOException e)
-        {
-            fail(e.getMessage());
-        }
+        COSUtils.closeDocumentQuietly(new IOCOSDocument());
     }
 
     protected void addToXref(COSDocument doc, COSObjectKey key, long value)