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

feat: support for querying pool/stake registration certs in validation rules #939

Merged
merged 1 commit into from
Mar 5, 2025

Conversation

agaffney
Copy link
Contributor

@agaffney agaffney commented Mar 5, 2025

We only want to collect a deposit on the first pool/stake registration cert for a particular key, so we need to query the ledger state to determine if there are previous certs.

Additionally, this commit:

  • moves the duplicated testLedgerState to a common place as MockLedgerState
  • adds support for mocking pool/stake registration queries to MockLedgerState

@agaffney agaffney requested a review from a team as a code owner March 5, 2025 22:01
@agaffney agaffney force-pushed the feat/validation-rules-query-certs branch from e5f803e to 6eaf223 Compare March 5, 2025 22:11
…n rules

We only want to collect a deposit on the first pool/stake registration
cert for a particular key, so we need to query the ledger state to
determine if there are previous certs.

Additionally, this commit:

* moves the duplicated testLedgerState to a common place as MockLedgerState
* adds support for mocking pool/stake registration queries to MockLedgerState

Signed-off-by: Aurora Gaffney <aurora@blinklabs.io>
@agaffney agaffney force-pushed the feat/validation-rules-query-certs branch from 6eaf223 to f79f36c Compare March 5, 2025 22:12
@agaffney agaffney merged commit e3c9893 into main Mar 5, 2025
9 checks passed
@agaffney agaffney deleted the feat/validation-rules-query-certs branch March 5, 2025 22:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants