add status stopped

Author: giulio93

internal/orchestrator/helpers.go

@@ -44,11 +44,11 @@ type AppStatusInfo struct {
 // For app that have at least 1 dependency, we calculate the overall state
 // as follow:
 //
-//	running: all running
-//	stopped: all stopped
-//	failed: at least one failed
-//	stopping: at least one stopping
-//	starting: at least one starting
+//		running: all running
+//		stopped: all stopped
+//		failed: at least one failed
+//	    stopped: at least one stopped
+//		starting: at least one starting
 func parseAppStatus(containers []container.Summary) []AppStatusInfo {
 	apps := make([]AppStatusInfo, 0, len(containers))
 	appsStatusMap := make(map[string][]Status)
@@ -93,6 +93,10 @@ func parseAppStatus(containers []container.Summary) []AppStatusInfo {
 			appendResult(appPath, StatusStopping)
 			continue
 		}
+		if slices.ContainsFunc(s, func(v Status) bool { return v == StatusStopped }) {
+			appendResult(appPath, StatusFailed)
+			continue
+		}
 		if slices.ContainsFunc(s, func(v Status) bool { return v == StatusStarting }) {
 			appendResult(appPath, StatusStarting)
 			continue

internal/orchestrator/helpers_test.go

@@ -46,7 +46,7 @@ func TestParseAppStatus(t *testing.T) {
 		},
 		{
 			name:           "failed container takes precedence over stopping and starting",
-			containerState: []container.ContainerState{container.StateRunning, container.StateDead, container.StateRemoving, container.StateRestarting},
+			containerState: []container.ContainerState{container.StateRunning, container.StateDead, container.StateRemoving, container.StateRestarting, container.StateExited},
 			want:           StatusFailed,
 		},
 		{
@@ -61,7 +61,7 @@ func TestParseAppStatus(t *testing.T) {
 		},
 		{
 			name:           "starting",
-			containerState: []container.ContainerState{container.StateRestarting, container.StateExited},
+			containerState: []container.ContainerState{container.StateRestarting},
 			want:           StatusStarting,
 		},
 	}