-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
CucumberPlugin.cucumber is an InputKey. #29
Comments
Thanks, I will take a look over the next few days. |
I will take a look at this, this evening. |
Still investigating, will setup a waffle board and bundle a few improvements into next release. |
Hi, I just did a re-write of the 'CucumberRunner'. If you run via the 'sbt test' then you don't need the plugin. The plugin is just used to provide a new command. I can look into this issue, but I am wondering if the plugin is really necessary, i.e. how many people need a separate command 'sbt cucumber' vs just running via 'sbt test'. My ex-firm use 'sbt cucumber', so I'll probably keep updating it. Will take a look shortly, delayed a bit with the runner re-write. |
We have a multi-module set-up, with a module containing our BDD tests in the main source tree, and some unit tests for that module in the test tree. So running sbt cucumber to run the BDD tests, and sbt test to run the unit tests (which test some aspects of the BDD code) works for us. |
Ok, I'll take another look at the plugin and tart it up a bit, may be a few days before I get chance (probably Monday). |
sbt prevents sequencing of InputKeys.
CucumberPlugin.cucumber is defined as an InputKey, because it can pass additional commandline parameters.
Would it be an improvement to have a TaskKey that runs all cucumber tests, and a separate InputKey to run specific tests/pass other parameters?
For example how there is a test TaskKey[Unit] and testOnly InputKey[Unit]
The text was updated successfully, but these errors were encountered: