chore: fix tests

Signed-off-by: Ivan Ivanov <[email protected]>
hashgraph · Nov 14, 2024 · d8bd3a9 · d8bd3a9
1 parent 5f2e437
commit d8bd3a9
Show file tree

Hide file tree

Showing 3 changed files with 38 additions and 5 deletions.
diff --git a/gradle/plugins/src/main/kotlin/com.hedera.gradle.java.gradle.kts b/gradle/plugins/src/main/kotlin/com.hedera.gradle.java.gradle.kts
@@ -37,6 +37,7 @@ testing.suites {
         testType = TestSuiteType.INTEGRATION_TEST
         targets.all {
             testTask {
+                failFast = true
                 group = "build"
                 systemProperty("CONFIG_FILE", providers.gradleProperty("CONFIG_FILE").getOrElse(""))
                 systemProperty("HEDERA_NETWORK", providers.gradleProperty("HEDERA_NETWORK").getOrElse(""))

diff --git a/sdk/src/main/java/com/hedera/hashgraph/sdk/MirrorNodeContractQuery.java b/sdk/src/main/java/com/hedera/hashgraph/sdk/MirrorNodeContractQuery.java
@@ -47,7 +47,7 @@ public class MirrorNodeContractQuery {
     private String senderEvmAddress = null;
     // The transaction callData
     private byte[] callData;
-    // The amount we are sending to payable functions
+    // The amount we are sending to the contract
     private long value;
     // The gas limit
     private long gasLimit;
@@ -170,6 +170,14 @@ public long getValue() {
         return this.value;
     }
 
+    /**
+     * Sets the amount of value (in tinybars or wei) to be sent to the contract in the transaction.
+     * <p>
+     * Use this to specify an amount for a payable function call.
+     *
+     * @param value the amount of value to send, in tinybars or wei
+     * @return {@code this}
+     */
     public MirrorNodeContractQuery setValue(long value) {
         this.value = value;
         return this;
@@ -179,6 +187,14 @@ public long getGasLimit() {
         return this.gasLimit;
     }
 
+    /**
+     * Sets the gas limit for the contract call.
+     * <p>
+     * This specifies the maximum amount of gas that the transaction can consume.
+     *
+     * @param gasLimit the maximum gas allowed for the transaction
+     * @return {@code this}
+     */
     public MirrorNodeContractQuery setGasLimit(long gasLimit) {
         this.gasLimit = gasLimit;
         return this;
@@ -188,6 +204,14 @@ public long getGasPrice() {
         return gasPrice;
     }
 
+    /**
+     * Sets the gas price to be used for the contract call.
+     * <p>
+     * This specifies the price of each unit of gas used in the transaction.
+     *
+     * @param gasPrice the gas price, in tinybars or wei, for each unit of gas
+     * @return {@code this}
+     */
     public MirrorNodeContractQuery setGasPrice(long gasPrice) {
         this.gasPrice = gasPrice;
         return this;
@@ -197,6 +221,14 @@ public long getBlockNumber() {
         return this.blockNumber;
     }
 
+    /**
+     * Sets the block number for the simulation of the contract call.
+     * <p>
+     * The block number determines the context of the contract call simulation within the blockchain.
+     *
+     * @param blockNumber the block number at which to simulate the contract call
+     * @return {@code this}
+     */
     public MirrorNodeContractQuery setBlockNumber(long blockNumber) {
         this.blockNumber = blockNumber;
         return this;

diff --git a/sdk/src/test/java/com/hedera/hashgraph/sdk/MirrorNodeContractQueryTest.java b/sdk/src/test/java/com/hedera/hashgraph/sdk/MirrorNodeContractQueryTest.java
@@ -143,7 +143,7 @@ void testCreateJsonPayloadAllFieldsSet() {
             gasPrice, value, blockNumber, estimate);
 
         JsonObject expectedJson = new JsonObject();
-        expectedJson.addProperty("data", "testData");
+        expectedJson.addProperty("data", "7465737444617461");
         expectedJson.addProperty("to", contractAddress);
         expectedJson.addProperty("estimate", estimate);
         expectedJson.addProperty("blockNumber", blockNumber);
@@ -170,7 +170,7 @@ void testCreateJsonPayloadOnlyRequiredFieldsSet() {
             gasPrice, value, blockNumber, estimate);
 
         JsonObject expectedJson = new JsonObject();
-        expectedJson.addProperty("data", "testData");
+        expectedJson.addProperty("data", "7465737444617461");
         expectedJson.addProperty("to", contractAddress);
         expectedJson.addProperty("estimate", estimate);
         expectedJson.addProperty("blockNumber", blockNumber);
@@ -193,7 +193,7 @@ void testCreateJsonPayloadSomeOptionalFieldsSet() {
             gasPrice, value, blockNumber, estimate);
 
         JsonObject expectedJson = new JsonObject();
-        expectedJson.addProperty("data", "testData");
+        expectedJson.addProperty("data", "7465737444617461");
         expectedJson.addProperty("to", contractAddress);
         expectedJson.addProperty("estimate", estimate);
         expectedJson.addProperty("blockNumber", blockNumber);
@@ -219,7 +219,7 @@ void testCreateJsonPayloadAllOptionalFieldsDefault() {
             gasPrice, value, blockNumber, estimate);
 
         JsonObject expectedJson = new JsonObject();
-        expectedJson.addProperty("data", "testData");
+        expectedJson.addProperty("data", "7465737444617461");
         expectedJson.addProperty("to", contractAddress);
         expectedJson.addProperty("estimate", estimate);
         expectedJson.addProperty("blockNumber", blockNumber);