Changeset 18853 in josm for trunk/test/unit/org/openstreetmap/josm/testutils

Timestamp:

2023-10-04T00:03:40+02:00 (22 months ago)

Author:

taylor.smock

Message:

See #16567: Update to JUnit 5

This removes new JOSMTestRules() with no additional setup and most
JOSMFixture calls.

Removing the bare JOSMTestRules speeds up the test suite since there are two
fewer System.gc() calls per test.

File:

• trunk/test/unit/org/openstreetmap/josm/testutils/annotations/JosmDefaults.java (modified) (1 diff)

trunk/test/unit/org/openstreetmap/josm/testutils/annotations/JosmDefaults.java

@@ -77 +77 @@
         /**
          * Clean up what test not using these test rules may have broken.
+         * See {@link org.openstreetmap.josm.JOSMFixture} for the most common reason why this needs to be called.
          */
-        private void cleanUpFromJosmFixture() {
+        private static void cleanUpFromJosmFixture() {
             MemoryManagerTest.resetState(true);
             JOSMTestRules.cleanLayerEnvironment();