Skip to content

Commit 3913616

Browse files
committed
fix: override JUMBO_TX_ENABLED to false for tests incompatible with Jumbo TX
Signed-off-by: Logan Nguyen <[email protected]>
1 parent 4bf23dc commit 3913616

File tree

2 files changed

+12
-0
lines changed

2 files changed

+12
-0
lines changed

packages/server/tests/acceptance/hbarLimiter.spec.ts

Lines changed: 8 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -37,6 +37,14 @@ import { AliasAccount } from '../types/AliasAccount';
3737
config({ path: resolve(__dirname, '../localAcceptance.env') });
3838

3939
describe('@hbarlimiter HBAR Limiter Acceptance Tests', function () {
40+
// With Jumbo TX enabled, deployment skips HFS (the main source of HBAR cost).
41+
// Combined with HIP-1084 (Zero-Cost EthereumTransaction on Success),
42+
// the cost difference becomes zero. As a result, no spending limit is applied
43+
// and this test fails.To avoid this, this suite overrides JUMBO_TX_ENABLED to false.
44+
overrideEnvsInMochaDescribe({
45+
JUMBO_TX_ENABLED: false,
46+
});
47+
4048
// @ts-ignore
4149
const {
4250
mirrorNode,

packages/server/tests/acceptance/rpc_batch1.spec.ts

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1492,6 +1492,10 @@ describe('@api-batch-1 RPC Server Acceptance Tests', function () {
14921492
});
14931493

14941494
it('should delete the file created while execute "eth_sendRawTransaction" to deploy a large contract', async function () {
1495+
// Jumbo TX skips HFS and writes contract directly to state, so no
1496+
// file exists to delete.To avoid this, this test overrides JUMBO_TX_ENABLED to false.
1497+
ConfigServiceTestHelper.dynamicOverride('JUMBO_TX_ENABLED', false);
1498+
14951499
const gasPrice = await relay.gasPrice();
14961500
const transaction = {
14971501
type: 2,

0 commit comments

Comments
 (0)