-
Notifications
You must be signed in to change notification settings - Fork 625
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: solomachine mock channel handshake, allow zero-height in channel handshake msgs #2776
chore: solomachine mock channel handshake, allow zero-height in channel handshake msgs #2776
Conversation
… non-zero proof height in channel handshake msgs
Left changelog section empty as this is being upstreamed to @colin-axner's branch at the moment. |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## colin/1874-sm-proof-height #2776 +/- ##
==============================================================
- Coverage 78.73% 78.69% -0.04%
==============================================================
Files 178 178
Lines 12295 12283 -12
==============================================================
- Hits 9680 9666 -14
- Misses 2188 2190 +2
Partials 427 427
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM just had a few suggestions/questions 👍
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excellent! You were lighting quick with this one! ⚡
// close init is not necessary as the solomachine implementation is mocked | ||
|
||
suite.solomachine.ChanCloseConfirm(suite.chainA, channelID) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If we turn the setup into a reusable func for testing receives and acks, we might want this last section to be a separate test
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I agree, I think we can do that in the next series of PRs for packet lifecycle stuff. I'll merge this to your branch now
Description
MsgChannelOpenTry
MsgChannelOpenAck
MsgChannelOpenConfirm
MsgChannelCloseConfirm
06-solomachine
closes: #2749 #2750
Commit Message / Changelog Entry
see the guidelines for commit messages. (view raw markdown for examples)
Before we can merge this PR, please make sure that all the following items have been
checked off. If any of the checklist items are not applicable, please leave them but
write a little note why.
docs/
) or specification (x/<module>/spec/
).godoc
comments.Files changed
in the Github PR explorer.Codecov Report
in the comment section below once CI passes.