* added the missing impl of json/events/junit/pretty - still need 'progress' and 'junit,pretty'
* added tests for "progress formatter"
* switched from tabs to spaces in the ambiguous steps error message
* rename some_scenarions_including_failing
to some_scenarios_including_failing
* changelog
* at some point someone changed the return type for nested steps from []string to godog.Steps but they forgot to adjust the type checks. The existing type checks were lax and unable to distinguish []string from godog.Steps but in a couple of places in the code the value is coerced to godog.Steps and so if someone returned []string then the code would blow up. Additionally there were some tests aroudn these types but they also had not been updated but the test was passing for the wrong reason - the particular test expected an error but the cause of the error wasn't the one the code expected.
* CHANGELOG.md
* use chatgpt to regen the top of the code based on the new tests
* use chatgpt to regen the top of the code based on the new tests
* corrected the error messages of the param checks to indicate that the problem is the function signature and not the args being passed to the function, also added numerous extra assertions on the precise error messages returned. Now that the precise error is being verified in the test I have improved certain error messages to that more accurate detail is included in the errors
* added further constraints to the step arg mapping tests
* removed redundant test
* include a step error result in the reported error even when the ctx is nil
* Minor doc and comment corrections
* Fixed bug where it was impossible to make attachments from 'after scenario' hook, also removed some dud comments.
* typo
* added basic detection for ambiguous steps, but causes an error and not yet recorded in the reports as 'Ambiguous', and no test cases figured out yet
* added initial support for detection of ambiguous steps - further work take a look at how cuke jvm report ambiguous steps and sets the step status to 'ambiguous' rather than my current solution which just blows the test up as a regular step error
* added suite_context_test and also introduced missing 'ambiguous' status to make cucumber jvm'
* update CHANGELOG for ambiguous step defs
* missed file from commit
* added internal/formatters/fmt_multi_test.go
* add tests for other peoples code
* added "ambigous" to the help text
* tests
* added some more tests for attachments
* Update internal/flags/flags.go
Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>
---------
Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>
* support multiple calls to the Attach() function from a single step
* run_progress_test.go changed so it's not sensitive to the name of the clone target directory
* applied code review comments also added _example/attachments
* applied code review comments also added _example/attachments
* applied code review comments also added _example/attachments
* Pretty Print when using rules (#440)
* Pretty Print when using rules (#440)
* fix a few formatting mistakes (#440)
* added test with rule and scenario outline (#440)
* Add example about incorrect project structure
This is to help reproduce #383
* Added some debugging statements
* Update go.sum for example project
* Made a cmd_run_test.go file in order to test and run the builderAndRunGodog function in order to see it fail
* added new assertion test
* Matt and I added debugging
Co-authored-by: Matt Wynne <matt@mattwynne.net>
* Matt and I tried to logging through the cobra command
by using cmd.OutOrStdout( )
* Improved some debugging
* Add a failing test for Builder that reproduces #383
* added new test for IncorrectProjectStructure #383
* Revert "Add a failing test for Builder that reproduces #383"
This reverts commit e5b26933b5d4e979009f8f4341448fa8322720d2.
* ignored vscode files
Co-authored-by: Matt Wynne <matt@cucumber.io>
* undid debugging changes
* undid debugging changes
* removed redundant test
* added check for incorrect project structure
we examined the output from running `go test` which tells us if we didn't
find any test files.
we tweaked the error message to follow the capitalization rules
Co-authored-by: Matt Wynne <matt@cucumber.io>
* Update internal/builder/builder_test.go
Co-authored-by: Matt Wynne <matt@mattwynne.net>
Co-authored-by: Matt Wynne <matt@cucumber.io>
Co-authored-by: Viacheslav Poturaev <nanopeni@gmail.com>
* Add new contextualized API for hooks and steps
* Make default context configurable
* Run AfterStep hooks even after failed steps, fixes#370
* Update CHANGELOG and README
* Add step result status to After hook, fixes#378
* Elaborate hooks documentation
* Add test to pass state between contextualized steps
* Update README with example of passing state between steps
* Fix step definition output for Data Tables
This fixes incorrect step definition output for Data Tables. The
PickleStepArgument_PickleTable type does not exist in the current
messages-go version.
* updated CHANGELOG.md
Issue: It is not possible to use function without return when
matching steps, resulting in a lot of Nil only error
returns.
Fix: Allows to provide empty result function by correctly matching
reflect Calls on step Handler.
When nothing is returned by the Handler, it will return
nil as if errors was nil.
* make pickleStepIDs unique accross multiple paths
* add test case for duplicate pickle ids
* go fmt
Co-authored-by: Rickard Englund <rickard.englund@skf.com>
* Improved the help text of the formatter flag in the run command
* Made some more changes to the help text for the run command
* made some small changes to the help text of the run and build cmd
