godog/test_context_test.go
Carl Menezes c95871f3ce
Switch from golint to staticcheck (#457)
* Switch from golint to staticcheck

* Remove unused function

run_test.go:618:6: func passingStepDefWithoutReturn is unused (U1000)

* Remove unused function

suite.go:421:6: func isEmptyFeature is unused (U1000)

* Fix unnecessary use of fmt.Sprintf

test_context_test.go:45:66: unnecessary use of fmt.Sprintf (S1039)
test_context_test.go:46:61: unnecessary use of fmt.Sprintf (S1039)

* Fix CI error

https://github.com/cucumber/godog/runs/5146601108?check_suite_focus=true#step:7:28

* Change CI to run staticcheck instead of golint

* Use staticcheck that definitely pass

* Fix CI staticcheck error

https://github.com/cucumber/godog/runs/5147133955?check_suite_focus=true#step:6:17

* Only run staticcheck for Go 1.17

Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>

* Add staticcheck linux binary

* Update Go module dependencies in _examples

* Use static check binary in bin for CI

* Reduce number of dependencies

Also add a note to CONTRIBUTING.md about the _examples module

* Pin the version of staticcheck

Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>

Co-authored-by: Matt Wynne <matt@cucumber.io>
Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>
2022-02-15 13:37:27 -08:00

67 строки
2,8 КиБ
Go

package godog
import (
"fmt"
"regexp"
"testing"
. "github.com/smartystreets/goconvey/convey"
)
func TestScenarioContext_Step(t *testing.T) {
Convey("When adding steps to ScenarioContext ", t, func() {
ctx := ScenarioContext{suite: &suite{}}
Convey("It should accept steps defined with regexp.Regexp", func() {
re := regexp.MustCompile(`(?:it is a test)?.{10}x*`)
So(func() { ctx.Step(re, okEmptyResult) }, ShouldNotPanic)
})
Convey("It should accept steps defined with bytes slice", func() {
So(func() { ctx.Step([]byte("(?:it is a test)?.{10}x*"), okEmptyResult) }, ShouldNotPanic)
})
Convey("It should accept steps handler with empty return", func() {
So(func() { ctx.Step(".*", okEmptyResult) }, ShouldNotPanic)
})
Convey("It should accept steps handler with error return", func() {
So(func() { ctx.Step(".*", okErrorResult) }, ShouldNotPanic)
})
Convey("It should accept steps handler with string slice return", func() {
So(func() { ctx.Step(".*", okSliceResult) }, ShouldNotPanic)
})
Convey("It should panic if step expression is neither a string, regex or byte slice", func() {
So(func() { ctx.Step(1251, okSliceResult) }, ShouldPanicWith, fmt.Sprintf("expecting expr to be a *regexp.Regexp or a string, got type: %T", 12))
})
Convey("It should panic if step handler", func() {
Convey("is not a function", func() {
So(func() { ctx.Step(".*", 124) }, ShouldPanicWith, fmt.Sprintf("expected handler to be func, but got: %T", 12))
})
Convey("has more than 2 return values", func() {
So(func() { ctx.Step(".*", nokLimitCase) }, ShouldPanicWith, "expected handler to return either zero, one or two values, but it has: 3")
So(func() { ctx.Step(".*", nokMore) }, ShouldPanicWith, "expected handler to return either zero, one or two values, but it has: 5")
})
Convey("return type is not an error or string slice or void", func() {
So(func() { ctx.Step(".*", nokInvalidReturnInterfaceType) }, ShouldPanicWith, "expected handler to return an error or context.Context, but got: interface")
So(func() { ctx.Step(".*", nokInvalidReturnSliceType) }, ShouldPanicWith, "expected handler to return []string for multistep, but got: []int")
So(func() { ctx.Step(".*", nokInvalidReturnOtherType) }, ShouldPanicWith, "expected handler to return an error or []string, but got: chan")
})
})
})
}
func okEmptyResult() {}
func okErrorResult() error { return nil }
func okSliceResult() []string { return nil }
func nokLimitCase() (string, int, error) { return "", 0, nil }
func nokMore() (int, int, int, int, error) { return 0, 0, 0, 0, nil }
func nokInvalidReturnInterfaceType() interface{} { return 0 }
func nokInvalidReturnSliceType() []int { return nil }
func nokInvalidReturnOtherType() chan int { return nil }