Add comprehensive test coverage for uncovered utility, protocol, and codec classes (#4)

* Initial plan

* Add comprehensive tests for ByteUtils, ThreadPoolService, SystemInformation, SecureRandomUtils, and LinuxSecureRandom

Co-authored-by: kyonRay <32325790+kyonRay@users.noreply.github.com>

* Add comprehensive tests for JsonRpcRequest and TopicTools

Co-authored-by: kyonRay <32325790+kyonRay@users.noreply.github.com>

* Address code review feedback - fix resource cleanup and remove redundant tests

Co-authored-by: kyonRay <32325790+kyonRay@users.noreply.github.com>

---------

Co-authored-by: copilot-swe-agent[bot] <198982749+Copilot@users.noreply.github.com>
Co-authored-by: kyonRay <32325790+kyonRay@users.noreply.github.com>

Author: Copilot

src/integration-test/java/org/fisco/bcos/sdk/v3/test/BcosSDKTest.java

@@ -66,7 +66,7 @@
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.test.contract.solidity.HelloWorld;
 import org.fisco.bcos.sdk.v3.transaction.codec.decode.TransactionDecoderService;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 import org.fisco.bcos.sdk.v3.transaction.pusher.TransactionPusherService;
 import org.fisco.bcos.sdk.v3.utils.Hex;
 import org.junit.Assert;

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/ContractTest.java

@@ -6,7 +6,7 @@
 import org.fisco.bcos.sdk.v3.model.ConstantConfig;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.test.contract.solidity.Incremental;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 import org.fisco.bcos.sdk.v3.transaction.nonce.DefaultNonceAndBlockLimitProvider;
 import org.fisco.bcos.sdk.v3.transaction.nonce.NonceAndBlockLimitProvider;
 import org.fisco.bcos.sdk.v3.utils.Hex;

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/CodecTest.java

@@ -33,7 +33,7 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 import org.fisco.bcos.sdk.v3.utils.StringUtils;
 
 @SuppressWarnings("unchecked")

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/ComplexCodecTest.java

@@ -29,7 +29,7 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 @SuppressWarnings("unchecked")
 public class ComplexCodecTest extends Contract {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/EventSubDemo.java

@@ -25,7 +25,7 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 @SuppressWarnings("unchecked")
 public class EventSubDemo extends Contract {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/HelloWorld.java

@@ -15,7 +15,7 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 public class HelloWorld extends Contract {
     public static final String[] BINARY_ARRAY = {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/Incremental.java

@@ -21,9 +21,9 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
 import org.fisco.bcos.sdk.v3.transaction.manager.transactionv1.ProxySignTransactionManager;
 import org.fisco.bcos.sdk.v3.transaction.manager.transactionv1.TransactionManager;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 @SuppressWarnings("unchecked")
 public class Incremental extends Contract {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/PayableTest.java

@@ -23,7 +23,7 @@
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
 import org.fisco.bcos.sdk.v3.transaction.manager.transactionv1.ProxySignTransactionManager;
 import org.fisco.bcos.sdk.v3.transaction.manager.transactionv1.TransactionManager;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 @SuppressWarnings("unchecked")
 public class PayableTest extends Contract {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/contract/solidity/SM2EvidenceVerify.java

@@ -23,7 +23,7 @@
 import org.fisco.bcos.sdk.v3.model.CryptoType;
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
 
 public class SM2EvidenceVerify extends Contract {
     public static final String[] BINARY_ARRAY = {

src/integration-test/java/org/fisco/bcos/sdk/v3/test/demo/AssembleTransactionWithRemoteSignProcessorSample.java

@@ -24,15 +24,15 @@
 import org.fisco.bcos.sdk.v3.model.TransactionReceipt;
 import org.fisco.bcos.sdk.v3.model.callback.TransactionCallback;
 import org.fisco.bcos.sdk.v3.test.transaction.mock.RemoteSignProviderMock;
-import org.fisco.bcos.sdk.v3.test.transaction.model.dto.CallRequest;
-import org.fisco.bcos.sdk.v3.test.transaction.model.dto.CallResponse;
-import org.fisco.bcos.sdk.v3.test.transaction.model.dto.TransactionResponse;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.ContractException;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.NoSuchTransactionFileException;
-import org.fisco.bcos.sdk.v3.test.transaction.model.exception.TransactionBaseException;
 import org.fisco.bcos.sdk.v3.transaction.codec.decode.TransactionDecoderService;
 import org.fisco.bcos.sdk.v3.transaction.manager.AssembleTransactionWithRemoteSignProcessor;
 import org.fisco.bcos.sdk.v3.transaction.manager.TransactionProcessorFactory;
+import org.fisco.bcos.sdk.v3.transaction.model.dto.CallRequest;
+import org.fisco.bcos.sdk.v3.transaction.model.dto.CallResponse;
+import org.fisco.bcos.sdk.v3.transaction.model.dto.TransactionResponse;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.ContractException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.NoSuchTransactionFileException;
+import org.fisco.bcos.sdk.v3.transaction.model.exception.TransactionBaseException;
 import org.fisco.bcos.sdk.v3.transaction.signer.RemoteSignProviderInterface;
 import org.fisco.bcos.sdk.v3.utils.Hex;