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.

Sign up for GitHub

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

Jump to bottom

feat - add own test coverage tool sanherib #13

Open
sandy-e opened this issue Feb 23, 2022 · 5 comments
Open

feat - add own test coverage tool sanherib #13

sandy-e opened this issue Feb 23, 2022 · 5 comments
Assignees
@sandy-e
Labels
feature

Comments

@sandy-e
Copy link
Collaborator

sandy-e commented Feb 23, 2022

No description provided.

@sandy-e sandy-e self-assigned this Feb 23, 2022
sandy-e added a commit that referenced this issue Feb 23, 2022
@sandy-e
feat - added own coverage tool (#13)
859903c
@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 23, 2022

100% coverage on Config::updateBindings@394-456@./src/main/java/com/jsoniter/spi/Config.java.

@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 23, 2022

The variable i also counts as CCN counter, just add two at the end and you get the corresponding CCN (18).

sandy-e added a commit that referenced this issue Feb 23, 2022
@sandy-e
feat - add coverage tool for new function (#13)
57b6871
@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 23, 2022 •
edited
Loading

New function added since the first one had 100% coverage.
Coverage: 18/20 statements from test class IterImplForStreamingTest.

Edit: 20/20 statements from one unit test. Switching functions to be able to implement additional unit tests that cover uncovered branches.

@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 24, 2022

Switching to

java/src/main/java/com/jsoniter/IterImplForStreaming.java

Line 317 in 235a737

public static Any readAny(JsonIterator iter) throws IOException {

sandy-e added a commit that referenced this issue Feb 24, 2022
@sandy-e
feat - add coverage tool for 3rd function (#13)
6e8aba1
@sandy-e
Copy link
Collaborator Author

sandy-e commented Feb 24, 2022

0/8 coverage.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@sandy-e sandy-e

Labels
feature
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sandy-e