Skip to content

Commit

Permalink
remove error parameter in getBusinessAppCR
Browse files Browse the repository at this point in the history
Signed-off-by: Megrez Lu <[email protected]>
  • Loading branch information
lujiajing1126 committed Jan 22, 2021
1 parent 5854c80 commit 6e7c287
Show file tree
Hide file tree
Showing 3 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion test/e2e/examples1_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -104,7 +104,7 @@ func (suite *ExamplesTestSuite) TestBusinessApp() {
require.NoError(t, err)

// Now deploy examples/business-application-injected-sidecar.yaml
businessAppCR := getBusinessAppCR(err)
businessAppCR := getBusinessAppCR()
defer os.Remove(businessAppCR.Name())
cmd := exec.Command("kubectl", "create", "--namespace", namespace, "--filename", businessAppCR.Name())
output, err := cmd.CombinedOutput()
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/istio_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -85,7 +85,7 @@ func (suite *IstioTestSuite) TestEnvoySidecar() {
require.NoError(t, err)

// Now deploy examples/business-application-injected-sidecar.yaml
businessAppCR := getBusinessAppCR(err)
businessAppCR := getBusinessAppCR()
defer os.Remove(businessAppCR.Name())
cmd := exec.Command("kubectl", "create", "--namespace", namespace, "--filename", businessAppCR.Name())
output, err := cmd.CombinedOutput()
Expand Down
2 changes: 1 addition & 1 deletion test/e2e/utils.go
Original file line number Diff line number Diff line change
Expand Up @@ -890,7 +890,7 @@ func waitForDeploymentAndUpdate(deploymentName, containerName string, update fun
})
}

func getBusinessAppCR(err error) *os.File {
func getBusinessAppCR() *os.File {
content, err := ioutil.ReadFile("../../examples/business-application-injected-sidecar.yaml")
require.NoError(t, err)
newContent := strings.Replace(string(content), "image: jaegertracing/vertx-create-span:operator-e2e-tests", "image: "+vertxExampleImage, 1)
Expand Down

0 comments on commit 6e7c287

Please sign in to comment.