-
Notifications
You must be signed in to change notification settings - Fork 213
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
add test Wallet has the same tip as network/information #829
Conversation
sync <- getNetworkInfo | ||
verify sync [ expectFieldEqual DSL.syncProgress Ready ] | ||
r <- getNetworkInfo | ||
let (Right epochNum) = fmap (epochNumber . tip) (snd r) |
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.
let (Right epochNum) = fmap (epochNumber . tip) (snd r) | |
let epochNum = getFromResponse (#tip . #epochNumber) |
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.
updated!
expectEventually' ctx state Ready w | ||
expectEventually' ctx (#tip . #epochNumber) epochNum w | ||
expectEventually' ctx (#tip . #slotNumber) slotNum w | ||
expectEventually' ctx (#tip . #height) blockHeight w |
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.
👍
f0fa3ef
to
1187d2e
Compare
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
bors r+ |
Merge conflict (retrying...) |
827: Add syncProgressRelativeToCurrentTime helper r=KtorZ a=Anviking # Issue Number #711 # Overview - [x] I made it easier to use `syncProgress` from StakePoolMetrics (or potential other places) by 1. adding a helper `syncProgressRelativeToTime :: SlotParameters -> BlockHeader -> SyncProgress` 2. Adding `slotParams :: BlockchainParameters -> SlotParameters` # Comments <!-- Additional comments or screenshots to attach if any --> <!-- Don't forget to: ✓ Self-review your changes to make sure nothing unexpected slipped through ✓ Assign yourself to the PR ✓ Assign one or several reviewer(s) ✓ Once created, link this PR to its corresponding ticket ✓ Acknowledge any changes required to the Wiki --> 829: add test Wallet has the same tip as network/information r=KtorZ a=piotr-iohk # Issue Number #819 # Overview <!-- Detail in a few bullet points the work accomplished in this PR --> - [ ] add test checking that wallet has the same tip as network/information endpoint # Comments <!-- Additional comments or screenshots to attach if any --> <!-- Don't forget to: ✓ Self-review your changes to make sure nothing unexpected slipped through ✓ Assign yourself to the PR ✓ Assign one or several reviewer(s) ✓ Once created, link this PR to its corresponding ticket ✓ Acknowledge any changes required to the Wiki --> Co-authored-by: Johannes Lund <johannes.lund@iohk.io> Co-authored-by: Piotr Stachyra <piotr.stachyra@iohk.io>
Build succeeded |
Issue Number
#819
Overview
Comments