DI - PoS

[tzemanovic]

Describe your changes

Dependency injection for proof_of_stake, replacing deps on:

• governance
• parameters
• trans_token

To help with these changes, the pos_queries mod was refactored out in de1555a

Indicate on which release or other PRs this topic is based on

Depends-On: #3482

• diff: https://github.com/anoma/namada/pull/3497/files/1fbb9fd40db1fd780056b596bb764dbfac7f78ed..aa570a1a036b4b7595acf2e3116812c4aeb72767

Checklist before merging to draft

• I have added a changelog
• Git history is in acceptable state

[codecov]

Codecov Report

Attention: Patch coverage is 73.55049% with 406 lines in your changes missing coverage. Please review.

Project coverage is 54.58%. Comparing base (1154023) to head (aa570a1).
Report is 29 commits behind head on main.

Files	Patch %	Lines
crates/tx_prelude/src/proof_of_stake.rs	0.00%	59 Missing ⚠️
crates/proof_of_stake/src/queries.rs	41.66%	49 Missing ⚠️
...s/node/src/shell/vote_extensions/val_set_update.rs	0.00%	32 Missing ⚠️
...rates/node/src/shell/vote_extensions/eth_events.rs	0.00%	26 Missing ⚠️
crates/proof_of_stake/src/lib.rs	91.58%	26 Missing ⚠️
crates/sdk/src/queries/vp/pos.rs	0.00%	23 Missing ⚠️
...ge/src/protocol/validation/validator_set_update.rs	8.33%	22 Missing ⚠️
...ridge/src/protocol/validation/bridge_pool_roots.rs	14.28%	18 Missing ⚠️
crates/token/src/lib.rs	25.00%	18 Missing ⚠️
crates/node/src/bench_utils.rs	0.00%	15 Missing ⚠️
... and 21 more

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #3497      +/-   ##
==========================================
+ Coverage   54.51%   54.58%   +0.06%     
==========================================
  Files         323      323              
  Lines      113288   113736     +448     
==========================================
+ Hits        61762    62083     +321     
- Misses      51526    51653     +127     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mergify]

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

⛓️ Depends-On Requirements

Requirement based on the presence of Depends-On in the body of the pull request

• depends-on=anoma/namada#3482 [⛓️ DI - governance #3482]

[tzemanovic]

the changes in epoched were the annoying ones as they end up bubbling up through a lot of fns that use these (we depend on a gov parameter for the OffsetMaxProposalPeriod epoch offset)

[tzemanovic]

@Mergifyio refresh

[mergify]

refresh

✅ Pull request refreshed

[mergify]

Hey @tzemanovic, your pull request has been dequeued due to the following reason: CHECKS_FAILED.
Sorry about that, but you can requeue the PR by using @mergifyio requeue if you think this was a mistake.