Skip to content
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

support built-in cucumber expressions #285

Merged

Conversation

qvdk
Copy link
Member

@qvdk qvdk commented Oct 2, 2018

Add the support of built-in cucumber expressions.
screenshot from 2018-10-03 00-41-00

Previously, this type of expression crash the gherkin editor as described into #278 and #267

Limitation: custom cucumber expressions are not detected.
screenshot from 2018-10-03 21-10-39


This path fixes also the issue #286.
When using a pattern like @Given("^(?>region|regions) of type (.*) (?>are|is) covering all references$"), the StepMatcher convert it as a malformed pattern ^(<p>|?>region|regions) of type (.*) (?>are|is) covering all references$.
More, it causes an exception with the side effect that the F3 key stop to work.

screenshot from 2018-10-04 00-16-26

@qvdk qvdk changed the title support cucumber expressions support built-in cucumber expressions Oct 2, 2018
@laeubi
Copy link
Contributor

laeubi commented Nov 5, 2018

We have a similar problem, can this be merged or is there anything left?

@laeubi
Copy link
Contributor

laeubi commented Nov 5, 2018

@johny2000uwb Can this be merged or is there anything let to be done? I can verify that this fixes the issue at least for my test cases.
Since I'd like to improve the support for cucumber expressions it would be cool if we can have this merged first (+it would fix my issue with cucumber expressions failing at all atm)

@laeubi
Copy link
Contributor

laeubi commented Nov 5, 2018

I can also verify that this fixes #286 (invalid regexp makes the parser crash)

@girijant girijant merged commit f980f0c into cucumber:master Nov 13, 2018
@aslakhellesoy
Copy link
Contributor

Hi @qvdk,

Thanks for your making your first contribution to Cucumber, and welcome to the Cucumber committers team! You can now push directly to this repo and all other repos under the cucumber organization! 🍾

In return for this generous offer we hope you will:

  • ✅ Continue to use branches and pull requests. When someone on the core team approves a pull request (yours or someone else's), you're welcome to merge it yourself.
  • 💚 Commit to setting a good example by following and upholding our code of conduct in your interactions with other collaborators and users.
  • 💬 Join the community Slack channel to meet the rest of the team and make yourself at home.
  • ℹ️ Don't feel obliged to help, just do what you can if you have the time and the energy.
  • 🙋 Ask if you need anything. We're looking for feedback about how to make the project more welcoming, so please tell us!

On behalf of the Cucumber core team,
Aslak Hellesøy
Creator of Cucumber

@qvdk
Copy link
Member Author

qvdk commented Nov 13, 2018

Hi @aslakhellesoy Thanks for the invitation !

@qvdk qvdk deleted the feature/support-cucumber-expression-fast branch November 15, 2018 13:34
@danedb
Copy link

danedb commented Nov 21, 2018

I really need this, is there a build available with this fix? Or when is next release scheduled?

never mind! I just saw the instructions to build from source. I'll just go ahead and do just that

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants