Skip to content

Commit

Permalink
fix: resolve comments made in PR #1229 for issue #1199
Browse files Browse the repository at this point in the history
Signed-off-by: Marc-Philippe Fuller <[email protected]>
  • Loading branch information
marcpfuller committed Nov 30, 2022
1 parent 3ad1ce6 commit c8c98ea
Show file tree
Hide file tree
Showing 7 changed files with 132 additions and 132 deletions.
4 changes: 2 additions & 2 deletions internal/app/service.go
Original file line number Diff line number Diff line change
Expand Up @@ -84,7 +84,7 @@ type Service struct {
config *common.ConfigurationStruct
lc logger.LoggingClient
usingConfigurablePipeline bool
runtime *runtime.AppServiceRuntime
runtime *runtime.FunctionsPipelineRuntime
webserver *webserver.WebServer
ctx contextGroup
deferredFunctions []bootstrap.Deferred
Expand Down Expand Up @@ -549,7 +549,7 @@ func (svc *Service) Initialize() error {
return fmt.Errorf("unable to parse Service.RequestTimeout configuration as a time duration: %s", err.Error())
}

svc.runtime = runtime.NewAppServiceRuntime(svc.serviceKey, svc.targetType, svc.dic)
svc.runtime = runtime.NewFunctionPipelineRuntime(svc.serviceKey, svc.targetType, svc.dic)

// Bootstrapping is complete, so now need to retrieve the needed objects from the containers.
svc.lc = bootstrapContainer.LoggingClientFrom(svc.dic.Get)
Expand Down
8 changes: 4 additions & 4 deletions internal/app/service_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -256,7 +256,7 @@ func TestSetupHTTPTrigger(t *testing.T) {
},
}

testRuntime := runtime.NewAppServiceRuntime("", nil, dic)
testRuntime := runtime.NewFunctionPipelineRuntime("", nil, dic)
testRuntime.SetDefaultFunctionsPipeline(nil)

sdk.runtime = testRuntime
Expand All @@ -276,7 +276,7 @@ func TestSetupMessageBusTrigger(t *testing.T) {
},
},
}
testRuntime := runtime.NewAppServiceRuntime("", nil, dic)
testRuntime := runtime.NewFunctionPipelineRuntime("", nil, dic)
testRuntime.SetDefaultFunctionsPipeline(nil)

sdk.runtime = testRuntime
Expand Down Expand Up @@ -305,7 +305,7 @@ func TestSetDefaultFunctionsPipelineOneTransform(t *testing.T) {
service := Service{
lc: lc,
dic: dic,
runtime: runtime.NewAppServiceRuntime("", nil, dic),
runtime: runtime.NewFunctionPipelineRuntime("", nil, dic),
config: &common.ConfigurationStruct{
Trigger: common.TriggerInfo{
Type: TriggerTypeMessageBus,
Expand All @@ -324,7 +324,7 @@ func TestService_AddFunctionsPipelineForTopics(t *testing.T) {
service := Service{
lc: lc,
dic: dic,
runtime: runtime.NewAppServiceRuntime("", nil, dic),
runtime: runtime.NewFunctionPipelineRuntime("", nil, dic),
config: &common.ConfigurationStruct{
Trigger: common.TriggerInfo{
Type: TriggerTypeMessageBus,
Expand Down
2 changes: 1 addition & 1 deletion internal/app/triggermessageprocessor.go
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ import (

type simpleTriggerServiceBinding struct {
*Service
*runtime.AppServiceRuntime
*runtime.FunctionsPipelineRuntime
}

func (b *simpleTriggerServiceBinding) SecretProvider() messaging.SecretDataProvider {
Expand Down
Loading

0 comments on commit c8c98ea

Please sign in to comment.