Operator has a chance of not cleaning up what it can

[KevinJCross]

Operator could be missing removing policies from dead apps.

on line 46 of application syncer function Operate()
We exit early rather than continuing over the rest of the list. This could cause a single continuous failure in the db to create a buildup of autoscaler policies without apps.

func (as ApplicationSynchronizer) Operate() {
	// Get all the application details from policyDB
	appIds, err := as.policyDb.GetAppIds()
	if err != nil {
		as.logger.Error("failed-to-get-apps", err)
		return
	}
	// For each app check if they really exist or not via CC api call
	for appID := range appIds {
		_, err = as.cfClient.GetApp(appID)
		if err != nil {
			as.logger.Error("failed-to-get-app-info", err)
			if models.IsNotFound(err) {
				// Application does not exist, lets clean up app details from policyDB
				err = as.policyDb.DeletePolicy(appID)
				if err != nil {
					as.logger.Error("failed-to-prune-non-existent-application-details", err)
	>>> here 		//TODO make this a continue and write a test.
					return
				}
				as.logger.Info("successfully-pruned-non-existent-applcation", lager.Data{"appid": appID})
			}
		}
	}
}