[sinttest] Move exit status logic into TestRunResult

Author: Flowdalic

smack-integration-test/src/main/java/org/igniterealtime/smack/inttest/SmackIntegrationTestFramework.java

@@ -114,11 +114,7 @@ public static void main(String[] args) throws IOException, KeyManagementExceptio
         SmackIntegrationTestFramework sinttest = new SmackIntegrationTestFramework(config);
         TestRunResult testRunResult = sinttest.run();
 
-        int exitStatus = testRunResult.failedIntegrationTests.isEmpty() ? 0 : 2;
-        if (config.failOnImpossibleTest && (!testRunResult.impossibleTestClasses.isEmpty() || !testRunResult.impossibleIntegrationTests.isEmpty())) {
-            exitStatus += 4;
-        }
-        System.exit(exitStatus);
+        System.exit(testRunResult.getExitStatus());
     }
 
     public static class JulTestRunResultProcessor implements TestRunResultProcessor {
@@ -214,7 +210,7 @@ public synchronized TestRunResult run()
             break;
         }
 
-        testRunResult = new TestRunResult();
+        testRunResult = new TestRunResult(config);
         sinttestDebugger = config.createSinttestDebugger(testRunResult.testRunStart, testRunResult.testRunId);
         // Create a connection manager *after* we created the testRunId (in testRunResult).
         this.connectionManager = new XmppConnectionManager(this);
@@ -744,6 +740,8 @@ public static final class TestRunResult {
 
         public final ZonedDateTime testRunStart = ZonedDateTime.now();
 
+        private final Configuration config;
+
         private final List<SuccessfulTest> successfulIntegrationTests = Collections.synchronizedList(new ArrayList<SuccessfulTest>());
         private final List<FailedTest> failedIntegrationTests = Collections.synchronizedList(new ArrayList<FailedTest>());
         private final List<TestNotPossible> impossibleIntegrationTests = Collections.synchronizedList(new ArrayList<TestNotPossible>());
@@ -755,7 +753,8 @@ public static final class TestRunResult {
 
         private final Map<Class<? extends AbstractSmackIntTest>, Throwable> impossibleTestClasses = new HashMap<>();
 
-        TestRunResult() {
+        TestRunResult(Configuration config) {
+            this.config = config;
         }
 
         public String getTestRunId() {
@@ -781,6 +780,15 @@ public List<TestNotPossible> getNotPossibleTests() {
         public Map<Class<? extends AbstractSmackIntTest>, Throwable> getImpossibleTestClasses() {
             return Collections.unmodifiableMap(impossibleTestClasses);
         }
+
+        public int getExitStatus() {
+          int exitStatus = failedIntegrationTests.isEmpty() ? 0 : 2;
+          if (config.failOnImpossibleTest && (!impossibleTestClasses.isEmpty() || !impossibleIntegrationTests.isEmpty())) {
+              exitStatus += 4;
+          }
+
+          return exitStatus;
+        }
     }
 
     final class PreparedTest {

smack-integration-test/src/test/java/org/igniterealtime/smack/inttest/DummySmackIntegrationTestFramework.java

@@ -1,6 +1,6 @@
 /**
  *
- * Copyright 2015-2024 Florian Schmaus
+ * Copyright 2015-2025 Florian Schmaus
  *
  * Licensed under the Apache License, Version 2.0 (the "License");
  * you may not use this file except in compliance with the License.
@@ -47,7 +47,7 @@ public DummySmackIntegrationTestFramework(Configuration configuration) throws Ke
             InstantiationException, IllegalAccessException, IllegalArgumentException, InvocationTargetException,
             NoSuchAlgorithmException, SmackException, IOException, XMPPException, InterruptedException {
         super(configuration);
-        testRunResult = new TestRunResult();
+        testRunResult = new TestRunResult(configuration);
     }
 
     @Override