added comments

Author: Mat Ryer

start/start.go

@@ -70,3 +70,14 @@ func MustAll(stopGrace time.Duration, startStoppers ...StartStopper) map[StartSt
 	}
 	return errs
 }
+
+// StopAll stops all StartStopper types and returns another channel
+// which will close once all things have finished stopping.
+// For more information, see stop.All.
+func StopAll(stopGrace time.Duration, startStoppers ...StartStopper) <-chan stop.Signal {
+	stoppers := make([]stop.Stopper, len(startStoppers))
+	for i, ss := range startStoppers {
+		stoppers[i] = ss.(stop.Stopper)
+	}
+	return stop.All(stopGrace, stoppers...)
+}

start/start_test.go

@@ -93,3 +93,19 @@ func TestMustAll(t *testing.T) {
 	require.False(t, s3.running)
 
 }
+
+func TestStopAll(t *testing.T) {
+
+	s1 := &TestStarter{}
+	s2 := &TestStarter{}
+	s3 := &ErrorStarter{}
+
+	start.All(s1, s2, s3)
+
+	<-start.StopAll(1*time.Second, s1, s2, s3)
+
+	require.False(t, s1.running)
+	require.False(t, s2.running)
+	require.False(t, s3.running)
+
+}