Avoid doing any xstream related work unless absolutely necessary

Relates to: #10081
quarkusio · Jun 22, 2020 · 8394406 · 8394406
1 parent 25c7a2a
commit 8394406
Showing 1 changed file with 10 additions and 5 deletions.
diff --git a/test-framework/junit5/src/main/java/io/quarkus/test/junit/internal/XStreamDeepClone.java b/test-framework/junit5/src/main/java/io/quarkus/test/junit/internal/XStreamDeepClone.java
@@ -9,13 +9,17 @@
  */
 public class XStreamDeepClone implements DeepClone {
 
-    private final XStream xStream;
+    private final Supplier<XStream> xStreamSupplier;
 
     public XStreamDeepClone(ClassLoader classLoader) {
-        xStream = new XStream();
-        XStream.setupDefaultSecurity(xStream);
-        xStream.allowTypesByRegExp(new String[] { ".*" });
-        xStream.setClassLoader(classLoader);
+        // avoid doing any work eagerly since the cloner is rarely used
+        xStreamSupplier = () -> {
+            XStream result = new XStream();
+            XStream.setupDefaultSecurity(result);
+            result.allowTypesByRegExp(new String[] { ".*" });
+            result.setClassLoader(classLoader);
+            return result;
+        };
     }
 
     public Object clone(Object objectToClone) {
@@ -40,6 +44,7 @@ public Object get() {
     }
 
     private Object doClone(Object objectToClone) {
+        XStream xStream = xStreamSupplier.get();
         final String serialized = xStream.toXML(objectToClone);
         final Object result = xStream.fromXML(serialized);
         if (result == null) {