-
Notifications
You must be signed in to change notification settings - Fork 43
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
Use Gherkin parser as soon as it becomes available #73
Comments
Work in progress here: https://github.com/pjlsergeant/gherkin/tree/master/perl |
This will break use of "Scenario" to mean "Scenario Outline", so I'm unsure about this. |
The Gherkin release notes suggest that in Gherkin 6 |
This item has already been logged as issue #73
After cucumber/common#694 gets merged the Gherkin parser has everything we need to drive cucumber tests: the generated |
I'm only inclined to go this way if we can have the protobuf library required to produce the wire protocol data be an optional component in the ecosystem. |
An alternative would be to test our parser with the central testing corpus. |
There's a fully updated Gherkin parser available now on CPAN! |
BTW, changing to the new parser also means to support the Rule keyword, bit at the same time has a big impact on the rest of the program due to the fact that Gherkin has a different setup for the AST of the feature file. For the execution, things will become simpler, because we will be handed pickles to be expanded into test cases and executed from the there. For the formatters (harnesses), things will be very different as they will need to combine AST data with test results to print scenarios and rules with their outcomes. |
As noted in cucumber/common#768, we need the pickles to be expanded, because the steps in the pickles don't hold a keyword whereas we have always used the keyword for dispatching the correct step function. |
With the merge of cucumber/common#1741, the data that we need is now available in the pickles! All we need is a release and some effort to replace our own parser with the one published by the Cucumber project! |
As of Gherkin 24.0.0 (released to CPAN today) all prerequisites are there to move to the parser from the Cucumber project! |
Great work @ehuelsmann :-D |
First step towards using Cucumber infrastructure taken: released 0.85 today which uses |
The cucumber project started "Gherkin3", a project which allows to consistently parse Gherkin .feature files across languages (https://github.com/cucumber/gherkin).
The text was updated successfully, but these errors were encountered: