Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update faucet handling for tests #1469

Open
7 tasks
wulfraem opened this issue Dec 2, 2024 · 0 comments
Open
7 tasks

Update faucet handling for tests #1469

wulfraem opened this issue Dec 2, 2024 · 0 comments
Assignees
Labels
No changelog Excludes PR from becoming part of any changelog Rust Related to the core Rust code. Becomes part of the Rust changelog.

Comments

@wulfraem
Copy link
Contributor

wulfraem commented Dec 2, 2024

Description

Allow using a funded account to distribute funds to test account instead of calling the faucet directly with every test account.

Motivation

Current implementation makes too many calls against the faucet, which can lead to getting put on an IP based denylist. To reduce those calls, we want to allow using a single funded account and transferring funds to test accounts created during tests.

To-do list

  • add logic to funds test accounts via a single account
  • behavior must be togglable via env
  • tests must still run; consider adding a mutix, if we run into transaction issues during parallel tests

Change checklist

Add an x to the boxes that are relevant to your changes, and delete any items that are not.

  • The feature or fix is implemented in Rust and across all bindings whereas possible.
  • The feature or fix has sufficient testing coverage
  • All tests and examples build and run locally as expected
  • Every piece of code has been document according to the documentation guidelines.
@wulfraem wulfraem added No changelog Excludes PR from becoming part of any changelog Rust Related to the core Rust code. Becomes part of the Rust changelog. labels Dec 2, 2024
@wulfraem wulfraem self-assigned this Dec 2, 2024
@wulfraem wulfraem moved this from Product Backlog to In Review in IOTA Identity - Framework Developments Dec 3, 2024
@wulfraem wulfraem moved this from In Review to Done in IOTA Identity - Framework Developments Dec 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
No changelog Excludes PR from becoming part of any changelog Rust Related to the core Rust code. Becomes part of the Rust changelog.
Projects
Development

No branches or pull requests

1 participant