Closed jonn-set closed 5 years ago
@BorisOsipov Can you review this and merge it please?
Hi, its been 2 months, appreciate you guys are busy but can someone have a look at this merge request please?
@jonn-set would you mind adding unit test for this?
@BorisOsipov, sure, will do.
I ran through the existing tests and was wondering if there's a way to check that the right number of scenarios have been picked up using CucumberAdapter. It looks like all the tests call run function to execute the tests, which only return 0 or 1, is there something else that I can use as I don't really need to run the whole test, just make sure the right scenarios are filtered for testing.
Thanks
@jonn-set I am not ready to drive deep into current testing suite. I would like to have corresponding test at least to make sure we won't have regression in future with this fix
@BorisOsipov I have added some unit tests, have a look and if you are happy with it, please merge.
Thank you
@BorisOsipov, thanks for merging, when will this be released?
@christian-bromann ping
@christian-bromann, was this released?
I am using
wdio-cucumber-framework
withcucumber 4
and have seen that passing tag expression that includes tags atExample
level is not getting filtered, instead it end up throwing an exception.If I pass a tag expression of
@F1 and @Sc01 and @ex01
, it will throw the below exception:Error
My feature file
If I remove the
@ex01
tag then it will work and the test runs. I was usingcucumber 1
and compatiblewdio-cucumber-framework
earlier with the old type tags and it use to work without any issues