Skip to content

Commit

Permalink
Fixed issue when chain binaries and account prefixes weren't of same …
Browse files Browse the repository at this point in the history
…length and added test between Gaia6 and Gaia7
  • Loading branch information
ljoss17 committed Nov 29, 2022
1 parent ab75855 commit be0a746
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 1 deletion.
3 changes: 3 additions & 0 deletions .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -76,6 +76,9 @@ jobs:
- package: .#gaia8 .#evmos
command: gaiad,evmosd
account_prefix: cosmos,evmos
- package: .#gaia6 .#gaia7
command: $(nix shell .#gaia6 -c which gaiad),$(nix shell .#gaia7 -c which gaiad)
account_prefix: cosmos

steps:
- uses: actions/checkout@v2
Expand Down
3 changes: 2 additions & 1 deletion tools/test-framework/src/chain/builder.rs
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ impl ChainBuilder {
// If there are more spawned chains than given chain binaries, take the N-th position modulo
// the number of chain binaries given.
let chain_number = chain_number % self.command_paths.len();
let account_number = chain_number % self.account_prefixes.len();

let chain_type = ChainType::from_str(&self.command_paths[chain_number])?;

Expand All @@ -112,7 +113,7 @@ impl ChainBuilder {
self.command_paths[chain_number].clone(),
chain_id,
home_path,
self.account_prefixes[chain_number].clone(),
self.account_prefixes[account_number].clone(),
rpc_port,
grpc_port,
grpc_web_port,
Expand Down

0 comments on commit be0a746

Please sign in to comment.