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

chore: add RFC9535 compliance test suite #81

Merged
merged 1 commit into from
Jan 1, 2025

Conversation

disintegrator
Copy link
Contributor

@disintegrator disintegrator commented Dec 31, 2024

This change adds the RFC9535 compliance test suite to the project from https://github.com/jsonpath-standard/jsonpath-compliance-test-suite. Many of the test cases are failing so the included test in lib.rs is suppressed for now and only used to print an informative summary.

CleanShot 2025-01-03 at 10 05 19@2x

This change adds the RFC9535 compliance test suite to the project from
https://github.com/jsonpath-standard/jsonpath-compliance-test-suite.
Many of the test cases are failing so the included test in lib.rs is
suppressed for now and only used to print an informative summary.
@besok
Copy link
Owner

besok commented Jan 1, 2025

Wow! Great! I missed this RFC completely! Thanks for the PR

@besok besok merged commit bfaa1f2 into besok:main Jan 1, 2025
5 checks passed
@disintegrator disintegrator deleted the rfc9535-compliance-tests branch January 3, 2025 00:16
@disintegrator
Copy link
Contributor Author

@besok thanks for merging! Just noticed the test workflow on GitHub is hiding println! output in tests. Here's what it looks like with regular cargo test:

CleanShot 2025-01-03 at 10 05 19@2x

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

Successfully merging this pull request may close these issues.

2 participants