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

[FEATURE] Change the JSON encoding for a seamless integration with JsonPath expressions #153

Closed
andrey-kuprianov opened this issue Jun 30, 2020 · 0 comments
Assignees
Labels
FMBT Feature: support for model-based testing new New issue to be triaged.
Milestone

Comments

@andrey-kuprianov
Copy link
Contributor

The current TLA+ JSON encoding works fine on itlself, but while trying to apply JsonPath expressions used in Jsonatr, e.g. in this spec, it turned out that JsonPath expression do not work when special symbols ('+', '=', etc.) are used as object keys -- and exactly this we do in our current JSON encoding, to shorten it a bit.

As the JSON encoding is anyway used for automated processing, we need to change it slightly, to have a more straightforward structure without using special symbols as keys, albeit a bit inflated in size. The proposed encoding should look like this file.

@andrey-kuprianov andrey-kuprianov added the new New issue to be triaged. label Jun 30, 2020
@andrey-kuprianov andrey-kuprianov added this to the MBT-v0.1-first-release milestone Jun 30, 2020
@andrey-kuprianov andrey-kuprianov self-assigned this Jun 30, 2020
andrey-kuprianov added a commit that referenced this issue Jul 28, 2020
* get rid of special symbols in object keys
* use objects instead of arrays for binary operators
* use key-value objects instead of arrays for pairs of expressions
andrey-kuprianov added a commit that referenced this issue Jul 28, 2020
@konnov konnov modified the milestones: MBT-v0.1-stateless, backlog2020 Dec 11, 2020
@konnov konnov added the FMBT Feature: support for model-based testing label Dec 11, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
FMBT Feature: support for model-based testing new New issue to be triaged.
Projects
None yet
Development

No branches or pull requests

2 participants