Author: tilman
Date: Sat Apr 5 18:32:29 2025
New Revision: 1924821
URL: http://svn.apache.org/viewvc?rev=1924821&view=rev
Log:
PDFBOX-5660: Sonar fix
Modified:
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdfwriter/ContentStreamWriterTest.java
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/PDFBoxHeadlessTest.java
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/TestExtractText.java
Modified:
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdfwriter/ContentStreamWriterTest.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdfwriter/ContentStreamWriterTest.java?rev=1924821&r1=1924820&r2=1924821&view=diff
==============================================================================
---
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdfwriter/ContentStreamWriterTest.java
(original)
+++
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdfwriter/ContentStreamWriterTest.java
Sat Apr 5 18:32:29 2025
@@ -33,10 +33,7 @@ import org.apache.pdfbox.pdmodel.PDPage;
import org.apache.pdfbox.pdmodel.common.PDStream;
import org.apache.pdfbox.rendering.PDFRenderer;
import org.apache.pdfbox.rendering.TestPDFToImage;
-import org.junit.jupiter.api.AfterAll;
-import org.junit.jupiter.api.AfterEach;
import org.junit.jupiter.api.BeforeAll;
-import org.junit.jupiter.api.BeforeEach;
import org.junit.jupiter.api.Test;
import org.junit.jupiter.api.parallel.Execution;
import org.junit.jupiter.api.parallel.ExecutionMode;
@@ -48,39 +45,21 @@ import org.junit.jupiter.api.parallel.Ex
@Execution(ExecutionMode.SAME_THREAD)
class ContentStreamWriterTest
{
- private final File testDirIn = new
File("target/test-output/contentstream/in");
- private final File testDirOut = new
File("target/test-output/contentstream/out");
-
- ContentStreamWriterTest()
- {
- testDirIn.mkdirs();
- testDirOut.mkdirs();
- }
+ private static final File TESTDIRIN = new
File("target/test-output/contentstream/in");
+ private static final File TESTDIROUT = new
File("target/test-output/contentstream/out");
@BeforeAll
- public static void setUpClass()
+ static void setUp()
{
+ TESTDIRIN.mkdirs();
+ TESTDIROUT.mkdirs();
+
// PDFBOX-5425: try to avoid "java.awt.color.CMMException: Unknown
profile ID"
ColorSpace csRGB = ColorSpace.getInstance(ColorSpace.CS_sRGB);
csRGB.toRGB(new float[] { 0, 0, 0 });
ColorSpace csXYZ = ColorSpace.getInstance(ColorSpace.CS_CIEXYZ);
csXYZ.toRGB(new float[] { 0, 0, 0 });
}
-
- @AfterAll
- public static void tearDownClass()
- {
- }
-
- @BeforeEach
- public void setUp()
- {
- }
-
- @AfterEach
- public void tearDown()
- {
- }
/**
* Test parse content stream, write back tokens and compare rendering.
@@ -98,7 +77,7 @@ class ContentStreamWriterTest
for (int i = 0; i < doc.getNumberOfPages(); ++i)
{
BufferedImage bim1 = r.renderImageWithDPI(i, 96);
- ImageIO.write(bim1, "png", new File(testDirIn, filename + "-"
+ (i + 1) + ".png"));
+ ImageIO.write(bim1, "png", new File(TESTDIRIN, filename + "-"
+ (i + 1) + ".png"));
PDPage page = doc.getPage(i);
PDStream newContent = new PDStream(doc);
try (OutputStream os =
newContent.createOutputStream(COSName.FLATE_DECODE))
@@ -109,12 +88,12 @@ class ContentStreamWriterTest
}
page.setContents(newContent);
}
- doc.save(new File(testDirIn, filename));
+ doc.save(new File(TESTDIRIN, filename));
}
- if (!TestPDFToImage.doTestFile(new File(testDirIn, filename),
testDirIn.getAbsolutePath(),
- testDirOut.getAbsolutePath()))
+ if (!TestPDFToImage.doTestFile(new File(TESTDIRIN, filename),
TESTDIRIN.getAbsolutePath(),
+ TESTDIROUT.getAbsolutePath()))
{
- fail("Rendering failed or is not identical, see in " + testDirOut);
+ fail("Rendering failed or is not identical, see in " + TESTDIROUT);
}
}
}
Modified:
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java?rev=1924821&r1=1924820&r2=1924821&view=diff
==============================================================================
---
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java
(original)
+++
pdfbox/trunk/pdfbox/src/test/java/org/apache/pdfbox/pdmodel/TestPDDocument.java
Sat Apr 5 18:32:29 2025
@@ -46,7 +46,7 @@ class TestPDDocument
private static final File TESTRESULTSDIR = new File("target/test-output");
@BeforeAll
- public static void setUp() throws Exception
+ static void setUp() throws Exception
{
TESTRESULTSDIR.mkdirs();
}
Modified:
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/PDFBoxHeadlessTest.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/PDFBoxHeadlessTest.java?rev=1924821&r1=1924820&r2=1924821&view=diff
==============================================================================
---
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/PDFBoxHeadlessTest.java
(original)
+++
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/PDFBoxHeadlessTest.java
Sat Apr 5 18:32:29 2025
@@ -41,13 +41,13 @@ class PDFBoxHeadlessTest
final ByteArrayOutputStream err = new ByteArrayOutputStream();
@BeforeAll
- public static void setHeadless()
+ static void setUp()
{
System.setProperty("java.awt.headless", "true");
- }
+ }
@BeforeEach
- public void setUpStreams()
+ void setUpStreams()
{
out.reset();
err.reset();
@@ -56,7 +56,7 @@ class PDFBoxHeadlessTest
}
@AfterEach
- public void restoreStreams()
+ void restoreStreams()
{
System.setOut(originalOut);
System.setErr(originalErr);
Modified:
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/TestExtractText.java
URL:
http://svn.apache.org/viewvc/pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/TestExtractText.java?rev=1924821&r1=1924820&r2=1924821&view=diff
==============================================================================
---
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/TestExtractText.java
(original)
+++
pdfbox/trunk/tools/src/test/java/org/apache/pdfbox/tools/TestExtractText.java
Sat Apr 5 18:32:29 2025
@@ -54,7 +54,7 @@ class TestExtractText
static String filename2 = null;
@BeforeAll
- public static void setupFilenames()
+ static void setupFilenames()
{
// the filename representation is platform dependent
filename1 = Paths.get(TESTFILE1).toString();
@@ -62,7 +62,7 @@ class TestExtractText
}
@BeforeEach
- public void setUpStreams()
+ void setUpStreams()
{
out.reset();
try
