Added cucumber-js-style Before and After hooks.

[visusnet]

Cucumber.js has Before and After hooks which can be used in conjunction with tags. This PR adds the same capability to cypress-cucumber-preprocessor:

Before(() => {
  // executed before every scenario
});

Before({ tags: '@tagged'}, () => {
  // executed before every scenario tagged with @tagged
});

After({ tags: '@tagged'}, () => {
  // executed after every scenario tagged with @tagged
});

It also allows asynchronous hooks, e.g.:

Before(async () => {
  // scenario is resumed only after this callback resolves
});

After(async () => {
  // scenario is completed only after this callback resolves
});

Before(() => {
  return new Promise(resolve => {
    setTimeout(() => {
      console.log('This will be logged before the scenario is resumed');
      resolve();
    }, 1000);
  });
});

[visusnet]

This should also fix #25.

[visusnet]

After some edge cases did not work, I had to do push some changes. This should work now. :)

[lgandecki]

Fantastic work. I didn't have a chance to try it out yet or review properly, just noticed that you commented out some tests. Otherwise it makes sense. @lucetius @sawasawasawa @rcywinski @dweller23 maybe one of you would like to checkout this branch, test the functionality and review the code?

[lgandecki]

any particular reason for commenting out this test? and the And and But below?

[lgandecki]

this one as well

[visusnet]

I have commented out these tests by mistake. I have force-pushed a new version without the comments.

[guiyomh]

Good job

[lgandecki]

What the heck @guiyomh? I'm reviewing this PR at the very moment!
No one picked it up from our team (consequences of summer time..), so decided to finally give this a try. :) Expect an update in an hour or so.

[lgandecki]

Looks great. I've tweaked some tests. Still thinking how to improve the readability of BeforeAndAfterSteps (pretty meta, hard to word it nicely, I know ;) )

The only thing I'm debating is whether we should overwrite the cypress exposed before/after.
I think it might be nicer to have them import-only with capital letters. (a bit confusing, but in line with what people use when writing cucumber files in pure cucumberjs)
This way, we also don't introduce any breaking changes for people that already use the mocha before/after steps.

[lgandecki]

I messed up the git commit history somehow.. :D But, you can see my changes here:
b232286
and here:
ddb5a46

I will think a bit more about improving the .feature file, otherwise I will merge and release

[lgandecki]

@visusnet this is what I came up with for the .feature file, please take a look , if you are fine with the changes I'd merge and release.
920aeec

[visusnet]

@lgandecki I‘m fine with your changes. Go ahead. :)

[lgandecki]

🎉 This PR is included in version 1.14.0 🎉

The release is available on:

• npm package (@latest dist-tag)
• GitHub release

Your semantic-release bot 📦🚀