Skip to content

Commit

Permalink
IWF-472: trying things
Browse files Browse the repository at this point in the history
  • Loading branch information
samuel27m committed Jan 28, 2025
1 parent 237ea83 commit 4c447d5
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 3 deletions.
4 changes: 2 additions & 2 deletions integ/helpers/errors.go
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@ package helpers
import "testing"

func FailTestWithError(error error, t *testing.T) {
t.Fatalf("%s - Test failed with error: %v", t.Name(), error)
t.Errorf("%s - Test failed with error: %v", t.Name(), error)
}

func FailTestWithErrorMessage(errorMessage string, t *testing.T) {
t.Fatalf("%s - Test failed with error: %s", t.Name(), errorMessage)
t.Errorf("%s - Test failed with error: %s", t.Name(), errorMessage)
}
5 changes: 4 additions & 1 deletion integ/workflow/wf_state_api_timeout/routers.go
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
package wf_state_api_timeout

import (
"fmt"
"github.com/gin-gonic/gin"
"github.com/indeedeng/iwf/gen/iwfidl"
"github.com/indeedeng/iwf/integ/helpers"
Expand Down Expand Up @@ -53,7 +54,9 @@ func (h *handler) ApiV1WorkflowStateStart(c *gin.Context, t *testing.T) {
return
}
}

fmt.Println(

Check failure on line 57 in integ/workflow/wf_state_api_timeout/routers.go

View workflow job for this annotation

GitHub Actions / golangci

[golangci] integ/workflow/wf_state_api_timeout/routers.go#L57

S1038: should use fmt.Printf instead of fmt.Println(fmt.Sprintf(...)) (but don't forget the newline) (gosimple)
Raw output
integ/workflow/wf_state_api_timeout/routers.go:57:2: S1038: should use fmt.Printf instead of fmt.Println(fmt.Sprintf(...)) (but don't forget the newline) (gosimple)
	fmt.Println(
	^
fmt.Sprintf("WorkflowType: %v", req.GetWorkflowType()),
)
helpers.FailTestWithErrorMessage("should not get here", t)
}

Expand Down

0 comments on commit 4c447d5

Please sign in to comment.