fix(testapp-support): Fixes to allow test-app to use notification listeners. (#66)

Author: yasirfolio3

optimizely/client/factory.go

@@ -32,6 +32,7 @@ import (
 type Options struct {
 	Context              context.Context
 	ProjectConfigManager optimizely.ProjectConfigManager
+	DecisionService      decision.DecisionService
 }
 
 // OptimizelyFactory is used to construct an instance of the OptimizelyClient
@@ -104,8 +105,12 @@ func (f OptimizelyFactory) ClientWithOptions(clientOptions Options) (*Optimizely
 		return client, errors.New("unable to instantiate client: no project config manager, SDK key, or a Datafile provided")
 	}
 
-	// @TODO: allow decision service to be passed in via options
-	client.decisionService = decision.NewCompositeService(notificationCenter)
+	if clientOptions.DecisionService != nil {
+		client.decisionService = clientOptions.DecisionService
+	} else {
+		client.decisionService = decision.NewCompositeService(notificationCenter)
+	}
+
 	client.isValid = true
 	return client, nil
 }

optimizely/decision/entities.go

@@ -39,7 +39,7 @@ type Source string
 
 const (
 	// Rollout - the decision came from a rollout
-	Rollout Source = "Rollout"
+	Rollout Source = "rollout"
 )
 
 // Decision contains base information about a decision