Skip to content

Use a separate account for k6 crypto allowance test #11419

@steven-sheehy

Description

@steven-sheehy

Problem

The DEFAULT_ACCOUNT_ID used in many tests is also used by the crypto allowance k6 test. It's difficult to find an account with a lot of activity and that consistently has allowances.

Solution

  • Add a DEFAULT_ACCOUNT_ID_CRYPTO_ALLOWANCE similar to token allowance and use it in accountsCryptoAllowance.js and accountsCryptoAllowanceSpender.js
  • Change accountsCryptoAllowance*.js to verify non-empty results
  • Update web3 hardcoded defaults in estimate.json and modificationFunctions.json and move default values to code.
  • Update DEFAULT_ACCOUNT_ID in deploy branch to 1682785
  • Update DEFAULT_ACCOUNT_ID_CRYPTO_ALLOWANCE in deploy branch with an account that we control that will have a permanent set of allowances

Alternatives

No response

Metadata

Metadata

Assignees

Projects

Status

👀 In review

Milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions