You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@jena.apache.org by an...@apache.org on 2026/05/01 08:10:22 UTC
(jena) 01/04: Fix adding a new system-wide test maker
This is an automated email from the ASF dual-hosted git repository.
afs pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/jena.git
commit 8417573509aeb332efa641a141fa774b4e884340
Author: Andy Seaborne <an...@apache.org>
AuthorDate: Wed Apr 22 11:51:31 2026 +0100
Fix adding a new system-wide test maker
---
.../test/java/org/apache/jena/arq/junit/manifest/TestMakers.java | 9 ++++-----
1 file changed, 4 insertions(+), 5 deletions(-)
diff --git a/jena-arq/src/test/java/org/apache/jena/arq/junit/manifest/TestMakers.java b/jena-arq/src/test/java/org/apache/jena/arq/junit/manifest/TestMakers.java
index 3d4ee22ec4..b37c8f7599 100644
--- a/jena-arq/src/test/java/org/apache/jena/arq/junit/manifest/TestMakers.java
+++ b/jena-arq/src/test/java/org/apache/jena/arq/junit/manifest/TestMakers.java
@@ -21,6 +21,7 @@
package org.apache.jena.arq.junit.manifest;
+import java.util.ArrayList;
import java.util.List;
import org.apache.jena.arq.junit.SurpressedTest;
@@ -52,7 +53,9 @@ public class TestMakers {
* Add a test maker to the system-wide test makers list
*/
public static void install(TestMaker testMaker) {
- systemSetup.add(testMaker);
+ List<TestMaker> next = new ArrayList<>(systemSetup.installed);
+ next.add(testMaker);
+ set(next);
}
public static void reset() {
@@ -77,10 +80,6 @@ public class TestMakers {
installed = testMakers;
}
- public void add(TestMaker testMaker) {
- installed.add(testMaker);
- }
-
public void clear() {
installed.clear();
}