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

double-0 number tests #104

Open
tswaters opened this issue Dec 20, 2019 · 0 comments
Open

double-0 number tests #104

tswaters opened this issue Dec 20, 2019 · 0 comments

Comments

@tswaters
Copy link

Hey thanks for the great collection of JSON test specs!

I'm writing a parser for a personal project and loaded your json files into my fixtures directory. This resulted in a lot of missed failed parsing cases that I was able to resolve.

One thing I noticed is there are a few "leading zero" tests -

n_number_-01.json
n_number_neg_int_starting_with_zero.json
n_number_with_leading_zero.json

One case I see not tested here is multiple zeros followed by a frac / exp. These cases, should, I think, result in failed parsing:

00.0
-00.0
00e0
-00e0

My parser was accepting these before I added special cases for them - it may be good to add them to the test suite.

tswaters added a commit to tswaters/JSONTestSuite that referenced this issue Dec 20, 2019
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