Upgrade to latest cucumber 3.2.1

[devth]

Not sure what all needs to happen to support upgrading but thought I'd start the conversation here.

[jsf-clabot]

All committers have signed the CLA.

[christian-bromann]

@devth as you can see in the tests, some of the integrations are failing. There are some breaking changes that need to get adapted here first. Any help is appreciated.

[devth]

Yep, I see. Any plans to make them? Should I dive in and try or is it a pretty involved task?

[christian-bromann]

Any plans to make them?

Not my highes priority right now

Should I dive in and try or is it a pretty involved task?

No, feel free to look into it

[devth]

Ok will do! (Reason is I want to use BeforeAll then realized wdio was using an older version before BeforeAll was introduced.)

[devth]

npm 5.x generates a package-lock.json to lock down dep versions. Want me to add it as part of this PR?

[christian-bromann]

Want me to add it as part of this PR?

Sure

[devth]

This might be as far as I get for now. Confused by test errors and it looks like it'll require a lot of changes.

[ashmind]

I did some research on this one — for 4.0.0, but I think it's the same issues.

The test fail with {stringInDoubleQuotes} is easy to fix — can be replaced with {string}.

However, there is also a change in how hooks are implemented. My understanding is that the hooks from before supported async handlers, but latest Cucumber uses EventEmitter without async support. I considered using Before/After instead, however that's only for scenarios. BeforeStep is considered (cucumber/cucumber-js#997), but not implemented yet.

I'm hardly an expert on this, but feels like this is blocked until there are some changes in Cucumber.

[christian-bromann]

Fixed by #104