Separate Querier and TxHandler

[Kayanski]

This PR aims at separating the querying and the transaction signing capabilities of CwEnv.

Closes ORC-61

Changes

1. Removed BankQuerier : Replaced by BankQuerierGetter and BankQuerier on top of it
2. Removed WasmCodeQuerier: Replaced by WasmQuerierGetter and WasmQuerier. Local hash function is also added in this trait

[cloudflare-workers-and-pages]

Deploying with    Cloudflare Pages

Latest commit:	ee2ab1d
Status:	✅  Deploy successful!
Preview URL:	https://83874985.cw-orchestrator.pages.dev
Branch Preview URL:	https://update-separate-query-tx.cw-orchestrator.pages.dev

View logs

[CyberHoward]

Deprecation issues

The block_info function of TxHandler is now marked as deprecated but we can't implement the exact same function on the QueryHandler trait because it would conflict with this function as long as it exists.

I think that we need to publish a breaking version for this change because of that !

Traits can have the same function. When calling the function the user needs to have either of the traits imported. If they are both imported they need to specify which trait they want to use. i.e.

TxHandler::block_info(chain)

So no breaking change required here afaik

[Kayanski]

Deprecation issues

The block_info function of TxHandler is now marked as deprecated but we can't implement the exact same function on the QueryHandler trait because it would conflict with this function as long as it exists.
I think that we need to publish a breaking version for this change because of that !

Traits can have the same function. When calling the function the user needs to have either of the traits imported. If they are both imported they need to specify which trait they want to use. i.e.

TxHandler::block_info(chain)

So no breaking change required here afaik

The error that show up at the following line shows exactly what I meant with this comment : https://github.com/AbstractSDK/cw-orchestrator/blob/update/separate-query-tx/contracts/counter/src/interface.rs#L48

[CyberHoward]

Can we maybe start working with milestones in github for breaking versions? I created one for 0.20

Have you merged anything that was breaking?

[Kayanski]

Can we maybe start working with milestones in github for breaking versions? I created one for 0.20
--> OK

Have you merged anything that was breaking?
--> No

[codecov]

Codecov Report

Attention: 491 lines in your changes are missing coverage. Please review.

Comparison is base (f39ab04) 66.4% compared to head (ee2ab1d) 65.4%.
Report is 2 commits behind head on main.

Additional details and impacted files

Files	Coverage Δ
cw-orch-daemon/src/error.rs	14.2% <ø> (ø)
...ges/cw-orch-core/src/contract/contract_instance.rs	81.8% <100.0%> (+3.4%)	⬆️
...ages/cw-orch-core/src/contract/interface_traits.rs	75.9% <100.0%> (+5.3%)	⬆️
...kages/cw-orch-core/src/environment/queriers/mod.rs	100.0% <100.0%> (ø)
...ages/cw-orch-core/src/environment/queriers/wasm.rs	100.0% <100.0%> (ø)
packages/cw-orch-core/src/environment/state.rs	55.5% <ø> (ø)
packages/cw-orch-mock/src/core.rs	93.7% <100.0%> (+6.2%)	⬆️
...kages/cw-orch-core/src/environment/queriers/env.rs	0.0% <0.0%> (ø)
cw-orch-daemon/src/tx_broadcaster.rs	76.7% <0.0%> (ø)
packages/cw-orch-core/src/environment/mut_env.rs	0.0% <0.0%> (ø)
... and 27 more

[Kayanski]

In standby until #318 is done

[CyberHoward]

Think some code can be removed and daemon queriers should be constructed with runtime by default

[CyberHoward]

Can it not get the runtime handle from the daemon object?

[Kayanski]

But it can. Keeping the same structure as for Queriers with traits