feat: reuse wait logic from RequestHelper

There is common logic to wait for given resource status, this commit
leverages that instead of custom logic for CE deletion.

Author: stefanoboriero

src/main/java/io/seqera/tower/cli/commands/computeenvs/DeleteCmd.java

@@ -30,6 +30,8 @@
 import java.util.Collections;
 import java.util.concurrent.TimeUnit;
 
+import static io.seqera.tower.cli.utils.ResponseHelper.waitStatus;
+
 @Command(
         name = "delete",
         description = "Delete a compute environment."
@@ -64,7 +66,7 @@ protected Response exec() throws ApiException {
             computeEnvsApi().deleteComputeEnv(id, wspId, null);
             deletedId = id;
             deletedWspId = wspId;
-            return new ComputeEnvDeleted(id, workspaceRef(wspId));
+            return new ComputeEnvDeleted(id, workspaceRef(wspId), wspId);
         } catch (ApiException e) {
             if (e.getCode() == 403) {
                 // Customize the forbidden message
@@ -80,34 +82,18 @@ protected Integer onBeforeExit(int exitCode, Response response) {
             return exitCode;
         }
 
+        ComputeEnvDeleted computeEnv = (ComputeEnvDeleted) response;
         boolean showProgress = app().output != OutputType.json;
 
         try {
-            long sleepMillis = 2000;
-            while (true) {
-                ComputeEnvStatus status = checkComputeEnvStatus(deletedId, deletedWspId);
-
-                if (status == null) {
-                    // CE is gone (404) - deletion succeeded
-                    if (showProgress) {
-                        app().getOut().println();
-                    }
-                    return CommandLine.ExitCode.OK;
-                }
-
-                if (status == ComputeEnvStatus.ERRORED) {
-                    app().getErr().println("ERROR: Compute environment disposal failed (status: ERRORED). AWS resources may not have been cleaned up.");
-                    return CommandLine.ExitCode.SOFTWARE;
-                }
-
-                if (showProgress) {
-                    app().getOut().print(".");
-                    app().getOut().flush();
-                }
-
-                TimeUnit.MILLISECONDS.sleep(sleepMillis);
-                sleepMillis = Math.min(sleepMillis + 1000, 120_000);
-            }
+            return waitStatus(
+                    app().getOut(),
+                    showProgress,
+                    ComputeEnvStatus.DELETED,
+                    ComputeEnvStatus.values(),
+                    () -> checkComputeEnvStatus(computeEnv.id, computeEnv.workspaceId),
+                    ComputeEnvStatus.ERRORED, ComputeEnvStatus.INVALID
+            );
         } catch (InterruptedException e) {
             Thread.currentThread().interrupt();
             return exitCode;

src/main/java/io/seqera/tower/cli/responses/computeenvs/ComputeEnvDeleted.java

@@ -22,9 +22,11 @@ public class ComputeEnvDeleted extends Response {
 
     public final String id;
     public final String workspaceRef;
+    public final Long workspaceId;
 
-    public ComputeEnvDeleted(String id, String workspaceRef) {
+    public ComputeEnvDeleted(String id, String workspaceRef, Long workspaceId) {
         this.id = id;
+        this.workspaceId = workspaceId;
         this.workspaceRef = workspaceRef;
     }
 

src/test/java/io/seqera/tower/cli/computeenvs/ComputeEnvsCmdTest.java

@@ -76,7 +76,7 @@ void testDelete(OutputType format, MockServerClient mock) {
         );
 
         ExecOut out = exec(format, mock, "compute-envs", "delete", "-i", "vYOK4vn7spw7bHHWBDXZ2");
-        assertOutput(format, out, new ComputeEnvDeleted("vYOK4vn7spw7bHHWBDXZ2", USER_WORKSPACE_NAME));
+        assertOutput(format, out, new ComputeEnvDeleted("vYOK4vn7spw7bHHWBDXZ2", USER_WORKSPACE_NAME, null));
     }
 
     @Test
@@ -733,7 +733,7 @@ void testDeleteWaitHappyPath(MockServerClient mock) {
         mock.when(
                 request().withMethod("GET").withPath("/compute-envs/vYOK4vn7spw7bHHWBDXZ2"), exactly(1)
         ).respond(
-                response().withStatusCode(404)
+                response().withStatusCode(200).withBody("{\"computeEnv\":{\"id\":\"vYOK4vn7spw7bHHWBDXZ2\",\"name\":\"demo\",\"platform\":\"aws-batch\",\"status\":\"DELETED\"}}").withContentType(MediaType.APPLICATION_JSON)
         );
 
         ExecOut out = exec(mock, "compute-envs", "delete", "-i", "vYOK4vn7spw7bHHWBDXZ2", "--wait");