Fix issue with -te flag. Version Bump

Author: valandi

CHANGELOG.md

@@ -1,5 +1,8 @@
 # Changelog
 
+## 3.6.3 - 2024/7/25
+- Fix issue in which using `-te` prevents Java process from exiting gracefully
+
 ## 3.5.4 - 2023/11/30
 - Resolve issue relating to universal executables not being available for all OSs
 

pom.xml

@@ -4,7 +4,7 @@
 
 	<groupId>com.applitools.imagetester</groupId>
 	<artifactId>ImageTester</artifactId>
-	<version>3.6.2</version>
+	<version>3.6.4</version>
 	<packaging>jar</packaging>
 	<properties>
 		<maven.compiler.source>1.8</maven.compiler.source>
@@ -63,7 +63,7 @@
 		<dependency>
 			<groupId>com.applitools</groupId>
 			<artifactId>eyes-images-java5</artifactId>
-			<version>5.66.5</version>
+			<version>5.69.3</version>
 		</dependency>
 		<dependency>
 			<groupId>commons-cli</groupId>
@@ -112,6 +112,11 @@
 			<version>1.7.32</version>
 			<scope>compile</scope>
 		</dependency>
+		<dependency>
+			<groupId>com.applitools</groupId>
+			<artifactId>eyes-sdk-core-java5</artifactId>
+			<version>5.70.0</version>
+		</dependency>
 	</dependencies>
 	<build>
 		<finalName>../jars/${project.artifactId}_${project.version}</finalName>

src/main/java/META-INF/MANIFEST.MF

@@ -1,4 +1,3 @@
 Manifest-Version: 1.0
 Main-Class: com.applitools.imagetester.ImageTester
-Implementation-Version: 3.6.2
-
+Implementation-Version: 3.6.4

src/main/java/com/applitools/imagetester/ImageTester.java

@@ -26,7 +26,7 @@
 import com.applitools.imagetester.lib.Utils;
 
 public class ImageTester {
-    private static final String cur_ver = "3.6.2";
+    private static final String cur_ver = "3.6.4";
 
     public static void main(String[] args) {
 

src/main/java/com/applitools/imagetester/lib/TestExecutor.java

@@ -56,6 +56,7 @@ public void enqueue(TestBase test, BatchInfo overrideBatch) {
     public void join() {
         int total = results_.size();
         int curr = 1;
+        boolean shouldExit = false;
         while (!results_.isEmpty()) {
             config_.logger.printProgress(curr++, total);
             ExecutorResult result = null;
@@ -65,10 +66,18 @@ public void join() {
                 config_.logger.reportException(e);
             } catch (ExecutionException e) {
                 config_.logger.reportException(e);
+                shouldExit = true;
                 if (config_.shouldThrowException) {
                     throw new RuntimeException("Eyes has reported a mismatch or test failure. \n" +
-                            "This exception is thrown because the '-te' flag was present, \n" +
-                            "which instructs ImageTester to throw exceptions if a test fails, or a mismatch is detected");
+                        "This exception is thrown because the '-te' flag was present, \n" +
+                        "which instructs ImageTester to throw exceptions if a test fails, or a mismatch is detected");
+                }
+            } finally {
+                if (shouldExit) {
+                    executorService_.shutdown();
+                    if (config_.shouldThrowException) {
+                        System.exit(1);
+                    }
                 }
             }