Merge remote-tracking branch 'remotes/origin/master' into address_already_in_use_error

1 files changed, 12 insertions, 4 deletions

diff --git a/service/reload/config_test.go b/service/reload/config_test.go
index cefb6544..600975d3 100644
--- a/service/reload/config_test.go
+++ b/service/reload/config_test.go
@@ -36,8 +36,17 @@ func Test_Fake_ServiceConfig(t *testing.T) {
 
 func Test_Interval(t *testing.T) {
 	services := make(map[string]ServiceConfig)
+	services["test"] = ServiceConfig{
+		Enabled:   false,
+		Recursive: false,
+		Patterns:  nil,
+		Dirs:      nil,
+		Ignore:    nil,
+		service:   nil,
+	}
+
 	cfg := &Config{
-		Interval: time.Millisecond,
+		Interval: time.Millisecond, // should crash here
 		Patterns: nil,
 		Services: services,
 	}
@@ -45,11 +54,10 @@ func Test_Interval(t *testing.T) {
 }
 
 func Test_NoServiceConfig(t *testing.T) {
-	services := make(map[string]ServiceConfig)
 	cfg := &Config{
-		Interval: time.Millisecond,
+		Interval: time.Second,
 		Patterns: nil,
-		Services: services,
+		Services: nil,
 	}
 	assert.Error(t, cfg.Valid())
 }