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

Validity of EXCLUDE tests #10

Open
stephenberry opened this issue Sep 4, 2024 · 1 comment
Open

Validity of EXCLUDE tests #10

stephenberry opened this issue Sep 4, 2024 · 1 comment

Comments

@stephenberry
Copy link
Contributor

The excluded tests should probably be flipped and included.

fail01_EXCLUDE.json -> This should actually pass. The latest JSON specification does not require only objects or arrays.
From the standard: JSON-text = ws value ws, where a value could be a string

fail18_EXCLUDE.json -> This should also pass as there are no depth limits, though limits are necessary for some DOM implementations to avoid stack overflows. I can understand the need to exclude this test for certain use cases.

@Loki-Astari
Copy link
Owner

Will have a look.

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

3 participants
@Loki-Astari @stephenberry and others