Merge pull request #8 from jianxiaoguo/master

feat(ps):add ps:stop/start command pretty print

Author: duanhongyi

ps/ps.go

@@ -127,11 +127,13 @@ func ByType(processes api.PodsList) api.PodTypes {
 
 		// Is processtype for process doesn't exist, create a new one
 		if !exists {
-			p := api.PodsList{}
-			status := "stopped"
-			if process.Name != "" {
-				p = api.PodsList{process}
-				status = "started"
+			p := api.PodsList{process}
+			status := "started"
+			if process.State == "stopped" && process.Replicas != 0 {
+				status = "stopped"
+			}
+			if process.Name == "" {
+				p = api.PodsList{}
 			}
 			pts = append(pts, api.PodType{
 				Type:     process.Type,

ps/ps_test.go

@@ -359,9 +359,9 @@ func TestByType(t *testing.T) {
 		{
 			Type: "abc",
 			PodsList: api.PodsList{
-				{Type: "abc", Name: "1", Started: started, Replicas: 3},
-				{Type: "abc", Name: "2", Started: started, Replicas: 3},
-				{Type: "abc", Name: "3", Started: started, Replicas: 3},
+				{Type: "abc", Name: "1", Started: started, Replicas: 3, State: "up"},
+				{Type: "abc", Name: "2", Started: started, Replicas: 3, State: "up"},
+				{Type: "abc", Name: "3", Started: started, Replicas: 3, State: "up"},
 			},
 			Replicas: 3,
 			Status:   "started",
@@ -375,36 +375,36 @@ func TestByType(t *testing.T) {
 		{
 			Type: "web",
 			PodsList: api.PodsList{
-				{Type: "web", Name: "test1", Started: started, Replicas: 3},
-				{Type: "web", Name: "test2", Started: started, Replicas: 3},
-				{Type: "web", Name: "test3", Started: started, Replicas: 3},
+				{Type: "web", Name: "test1", Started: started, Replicas: 3, State: "up"},
+				{Type: "web", Name: "test2", Started: started, Replicas: 3, State: "up"},
+				{Type: "web", Name: "test3", Started: started, Replicas: 3, State: "up"},
 			},
 			Replicas: 3,
 			Status:   "started",
 		},
 		{
 			Type: "worker",
 			PodsList: api.PodsList{
-				{Type: "worker", Name: "a", Started: started, Replicas: 3},
-				{Type: "worker", Name: "b", Started: started, Replicas: 3},
-				{Type: "worker", Name: "c", Started: started, Replicas: 3},
+				{Type: "worker", Name: "a", Started: started, Replicas: 3, State: "up"},
+				{Type: "worker", Name: "b", Started: started, Replicas: 3, State: "up"},
+				{Type: "worker", Name: "c", Started: started, Replicas: 3, State: "up"},
 			},
 			Replicas: 3,
 			Status:   "started",
 		},
 	}
 
 	input := api.PodsList{
-		{Type: "worker", Name: "c", Started: started, Replicas: 3},
-		{Type: "abc", Name: "2", Started: started, Replicas: 3},
-		{Type: "worker", Name: "b", Started: started, Replicas: 3},
-		{Type: "web", Name: "test1", Started: started, Replicas: 3},
-		{Type: "web", Name: "test3", Started: started, Replicas: 3},
-		{Type: "abc", Name: "1", Started: started, Replicas: 3},
-		{Type: "worker", Name: "a", Started: started, Replicas: 3},
-		{Type: "abc", Name: "3", Started: started, Replicas: 3},
-		{Type: "web", Name: "test2", Started: started, Replicas: 3},
-		{Type: "job", Replicas: 3},
+		{Type: "worker", Name: "c", Started: started, Replicas: 3, State: "up"},
+		{Type: "abc", Name: "2", Started: started, Replicas: 3, State: "up"},
+		{Type: "worker", Name: "b", Started: started, Replicas: 3, State: "up"},
+		{Type: "web", Name: "test1", Started: started, Replicas: 3, State: "up"},
+		{Type: "web", Name: "test3", Started: started, Replicas: 3, State: "up"},
+		{Type: "abc", Name: "1", Started: started, Replicas: 3, State: "up"},
+		{Type: "worker", Name: "a", Started: started, Replicas: 3, State: "up"},
+		{Type: "abc", Name: "3", Started: started, Replicas: 3, State: "up"},
+		{Type: "web", Name: "test2", Started: started, Replicas: 3, State: "up"},
+		{Type: "job", Replicas: 3, State: "stopped"},
 	}
 
 	actual := ByType(input)