proofs: port TestInteropFaultProofs action test to devstack#19163
proofs: port TestInteropFaultProofs action test to devstack#19163
Conversation
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## develop #19163 +/- ##
===========================================
+ Coverage 75.2% 75.6% +0.3%
===========================================
Files 692 504 -188
Lines 71528 60591 -10937
===========================================
- Hits 53828 45824 -8004
+ Misses 17556 14767 -2789
+ Partials 144 0 -144
Flags with carried forward coverage won't be shown. Click here to find out more.
🚀 New features to boost your workflow:
|
0b31d5f to
f81b9cd
Compare
ajsutton
left a comment
There was a problem hiding this comment.
Looks good generally - but the first couple of tests are failing (in CI and for me locally).
I thought I checked this locally. I can confirm this. And it looks like a bug in kona (or divergence from op-program) where it handles the super state transitions unexpectedly when interop is disabled. |
…rt of target block
|
@ajsutton it should be fixed now. I've tweaked kona to follow op-program behavior - optimism/op-program/client/interop/interop.go Lines 101 to 103 in 3235773 Which I believe is correct. |
Co-authored-by: graphite-app[bot] <96075541+graphite-app[bot]@users.noreply.github.com>
ajsutton
left a comment
There was a problem hiding this comment.
LGTM. Nice to see the tests providing value.
TestInteropFaultProofsaction test scenario in a supernode devstack.part of #19010