From ebe86254782ccf4658881f32ed0c928bbf3ed950 Mon Sep 17 00:00:00 2001 From: Anton Kaliaev Date: Sun, 3 Feb 2019 17:04:22 +0400 Subject: [PATCH] fixes after my second review --- consensus/state_test.go | 1 - state/execution_test.go | 2 +- 2 files changed, 1 insertion(+), 2 deletions(-) diff --git a/consensus/state_test.go b/consensus/state_test.go index 3ce791669..e51ea1247 100644 --- a/consensus/state_test.go +++ b/consensus/state_test.go @@ -1570,6 +1570,5 @@ func subscribe(eventBus *types.EventBus, q tmpubsub.Query) <-chan tmpubsub.Messa if err != nil { panic(fmt.Sprintf("failed to subscribe %s to %v", testSubscriber, q)) } - // XXX: return a subscription? return sub.Out() } diff --git a/state/execution_test.go b/state/execution_test.go index 6146d3ba7..a33757060 100644 --- a/state/execution_test.go +++ b/state/execution_test.go @@ -349,7 +349,7 @@ func TestEndBlockValidatorUpdates(t *testing.T) { assert.EqualValues(t, 10, event.ValidatorUpdates[0].VotingPower) } case <-updatesSub.Cancelled(): - t.Fatal(fmt.Sprintf("updatesSub was cancelled (reason: %v)", updatesSub.Err())) + t.Fatalf("updatesSub was cancelled (reason: %v)", updatesSub.Err()) case <-time.After(1 * time.Second): t.Fatal("Did not receive EventValidatorSetUpdates within 1 sec.") }