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

Test our examples against cucumber #228

Open
ciaranmcnulty opened this issue Feb 17, 2021 · 3 comments
Open

Test our examples against cucumber #228

ciaranmcnulty opened this issue Feb 17, 2021 · 3 comments

Comments

@ciaranmcnulty
Copy link
Contributor

Now we're running the cucumber testate against Behat, we should also run the Behat test data against Cucumber.

The loader written for those tests should be reusable, the only downside is it'll require the gherkin go binary to execute so maybe should be in an off-by-default set of tests

@stof
Copy link
Member

stof commented Feb 17, 2021

which test data are you talking about ?

@stof
Copy link
Member

stof commented Feb 17, 2021

If you mean files in https://github.com/Behat/Gherkin/tree/master/tests/Behat/Gherkin/Fixtures/features, I think a better way would be to identify cases that are not covered by the cucumber tests but are covered by our fixtures, and to upstream those to cucumber (this way, cucumber parsers would also run against them).

And we would remove any duplicate (which means that once all our upstreaming is accepted, this folder would be empty)

@ciaranmcnulty
Copy link
Contributor Author

I mean load the Yaml etalon files, run the Go binary against the equivalent features, then see where they diverge.

This could indeed lead to moving them upstream 👍 I think short term having the tests locally will make sense as we can do that immediately.

We'll need to see what the cucumber team's feelings are towards having lots more examples

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

No branches or pull requests

3 participants