From 6316e1241e5132728d993a238059ff170a34ae16 Mon Sep 17 00:00:00 2001 From: Dev Ojha Date: Wed, 31 Oct 2018 04:04:46 -0700 Subject: [PATCH] Merge PR #2644: Simulation: Print last block when there is an error There was an off by one error in the log printing function previously --- x/mock/simulation/random_simulate_blocks.go | 2 +- x/mock/simulation/util.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/x/mock/simulation/random_simulate_blocks.go b/x/mock/simulation/random_simulate_blocks.go index 2506af377..bbe27bf6c 100644 --- a/x/mock/simulation/random_simulate_blocks.go +++ b/x/mock/simulation/random_simulate_blocks.go @@ -221,11 +221,11 @@ func createBlockSimulator(testingMode bool, tb testing.TB, t *testing.T, event f accounts []Account, header abci.Header, logWriter func(string)) (opCount int) { for j := 0; j < blocksize; j++ { logUpdate, futureOps, err := selectOp(r)(r, app, ctx, accounts, event) + logWriter(logUpdate) if err != nil { displayLogs() tb.Fatalf("error on operation %d within block %d, %v", header.Height, opCount, err) } - logWriter(logUpdate) queueOperations(operationQueue, timeOperationQueue, futureOps) if testingMode { diff --git a/x/mock/simulation/util.go b/x/mock/simulation/util.go index e82c1bf5e..84a4ad2a3 100644 --- a/x/mock/simulation/util.go +++ b/x/mock/simulation/util.go @@ -134,7 +134,7 @@ func logPrinter(testingmode bool, logs []*strings.Builder) func() { for i := 0; i < len(logs); i++ { // We're passed the last created block if logs[i] == nil { - numLoggers = i - 1 + numLoggers = i break } }