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

Separate n_* tests that are about multiple texts in one file #91

Open
nicowilliams opened this issue Apr 22, 2019 · 1 comment
Open

Comments

@nicowilliams
Copy link

Tests like parsers/test_ccan_json/json/_test/nst_files/n_31.json (of which there are many) fail with jq only because jq permits multiple JSON texts in one input stream.

I'm not saying this is permissible JSON, just that it's an entire category of tests that jq will always fail (by design), so it would be best to separate them, kinda like i_* tests.

@nicowilliams
Copy link
Author

nicowilliams commented Apr 22, 2019

E.g., test_parsing/n_structure_double_array.json ([][]), and test_parsing/n_structure_object_with_trailing_garbage.json ({"a": true} "x"). Also, parsers/test_ccan_json/json/_test/nst_files/n_22.json ("3" "4").

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

No branches or pull requests

1 participant