You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
LocalOrchestrationAccount uses the getBalance method from vats/localchain.js, which calls out to vbank. This presents a challenge if the caller is trying to work with an asset that's not recognized by vbank:
Also, it doesn't seem Bank (vbank) has an interface for enumerating purses. The implementation may require iterating over every known asset, which is not ideal.
Description of the Design
Security Considerations
Scaling Considerations
Test Plan
Upgrade Considerations
The text was updated successfully, but these errors were encountered:
refs: #9610
## Description
- adds `.getBalances()` method to `cosmos-orchestration-accounts.js`
- adds e2e tests of `CosmosOrchAccount` `getBalance()` and `getBalances()` in `multichain-testing`
- moves `*query` invitations used for testing out of `src/examples/basic-flows` contract into its own contract stored in `src/fixtures/query-flows.contract.js`
### Security Considerations
n/a, using existing powers
### Scaling Considerations
n/a
### Documentation Considerations
n/a
### Testing Considerations
Includes unit tests with high fidelity mocks and e2e tests.
### Upgrade Considerations
n/a, unreleased code
What is the Problem Being Solved?
This ticket captures three related items:
CosmosOrchestrationAccount and LocalOrchestrationAccount do not have the
getBalances
interface implemented (only getBalance).LocalOrchestrationAccount
uses thegetBalance
method fromvats/localchain.js
, which calls out to vbank. This presents a challenge if the caller is trying to work with an asset that's not recognized by vbank:agoric-sdk/packages/vats/src/localchain.js
Lines 113 to 117 in b5cf8bd
Also, it doesn't seem
Bank
(vbank) has an interface for enumerating purses. The implementation may require iterating over every known asset, which is not ideal.Description of the Design
Security Considerations
Scaling Considerations
Test Plan
Upgrade Considerations
The text was updated successfully, but these errors were encountered: