This is an automated email from the ASF dual-hosted git repository. desruisseaux pushed a commit to branch geoapi-4.0 in repository https://gitbox.apache.org/repos/asf/sis.git
commit a1d8ed95d917935cb3494edfbdcbe646dba08bc0 Author: Martin Desruisseaux <[email protected]> AuthorDate: Fri Oct 10 15:12:08 2025 +0200 Make non-optional some tests that are not so slow according JUnit report. Fix also a remaining EPSG test which could be accidentally skipped. --- .../referencing/AbstractIdentifiedObjectTest.java | 1 + .../referencing/AbstractReferenceSystemTest.java | 8 ++-- .../org/apache/sis/referencing/BuilderTest.java | 3 +- .../sis/referencing/ImmutableIdentifierTest.java | 1 + .../sis/referencing/NamedIdentifierTest.java | 5 +- .../referencing/factory/sql/EPSGFactoryTest.java | 53 ++++++++++------------ .../referencing/factory/sql/EPSGInstallerTest.java | 9 +++- .../DefaultCoordinateOperationFactoryTest.java | 16 ++++--- .../operation/provider/ProvidersTest.java | 4 +- .../sis/test/integration/ConsistencyTest.java | 3 +- .../sis/test/integration/MetadataVerticalTest.java | 3 +- .../test/org/apache/sis/test/Benchmark.java | 2 - .../test/org/apache/sis/test/TestCase.java | 6 +-- 13 files changed, 57 insertions(+), 57 deletions(-) diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractIdentifiedObjectTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractIdentifiedObjectTest.java index fef7aec5d4..e341bbb439 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractIdentifiedObjectTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractIdentifiedObjectTest.java @@ -46,6 +46,7 @@ import org.opengis.metadata.Identifier; * * @author Martin Desruisseaux (IRD, Geomatys) */ +@SuppressWarnings("exports") public final class AbstractIdentifiedObjectTest extends TestCase { /** * Creates a new test case. diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractReferenceSystemTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractReferenceSystemTest.java index cd16bd8426..d9b80f2d15 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractReferenceSystemTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/AbstractReferenceSystemTest.java @@ -16,7 +16,6 @@ */ package org.apache.sis.referencing; -import java.util.Map; import java.util.HashMap; import java.util.Locale; import java.time.LocalDate; @@ -49,6 +48,7 @@ import static org.opengis.referencing.ObjectDomain.*; * * @author Martin Desruisseaux (IRD, Geomatys) */ +@SuppressWarnings("exports") public final class AbstractReferenceSystemTest extends TestCase { /** * Creates a new test case. @@ -61,7 +61,7 @@ public final class AbstractReferenceSystemTest extends TestCase { */ @Test public void testCreateFromMap() { - final Map<String,Object> properties = new HashMap<>(); + final var properties = new HashMap<String,Object>(); assertNull(properties.put("name", "This is a name")); assertNull(properties.put("scope", "This is a scope")); assertNull(properties.put("scope_fr", "Valide dans ce domaine")); @@ -85,7 +85,7 @@ public final class AbstractReferenceSystemTest extends TestCase { */ @Test public void testSerialization() { - final Map<String,Object> properties = new HashMap<>(8); + final var properties = new HashMap<String,Object>(8); assertNull(properties.put("code", "4326")); assertNull(properties.put("codeSpace", "EPSG")); assertNull(properties.put("scope", "This is a scope")); @@ -104,7 +104,7 @@ public final class AbstractReferenceSystemTest extends TestCase { */ @Test public void testWKT() { - final Map<String,Object> properties = new HashMap<>(8); + final var properties = new HashMap<String,Object>(8); assertNull(properties.put(NAME_KEY, "My “object”.")); assertNull(properties.put(SCOPE_KEY, "Large scale topographic mapping and cadastre.")); assertNull(properties.put(REMARKS_KEY, "注です。")); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/BuilderTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/BuilderTest.java index e075067cac..b75948ca6b 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/BuilderTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/BuilderTest.java @@ -41,6 +41,7 @@ import org.apache.sis.test.TestCase; * * @author Martin Desruisseaux (Geomatys) */ +@SuppressWarnings("exports") public final class BuilderTest extends TestCase { /** * Creates a new test case. @@ -72,7 +73,7 @@ public final class BuilderTest extends TestCase { * Setting the same codespace should have no effect, while attempt to * set a new codespace after we added a name shall not be allowed. */ - final SimpleCitation IOGP = new SimpleCitation("IOGP"); + final var IOGP = new SimpleCitation("IOGP"); builder.setCodeSpace(Citations.EPSG, "EPSG"); var exception = assertThrows(IllegalStateException.class, () -> builder.setCodeSpace(IOGP, "EPSG")); assertMessageContains(exception, Identifier.AUTHORITY_KEY); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/ImmutableIdentifierTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/ImmutableIdentifierTest.java index e0f36f1262..0b0365681c 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/ImmutableIdentifierTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/ImmutableIdentifierTest.java @@ -49,6 +49,7 @@ import static org.opengis.metadata.Identifier.*; * * @author Martin Desruisseaux (Geomatys) */ +@SuppressWarnings("exports") public final class ImmutableIdentifierTest extends TestCase { /** * Creates a new test case. diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/NamedIdentifierTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/NamedIdentifierTest.java index 9f3eaea041..b289fa6873 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/NamedIdentifierTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/NamedIdentifierTest.java @@ -41,6 +41,7 @@ import org.opengis.metadata.Identifier; * * @author Martin Desruisseaux (Geomatys) */ +@SuppressWarnings("exports") public final class NamedIdentifierTest extends TestCase { /** * Creates a new test case. @@ -54,7 +55,7 @@ public final class NamedIdentifierTest extends TestCase { */ @Test public void testCreateFromCode() { - final NamedIdentifier identifier = new NamedIdentifier(Citations.EPSG, "EPSG", "4326", "8.3", null); + final var identifier = new NamedIdentifier(Citations.EPSG, "EPSG", "4326", "8.3", null); Validators.validate((Identifier) identifier); Validators.validate((GenericName) identifier); @@ -80,7 +81,7 @@ public final class NamedIdentifierTest extends TestCase { public void testCreateFromName() { final NameFactory factory = DefaultNameFactory.provider(); final NameSpace scope = factory.createNameSpace(factory.createLocalName(null, "IOGP"), null); - final NamedIdentifier identifier = new NamedIdentifier(factory.createGenericName(scope, "EPSG", "4326")); + final var identifier = new NamedIdentifier(factory.createGenericName(scope, "EPSG", "4326")); Validators.validate((Identifier) identifier); Validators.validate((GenericName) identifier); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGFactoryTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGFactoryTest.java index 764afe751d..99d047429c 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGFactoryTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGFactoryTest.java @@ -607,12 +607,10 @@ public final class EPSGFactoryTest extends TestCaseWithLogs { assertFalse(verticalDatum.containsAll(datum), "Vertical datum should be a subset of datum."); // Iteration should stop at the first mismatch. assertTrue (datum.containsAll(verticalDatum), "Vertical datum should be a subset of datum."); // Iteration should over a small set (vertical datum). - if (RUN_EXTENSIVE_TESTS) { - assertTrue(geodeticDatum.size() >= 445, "size() consistency check."); - assertTrue(datum.size() > geodeticDatum.size(), "Geodetic reference frame should be a subset of datum."); - assertTrue(datum.size() > verticalDatum.size(), "Vertical datum should be a subset of datum."); - assertTrue(datum.containsAll(geodeticDatum), "Geodetic reference frame should be a subset of datum."); - } + assertTrue(geodeticDatum.size() >= 445, "size() consistency check."); + assertTrue(datum.size() > geodeticDatum.size(), "Geodetic reference frame should be a subset of datum."); + assertTrue(datum.size() > verticalDatum.size(), "Vertical datum should be a subset of datum."); + assertTrue(datum.containsAll(geodeticDatum), "Geodetic reference frame should be a subset of datum."); /* * COORDINATE REFERENCE SYSTEMS - There is thousands of CRS, so we avoid all tests that may require @@ -622,34 +620,31 @@ public final class EPSGFactoryTest extends TestCaseWithLogs { assertFalse (crs.isEmpty(), "CRSs not found."); assertTrue (crs.contains("4326"), "Shall contain WGS84."); assertTrue (crs.contains("3395"), "Shall contain World Mercator."); - if (RUN_EXTENSIVE_TESTS) { - assertTrue (crs.size() >= 4175); // Cause a scanning of the full table. - assertEquals(crs.size(), crs.size()); - } + + assertTrue (crs.size() >= 4175); // Cause a scanning of the full table. + assertEquals(crs.size(), crs.size()); final Set<String> geographicCRS = factory.getAuthorityCodes(GeographicCRS.class); assertFalse(geographicCRS.isEmpty(), "GeographicCRSs not found."); assertTrue (geographicCRS.contains("4326"), "Shall contain WGS84."); assertFalse(geographicCRS.contains("4978"), "Shall not contain geocentric CRS."); assertFalse(geographicCRS.contains("3395"), "Shall not contain projected CRS."); - if (RUN_EXTENSIVE_TESTS) { - assertTrue (geographicCRS.size() >= 468, "size() consistency check."); - assertTrue (geographicCRS.size() < crs.size(), "Geographic CRS should be a subset of CRS."); - assertFalse(geographicCRS.containsAll(crs), "Geographic CRS should be a subset of CRS."); - assertTrue (crs.containsAll(geographicCRS), "Geographic CRS should be a subset of CRS."); - } + + assertTrue (geographicCRS.size() >= 468, "size() consistency check."); + assertTrue (geographicCRS.size() < crs.size(), "Geographic CRS should be a subset of CRS."); + assertFalse(geographicCRS.containsAll(crs), "Geographic CRS should be a subset of CRS."); + assertTrue (crs.containsAll(geographicCRS), "Geographic CRS should be a subset of CRS."); final Set<String> projectedCRS = factory.getAuthorityCodes(ProjectedCRS.class); assertFalse(projectedCRS.isEmpty(), "ProjectedCRSs not found."); assertFalse(projectedCRS.contains("4326"), "Shall not contain geographic CRS."); assertTrue (projectedCRS.contains("3395"), "Shall contain World Mercator."); - if (RUN_EXTENSIVE_TESTS) { - assertTrue (projectedCRS.size() >= 3441, "size() consistency check."); - assertTrue (projectedCRS.size() < crs.size(), "Projected CRS should be a subset of CRS."); - assertFalse(projectedCRS.containsAll(crs), "Projected CRS should be a subset of CRS."); - assertTrue (crs.containsAll(projectedCRS), "Projected CRS should be a subset of CRS."); - assertTrue (Collections.disjoint(geographicCRS, projectedCRS), "Projected CRS cannot be Geographic CRS."); - } + + assertTrue (projectedCRS.size() >= 3441, "size() consistency check."); + assertTrue (projectedCRS.size() < crs.size(), "Projected CRS should be a subset of CRS."); + assertFalse(projectedCRS.containsAll(crs), "Projected CRS should be a subset of CRS."); + assertTrue (crs.containsAll(projectedCRS), "Projected CRS should be a subset of CRS."); + assertTrue (Collections.disjoint(geographicCRS, projectedCRS), "Projected CRS cannot be Geographic CRS."); /* * COORDINATE OPERATIONS - There is thousands of operations, so we avoid all tests that may require @@ -676,13 +671,11 @@ public final class EPSGFactoryTest extends TestCaseWithLogs { assertTrue (conversions .contains("16001"), "Shall contain “UTM zone 1N”"); assertFalse(transformations.contains("16001"), "Shall not contain “UTM zone 1N”"); - if (RUN_EXTENSIVE_TESTS) { - assertTrue (conversions .size() < operations .size(), "Conversions shall be a subset of operations."); - assertTrue (transformations.size() < operations .size(), "Transformations shall be a subset of operations."); - assertTrue (operations .containsAll(conversions), "Conversion shall be a subset of operations."); - assertTrue (operations .containsAll(transformations), "Transformations shall be a subset of operations."); - assertTrue (Collections.disjoint(conversions, transformations), "Conversions cannot be transformations."); - } + assertTrue (conversions .size() < operations .size(), "Conversions shall be a subset of operations."); + assertTrue (transformations.size() < operations .size(), "Transformations shall be a subset of operations."); + assertTrue (operations .containsAll(conversions), "Conversion shall be a subset of operations."); + assertTrue (operations .containsAll(transformations), "Transformations shall be a subset of operations."); + assertTrue (Collections.disjoint(conversions, transformations), "Conversions cannot be transformations."); // We are cheating here since we are breaking generic type check. // However, in the particular case of our EPSG factory, it works. diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGInstallerTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGInstallerTest.java index eccaa9843b..bd4bcc8e48 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGInstallerTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/factory/sql/EPSGInstallerTest.java @@ -89,7 +89,13 @@ public final class EPSGInstallerTest extends TestCaseWithLogs { * Tests the creation of an EPSG database on Derby. * This test is skipped if the SQL scripts are not found. * - * <p>See {@link TestDatabase} Javadoc if there is a need to inspect the content of that in-memory database.</p> + * <h4>Debugging</h4> + * See {@link TestDatabase} Javadoc if there is a need to inspect the content of that in-memory database. + * + * <h4>Test performance</h4> + * This test is slower than the same tests on other database engines. Furthermore, it is redundant + * with the build of the {@code org.apache.sis.referencing.database} module when output is clean. + * For these reasons, this test is skipped unless extensive tests are enabled. * * @throws Exception if an error occurred while creating the database. */ @@ -132,7 +138,6 @@ public final class EPSGInstallerTest extends TestCaseWithLogs { */ @Test public void testCreationOnH2() throws Exception { - assumeExtensiveTestsEnabled(); final InstallationScriptProvider scripts = getScripts(); // Needs to be invoked first. try (TestDatabase db = TestDatabase.createOnH2("EPSGInstaller")) { createAndTest(db.source, scripts); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactoryTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactoryTest.java index b4e9223c46..3b637f77a1 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactoryTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/DefaultCoordinateOperationFactoryTest.java @@ -57,6 +57,7 @@ import static org.apache.sis.referencing.Assertions.assertEpsgNameAndIdentifierE * * @author Martin Desruisseaux (Geomatys) */ +@SuppressWarnings("exports") @Execution(ExecutionMode.SAME_THREAD) @TestInstance(TestInstance.Lifecycle.PER_CLASS) public final class DefaultCoordinateOperationFactoryTest extends MathTransformTestCase { @@ -188,8 +189,8 @@ public final class DefaultCoordinateOperationFactoryTest extends MathTransformTe * prime meridian. This is the same test as {@link #testProjectionAndLongitudeRotation()}, * with extra dimension which should be just dropped. * - * <p>This tests requires the EPSG database, because it requires the coordinate operation - * path which is defined there.</p> + * <p>Accurate test requires the EPSG database, because it requires the coordinate operation + * path which is defined there. If the database is absent, the test will be more approximate.</p> * * @throws ParseException if a CRS used in this test cannot be parsed. * @throws FactoryException if the operation cannot be created. @@ -212,15 +213,16 @@ public final class DefaultCoordinateOperationFactoryTest extends MathTransformTe " TimeUnit[“day”, 86400]]]"); final CoordinateReferenceSystem targetCRS = parse("$Mercator"); - final CoordinateOperation operation = factory.createOperation(sourceCRS, targetCRS, null); - assertSame (sourceCRS, operation.getSourceCRS()); - assertSame (targetCRS, operation.getTargetCRS()); - assertInstanceOf(ConcatenatedOperation.class, operation); + final ConcatenatedOperation operation = assertInstanceOf( + ConcatenatedOperation.class, + factory.createOperation(sourceCRS, targetCRS, null)); + assertSame(sourceCRS, operation.getSourceCRS()); + assertSame(targetCRS, operation.getTargetCRS()); /* * The accuracy of the coordinate operation depends on whether a path has been found with the help * of the EPSG database. See testProjectionAndLongitudeRotation() for more information. */ - final boolean isUsingEpsgFactory = verifyParametersNTF(((ConcatenatedOperation) operation).getOperations(), 2); + final boolean isUsingEpsgFactory = verifyParametersNTF(operation.getOperations(), 2); assertEquals(isUsingEpsgFactory ? 2 : PositionalAccuracyConstant.UNKNOWN_ACCURACY, CRS.getLinearAccuracy(operation)); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/provider/ProvidersTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/provider/ProvidersTest.java index 6b89144a18..430de3c4d3 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/provider/ProvidersTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/referencing/operation/provider/ProvidersTest.java @@ -39,7 +39,6 @@ import org.apache.sis.referencing.factory.sql.EPSGFactory; // Test dependencies import org.junit.jupiter.api.Test; import static org.junit.jupiter.api.Assertions.*; -import static org.junit.jupiter.api.Assumptions.abort; import org.apache.sis.test.TestCase; // Specific to the geoapi-3.1 and geoapi-4.0 branches: @@ -258,12 +257,11 @@ public final class ProvidersTest extends TestCase { */ @Test public void compareWithEPSG() throws ReflectiveOperationException, FactoryException { - assumeExtensiveTestsEnabled(); final EPSGFactory factory; try { factory = (EPSGFactory) CRS.getAuthorityFactory(Constants.EPSG); } catch (ClassCastException e) { - abort("This test requires the EPSG geodetic dataset."); + assumeConnectionToEPSG(false); throw e; } final var methodAliases = new HashMap<AbstractProvider, String[]>(256); diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/ConsistencyTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/ConsistencyTest.java index 254aa21083..eebf5bafec 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/ConsistencyTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/ConsistencyTest.java @@ -114,7 +114,8 @@ public final class ConsistencyTest extends TestCase { } /** - * Verifies the WKT consistency of all CRS instances. + * Verifies the <abbr>WKT</abbr> consistency of all <abbr>CRS</abbr> instances. + * This check is slow and executed only if extensive tests are enabled. * * @throws FactoryException if an error other than "unsupported operation method" occurred. */ diff --git a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/MetadataVerticalTest.java b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/MetadataVerticalTest.java index b933292c8f..97fe5a8973 100644 --- a/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/MetadataVerticalTest.java +++ b/endorsed/src/org.apache.sis.referencing/test/org/apache/sis/test/integration/MetadataVerticalTest.java @@ -39,7 +39,6 @@ import org.opengis.referencing.datum.VerticalDatum; import org.apache.sis.system.Loggers; import org.apache.sis.xml.NilObject; import org.apache.sis.xml.NilReason; -import org.apache.sis.temporal.TemporalDate; // Test dependencies import org.junit.jupiter.api.Test; @@ -91,7 +90,7 @@ public final class MetadataVerticalTest extends TestCase.WithLogs { assertEquals("20090901", metadata.getMetadataIdentifier().getCode()); assertEquals(Locale.ENGLISH, getSingleton(metadata.getLocalesAndCharsets().keySet())); assertEquals(StandardCharsets.UTF_8, getSingleton(metadata.getLocalesAndCharsets().values())); - assertEquals(LocalDate.of(2014, 1, 4), TemporalDate.toTemporal(getSingleton(metadata.getDateInfo()).getDate())); + assertEquals(LocalDate.of(2014, 1, 4), getSingleton(metadata.getDateInfo()).getReferenceDate()); /* * <gmd:contact> * <gmd:CI_ResponsibleParty> diff --git a/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/Benchmark.java b/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/Benchmark.java index 33f945c644..d14f02ec75 100644 --- a/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/Benchmark.java +++ b/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/Benchmark.java @@ -40,8 +40,6 @@ public @interface Benchmark { /** * The value to use in {@link org.junit.jupiter.api.Tag} annotations * for tests that are also benchmarks. - * - * @see TestCase#RUN_EXTENSIVE_TESTS */ String TAG = "Benchmark"; } diff --git a/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/TestCase.java b/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/TestCase.java index 488c5f4cca..22537f344d 100644 --- a/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/TestCase.java +++ b/endorsed/src/org.apache.sis.util/test/org/apache/sis/test/TestCase.java @@ -85,7 +85,7 @@ public abstract class TestCase { * * @see #assumeExtensiveTestsEnabled() */ - public static final boolean RUN_EXTENSIVE_TESTS; + private static final boolean RUN_EXTENSIVE_TESTS; /** * Whether the tests shall use the <abbr>EPSG</abbr> geodetic dataset. @@ -93,7 +93,7 @@ public abstract class TestCase { * * @see #assumeConnectionToEPSG(boolean) */ - public static final boolean REQUIRE_EPSG_DATABASE; + protected static final boolean REQUIRE_EPSG_DATABASE; /** * Whether the tests can use the PostgreSQL database on the local host. @@ -105,7 +105,7 @@ public abstract class TestCase { * Whether the tests is allowed to popup a widget. * Only a few tests provide visualization widget. */ - public static final boolean SHOW_WIDGET; + protected static final boolean SHOW_WIDGET; /** * Whether the tests should print debugging information.
