create unit tests for various number formats #691
Merged
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
For issue #657 Unit tests should check various number formats
This adds tests for various number formats (namely: decimal with and without a decimal point, exponent with both capital and lowercase E, both in the positive and negative) for the various
get*
andopt*
number methods of JSONObject.It does NOT add hex, floating point hex, or octal, as these are not supported JSON features (see https://stackoverflow.com/a/52671839/6323312). JSONObject also does not support hex, floating point hex, or octal.
I can add these additional number format tests if wanted, though.