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

Tests n_172 and n_175 are the same #92

Open
nicowilliams opened this issue Apr 22, 2019 · 0 comments
Open

Tests n_172 and n_175 are the same #92

nicowilliams opened this issue Apr 22, 2019 · 0 comments

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.

$ sha256sum parsers/test_ccan_json/json/_test/nst_files/n_17[25].json
c28f64ade65fab04e9ced46a826c6779188f7754c604fb3f4c3e8b80e3418148  parsers/test_ccan_json/json/_test/nst_files/n_172.json
c28f64ade65fab04e9ced46a826c6779188f7754c604fb3f4c3e8b80e3418148  parsers/test_ccan_json/json/_test/nst_files/n_175.json
$ 
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