Skip to content

Commit

Permalink
feat(orchestration): add more chain infos for fusdc (#10845)
Browse files Browse the repository at this point in the history
closes #10736
refs #10629

## Description

- Adds remaining cosmos chains for fusdc as listed in https://github.com/Agoric/product-tasks/issues/186#issuecomment-2442824919.
- Adds `icaEnabled` property to `CosmosChainInfo`
- Makes the connection infos in vstorage support chain IDs with underscores (`_`). Since we use underscores as a separator in the vstorage key, we sanitize the chain IDs by doubling the underscores (similar to double-quote in csv)
- Don't use all the chain infos in bootstrap tests, only a subset, since CI can't handle the input size

### Security Considerations

We're adding non-ica-enabled chains now, so #10629 is relevant.

### Scaling Considerations

`packages/orchestration/src/fetched-chain-info.js` is quite large, hopefully not too large?

### Documentation Considerations

Not sure if these chains need to be documented somewhere.

### Testing Considerations

Just did `yarn codegen`. Not sure how to test this for real.

### Upgrade Considerations

Updating the chain infos will presumably require a core-eval
  • Loading branch information
mergify[bot] authored Jan 22, 2025
2 parents 3d4f71f + bcc450c commit 8fb6eaf
Show file tree
Hide file tree
Showing 20 changed files with 21,220 additions and 1,884 deletions.
6 changes: 3 additions & 3 deletions packages/boot/test/bootstrapTests/orchestration.test.ts
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ import type { start as startStakeIca } from '@agoric/orchestration/src/examples/
import type { Instance } from '@agoric/zoe/src/zoeService/utils.js';
import type { TestFn } from 'ava';
import { SIMULATED_ERRORS } from '@agoric/vats/tools/fake-bridge.js';
import fetchedChainInfo from '@agoric/orchestration/src/fetched-chain-info.js';
import { buildVTransferEvent } from '@agoric/orchestration/tools/ibc-mocks.js';
import { BridgeId } from '@agoric/internal';
import { makeTestAddress } from '@agoric/orchestration/tools/make-test-address.js';
Expand All @@ -25,6 +24,7 @@ import {
insistManagerType,
makeSwingsetHarness,
} from '../../tools/supports.js';
import { minimalChainInfos } from '../tools/chainInfo.js';

const test: TestFn<
WalletFactoryTestContext & {
Expand Down Expand Up @@ -331,7 +331,7 @@ test.serial('basic-flows', async t => {
'@agoric/builders/scripts/orchestration/init-basic-flows.js',
[
'--chainInfo',
JSON.stringify(withChainCapabilities(fetchedChainInfo)),
JSON.stringify(withChainCapabilities(minimalChainInfos)),
'--assetInfo',
JSON.stringify([
[
Expand Down Expand Up @@ -556,7 +556,7 @@ test.serial('basic-flows - portfolio holder', async t => {
'@agoric/builders/scripts/orchestration/init-basic-flows.js',
[
'--chainInfo',
JSON.stringify(withChainCapabilities(fetchedChainInfo)),
JSON.stringify(withChainCapabilities(minimalChainInfos)),
'--assetInfo',
JSON.stringify([
[
Expand Down
Loading

0 comments on commit 8fb6eaf

Please sign in to comment.