Sonar fixes

JUnit5 test classes and methods should have default package visibility

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

src/ooxml/testcases/org/apache/poi/extractor/ooxml/TestExtractorFactory.java

@@ -58,7 +58,7 @@ import org.junit.jupiter.params.provider.ValueSource;
 /**
  * Test that the extractor factory plays nicely
  */
-public class TestExtractorFactory {
+class TestExtractorFactory {
 
     private static final POIDataSamples ssTests = POIDataSamples.getSpreadSheetInstance();
     private static final File xls = getFileAndCheck(ssTests, "SampleSS.xls");

src/ooxml/testcases/org/apache/poi/ooxml/TestDetectAsOOXML.java

@@ -40,7 +40,7 @@ import org.junit.jupiter.params.provider.CsvSource;
  *  documents
  */
 @SuppressWarnings("deprecation")
-public class TestDetectAsOOXML {
+class TestDetectAsOOXML {
     @Test
 	void testOpensProperly() throws IOException, InvalidFormatException {
     	try (InputStream is = openSampleFileStream("sample.xlsx");

src/ooxml/testcases/org/apache/poi/ooxml/TestEmbedded.java

@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Class to test that we handle embeded bits in OOXML files properly
  */
-public class TestEmbedded {
+class TestEmbedded {
 	@Test
 	void testExcel() throws Exception {
 		POIXMLDocument doc = new XSSFWorkbook(

src/ooxml/testcases/org/apache/poi/ooxml/TestTriggerCoverage.java

@@ -56,7 +56,7 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
  * Test to trigger code-execution of various parts so
  * that all required elements are inclueded in the ooxml-schema-lite package
  */
-public class TestTriggerCoverage {
+class TestTriggerCoverage {
     private static final Set<String> FAILING = new HashSet<>();
     static {
         FAILING.add("stress025.docx");

src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestPackagingURIHelper.java

@@ -24,7 +24,7 @@ import java.net.URI;
 import org.apache.poi.openxml4j.exceptions.InvalidFormatException;
 import org.junit.jupiter.api.Test;
 
-public class TestPackagingURIHelper {
+class TestPackagingURIHelper {
 
     /**
      * Test relativizePartName() method.

src/ooxml/testcases/org/apache/poi/openxml4j/opc/TestRelationships.java

@@ -36,7 +36,7 @@ import org.apache.poi.util.POILogger;
 import org.junit.jupiter.api.Test;
 
 
-public class TestRelationships {
+class TestRelationships {
 	private static final String HYPERLINK_REL_TYPE =
 		"http://schemas.openxmlformats.org/officeDocument/2006/relationships/hyperlink";
 	private static final String COMMENTS_REL_TYPE =

src/ooxml/testcases/org/apache/poi/openxml4j/opc/compliance/TestOPCCompliancePackageModel.java

@@ -44,7 +44,7 @@ import org.junit.jupiter.api.Test;
  * M1.11 : A package implementer shall neither create nor recognize a part with
  * a part name derived from another part name by appending segments to it.
  */
-public class TestOPCCompliancePackageModel {
+class TestOPCCompliancePackageModel {
 
     private static File TESTFILE;
 

src/ooxml/testcases/org/apache/poi/openxml4j/opc/internal/marshallers/TestZipPackagePropertiesMarshaller.java

@@ -34,7 +34,7 @@ import org.apache.poi.openxml4j.opc.internal.PackagePropertiesPart;
 import org.apache.poi.openxml4j.opc.internal.PartMarshaller;
 import org.junit.jupiter.api.Test;
 
-public class TestZipPackagePropertiesMarshaller {
+class TestZipPackagePropertiesMarshaller {
     private final PartMarshaller marshaller = new ZipPackagePropertiesMarshaller();
 
     private boolean marshall() throws OpenXML4JException {

src/ooxml/testcases/org/apache/poi/openxml4j/util/TestZipSecureFile.java

@@ -27,7 +27,7 @@ import java.util.Enumeration;
 
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class TestZipSecureFile {
+class TestZipSecureFile {
     @Test
     void testThresholdInputStream() throws Exception {
         // This fails in Java 10 because our reflection injection of the ThresholdInputStream causes a

src/ooxml/testcases/org/apache/poi/poifs/crypt/dsig/TestSignatureConfig.java

@@ -25,7 +25,7 @@ import org.apache.poi.poifs.crypt.HashAlgorithm;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-public class TestSignatureConfig {
+class TestSignatureConfig {
 
     @Test
     @Disabled("failing in automated builds, due to issues loading security classes")

src/ooxml/testcases/org/apache/poi/poifs/crypt/dsig/TestSignatureInfo.java

@@ -161,7 +161,7 @@ import org.w3.x2000.x09.xmldsig.ReferenceType;
 import org.w3.x2000.x09.xmldsig.SignatureDocument;
 import org.w3c.dom.Document;
 
-public class TestSignatureInfo {
+class TestSignatureInfo {
     private static final POILogger LOG = POILogFactory.getLogger(TestSignatureInfo.class);
     private static final POIDataSamples testdata = POIDataSamples.getXmlDSignInstance();
 

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestAgileEncryptionParameters.java

@@ -45,7 +45,7 @@ import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 
-public class TestAgileEncryptionParameters {
+class TestAgileEncryptionParameters {
 
     static byte[] testData;
 

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestDecryptor.java

@@ -44,7 +44,7 @@ import org.apache.poi.poifs.filesystem.POIFSFileSystem;
 import org.apache.poi.util.IOUtils;
 import org.junit.jupiter.api.Test;
 
-public class TestDecryptor {
+class TestDecryptor {
     private static final POIDataSamples samples = POIDataSamples.getPOIFSInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestEncryptionInfo.java

@@ -28,7 +28,7 @@ import org.apache.poi.poifs.crypt.HashAlgorithm;
 import org.apache.poi.poifs.filesystem.POIFSFileSystem;
 import org.junit.jupiter.api.Test;
 
-public class TestEncryptionInfo {
+class TestEncryptionInfo {
     @Test
     void testEncryptionInfo() throws IOException {
         POIFSFileSystem fs = new POIFSFileSystem(POIDataSamples.getPOIFSInstance().openResourceAsStream("protect.xlsx"));

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestEncryptor.java

@@ -66,7 +66,7 @@ import org.apache.poi.xwpf.usermodel.XWPFParagraph;
 import org.junit.jupiter.api.Disabled;
 import org.junit.jupiter.api.Test;
 
-public class TestEncryptor {
+class TestEncryptor {
     @Test
     void binaryRC4Encryption() throws Exception {
         // please contribute a real sample file, which is binary rc4 encrypted

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestHxxFEncryption.java

@@ -44,7 +44,7 @@ import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 
-public class TestHxxFEncryption {
+class TestHxxFEncryption {
     public static Stream<Arguments> data() throws IOException {
         final String base64 =
             "H4sIAAAAAAAAAF1Uu24bMRDs/RULVwkgCUhSpHaZwkDgpHJH8fZ0G/Nx4ZI6y13yG/mRfIb9R5mlZFlIpdPtcnZmdnjPf57/vvx6+f3h6obuv3"+

src/ooxml/testcases/org/apache/poi/poifs/crypt/tests/TestSecureTempZip.java

@@ -43,7 +43,7 @@ import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.apache.xmlbeans.XmlException;
 import org.junit.jupiter.api.Test;
 
-public class TestSecureTempZip {
+class TestSecureTempZip {
 
     /**
      * Test case for #59841 - this is an example on how to use encrypted temp files,

src/ooxml/testcases/org/apache/poi/sl/tests/TestFonts.java

@@ -53,7 +53,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Test rendering - specific to font handling
  */
-public class TestFonts {
+class TestFonts {
     private static final POIDataSamples _slTests = POIDataSamples.getSlideShowInstance();
 
     private static final String JPTEXT =

src/ooxml/testcases/org/apache/poi/sl/tests/TestHeadersFooters.java

@@ -33,7 +33,7 @@ import org.apache.poi.sl.usermodel.TextParagraph;
 import org.apache.poi.sl.usermodel.TextShape;
 import org.junit.jupiter.api.Test;
 
-public class TestHeadersFooters {
+class TestHeadersFooters {
     @Test
     void bug58144c() throws IOException {
         SlideShow<?,?> ppt = openSampleSlideshow("bug58144-headers-footers-2007.pptx");

src/ooxml/testcases/org/apache/poi/sl/tests/TestOleShape.java

@@ -61,7 +61,7 @@ import org.junit.jupiter.params.ParameterizedTest;
 import org.junit.jupiter.params.provider.Arguments;
 import org.junit.jupiter.params.provider.MethodSource;
 
-public class TestOleShape {
+class TestOleShape {
     private static final String PDF_SAMPLE =
         "H4sIAAAAAAAAAJWUezRUWxzHe+o2FXncVtxLpxi3FPOeKYspjMdM5J1S4TTOaDIzxzpzJo9CUrnrSiUxIeT" +
         "9jB7yqInihrhepTwqt1AT5VZCC7XcY0LWcv+5Z521zz6fvX+/vb/7t9cX78CyMiQZ0XD4W4OFEzgKQATgg4" +

src/ooxml/testcases/org/apache/poi/sl/tests/TestSlide.java

@@ -35,7 +35,7 @@ import org.apache.poi.sl.usermodel.SlideShowFactory;
 import org.apache.poi.xslf.usermodel.XMLSlideShow;
 import org.junit.jupiter.api.Test;
 
-public class TestSlide {
+class TestSlide {
 
     @Test
     void hideHSLF() throws IOException {

src/ooxml/testcases/org/apache/poi/sl/tests/TestTable.java

@@ -43,7 +43,7 @@ import org.apache.poi.sl.usermodel.TextShape.TextDirection;
 import org.apache.poi.xslf.usermodel.XMLSlideShow;
 import org.junit.jupiter.api.Test;
 
-public class TestTable {
+class TestTable {
 
     @Test
     void colWidthRowHeight() throws IOException {

src/ooxml/testcases/org/apache/poi/sl/tests/draw/TestDrawPictureShape.java

@@ -40,7 +40,7 @@ import org.apache.poi.util.Units;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
-public class TestDrawPictureShape {
+class TestDrawPictureShape {
     final static POIDataSamples ssSamples = POIDataSamples.getSlideShowInstance();
 
     private static boolean xslfOnly;

src/ooxml/testcases/org/apache/poi/ss/tests/extractor/TestEmbeddedExtractor.java

@@ -40,7 +40,7 @@ import org.apache.poi.ss.usermodel.Workbook;
 import org.apache.poi.ss.usermodel.WorkbookFactory;
 import org.junit.jupiter.api.Test;
 
-public class TestEmbeddedExtractor {
+class TestEmbeddedExtractor {
     private static final POIDataSamples samples = POIDataSamples.getSpreadSheetInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/ss/tests/format/TestCellFormatPart.java

@@ -54,7 +54,7 @@ import org.junit.jupiter.api.Test;
  * flag "Categories" is not empty, only tests that have at least one category
  * listed in "Categories" are run.
  */
-public class TestCellFormatPart {
+class TestCellFormatPart {
     private static final Pattern NUMBER_EXTRACT_FMT = Pattern.compile(
             "([-+]?[0-9]+)(\\.[0-9]+)?.*(?:(e).*?([+-]?[0-9]+))",
             Pattern.CASE_INSENSITIVE);

src/ooxml/testcases/org/apache/poi/ss/tests/formula/TestFormulaParser.java

@@ -46,7 +46,7 @@ import org.junit.jupiter.api.Test;
  *
  * @author David North
  */
-public class TestFormulaParser {
+class TestFormulaParser {
 
     @Test
     void testHSSFFailsForOver65536() {

src/ooxml/testcases/org/apache/poi/ss/tests/formula/TestStructuredReferences.java

@@ -42,7 +42,7 @@ import org.junit.jupiter.api.Test;
  *         Excel Structured Reference Syntax
  *      </a>
  */
-public class TestStructuredReferences {
+class TestStructuredReferences {
 
     /**
      * Test the regular expression used in INDIRECT() evaluation to recognize structured references

src/ooxml/testcases/org/apache/poi/ss/tests/formula/functions/TestSumifsXSSF.java

@@ -33,7 +33,7 @@ import org.junit.jupiter.api.Test;
 /**
  *
  */
-public class TestSumifsXSSF {
+class TestSumifsXSSF {
 
     /**
      * handle null cell predicate

src/ooxml/testcases/org/apache/poi/ss/tests/formula/functions/TestVlookup.java

@@ -37,7 +37,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Test the VLOOKUP function
  */
-public class TestVlookup {
+class TestVlookup {
 
     @Test
     void testFullColumnAreaRef61841() throws IOException {

src/ooxml/testcases/org/apache/poi/ss/tests/usermodel/TestEmbedOLEPackage.java

@@ -65,7 +65,7 @@ import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.Test;
 
-public class TestEmbedOLEPackage {
+class TestEmbedOLEPackage {
     private static byte[] samplePPT, samplePPTX, samplePNG;
 
     private static final POIDataSamples ssamples = POIDataSamples.getSpreadSheetInstance();

src/ooxml/testcases/org/apache/poi/ss/tests/usermodel/TestXSSFRangeCopier.java

@@ -35,7 +35,7 @@ import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-public class TestXSSFRangeCopier extends BaseTestRangeCopier {
+class TestXSSFRangeCopier extends BaseTestRangeCopier {
     public TestXSSFRangeCopier() {
         super();
         workbook = new XSSFWorkbook();

src/ooxml/testcases/org/apache/poi/ss/tests/util/TestSXSSFCellUtil.java

@@ -20,7 +20,7 @@ package org.apache.poi.ss.tests.util;
 import org.apache.poi.ss.util.BaseTestCellUtil;
 import org.apache.poi.xssf.SXSSFITestDataProvider;
 
-public class TestSXSSFCellUtil extends BaseTestCellUtil {
+class TestSXSSFCellUtil extends BaseTestCellUtil {
     public TestSXSSFCellUtil() {
         super(SXSSFITestDataProvider.instance);
     }

src/ooxml/testcases/org/apache/poi/ss/tests/util/TestXSSFCellUtil.java

@@ -20,7 +20,7 @@ package org.apache.poi.ss.tests.util;
 import org.apache.poi.ss.util.BaseTestCellUtil;
 import org.apache.poi.xssf.XSSFITestDataProvider;
 
-public class TestXSSFCellUtil extends BaseTestCellUtil {
+class TestXSSFCellUtil extends BaseTestCellUtil {
     public TestXSSFCellUtil() {
         super(XSSFITestDataProvider.instance);
     }

src/ooxml/testcases/org/apache/poi/ss/tests/util/TestXSSFPropertyTemplate.java

@@ -35,7 +35,7 @@ import org.apache.poi.ss.util.PropertyTemplate;
 import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 
-public class TestXSSFPropertyTemplate {
+class TestXSSFPropertyTemplate {
 
     @Test
     void applyBorders() throws IOException {

src/ooxml/testcases/org/apache/poi/util/tests/TestIdentifierManager.java

@@ -26,7 +26,7 @@ import static org.junit.jupiter.api.Assertions.assertTrue;
 import org.apache.poi.ooxml.util.IdentifierManager;
 import org.junit.jupiter.api.Test;
 
-public class TestIdentifierManager {
+class TestIdentifierManager {
     @Test
     void testBasic()
     {

src/ooxml/testcases/org/apache/poi/util/tests/TestTempFileThreaded.java

@@ -40,7 +40,7 @@ import org.junit.jupiter.api.BeforeAll;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-public class TestTempFileThreaded {
+class TestTempFileThreaded {
     private static final int NUMBER_OF_THREADS = 10;
     private static final int NUMBER_OF_TESTS = 200;
 

src/ooxml/testcases/org/apache/poi/xddf/usermodel/TestNecessaryOOXMLClasses.java

@@ -29,7 +29,7 @@ import org.openxmlformats.schemas.drawingml.x2006.chart.*;
 import org.openxmlformats.schemas.drawingml.x2006.main.*;
 
 // aim is to get these classes loaded and included in poi-ooxml-lite.jar
-public class TestNecessaryOOXMLClasses {
+class TestNecessaryOOXMLClasses {
 
     @Test
     void testProblemClasses() {

src/ooxml/testcases/org/apache/poi/xddf/usermodel/TestXDDFColor.java

@@ -37,7 +37,7 @@ import org.xmlunit.builder.DiffBuilder;
 import org.xmlunit.builder.Input;
 import org.xmlunit.diff.Diff;
 
-public class TestXDDFColor {
+class TestXDDFColor {
     private static final String XMLNS = "xmlns:a=\"http://schemas.openxmlformats.org/drawingml/2006/main\"/>";
 
     @Test

src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChart.java

@@ -26,7 +26,7 @@ import org.apache.poi.ooxml.POIXMLRelation;
 import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.drawingml.x2006.chart.CTChartSpace;
 
-public class TestXDDFChart {
+class TestXDDFChart {
     @Test
     void testConstruct() {
         // minimal test to cause ooxml-lite to include all the classes in poi-ooxml-lite

src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChartExtensionList.java

@@ -21,7 +21,7 @@ import static org.junit.jupiter.api.Assertions.assertNotNull;
 
 import org.junit.jupiter.api.Test;
 
-public class TestXDDFChartExtensionList {
+class TestXDDFChartExtensionList {
 
     @Test
     void getXmlObject() {

src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFChartRemoveSeries.java

@@ -37,7 +37,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Test case for bug 63153
  */
-public class TestXDDFChartRemoveSeries {
+class TestXDDFChartRemoveSeries {
     final File resultDir = new File("build/custom-reports-test");
     String procName = null;
     String fileName = null;

src/ooxml/testcases/org/apache/poi/xddf/usermodel/chart/TestXDDFDataSourcesFactory.java

@@ -31,7 +31,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Tests for {@link XDDFDataSourcesFactory}.
  */
-public class TestXDDFDataSourcesFactory {
+class TestXDDFDataSourcesFactory {
 
     private static final Object[][] numericCells = {
             {0.0,      1.0,       2.0,     3.0,      4.0},

src/ooxml/testcases/org/apache/poi/xddf/usermodel/text/TestXDDFTextBodyProperties.java

@@ -26,7 +26,7 @@ import org.apache.poi.util.Units;
 import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.drawingml.x2006.main.CTTextBodyProperties;
 
-public class TestXDDFTextBodyProperties {
+class TestXDDFTextBodyProperties {
 
     @Test
     void testProperties() throws IOException {

src/ooxml/testcases/org/apache/poi/xddf/usermodel/text/TestXDDFTextRun.java

@@ -40,7 +40,7 @@ import org.apache.poi.xssf.usermodel.XSSFTextBox;
 import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 
-public class TestXDDFTextRun {
+class TestXDDFTextRun {
 
     @Test
     void testTextRunPropertiesInSlide() throws IOException {

src/ooxml/testcases/org/apache/poi/xdgf/extractor/TestXDGFVisioExtractor.java

@@ -29,7 +29,7 @@ import org.junit.jupiter.api.AfterEach;
 import org.junit.jupiter.api.BeforeEach;
 import org.junit.jupiter.api.Test;
 
-public class TestXDGFVisioExtractor {
+class TestXDGFVisioExtractor {
 
     private static final POIDataSamples SAMPLES = POIDataSamples.getDiagramInstance();
     private OPCPackage pkg;

src/ooxml/testcases/org/apache/poi/xslf/TestNecessaryOOXMLClasses.java

@@ -27,7 +27,7 @@ import org.apache.xmlbeans.XmlObject;
 import org.junit.jupiter.api.Test;
 
 // aim is to get these classes loaded and included in poi-ooxml-lite.jar
-public class TestNecessaryOOXMLClasses {
+class TestNecessaryOOXMLClasses {
 
     @Test
     void testProblemClasses() {

src/ooxml/testcases/org/apache/poi/xslf/TestXSLFBugs.java

@@ -112,7 +112,7 @@ import org.junit.jupiter.api.function.ThrowingSupplier;
 import org.openxmlformats.schemas.drawingml.x2006.main.CTOuterShadowEffect;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTShape;
 
-public class TestXSLFBugs {
+class TestXSLFBugs {
     private static final POIDataSamples slTests = POIDataSamples.getSlideShowInstance();
 
     private static boolean xslfOnly;

src/ooxml/testcases/org/apache/poi/xslf/TestXSLFSlideShow.java

@@ -42,7 +42,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.CTSlideMasterIdListE
 
 import static org.junit.jupiter.api.Assertions.*;
 
-public class TestXSLFSlideShow {
+class TestXSLFSlideShow {
     private static final POIDataSamples slTests = POIDataSamples.getSlideShowInstance();
     private OPCPackage pack;
 

src/ooxml/testcases/org/apache/poi/xslf/extractor/TestXSLFPowerPointExtractor.java

@@ -39,7 +39,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Tests for XSLFPowerPointExtractor
  */
-public class TestXSLFPowerPointExtractor {
+class TestXSLFPowerPointExtractor {
     private static final POIDataSamples slTests = POIDataSamples.getSlideShowInstance();
 
 	/**

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestPPTX2PNG.java

@@ -41,7 +41,7 @@ import org.junit.jupiter.params.provider.MethodSource;
  * Test class for testing PPTX2PNG utility which renders .ppt and .pptx slideshows
  */
 @SuppressWarnings("ConstantConditions")
-public class TestPPTX2PNG {
+class TestPPTX2PNG {
     private static boolean xslfOnly;
     private static final POIDataSamples samples = POIDataSamples.getSlideShowInstance();
     private static final File basedir = null;

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXMLSlideShow.java

@@ -37,7 +37,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.CTNotesMasterIdListE
 import org.openxmlformats.schemas.presentationml.x2006.main.CTSlideIdListEntry;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTSlideMasterIdListEntry;
 
-public class TestXMLSlideShow extends BaseTestSlideShow<XSLFShape, XSLFTextParagraph> {
+class TestXMLSlideShow extends BaseTestSlideShow<XSLFShape, XSLFTextParagraph> {
     private OPCPackage pack;
 
     @Override

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFAutoShape.java

@@ -36,7 +36,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.CTTextCharacterProperties
 import org.openxmlformats.schemas.drawingml.x2006.main.STTextStrikeType;
 import org.openxmlformats.schemas.drawingml.x2006.main.STTextUnderlineType;
 
-public class TestXSLFAutoShape {
+class TestXSLFAutoShape {
     @Test
     void testTextBodyProperies() throws IOException {
         XMLSlideShow ppt = new XMLSlideShow();

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFBackground.java

@@ -26,7 +26,7 @@ import java.io.IOException;
 
 import static org.junit.jupiter.api.Assertions.*;
 
-public class TestXSLFBackground {
+class TestXSLFBackground {
     @Test
     void testNoFillBackground() throws IOException {
         XMLSlideShow pptx = new XMLSlideShow();

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFChart.java

@@ -61,7 +61,7 @@ import org.junit.jupiter.api.Test;
 /**
  * a modified version from POI-examples
  */
-public class TestXSLFChart {
+class TestXSLFChart {
     @Test
     void testFillPieChartTemplate() throws IOException {
         XMLSlideShow pptx = XSLFTestDataSamples.openSampleDocument("pie-chart.pptx");

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFColor.java

@@ -32,7 +32,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.STPresetColorVal;
 import org.openxmlformats.schemas.drawingml.x2006.main.STSchemeColorVal;
 import org.openxmlformats.schemas.drawingml.x2006.main.STSystemColorVal;
 
-public class TestXSLFColor {
+class TestXSLFColor {
 
     @Test
     void testGetters() {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFConnectorShape.java

@@ -38,7 +38,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.STLineEndWidth;
 import org.openxmlformats.schemas.drawingml.x2006.main.STShapeType;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTConnector;
 
-public class TestXSLFConnectorShape {
+class TestXSLFConnectorShape {
 
     @Test
     void testLineDecorations() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFExamples.java

@@ -55,7 +55,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.STTLTimeNodeType;
  * Reimplement and test examples classes so the necessary XmlBeans schema objects are included
  * in the lite schema
  */
-public class TestXSLFExamples {
+class TestXSLFExamples {
     @Test
     void LinkVideoToPptx() throws IOException, URISyntaxException {
         String videoFileName = "file_example_MP4_640_3MG.mp4";

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFFreeformShape.java

@@ -36,7 +36,7 @@ import java.io.IOException;
 import static org.apache.poi.xslf.usermodel.TestXSLFSimpleShape.getSpPr;
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class TestXSLFFreeformShape {
+class TestXSLFFreeformShape {
 
     @Test
     void testSetPath() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFGroupShape.java

@@ -26,7 +26,7 @@ import java.awt.geom.Rectangle2D;
 import org.apache.poi.xslf.XSLFTestDataSamples;
 import org.junit.jupiter.api.Test;
 
-public class TestXSLFGroupShape {
+class TestXSLFGroupShape {
 
     @Test
     void testCreateShapes() throws Exception {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFHyperlink.java

@@ -30,7 +30,7 @@ import org.apache.poi.openxml4j.opc.TargetMode;
 import org.apache.poi.xslf.XSLFTestDataSamples;
 import org.junit.jupiter.api.Test;
 
-public class TestXSLFHyperlink {
+class TestXSLFHyperlink {
 
     @Test
     void testRead() throws IOException{

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFNotes.java

@@ -26,7 +26,7 @@ import org.apache.poi.sl.usermodel.Placeholder;
 import org.apache.poi.xslf.XSLFTestDataSamples;
 import org.junit.jupiter.api.Test;
 
-public class TestXSLFNotes {
+class TestXSLFNotes {
 
     @Test
     void createNewNote() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFPictureShape.java

@@ -45,7 +45,7 @@ import org.apache.poi.xslf.util.PPTX2PNG;
 import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTPicture;
 
-public class TestXSLFPictureShape {
+class TestXSLFPictureShape {
     private static POIDataSamples _slTests = POIDataSamples.getSlideShowInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFShape.java

@@ -27,7 +27,7 @@ import org.apache.poi.xslf.XSLFTestDataSamples;
 import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.drawingml.x2006.main.STTextUnderlineType;
 
-public class TestXSLFShape {
+class TestXSLFShape {
 
     @Test
     void testReadTextShapes() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFShapeContainer.java

@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
 /**
  * test common operations on containers of shapes (sheets and groups of shapes)
  */
-public class TestXSLFShapeContainer {
+class TestXSLFShapeContainer {
 
     void verifyContainer(XSLFShapeContainer container) {
         container.clear();

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFSheet.java

@@ -29,7 +29,7 @@ import org.junit.jupiter.api.Test;
  *
  * @author Yegor Kozlov
  */
-public class TestXSLFSheet {
+class TestXSLFSheet {
 
     @Test
     void testCreateShapes() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFSimpleShape.java

@@ -47,7 +47,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.STLineCap;
 import org.openxmlformats.schemas.drawingml.x2006.main.STPresetLineDashVal;
 import org.openxmlformats.schemas.drawingml.x2006.main.STShapeType;
 
-public class TestXSLFSimpleShape {
+class TestXSLFSimpleShape {
 
     @Test
     void testLineStyles() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFSlide.java

@@ -34,7 +34,7 @@ import org.junit.jupiter.api.Test;
 /**
  * @author Yegor Kozlov
  */
-public class TestXSLFSlide {
+class TestXSLFSlide {
 
     @Test
     void testReadShapes() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFSlideShow.java

@@ -31,7 +31,7 @@ import org.junit.jupiter.api.Test;
 /**
  * @author Yegor Kozlov
  */
-public class TestXSLFSlideShow {
+class TestXSLFSlideShow {
     @Test
     void testCreateSlide() throws IOException {
         XMLSlideShow  ppt = new XMLSlideShow();

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTable.java

@@ -47,7 +47,7 @@ import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.drawingml.x2006.main.CTTableCell;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTGraphicalObjectFrame;
 
-public class TestXSLFTable {
+class TestXSLFTable {
 
     @Test
     void testResize() throws Exception {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTableRow.java

@@ -28,7 +28,7 @@ import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.drawingml.x2006.main.CTTableCell;
 import org.openxmlformats.schemas.drawingml.x2006.main.CTTableRow;
 
-public class TestXSLFTableRow {
+class TestXSLFTableRow {
 
     private static XMLSlideShow ppt;
     private static XSLFTable tbl;

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTableStyles.java

@@ -23,7 +23,7 @@ import java.io.IOException;
 
 import org.junit.jupiter.api.Test;
 
-public class TestXSLFTableStyles {
+class TestXSLFTableStyles {
 
     @Test
     void testRead() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTextBox.java

@@ -28,7 +28,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.CTTextCharacterProperties
 /**
  * @author Yegor Kozlov
  */
-public class TestXSLFTextBox {
+class TestXSLFTextBox {
 
     @Test
     void testPlaceholder() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTextParagraph.java

@@ -41,7 +41,7 @@ import org.junit.jupiter.api.Test;
 /**
  * @author Yegor Kozlov
  */
-public class TestXSLFTextParagraph {
+class TestXSLFTextParagraph {
     // private static POILogger _logger = POILogFactory.getLogger(XSLFTextParagraph.class);
 
     static class DrawTextParagraphProxy extends DrawTextParagraph {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTextRun.java

@@ -39,7 +39,7 @@ import org.openxmlformats.schemas.presentationml.x2006.main.CTShape;
 /**
  * @author Yegor Kozlov
  */
-public class TestXSLFTextRun {
+class TestXSLFTextRun {
 
     @Test
     void testRunProperties() throws IOException {

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTextShape.java

@@ -52,7 +52,7 @@ import org.openxmlformats.schemas.drawingml.x2006.main.STTextAlignType;
 import org.openxmlformats.schemas.presentationml.x2006.main.CTPlaceholder;
 import org.openxmlformats.schemas.presentationml.x2006.main.STPlaceholderType;
 
-public class TestXSLFTextShape {
+class TestXSLFTextShape {
     private static boolean xslfOnly;
 
     @BeforeAll

src/ooxml/testcases/org/apache/poi/xslf/usermodel/TestXSLFTheme.java

@@ -37,7 +37,7 @@ import org.junit.jupiter.api.Test;
  *
  * @author Yegor Kozlov
  */
-public class TestXSLFTheme {
+class TestXSLFTheme {
     @Test
     void testRead(){
         XMLSlideShow ppt = XSLFTestDataSamples.openSampleDocument("themes.pptx");

src/ooxml/testcases/org/apache/poi/xssf/TestNecessaryOOXMLClasses.java

@@ -24,7 +24,7 @@ import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTRow;
 import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTSheetData;
 
 // aim is to get these classes loaded and included in poi-ooxml-lite.jar
-public class TestNecessaryOOXMLClasses {
+class TestNecessaryOOXMLClasses {
 
     @Test
     void testProblemClasses() {

src/ooxml/testcases/org/apache/poi/xssf/TestSheetProtection.java

@@ -27,7 +27,7 @@ import java.io.IOException;
 import static org.junit.jupiter.api.Assertions.assertFalse;
 import static org.junit.jupiter.api.Assertions.assertTrue;
 
-public class TestSheetProtection {
+class TestSheetProtection {
 	private XSSFWorkbook workbook;
 	private XSSFSheet sheet;
 

src/ooxml/testcases/org/apache/poi/xssf/TestWorkbookProtection.java

@@ -44,7 +44,7 @@ import org.apache.poi.xssf.usermodel.XSSFSheet;
 import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 
-public class TestWorkbookProtection {
+class TestWorkbookProtection {
 
     @Test
     void workbookAndRevisionPassword() throws Exception {

src/ooxml/testcases/org/apache/poi/xssf/TestXSSFCloneSheet.java

@@ -33,7 +33,7 @@ import org.junit.jupiter.api.Test;
 
 import java.io.IOException;
 
-public class TestXSSFCloneSheet  extends BaseTestCloneSheet {
+class TestXSSFCloneSheet  extends BaseTestCloneSheet {
     public TestXSSFCloneSheet() {
         super(HSSFITestDataProvider.instance);
     }

src/ooxml/testcases/org/apache/poi/xssf/TestXSSFOffset.java

@@ -27,7 +27,7 @@ import org.junit.jupiter.api.Test;
 
 import static org.junit.jupiter.api.Assertions.assertEquals;
 
-public class TestXSSFOffset {
+class TestXSSFOffset {
 
     @Test
     void testOffsetWithEmpty23Arguments() throws IOException {

src/ooxml/testcases/org/apache/poi/xssf/binary/TestXSSFBSharedStringsTable.java

@@ -27,7 +27,7 @@ import org.apache.poi.openxml4j.opc.OPCPackage;
 import org.apache.poi.openxml4j.opc.PackagePart;
 import org.junit.jupiter.api.Test;
 
-public class TestXSSFBSharedStringsTable {
+class TestXSSFBSharedStringsTable {
     private static POIDataSamples _ssTests = POIDataSamples.getSpreadSheetInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/xssf/binary/TestXSSFBSheetHyperlinkManager.java

@@ -29,7 +29,7 @@ import org.apache.poi.xssf.eventusermodel.XSSFBReader;
 import org.apache.poi.xssf.eventusermodel.XSSFReader;
 import org.junit.jupiter.api.Test;
 
-public class TestXSSFBSheetHyperlinkManager {
+class TestXSSFBSheetHyperlinkManager {
     private static POIDataSamples _ssTests = POIDataSamples.getSpreadSheetInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/xssf/eventusermodel/TestXSSFBReader.java

@@ -34,7 +34,7 @@ import org.apache.poi.xssf.binary.XSSFBStylesTable;
 import org.apache.poi.xssf.usermodel.XSSFComment;
 import org.junit.jupiter.api.Test;
 
-public class TestXSSFBReader {
+class TestXSSFBReader {
 
     private static POIDataSamples _ssTests = POIDataSamples.getSpreadSheetInstance();
 

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFBEventBasedExcelExtractor.java

@@ -33,7 +33,7 @@ import java.nio.file.Files;
 /**
  * Tests for {@link XSSFBEventBasedExcelExtractor}
  */
-public class TestXSSFBEventBasedExcelExtractor {
+class TestXSSFBEventBasedExcelExtractor {
     protected XSSFEventBasedExcelExtractor getExtractor(String sampleName) throws Exception {
         return new XSSFBEventBasedExcelExtractor(XSSFTestDataSamples.
                 openSamplePackage(sampleName));

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFEventBasedExcelExtractor.java

@@ -38,7 +38,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Tests for {@link XSSFEventBasedExcelExtractor}
  */
-public class TestXSSFEventBasedExcelExtractor {
+class TestXSSFEventBasedExcelExtractor {
 	protected XSSFEventBasedExcelExtractor getExtractor(String sampleName) throws Exception {
         return new XSSFEventBasedExcelExtractor(XSSFTestDataSamples.
                 openSamplePackage(sampleName));

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFEventBasedExcelExtractorUsingFactory.java

@@ -21,7 +21,7 @@ import org.apache.poi.hssf.HSSFTestDataSamples;
 import org.junit.jupiter.api.AfterEach;
 
 
-public class TestXSSFEventBasedExcelExtractorUsingFactory extends TestXSSFEventBasedExcelExtractor {
+class TestXSSFEventBasedExcelExtractorUsingFactory extends TestXSSFEventBasedExcelExtractor {
 	@Override
 	protected final XSSFEventBasedExcelExtractor getExtractor(String sampleName) throws Exception {
 		ExtractorFactory.setAllThreadsPreferEventExtractors(true);

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFExcelExtractor.java

@@ -40,7 +40,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Tests for {@link XSSFExcelExtractor}
  */
-public class TestXSSFExcelExtractor {
+class TestXSSFExcelExtractor {
 	protected XSSFExcelExtractor getExtractor(String sampleName) {
 		return new XSSFExcelExtractor(XSSFTestDataSamples.openSampleWorkbook(sampleName));
 	}

src/ooxml/testcases/org/apache/poi/xssf/extractor/TestXSSFImportFromXML.java

@@ -42,7 +42,7 @@ import org.junit.jupiter.api.Test;
 import org.junit.jupiter.api.Timeout;
 import org.xml.sax.SAXException;
 
-public class TestXSSFImportFromXML {
+class TestXSSFImportFromXML {
 
     @Test
     void testImportFromXML() throws IOException, XPathExpressionException, SAXException{

src/ooxml/testcases/org/apache/poi/xssf/io/TestLoadSaveXSSF.java

@@ -32,7 +32,7 @@ import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 
 
-public class TestLoadSaveXSSF {
+class TestLoadSaveXSSF {
     private static final POIDataSamples _ssSamples = POIDataSamples.getSpreadSheetInstance();
 
     @Test

src/ooxml/testcases/org/apache/poi/xssf/model/TestCommentsTable.java

@@ -47,7 +47,7 @@ import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTComments;
 import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTRst;
 
 
-public class TestCommentsTable {
+class TestCommentsTable {
 
     private static final String TEST_A2_TEXT = "test A2 text";
     private static final String TEST_A1_TEXT = "test A1 text";

src/ooxml/testcases/org/apache/poi/xssf/model/TestThemesTable.java

@@ -44,7 +44,7 @@ import org.apache.poi.xssf.usermodel.XSSFWorkbook;
 import org.junit.jupiter.api.Test;
 import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTColor;
 
-public class TestThemesTable {
+class TestThemesTable {
     private static final String testFileComplex = "Themes2.xlsx";
     // TODO .xls version available too, add HSSF support then check
 

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestAutoSizeColumnTracker.java

@@ -46,7 +46,7 @@ import org.junit.jupiter.api.Test;
  *
  * see Bug #57450 which reported the original misbehaviour
  */
-public class TestAutoSizeColumnTracker {
+class TestAutoSizeColumnTracker {
 
     private SXSSFSheet sheet;
     private SXSSFWorkbook workbook;

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFCell.java

@@ -52,7 +52,7 @@ import org.openxmlformats.schemas.spreadsheetml.x2006.main.CTRst;
  * Tests various functionality having to do with {@link SXSSFCell}.  For instance support for
  * particular datatypes, etc.
  */
-public class TestSXSSFCell extends BaseTestXCell {
+class TestSXSSFCell extends BaseTestXCell {
 
     public TestSXSSFCell() {
         super(SXSSFITestDataProvider.instance);

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFDataValidation.java

@@ -29,7 +29,7 @@ import org.apache.poi.ss.util.CellRangeAddressList;
 import org.apache.poi.xssf.SXSSFITestDataProvider;
 import org.junit.jupiter.api.Test;
 
-public class TestSXSSFDataValidation extends BaseTestDataValidation {
+class TestSXSSFDataValidation extends BaseTestDataValidation {
 
     public TestSXSSFDataValidation(){
         super(SXSSFITestDataProvider.instance);

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFEvaluationSheet.java

@@ -28,7 +28,7 @@ import java.util.ArrayList;
 import java.util.List;
 import java.util.Map;
 
-public class TestSXSSFEvaluationSheet extends BaseTestXEvaluationSheet {
+class TestSXSSFEvaluationSheet extends BaseTestXEvaluationSheet {
     private List<Workbook> workbooksToClose = new ArrayList<>();
 
     @AfterEach

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFHyperlink.java

@@ -31,7 +31,7 @@ import org.apache.poi.xssf.usermodel.XSSFHyperlink;
  *
  * @author Yegor Kozlov
  */
-public class TestSXSSFHyperlink extends BaseTestHyperlink {
+class TestSXSSFHyperlink extends BaseTestHyperlink {
 
     public TestSXSSFHyperlink() {
         super(SXSSFITestDataProvider.instance);

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFSheetAutoSizeColumn.java

@@ -50,7 +50,7 @@ import org.junit.jupiter.params.provider.MethodSource;
  *
  * see Bug #57450 which reported the original misbehaviour
  */
-public class TestSXSSFSheetAutoSizeColumn {
+class TestSXSSFSheetAutoSizeColumn {
 
     private static final String SHORT_CELL_VALUE = "Ben";
     private static final String LONG_CELL_VALUE = "B Be Ben Beni Benif Benify Benif Beni Ben Be B";

src/ooxml/testcases/org/apache/poi/xssf/streaming/TestSXSSFUnicodeSurrogates.java

@@ -31,7 +31,7 @@ import java.io.FileInputStream;
 import java.io.FileOutputStream;
 import java.io.IOException;
 
-public class TestSXSSFUnicodeSurrogates {
+class TestSXSSFUnicodeSurrogates {
 
     // "𝝊𝝋𝝌𝝍𝝎𝝏𝝐𝝑𝝒𝝓𝝔𝝕𝝖𝝗𝝘𝝙𝝚𝝛𝝜𝝝𝝞𝝟𝝠𝝡𝝢𝝣𝝤𝝥𝝦𝝧𝝨𝝩𝝪𝝫𝝬𝝭𝝮𝝯𝝰𝝱𝝲𝝳𝝴𝝵𝝶𝝷𝝸𝝹𝝺";
     private static final String unicodeText =

src/ooxml/testcases/org/apache/poi/xssf/usermodel/TestTableStyles.java

@@ -35,7 +35,7 @@ import org.junit.jupiter.api.Test;
 /**
  * Test built-in table styles
  */
-public class TestTableStyles {
+class TestTableStyles {
 
     /**
      * Test that a built-in style is initialized properly