Skip to content

Commit e15cbbf

Browse files
committed
fix the bug of VmConfigTest
1 parent 99dcd91 commit e15cbbf

1 file changed

Lines changed: 4 additions & 15 deletions

File tree

common/src/test/java/org/tron/core/config/args/VmConfigTest.java

Lines changed: 4 additions & 15 deletions
Original file line numberDiff line numberDiff line change
@@ -90,8 +90,8 @@ public void testEstimateEnergyMaxRetryBoundaryValues() {
9090
}
9191

9292
// ===========================================================================
93-
// Constant-call timeout (issue #6681). The validation rule: any positive
94-
// value that fits VM deadline conversion is accepted, but zero/negative is
93+
// Constant-call timeout (issue #6681). The validation rule: any zero or positive
94+
// value that fits VM deadline conversion is accepted, but negative is
9595
// rejected ONLY when the operator explicitly set the property in their
9696
// config. Absence keeps the in-Java default (0L = "share the
9797
// block-processing deadline").
@@ -107,6 +107,8 @@ public void testConstantCallTimeoutDefaultWhenAbsent() {
107107

108108
@Test
109109
public void testConstantCallTimeoutAcceptsAnyPositiveValue() {
110+
assertEquals(0L, VmConfig.fromConfig(
111+
withRef("vm { constantCallTimeoutMs = 0 }")).getConstantCallTimeoutMs());
110112
assertEquals(1L, VmConfig.fromConfig(
111113
withRef("vm { constantCallTimeoutMs = 1 }")).getConstantCallTimeoutMs());
112114
assertEquals(50L, VmConfig.fromConfig(
@@ -117,19 +119,6 @@ public void testConstantCallTimeoutAcceptsAnyPositiveValue() {
117119
withRef("vm { constantCallTimeoutMs = 5000 }")).getConstantCallTimeoutMs());
118120
}
119121

120-
@Test
121-
public void testConstantCallTimeoutZeroRejectedWhenExplicitlyConfigured() {
122-
// Operator wrote `= 0` in config -> treated as a misconfiguration even
123-
// though it equals the in-Java default. Forces an explicit positive value.
124-
try {
125-
VmConfig.fromConfig(withRef("vm { constantCallTimeoutMs = 0 }"));
126-
org.junit.Assert.fail("expected IllegalArgumentException for explicit 0");
127-
} catch (IllegalArgumentException ex) {
128-
org.junit.Assert.assertTrue(ex.getMessage(),
129-
ex.getMessage().contains("constantCallTimeoutMs"));
130-
}
131-
}
132-
133122
@Test
134123
public void testConstantCallTimeoutNegativeRejected() {
135124
try {

0 commit comments

Comments
 (0)