Merge branch 'main' into unit-tests

Author: navin772

src/main/java/io/github/lambdatest/gradle/LambdaTestTask.java

@@ -91,28 +91,23 @@ public void runLambdaTest() {
         try {
             if (appIdFuture != null) {
                 appId = appIdFuture.join();
-                if (!progressEnabled) {
-                    logger.info("App uploaded successfully with ID: {}", appId);
-                }
             }
 
             if (testSuiteIdFuture != null) {
                 testSuiteId = testSuiteIdFuture.join();
-                if (!progressEnabled) {
-                    logger.info("Test suite uploaded successfully with ID: {}", testSuiteId);
-                }
             }
 
-            // Clear progress display if enabled
+            // Clear progress display if enabled, then show success messages
             if (progressEnabled) {
                 ProgressTracker.cleanup();
-                // Show success messages after progress cleanup
-                if (appIdFuture != null) {
-                    logger.info("App uploaded successfully with ID: {}", appId);
-                }
-                if (testSuiteIdFuture != null) {
-                    logger.info("Test suite uploaded successfully with ID: {}", testSuiteId);
-                }
+            }
+
+            // Show success messages (unified flow for both progress and non-progress cases)
+            if (appIdFuture != null) {
+                logger.info("App uploaded successfully with ID: {}", appId);
+            }
+            if (testSuiteIdFuture != null) {
+                logger.info("Test suite uploaded successfully with ID: {}", testSuiteId);
             }
         } catch (CompletionException e) {
             // Cleanup progress display on error

src/main/java/io/github/lambdatest/gradle/LambdaUploaderTask.java

@@ -59,34 +59,25 @@ public void uploadApkToLambdaTest() {
         try {
             if (appIdFuture != null) {
                 appId = appIdFuture.join();
-                if (!progressEnabled) {
-                    logger.lifecycle(
-                            "\u001B[32mApp uploaded successfully with ID: {}\u001B[0m", appId);
-                }
             }
 
             if (testSuiteIdFuture != null) {
                 testSuiteId = testSuiteIdFuture.join();
-                if (!progressEnabled) {
-                    logger.lifecycle(
-                            "\u001B[32mTest suite uploaded successfully with ID: {}\u001B[0m",
-                            testSuiteId);
-                }
             }
 
-            // Clear progress display if enabled
+            // Clear progress display if enabled, then show success messages
             if (progressEnabled) {
                 ProgressTracker.cleanup();
-                // Show success messages after progress cleanup
-                if (appIdFuture != null) {
-                    logger.lifecycle(
-                            "\u001B[32mApp uploaded successfully with ID: {}\u001B[0m", appId);
-                }
-                if (testSuiteIdFuture != null) {
-                    logger.lifecycle(
-                            "\u001B[32mTest suite uploaded successfully with ID: {}\u001B[0m",
-                            testSuiteId);
-                }
+            }
+
+            // Show success messages (unified flow for both progress and non-progress cases)
+            if (appIdFuture != null) {
+                logger.lifecycle("\u001B[32mApp uploaded successfully with ID: {}\u001B[0m", appId);
+            }
+            if (testSuiteIdFuture != null) {
+                logger.lifecycle(
+                        "\u001B[32mTest suite uploaded successfully with ID: {}\u001B[0m",
+                        testSuiteId);
             }
         } catch (CompletionException e) {
             // Cleanup progress display on error