return an error if execution period is unset

health.go

@@ -59,12 +59,19 @@ type health struct {
 }
 
 func (h *health) RegisterCheck(check Check, opts ...CheckOption) error {
-	if check == nil || check.Name() == "" {
-		return errors.Errorf("misconfigured check %v", check)
+	if check == nil {
+		return errors.New("check must not be nil")
+	}
+	if check.Name() == "" {
+		return errors.New("check name must not be empty")
 	}
 
 	cfg := h.initCheckConfig(opts)
 
+	if cfg.executionPeriod <= 0 {
+		return errors.New("execution period must be greater than 0")
+	}
+
 	// checks are initially failing by default, but we allow overrides...
 	var initialErr error
 	if !cfg.initiallyPassing {

health_test.go

@@ -52,6 +52,21 @@ func TestHealthWithBogusCheck(t *testing.T) {
 	assert.Empty(t, results, "results after bogus register")
 }
 
+func TestRegisterCheckValidations(t *testing.T) {
+	h := New()
+
+	// should return an error for nil check
+	assert.EqualError(t, h.RegisterCheck(nil), "check must not be nil")
+	// should return an error for missing check name
+	assert.EqualError(t, h.RegisterCheck(&checks.CustomCheck{}), "check name must not be empty")
+	// Should return an error for missing execution period
+	assert.EqualError(t, h.RegisterCheck(&checks.CustomCheck{CheckName: "non-empty"}), "execution period must be greater than 0")
+
+	hWithExecPeriod := New(ExecutionPeriod(1 * time.Minute))
+	assert.NoError(t, hWithExecPeriod.RegisterCheck(&checks.CustomCheck{CheckName: "non-empty"}))
+
+}
+
 func TestRegisterDeregister(t *testing.T) {
 	leaktest.Check(t)
 

options.go

@@ -37,11 +37,7 @@ func WithHealthListeners(listener ...HealthListener) HealthOption {
 // WithDefaults sets all the Health object settings. It's not required to use this as no options is always default
 // This is a simple placeholder for any future defaults
 func WithDefaults() HealthOption {
-	return healthOptionFunc(func(h *health) {
-		if h.defaultExecutionPeriod <= 0 {
-			h.defaultExecutionPeriod = 1 * time.Minute
-		}
-	})
+	return healthOptionFunc(func(h *health) {})
 }
 
 // CheckOption configures a health check using the functional options paradigm