cerc-io/lotus
16
0
Fork 0
Code Issues Pull Requests Packages Projects Releases 27 Wiki Activity

events: Fix double apply in events.Called as well

Browse Source
This commit is contained in:
Łukasz Magiera 2019-12-04 13:58:22 +01:00
parent 7d80cbfbdf
commit 631a9cd3b6
2 changed files with 10 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
chain/events/events_called.go
View File

@ -183,6 +183,10 @@ func (e *calledEvents) applyWithConfidence(ts *types.TipSet) {
} }
for _, event := range events { for _, event := range events {
if event.called {
continue
}
trigger := e.triggers[event.trigger] trigger := e.triggers[event.trigger]
if trigger.disabled { if trigger.disabled {
continue continue

6
chain/events/events_test.go
View File

@ -604,6 +604,12 @@ func TestCalled(t *testing.T) {
require.Equal(t, false, reverted) require.Equal(t, false, reverted)
applied = false applied = false
// dip below confidence
fcs.advance(2, 2, nil) // H=10 (confidence=3, apply)
require.Equal(t, false, applied)
require.Equal(t, false, reverted)
require.Equal(t, uint64(7), appliedTs.Height()) require.Equal(t, uint64(7), appliedTs.Height())
require.Equal(t, "bafkqaaa", appliedTs.Blocks()[0].Messages.String()) require.Equal(t, "bafkqaaa", appliedTs.Blocks()[0].Messages.String())
require.Equal(t, uint64(10), appliedH) require.Equal(t, uint64(10), appliedH)