You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pdfbox.apache.org by ti...@apache.org on 2020/09/07 18:21:30 UTC
svn commit: r1881543 -
/pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java
Author: tilman
Date: Mon Sep 7 18:21:30 2020
New Revision: 1881543
URL: http://svn.apache.org/viewvc?rev=1881543&view=rev
Log:
PDFBOX-4892: SonarQube fix
Modified:
pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java
Modified: pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java?rev=1881543&r1=1881542&r2=1881543&view=diff
==============================================================================
--- pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java (original)
+++ pdfbox/branches/2.0/pdfbox/src/test/java/org/apache/pdfbox/multipdf/PDFMergerUtilityTest.java Mon Sep 7 18:21:30 2020
@@ -54,6 +54,7 @@ import org.apache.pdfbox.pdmodel.interac
import org.apache.pdfbox.pdmodel.interactive.form.PDAcroForm;
import org.apache.pdfbox.pdmodel.interactive.form.PDField;
import org.apache.pdfbox.rendering.PDFRenderer;
+import org.junit.Assert;
/**
* Test suite for PDFMergerUtility.
@@ -818,12 +819,12 @@ public class PDFMergerUtilityTest extend
COSBase item = kdict.getItem(COSName.OBJ);
if (item instanceof COSObject)
{
- assertTrue("Annotation page is not in the page tree: " + item, pageTree.indexOf(page) != -1);
+ Assert.assertNotEquals("Annotation page is not in the page tree: " + item, -1, pageTree.indexOf(page));
}
else
{
// don't display because of stack overflow
- assertTrue("Annotation page is not in the page tree", pageTree.indexOf(page) != -1);
+ Assert.assertNotEquals("Annotation page is not in the page tree", -1, pageTree.indexOf(page));
}
}
}
@@ -909,7 +910,7 @@ public class PDFMergerUtilityTest extend
PDPage page = structureElement.getPage();
if (page != null)
{
- assertTrue("Page is not in the page tree", pageTree.indexOf(page) != -1);
+ Assert.assertNotEquals("Page is not in the page tree", -1, pageTree.indexOf(page));
}
}
}