-
Notifications
You must be signed in to change notification settings - Fork 615
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
Setup Kover for validation of each module #2674
Conversation
Also, slightly increased coverage Resolves #2634
build.gradle.kts
Outdated
disable() | ||
} | ||
|
||
currentProject { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can we extract Kover configuration to a separate file?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure
|
||
|
||
@Test | ||
fun testEncodingAndDecoding() { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Theoretically, IO's encodeToSink/decodeFromSource
should also be covered by json-tests
module. Isn't this the case?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
In json-tests
, methods with a different signature are being tested (with explicit transfer of the serializer)
Also, slightly increased coverage
Resolves #2634