Skip to content

Commit

Permalink
fix unit test expectations for context
Browse files Browse the repository at this point in the history
  • Loading branch information
bvtujo committed Jun 26, 2020
1 parent be1134a commit 89d1469
Show file tree
Hide file tree
Showing 3 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions internal/pkg/manifest/lb_web_svc_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ func TestLoadBalancedWebSvc_MarshalBinary(t *testing.T) {
mockDependencies: func(ctrl *gomock.Controller, manifest *LoadBalancedWebService) {
m := mocks.NewMockParser(ctrl)
manifest.parser = m
m.EXPECT().Parse(lbWebSvcManifestPath, *manifest).Return(nil, errors.New("some error"))
m.EXPECT().Parse(lbWebSvcManifestPath, *manifest, gomock.Any()).Return(nil, errors.New("some error"))
},

wantedError: errors.New("some error"),
Expand All @@ -35,7 +35,7 @@ func TestLoadBalancedWebSvc_MarshalBinary(t *testing.T) {
mockDependencies: func(ctrl *gomock.Controller, manifest *LoadBalancedWebService) {
m := mocks.NewMockParser(ctrl)
manifest.parser = m
m.EXPECT().Parse(lbWebSvcManifestPath, *manifest).Return(&template.Content{Buffer: bytes.NewBufferString("hello")}, nil)
m.EXPECT().Parse(lbWebSvcManifestPath, *manifest, gomock.Any()).Return(&template.Content{Buffer: bytes.NewBufferString("hello")}, nil)

},

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ image:
# Path to your service's Dockerfile.
build: ./subscribers/Dockerfile
# Path to the build context directory relative to workspace root.
context: ./subscribers
context: subscribers
# Port exposed through your container to route traffic to it.
port: 8080
healthcheck:
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ image:
# Path to your service's Dockerfile.
build: ./subscribers/Dockerfile
# Path to the build context directory relative to workspace root.
context: ./subscribers
context: subscribers
# Port exposed through your container to route traffic to it.
port: 8080

Expand Down

0 comments on commit 89d1469

Please sign in to comment.