feat(test): optimize tests coverage

build.gradle

@@ -37,6 +37,7 @@ subprojects {
         implementation group: 'org.slf4j', name: 'slf4j-api', version: '1.7.25'
         implementation group: 'org.slf4j', name: 'jcl-over-slf4j', version: '1.7.25'
         implementation group: 'ch.qos.logback', name: 'logback-classic', version: '1.2.9'
+        implementation group: 'com.google.guava', name: 'guava', version: '30.1-jre'
         implementation "com.google.code.findbugs:jsr305:3.0.0"
         implementation group: 'org.springframework', name: 'spring-context', version: '5.3.18'
         implementation group: 'org.springframework', name: 'spring-tx', version: '5.3.18'
@@ -52,7 +53,10 @@ subprojects {
         testAnnotationProcessor 'org.projectlombok:lombok:1.18.12'
 
         testImplementation group: 'junit', name: 'junit', version: '4.13.2'
-        testImplementation "org.mockito:mockito-core:2.13.0"
+        testImplementation "org.mockito:mockito-core:3.10.0"
+        testImplementation group: 'org.powermock', name: 'powermock-module-junit4', version: '2.0.9'
+        testImplementation group: 'org.powermock', name: 'powermock-api-mockito2', version: '2.0.9'
+
     }
 
     task sourcesJar(type: Jar, dependsOn: classes) {
@@ -67,6 +71,12 @@ subprojects {
         reproducibleFileOrder = true
         duplicatesStrategy = DuplicatesStrategy.INCLUDE // allow duplicates
     }
+
+    configurations.all {
+        resolutionStrategy {
+            force group: 'com.google.guava', name: 'guava', version: '30.1-jre'
+        }
+    }
 }
 
 task copyToParent(type: Copy) {

framework/build.gradle

@@ -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"
 }
 
 gitProperties.failOnNoGitDirectory = false;
 
 apply plugin: 'application'
 apply plugin: 'checkstyle'
+apply plugin: "io.github.surpsg.offlins"
 
 mainClassName = 'org.tron.program.FullNode'
 
@@ -16,7 +18,7 @@ def versions = [
 ]
 
 jacoco {
-    toolVersion = "0.8.1"
+    toolVersion = "0.8.8"
 }
 
 
@@ -121,10 +123,12 @@ test {
     testLogging {
         exceptionFormat = 'full'
     }
+    /*
     jacoco {
         destinationFile = file("$buildDir/jacoco/jacocoTest.exec")
         classDumpDir = file("$buildDir/jacoco/classpathdumps")
     }
+    */
     if (isWindows()) {
         exclude '**/ShieldedTransferActuatorTest.class'
         exclude '**/BackupDbUtilTest.class'
@@ -141,6 +145,7 @@ test {
     }
 }
 
+/*
 jacocoTestReport {
     reports {
         xml.enabled true
@@ -149,6 +154,26 @@ jacocoTestReport {
     }
     getExecutionData().setFrom(fileTree('../framework/build/jacoco').include("**.exec"))
 }
+ */
+
+offlinsCoverage {
+    jacocoVersion = '0.8.8' // Optional. By default `0.8.8`
+
+    reports {
+        html.enabled.set true // Optional. By default `true`
+        html.location.set project.file('build/reports/jacoco/test/html') // Optional. By default `build/reports/jacoco/html`
+
+        xml.enabled.set true // Optional. By default `false`
+        xml.location.set project.file('build/reports/jacoco/test/jacocoTestReport.xml')
+        // Optional. By default `build/reports/jacoco/coverageReport.xml`
+
+        csv.enabled.set true // Optional. By default `false`
+        csv.location.set project.file('build/reports/jacoco/test/csvCoverage.csv')
+        // Optional. By default `build/reports/jacoco/coverageReport.csv`
+    }
+}
+
+
 
 def binaryRelease(taskName, jarName, mainClass) {
     return tasks.create("${taskName}", Jar) {

framework/src/test/java/org/tron/common/logsfilter/TransactionLogTriggerCapsuleMockTest.java

@@ -0,0 +1,130 @@
+package org.tron.common.logsfilter;
+
+import static org.powermock.api.mockito.PowerMockito.mock;
+import static org.powermock.api.mockito.PowerMockito.spy;
+import static org.powermock.api.mockito.PowerMockito.when;
+
+import com.google.protobuf.ByteString;
+import java.util.ArrayList;
+import java.util.HashMap;
+import java.util.List;
+
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Before;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mockito;
+import org.powermock.core.classloader.annotations.PrepareForTest;
+import org.powermock.modules.junit4.PowerMockRunner;
+import org.powermock.reflect.Whitebox;
+import org.tron.common.logsfilter.capsule.TransactionLogTriggerCapsule;
+import org.tron.common.logsfilter.trigger.InternalTransactionPojo;
+import org.tron.common.runtime.InternalTransaction;
+import org.tron.common.runtime.ProgramResult;
+import org.tron.common.runtime.RuntimeImpl;
+import org.tron.common.utils.Sha256Hash;
+import org.tron.core.capsule.BlockCapsule;
+import org.tron.core.capsule.ReceiptCapsule;
+import org.tron.core.capsule.TransactionCapsule;
+import org.tron.core.db.TransactionTrace;
+import org.tron.p2p.utils.ByteArray;
+import org.tron.protos.Protocol;
+import org.tron.protos.contract.BalanceContract;
+
+@RunWith(PowerMockRunner.class)
+@PrepareForTest({
+    TransactionLogTriggerCapsule.class,
+    TransactionTrace.class
+})
+public class TransactionLogTriggerCapsuleMockTest {
+
+  private static final String OWNER_ADDRESS = "41548794500882809695a8a687866e76d4271a1abc";
+  private static final String RECEIVER_ADDRESS = "41abd4b9367799eaa3197fecb144eb71de1e049150";
+  private static final String CONTRACT_ADDRESS = "111111";
+
+  private TransactionCapsule transactionCapsule;
+  private BlockCapsule blockCapsule;
+
+  @Before
+  public void setup() {
+    blockCapsule = new BlockCapsule(1,
+        Sha256Hash.ZERO_HASH,
+        System.currentTimeMillis(),
+        Sha256Hash.ZERO_HASH.getByteString()
+    );
+  }
+
+  @After
+  public void  clearMocks() {
+    Mockito.framework().clearInlineMocks();
+  }
+
+
+  @Test
+  public void testConstructorWithTransactionTrace() {
+    BalanceContract.TransferContract.Builder builder2 =
+        BalanceContract.TransferContract.newBuilder()
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+            .setToAddress(ByteString.copyFrom(ByteArray.fromHexString(RECEIVER_ADDRESS)));
+    transactionCapsule = spy(new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.TransferContract));
+
+    TransactionTrace trace = mock(TransactionTrace.class);
+    ReceiptCapsule receiptCapsule = new ReceiptCapsule(Sha256Hash.ZERO_HASH);
+    RuntimeImpl runtime =  mock(RuntimeImpl.class);
+    List<Protocol.TransactionInfo.Log> logs = new ArrayList<>();
+    logs.add(Protocol.TransactionInfo.Log.newBuilder()
+        .setAddress(ByteString.copyFrom("address".getBytes()))
+        .setData(ByteString.copyFrom("data".getBytes()))
+        .addTopics(ByteString.copyFrom("topic".getBytes()))
+        .build());
+
+    Protocol.TransactionInfo.Builder builder = Protocol.TransactionInfo.newBuilder()
+        .addAllLog(logs);
+
+    ProgramResult programResult = ProgramResult.createEmpty();
+    programResult.setHReturn("hreturn".getBytes());
+    programResult.setContractAddress(CONTRACT_ADDRESS.getBytes());
+
+    when(transactionCapsule.getTrxTrace()).thenReturn(trace);
+    when(trace.getReceipt()).thenReturn(receiptCapsule);
+    when(trace.getRuntime()).thenReturn(runtime);
+    when(runtime.getResult()).thenReturn(programResult);
+
+    transactionCapsule.setTrxTrace(trace);
+
+    TransactionLogTriggerCapsule triggerCapsule = new TransactionLogTriggerCapsule(
+        transactionCapsule, blockCapsule,0,0,0,
+        builder.build(),0);
+
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger());
+  }
+
+  @Test
+  public void testGetInternalTransactionList() throws Exception {
+    BalanceContract.TransferContract.Builder builder2 =
+        BalanceContract.TransferContract.newBuilder()
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+            .setToAddress(ByteString.copyFrom(ByteArray.fromHexString(RECEIVER_ADDRESS)));
+    transactionCapsule = new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.TransferContract);
+    InternalTransaction internalTransaction = new InternalTransaction(
+        "parentHash".getBytes(), 10, 0,
+        "sendAddress".getBytes(),
+        "transferToAddress".getBytes(),
+        100L, "data".getBytes(), "note",
+        0L, new HashMap<>()
+    );
+    List<InternalTransaction> internalTransactionList = new ArrayList<>();
+    internalTransactionList.add(internalTransaction);
+    TransactionLogTriggerCapsule triggerCapsule =
+        new TransactionLogTriggerCapsule(transactionCapsule, blockCapsule);
+
+    List<InternalTransactionPojo> pojoList = Whitebox.invokeMethod(triggerCapsule,
+        "getInternalTransactionList", internalTransactionList);
+
+    Assert.assertNotNull(pojoList);
+  }
+
+}

framework/src/test/java/org/tron/common/logsfilter/TransactionLogTriggerCapsuleTest.java

@@ -13,13 +13,16 @@
 import org.tron.core.capsule.TransactionCapsule;
 import org.tron.p2p.utils.ByteArray;
 import org.tron.protos.Protocol;
+import org.tron.protos.contract.AssetIssueContractOuterClass;
 import org.tron.protos.contract.BalanceContract;
 import org.tron.protos.contract.Common;
+import org.tron.protos.contract.SmartContractOuterClass;
 
 public class TransactionLogTriggerCapsuleTest {
 
   private static final String OWNER_ADDRESS = "41548794500882809695a8a687866e76d4271a1abc";
   private static final String RECEIVER_ADDRESS = "41abd4b9367799eaa3197fecb144eb71de1e049150";
+  private static final String CONTRACT_ADDRESS = "111111";
 
   public TransactionCapsule transactionCapsule;
   public BlockCapsule blockCapsule;
@@ -41,7 +44,6 @@ public void testConstructorWithUnfreezeBalanceTrxCapsule() {
     Protocol.TransactionInfo.Builder builder = Protocol.TransactionInfo.newBuilder();
     builder.setUnfreezeAmount(TRX_PRECISION + 1000);
 
-
     TransactionLogTriggerCapsule triggerCapsule = new TransactionLogTriggerCapsule(
         transactionCapsule, blockCapsule,0,0,0,
         builder.build(),0);
@@ -175,4 +177,70 @@ public void testConstructorWithCancelAllUnfreezeTrxCapsule() {
         triggerCapsule.getTransactionLogTrigger().getExtMap().get(BANDWIDTH.name()).longValue());
   }
 
+  @Test
+  public void testConstructorWithTransferCapsule() {
+    BalanceContract.TransferContract.Builder builder2 =
+        BalanceContract.TransferContract.newBuilder()
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+            .setToAddress(ByteString.copyFrom(ByteArray.fromHexString(RECEIVER_ADDRESS)));
+    transactionCapsule = new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.TransferContract);
+
+    TransactionLogTriggerCapsule triggerCapsule =
+        new TransactionLogTriggerCapsule(transactionCapsule, blockCapsule);
+
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getFromAddress());
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getToAddress());
+  }
+
+  @Test
+  public void testConstructorWithTransferAssetCapsule() {
+    AssetIssueContractOuterClass.TransferAssetContract.Builder builder2 =
+        AssetIssueContractOuterClass.TransferAssetContract.newBuilder()
+            .setAssetName(ByteString.copyFrom("AssetName".getBytes()))
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+            .setToAddress(ByteString.copyFrom(ByteArray.fromHexString(RECEIVER_ADDRESS)));
+    transactionCapsule = new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.TransferAssetContract);
+
+    TransactionLogTriggerCapsule triggerCapsule =
+        new TransactionLogTriggerCapsule(transactionCapsule, blockCapsule);
+
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getFromAddress());
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getToAddress());
+  }
+
+  @Test
+  public void testConstructorWithTriggerSmartContract() {
+    SmartContractOuterClass.TriggerSmartContract.Builder builder2 =
+        SmartContractOuterClass.TriggerSmartContract.newBuilder()
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)))
+            .setContractAddress(ByteString.copyFrom(ByteArray.fromHexString(CONTRACT_ADDRESS)));
+    transactionCapsule = new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.TriggerSmartContract);
+
+    TransactionLogTriggerCapsule triggerCapsule =
+        new TransactionLogTriggerCapsule(transactionCapsule, blockCapsule);
+
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getFromAddress());
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getToAddress());
+  }
+
+  @Test
+  public void testConstructorWithCreateSmartContract() {
+    SmartContractOuterClass.CreateSmartContract.Builder builder2 =
+        SmartContractOuterClass.CreateSmartContract.newBuilder()
+            .setOwnerAddress(ByteString.copyFrom(ByteArray.fromHexString(OWNER_ADDRESS)));
+    transactionCapsule = new TransactionCapsule(builder2.build(),
+        Protocol.Transaction.Contract.ContractType.CreateSmartContract);
+
+    TransactionLogTriggerCapsule triggerCapsule =
+        new TransactionLogTriggerCapsule(transactionCapsule, blockCapsule);
+
+    Assert.assertNotNull(triggerCapsule.getTransactionLogTrigger().getFromAddress());
+  }
+
+
+
+
 }

framework/src/test/java/org/tron/common/runtime/RuntimeImplMockTest.java

@@ -0,0 +1,57 @@
+package org.tron.common.runtime;
+
+import lombok.extern.slf4j.Slf4j;
+import org.junit.After;
+import org.junit.Assert;
+import org.junit.Test;
+import org.junit.runner.RunWith;
+import org.mockito.Mockito;
+import org.powermock.core.classloader.annotations.PrepareForTest;
+import org.powermock.modules.junit4.PowerMockRunner;
+import org.powermock.reflect.Whitebox;
+import org.tron.core.vm.program.Program;
+
+@RunWith(PowerMockRunner.class)
+@PrepareForTest({RuntimeImpl.class})
+@Slf4j
+public class RuntimeImplMockTest {
+  @After
+  public void  clearMocks() {
+    Mockito.framework().clearInlineMocks();
+  }
+
+  @Test
+  public void testSetResultCode1() throws Exception {
+    RuntimeImpl runtime = new RuntimeImpl();
+    ProgramResult programResult = new ProgramResult();
+
+    Program.BadJumpDestinationException badJumpDestinationException
+        = new Program.BadJumpDestinationException("Operation with pc isn't 'JUMPDEST': PC[%d];", 0);
+    programResult.setException(badJumpDestinationException);
+    Whitebox.invokeMethod(runtime,"setResultCode", programResult);
+
+    Program.OutOfTimeException outOfTimeException
+        = new Program.OutOfTimeException("CPU timeout for 0x0a executing");
+    programResult.setException(outOfTimeException);
+    Whitebox.invokeMethod(runtime,"setResultCode", programResult);
+
+    Program.PrecompiledContractException precompiledContractException
+        = new Program.PrecompiledContractException("precompiled contract exception");
+    programResult.setException(precompiledContractException);
+    Whitebox.invokeMethod(runtime,"setResultCode", programResult);
+
+    Program.StackTooSmallException stackTooSmallException
+        = new Program.StackTooSmallException("Expected stack size %d but actual %d;", 100, 10);
+    programResult.setException(stackTooSmallException);
+    Whitebox.invokeMethod(runtime,"setResultCode", programResult);
+
+    Program.JVMStackOverFlowException jvmStackOverFlowException
+        = new Program.JVMStackOverFlowException();
+    programResult.setException(jvmStackOverFlowException);
+    Whitebox.invokeMethod(runtime,"setResultCode", programResult);
+
+    Assert.assertTrue(true);
+  }
+
+}
+