Comments (4)
I agree we should focus on the consensus test suite.
Related: What's going to be the arbiter on the one and only JSONPath truth in the end. The written standard, or the conjoined implementation?
I think it has to be the written standard in the end as we can't afford to maintain code indefinitely. This is the view taken by the IETF in RFC 7942 which advises stripping out the implementations section before promoting an Internet Draft to a RFC.
from json-path-comparison.
I've changed Proposal A's handling. (Although I haven't spent time on refactoring, which might be necessary now that the logic is more complex.)
I've also added 4 test cases for begin and end for positive and negative steps each.
From my side I feel all is done.
from json-path-comparison.
Love the fuzzer, thanks for flagging.
I think we can agree that Proposal A's implementation in this part does not cover this input. At the time of writing it was more important for me to find a concise way of expressing the general intent, not so much trying to capture all cross functional requirements (like speed or low memory consumption).
The reason for this is that I don't believe that JSONPath can be implemented in exactly one way, but on the contrary, different implementations might focus on different aspects with different tradeoffs.
The case you flagged is somewhat borderline, so maybe we change this.
Related: What's going to be the arbiter on the one and only JSONPath truth in the end. The written standard, or the conjoined implementation?
from json-path-comparison.
Actually, now that I fixed the issue in the Proposal (had to add 4 test cases), I can see value in calling out this detail in a standard. Or at least to make sure this is captured in the reference test suite.
On that note, I'm not sure whether we agree on which document will become the test suite we want to push. Currently the Proposal A's own test suite is lagging a bit behind the one from the consensus. My personal goal was to drive forward the one from the consensus, which is why I'm adding more queries to the consensus for every issue you raise.
What do you think?
In that sense, I'd like to keep this issue open until I added the 4 necessary queries to the consensus itself.
from json-path-comparison.
Related Issues (20)
- Is it possible to provide JSONPath test cases based on consensus results HOT 7
- Show footnote 4 if applicable on the query detail page
- Need test for filter expression checking for local key in array with a null value
- Alignment with spec in its current state (a report) HOT 7
- Incorporating and merging with the compliance test suite HOT 3
- Add nimma
- Include github.com/SteelBridgeLabs/jsonpath HOT 1
- Failing build of Java implementations in the docker container HOT 1
- Add serde_json_path HOT 2
- Add jpt HOT 2
- Bump JsonPath.Net to v1.0.0 HOT 3
- Comparing dotNET_JsonPathLib questionable
- Project governance HOT 7
- Link to reference implementation
- Support for path axis navigation HOT 11
- Issue with display of queries containing the * character HOT 1
- Expand on type of consensus HOT 2
- Tests for root reference in filter expressions. HOT 3
- Analysis HOT 2
- provide key for table contents HOT 8
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from json-path-comparison.