Skip to content

Commit

Permalink
More fixes
Browse files Browse the repository at this point in the history
  • Loading branch information
terryyylim committed Mar 2, 2023
1 parent e04b26a commit 468dad8
Show file tree
Hide file tree
Showing 2 changed files with 13 additions and 7 deletions.
19 changes: 13 additions & 6 deletions management-service/controller/configuration_controller_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -22,11 +22,15 @@ func (s *ConfigurationControllerTestSuite) SetupSuite() {

pubSubConfigProject := "dev"
pubSubConfigTopicName := "xp-update"
messageQueueKind := schema.MessageQueueKindPubsub

treatmentServicePluginConfiguration := schema.TreatmentServiceConfig{
PubSub: &schema.PubSub{
Project: &pubSubConfigProject,
TopicName: &pubSubConfigTopicName,
MessageQueueConfig: &schema.MessageQueueConfig{
Kind: &messageQueueKind,
PubSub: &schema.PubSub{
Project: &pubSubConfigProject,
TopicName: &pubSubConfigTopicName,
},
},
SegmenterConfig: &schema.SegmenterConfig{
"s2_ids": map[string]interface{}{
Expand Down Expand Up @@ -65,9 +69,12 @@ func (s *ConfigurationControllerTestSuite) TestGetTreatmentServicePluginConfig()
body, err := io.ReadAll(resp.Body)
s.Suite.Require().NoError(err)
s.Suite.Assert().JSONEq(`{"data":{
"pub_sub":{
"project":"dev",
"topic_name":"xp-update"
"message_queue_config": {
"kind":"pubsub",
"pub_sub":{
"project":"dev",
"topic_name":"xp-update"
}
},
"segmenter_config":{
"s2_ids":{
Expand Down
1 change: 0 additions & 1 deletion plugins/turing/runner/experiment_runner.go
Original file line number Diff line number Diff line change
Expand Up @@ -257,7 +257,6 @@ func (er *experimentRunner) startBackgroundServices(
}
}()
}
return
}

func (er *experimentRunner) getRequestParams(
Expand Down

0 comments on commit 468dad8

Please sign in to comment.