feat: Add support for step argument (#149) (#150)

* feat: Add support for step argument (#149)

* style: update linter

.github/workflows/ci.yml

@@ -18,14 +18,14 @@ jobs:
         uses: golangci/golangci-lint-action@v3
         with:
           # Required: the version of golangci-lint is required and must be specified without patch version: we always use the latest patch version.
-          version: v1.47
+          version: v1.55
 
   coverage:
     name: Build
     runs-on: ubuntu-latest
     strategy:
       matrix:
-        go: ['1.12', '1.13', '1.14', '1.15', '1.16', '1.17', '1.18']
+        go: ['1.12', '1.13', '1.14', '1.15', '1.16', '1.17', '1.18', '1.19', '1.20', '1.21']
     env:
       GOFLAGS: -mod=readonly
       GOPROXY: https://proxy.golang.org

.golangci.yml

@@ -16,6 +16,7 @@ linters:
     - golint
     - scopelint
     - gosec
+    - depguard
 
     # Disabled after upgrading golangci-lint
     # Enable them after issues are resolved

context_test.go

@@ -3,14 +3,14 @@ package gobdd
 import (
 	"testing"
 
-	"github.com/stretchr/testify/assert"
+	"github.com/stretchr/testify/require"
 )
 
 func TestContextNilInGetError(t *testing.T) {
 	ctx := NewContext()
 	ctx.Set("err", nil)
 
 	res, err := ctx.GetError("err")
-	assert.NoError(t, err)
-	assert.Nil(t, res)
+	require.NoError(t, err)
+	require.NoError(t, res)
 }

features/argument.feature

@@ -0,0 +1,21 @@
+Feature: Argument feature
+  Scenario: compare text with argument
+    When I concat text "Hello " and argument:
+      """
+      World!
+      """
+    Then the result should equal argument:
+      """
+      Hello World!
+      """
+  Scenario: compare text with multiline argument
+    When I concat text "Hello " and argument:
+      """
+      New
+      World!
+      """
+    Then the result should equal argument:
+      """
+      Hello New
+      World!
+      """

gobdd.go

@@ -152,12 +152,12 @@ type stepDef struct {
 }
 
 type StepTest interface {
-	Log(...interface{})
-	Logf(string, ...interface{})
-	Fatal(...interface{})
-	Fatalf(string, ...interface{})
-	Errorf(string, ...interface{})
-	Error(...interface{})
+	Log(args ...interface{})
+	Logf(format string, args ...interface{})
+	Fatal(args ...interface{})
+	Fatalf(format string, args ...interface{})
+	Errorf(format string, args ...interface{})
+	Error(args ...interface{})
 
 	Fail()
 	FailNow()
@@ -541,6 +541,10 @@ func (s *Suite) runStep(ctx Context, t *testing.T, step *msgs.GherkinDocument_Fe
 	}
 
 	params := def.expr.FindSubmatch([]byte(step.Text))[1:]
+	if argument, ok := step.Argument.(*msgs.GherkinDocument_Feature_Step_DocString_); ok {
+		params = append(params, []byte(argument.DocString.Content))
+	}
+
 	t.Run(fmt.Sprintf("%s %s", strings.TrimSpace(step.Keyword), step.Text), func(t *testing.T) {
 		// NOTE consider passing t as argument to step hooks
 		ctx.Set(TestingTKey{}, t)

gobdd_go1_16.go

@@ -13,7 +13,7 @@ import (
 func WithFeaturesFS(fs fs.FS, path string) func(*SuiteOptions) {
 	return func(options *SuiteOptions) {
 		options.featureSource = fsFeatureSource{
-			fs:      fs,
+			fs:   fs,
 			path: path,
 		}
 	}

gobdd_test.go

@@ -68,6 +68,14 @@ func TestParameterTypes(t *testing.T) {
 	suite.Run()
 }
 
+func TestArguments(t *testing.T) {
+	suite := NewSuite(t, WithFeaturesPath("features/argument.feature"))
+	suite.AddStep(`the result should equal argument:`, checkt)
+	suite.AddStep(`I concat text {text} and argument:`, concat)
+
+	suite.Run()
+}
+
 func TestScenarioOutlineExecutesAllTests(t *testing.T) {
 	c := 0
 	suite := NewSuite(t, WithFeaturesPath("features/outline.feature"))