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

test - unit test for Config::updateBindings@394-456@./src/main/java/com/jsoniter/spi/Config.java #14

Open
sandy-e opened this issue Feb 23, 2022 · 3 comments
Assignees
Labels

Comments

@sandy-e
Copy link
Collaborator

sandy-e commented Feb 23, 2022

No description provided.

@sandy-e sandy-e added the test label Feb 23, 2022
@sandy-e sandy-e self-assigned this Feb 23, 2022
@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 23, 2022

This function has 100% coverage as measured by my own tool, so switching to IterImplForStreaming::readNumber@571-620@./src/main/java/com/jsoniter/IterImplForStreaming.java instead for writing new unit tests.

sandy-e added a commit that referenced this issue Feb 23, 2022
@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 24, 2022

One unit test in commit 235a737 increased coverage from 18/20 to 20/20. Switching to

public static Any readAny(JsonIterator iter) throws IOException {
in the same file for the remaining unit tests.

It is not as complex as the most complex functions but still has CCN = 9 which means a high complex function.

@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 24, 2022

Coverage 0/8 --> 6/8, i.e. from 0% coverage to 75%.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

1 participant