Skip to content

Commit

Permalink
Update Nix and CI dependencies (#2523)
Browse files Browse the repository at this point in the history
* Update Nix and CI dependencies

* Fix new account sequence mismatch error pattern

* Update Cosmos.nix
  • Loading branch information
soareschen authored Aug 8, 2022
1 parent 5df3547 commit 190d810
Show file tree
Hide file tree
Showing 5 changed files with 111 additions and 65 deletions.
3 changes: 3 additions & 0 deletions .github/workflows/integration.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -62,6 +62,9 @@ jobs:
- package: ibc-go-v4-simapp
command: simd
account_prefix: cosmos
- package: ibc-go-v5-simapp
command: simd
account_prefix: cosmos
- package: osmosis
command: osmosisd
account_prefix: osmo
Expand Down
5 changes: 2 additions & 3 deletions Cargo.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

156 changes: 98 additions & 58 deletions flake.lock

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

1 change: 1 addition & 0 deletions flake.nix
Original file line number Diff line number Diff line change
Expand Up @@ -39,6 +39,7 @@
ibc-go-v2-simapp
ibc-go-v3-simapp
ibc-go-v4-simapp
ibc-go-v5-simapp
apalache
;

Expand Down
11 changes: 7 additions & 4 deletions relayer/src/error.rs
Original file line number Diff line number Diff line change
Expand Up @@ -571,10 +571,7 @@ impl GrpcStatusSubdetail {
/// there are hermes code changes such that the E < G case is not previously caught anymore,
/// then this predicate will catch all "account sequence mismatch" errors
pub fn is_account_sequence_mismatch_that_requires_refresh(&self) -> bool {
self.status
.message()
.trim_start()
.starts_with("account sequence mismatch")
self.status.message().contains("account sequence mismatch")
}

/// Check whether this gRPC error matches:
Expand Down Expand Up @@ -653,6 +650,12 @@ mod tests {
"account sequence mismatch, expected 200, got 100 --> this part has changed",
result: Some((200, 100)),
},
Test {
name: "good changed mismatch error, expected > got",
message:
"codespace sdk code 32: incorrect account sequence: account sequence mismatch, expected 200, got 100",
result: Some((200, 100)),
},
Test {
name: "bad mismatch error, bad expected",
message:
Expand Down

0 comments on commit 190d810

Please sign in to comment.