tags at Example causing exception about wdio-cucumber-framework HOT 9 CLOSED

@BorisOsipov Can you review this and merge it please?

from wdio-cucumber-framework.

Hi, its been 2 months, appreciate you guys are busy but can someone have a look at this merge request please?

from wdio-cucumber-framework.

@jonn-set would you mind adding unit test for this?

from wdio-cucumber-framework.

@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

from wdio-cucumber-framework.

@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

from wdio-cucumber-framework.

@BorisOsipov I have added some unit tests, have a look and if you are happy with it, please merge.

Thank you

from wdio-cucumber-framework.

@BorisOsipov, thanks for merging, when will this be released?

from wdio-cucumber-framework.

@christian-bromann ping

from wdio-cucumber-framework.

@christian-bromann, was this released?

from wdio-cucumber-framework.