feat(test): improve tests coverage #6034
Conversation
| @@ -2,12 +2,14 @@ plugins { | |||
| id "org.gradle.test-retry" version "1.5.9" | |||
| id "org.sonarqube" version "2.6" | |||
| id "com.gorylenko.gradle-git-properties" version "2.4.1" | |||
| id "io.github.surpsg.offlins" version "0.3.0" | |||
There was a problem hiding this comment.
What is the purpose of introducing this dependency?
There was a problem hiding this comment.
To solve the incompatibility problem between jacoco and powermock
| @@ -121,10 +123,12 @@ test { | |||
| testLogging { | |||
| exceptionFormat = 'full' | |||
| } | |||
| /* | |||
There was a problem hiding this comment.
Use the report part configuration of the offlinsCoverage module
| } | ||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Why change the default output path?
There was a problem hiding this comment.
The default output path configured by offlinsCoverage is inconsistent with the original configuration of jacoco. In order to maintain the consistency of the coverage report document path.
framework/src/test/java/org/tron/common/logsfilter/TransactionLogTriggerCapsuleMockTest.java
Outdated
Show resolved
Hide resolved
framework/src/test/java/org/tron/common/logsfilter/TransactionLogTriggerCapsuleMockTest.java
Outdated
Show resolved
Hide resolved
| } | ||
| } | ||
|
|
||
|
|
There was a problem hiding this comment.
Can the Jacoco coverage of each module still be output locally after this change?
There was a problem hiding this comment.
Yes, and the path is still the same
What does this PR do?
improve tests coverage
Why are these changes required?
This PR has been tested by:
Follow up
Extra details