Update internal/orchestrator/orchestrator.go

Co-authored-by: Luca Rinaldi <l.rinaldi@arduino.cc>

Author: giulio93

internal/api/handlers/app_delete.go

@@ -19,11 +19,12 @@ import (
 	"log/slog"
 	"net/http"
 
+	"github.com/docker/cli/cli/command"
+
 	"github.com/arduino/arduino-app-cli/internal/api/models"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator"
 	"github.com/arduino/arduino-app-cli/internal/orchestrator/app"
 	"github.com/arduino/arduino-app-cli/internal/render"
-	"github.com/docker/cli/cli/command"
 )
 
 func HandleAppDelete(

internal/e2e/daemon/app_test.go

@@ -438,7 +438,8 @@ func TestDeleteApp(t *testing.T) {
 		deleteResp, err := httpClient.DeleteApp(t.Context(), appToDeleteId)
 		require.NoError(t, err)
 		defer deleteResp.Body.Close()
-		require.Equal(t, http.StatusOK, deleteResp.StatusCode)
+		require.Equal(t, http.StatusInternalServerError, deleteResp.StatusCode)
+		fmt.Println(deleteResp.Body)
 
 		getResp, err := httpClient.GetAppDetails(t.Context(), appToDeleteId)
 		require.NoError(t, err)

internal/orchestrator/orchestrator.go

@@ -388,7 +388,7 @@ func stopAppWithCmd(ctx context.Context, docker command.Cli, app app.ArduinoApp,
 			return
 		}
 		if appStatus.Status != StatusStarting && appStatus.Status != StatusRunning {
-			yield(StreamMessage{error: fmt.Errorf("App %q is not running", app.Name)})
+			yield(StreamMessage{error: fmt.Errorf("app %q is not running", app.Name)})
 			return
 		}
 
@@ -406,7 +406,6 @@ func stopAppWithCmd(ctx context.Context, docker command.Cli, app app.ArduinoApp,
 			}
 		})
 
-		fmt.Printf("this is the main sketch path:%s", app.MainSketchPath)
 		if app.MainSketchPath != nil {
 			// TODO: check that the app sketch is running before attempting to stop it.