Merge pull request #480 from kool-dev/kool-status-tweaks

`kool status` improvement

Author: fabriciojs

commands/status.go

@@ -50,7 +50,7 @@ func NewKoolStatus() *KoolStatus {
 		checker.NewChecker(defaultKoolService.shell),
 		network.NewHandler(defaultKoolService.shell),
 		environment.NewEnvStorage(),
-		builder.NewCommand("docker", "compose", "ps", "--all", "--services"),
+		builder.NewCommand("docker", "compose", "config", "--services"),
 		builder.NewCommand("docker", "compose", "ps", "--all", "--quiet"),
 		builder.NewCommand("docker", "ps", "--all", "--format", "{{.Status}}|{{.Ports}}"),
 		shell.NewTableWriter(),
@@ -65,7 +65,9 @@ func (s *KoolStatus) Execute(args []string) (err error) {
 		return
 	}
 
-	if services, err = s.getServices(); err != nil || len(services) == 0 {
+	if services, err = s.getServices(); err != nil {
+		return
+	} else if len(services) == 0 {
 		s.Shell().Warning("No services found.")
 		return
 	}

commands/status_test.go

@@ -166,13 +166,27 @@ func TestFailedGetServicesStatusCommand(t *testing.T) {
 
 	assertExecGotError(t, cmd, "exec err")
 
-	expected := "No services found."
+	expected := ""
 
 	output := fmt.Sprint(f.shell.(*shell.FakeShell).WarningOutput...)
 
 	if output != expected {
 		t.Errorf("Expected '%s', got '%s'", expected, output)
 	}
+
+	f.getServicesCmd.(*builder.FakeCommand).MockExecError = nil
+
+	if err := cmd.Execute(); err != nil {
+		t.Errorf("unexpected error: %v", err)
+	}
+
+	expected = "No services found."
+
+	output = fmt.Sprint(f.shell.(*shell.FakeShell).WarningOutput...)
+
+	if output != expected {
+		t.Errorf("Expected '%s', got '%s'", expected, output)
+	}
 }
 
 func TestFailedDependenciesStatusCommand(t *testing.T) {