search

hiltontj / serde_json_path

Query serde_json Values with JSONPath
https://serdejsonpath.live/
MIT License
51 stars 4 forks source link

Include the JSONPath Compliance Test Suite in tests #5

Closed hiltontj closed 1 year ago

hiltontj commented 1 year ago

The purpose of this issue is to incorporate the JSONPath compliance test-suite into the crate's tests to ensure compliance with the standard.

Currently, the CTS contains tests for the function extension point, which serde_json_path does not yet support. Nonetheless, running the full CTS will be a good exercise for finding any holes in serde_json_path's implementation.

hiltontj commented 1 year ago

This was addressed, in large part, by #7 . However, it deserves to remain open, as serde_json_path is not yet fully compliant. There are several tests in the CTS that validate function extension capability, which serde_json_path does not yet have (see #1 and #8).

For the time being, the compliance test case is #[ignore]'d, and this issue can remain open.

hiltontj commented 1 year ago

32 did re-enable the compliance test suite, and all are passing, however, there are a couple of tests that have been added recently that I need to check, so will do so in a new PR. Then this should be able to close.