Skip to content

Resolve implicit gas limits in transaction tests #3489

Description

@danceratopz

Problem

State and blockchain test fillers resolve an omitted transaction gas limit before signing. TransactionTest does not. As a result, transaction tests must provide an otherwise arbitrary gas limit or signing fails with gas_limit must be set to sign a transaction.

Proposed change

Resolve omitted gas limits in TransactionTest before signing, using the shared implicit gas-limit behavior and a documented transaction-test gas budget. Keep explicit gas limits unchanged for boundary and invalid-transaction tests.

Add coverage for omitted and explicit limits, then remove redundant gas limits from transaction-test unit tests. Verify and document any fixture changes.

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Labels

A-test-fixturesArea: execution_testing.fixturesC-featCategory: an improvement or new feature

Type

No type

Projects

No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions