-
-
Notifications
You must be signed in to change notification settings - Fork 689
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
JSON formatter specification #187
Comments
Briefly: see #172 The idea is to move away from the big blog of JSON that combines AST with test results, and use a series of smaller events instead. I'd like to deprecate then remove the JSON formatter from Cucumber Ruby ASAP. If we can standardise around one protocol, we can build a single set of renderers / formatters that consume the protocol and produce human-readable output. I'm glad to have you in the loop on this @charlierudolph. |
I think the new style of the JSON formatter (which creates one JSON file - as opposed to the set of JSON event in the event protocol), should be base the (list of) pickles emitted by the compiler. This data need to be augmented with the data of the step definitions, hooks and results. That being said, since there are tools parsing the current output of the JSON formatter, that option needs still to be available (for some time), but deprecated. |
IMO the JSON formatter will be replaced by the event protocol, and we shouldn't need both in the end. I don't think there's any need to write a new JSON formatter. |
We could write a small utility that builds the legacy JSON report from events |
Yes, a formatter, because that is what a formatter is - a utility that build a report from events. |
One thing that always made me a bit uneasy related to the discussion of "the event protocol" is the suspicion that we mix up concept/data/events belonging in different bounded context in the same bucket. AFAIK "protocol" includes the concept of allowed sequence of things. Mentally I imagine massive parallel execution of pickles on a grid of executors. In such situation there could be 528 unmatched |
@brasmusson interesting feedback. Would we be better having that discussion in #172? |
Closing this since the JSON format is being phased out |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Do we have a document anywhere that documents what we would like cucumber implementations to export in the JSON formatter?
In cucumber-js we were mainly following cucumber-ruby and I made a few changes in 2.0 (removing the "type" from scenarios as that is the only thing exported and converting the duration from nanoseconds to milliseconds) that I have received a few issues on
If we would like this behavior to be shared between the different cucumber implementations I would prefer we have a JSON schema somewhere that the implementations can test themselves against.
I'm happy to lead this and create an initial schema that is based off of the current output of the different projects.
@aslakhellesoy @jbpros @mattwynne
The text was updated successfully, but these errors were encountered: