Minor tweaks

api/healthcheck/healthcheck_controller.go

@@ -62,7 +62,7 @@ func (c *controller) healthCheck(r *web.Request) (*web.Response, error) {
 
 func (c *controller) aggregate(state map[string]h.State) *health.Health {
 	if len(state) == 0 {
-		return health.New().WithDetail("error", "no health indicators registered").Unknown()
+		return health.New().WithDetail("error", "no health indicators registered")
 	}
 	overallStatus := health.StatusUp
 	for i, v := range state {

pkg/health/registry_test.go

@@ -61,9 +61,7 @@ var _ = Describe("Healthcheck Registry", func() {
 			Expect(newIndicator.Interval()).To(Equal(DefaultIndicatorSettings().Interval))
 			Expect(newIndicator.FailuresTreshold()).To(Equal(DefaultIndicatorSettings().FailuresTreshold))
 		})
-	})
 
-	When("When configure indicators", func() {
 		It("Should configure with provided settings", func() {
 			newIndicator := &testIndicator{}
 			registry.HealthIndicators = append(registry.HealthIndicators, newIndicator)

pkg/health/settings_test.go

@@ -84,4 +84,16 @@ var _ = Describe("Healthcheck Settings", func() {
 			assertValidationErrorOccured(err)
 		})
 	})
+
+	When("Indicator with positive treshold and interval > 30", func() {
+		It("Should be considered valid", func() {
+			interval = 30
+			failuresTreshold = 3
+			registerIndicatorSettings()
+
+			err := settings.Validate()
+
+			Expect(err).ShouldNot(HaveOccurred())
+		})
+	})
 })

pkg/health/types.go

@@ -127,24 +127,6 @@ func (h *Health) WithDetail(key string, val interface{}) *Health {
 	return h
 }
 
-// Up sets the health status to up
-func (h *Health) Up() *Health {
-	h.Status = StatusUp
-	return h
-}
-
-// Down sets the health status to down
-func (h *Health) Down() *Health {
-	h.Status = StatusDown
-	return h
-}
-
-// Unknown sets the health status to unknown
-func (h *Health) Unknown() *Health {
-	h.Status = StatusUnknown
-	return h
-}
-
 // WithDetails adds the given details to the health
 func (h *Health) WithDetails(details map[string]interface{}) *Health {
 	for k, v := range details {