Merge pull request #19 from Financial-Times/respect-cache-refresh-period

Take into consideration category refresh period settings

cachingController.go

@@ -52,6 +52,18 @@ func (c *healthCheckController) updateCachedHealth(services map[string]service,
 			newMService := newMeasuredService(service)
 			c.measuredServices[service.name] = newMService
 			refreshPeriod := findShortestPeriod(categories)
+			categories, err := c.healthCheckService.getCategories()
+			if err != nil {
+				warnLogger.Printf("Cannot read categories: [%v]\n Using minimum refresh period for service [%s]", err, service.name)
+			} else {
+				for _, category := range categories {
+					if isStringInSlice(service.name, category.services) {
+						refreshPeriod = category.refreshPeriod
+						break
+					}
+				}
+			}
+			infoLogger.Printf("Scheduling check for service [%s] with refresh period [%v].\n", service.name, refreshPeriod)
 			go c.scheduleCheck(newMService, refreshPeriod, time.NewTimer(0))
 		}
 	}

controller_test.go

@@ -222,6 +222,7 @@ func TestRemoveAckServiceErr(t *testing.T) {
 }
 
 func TestRemoveAckHappyFlow(t *testing.T) {
+	initLogs(os.Stdout, os.Stdout, os.Stderr)
 	controller := initializeMockController("test", nil)
 	err := controller.removeAck(validService)
 	assert.Nil(t, err)