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/09 18:38:05 UTC
svn commit: r1881595 - in
/pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos:
TestCOSFloat.java TestCOSString.java
Author: tilman
Date: Wed Sep 9 18:38:04 2020
New Revision: 1881595
URL: http://svn.apache.org/viewvc?rev=1881595&view=rev
Log:
PDFBOX-4892: SonarQube fix
Modified:
pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java
pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSString.java
Modified: pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java?rev=1881595&r1=1881594&r2=1881595&view=diff
==============================================================================
--- pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java (original)
+++ pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSFloat.java Wed Sep 9 18:38:04 2020
@@ -110,6 +110,8 @@ public class TestCOSFloat extends TestCO
@Override
void runTest(float num)
{
+ @SuppressWarnings({"java:S5863"}) // don't flag tests for reflexivity
+
COSFloat test1 = new COSFloat(num);
COSFloat test2 = new COSFloat(num);
COSFloat test3 = new COSFloat(num);
Modified: pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSString.java
URL: http://svn.apache.org/viewvc/pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSString.java?rev=1881595&r1=1881594&r2=1881595&view=diff
==============================================================================
--- pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSString.java (original)
+++ pdfbox/branches/issue45/pdfbox/src/test/java/org/apache/pdfbox/cos/TestCOSString.java Wed Sep 9 18:38:04 2020
@@ -335,9 +335,13 @@ public class TestCOSString extends TestC
/**
* Test testCompareFromHexString() - tests that Strings created from hex
* compare correctly (PDFBOX-2401)
+ *
+ * @throws java.io.IOException
*/
public void testCompareFromHexString() throws IOException
{
+ @SuppressWarnings({"java:S5863"}) // don't flag tests for reflexivity
+
COSString test1 = COSString.parseHex("000000FF000000");
COSString test2 = COSString.parseHex("000000FF00FFFF");
Assert.assertEquals(test1, test1);