Skip to content

Commit

Permalink
(fix) Remove t.Skip which relate TestSimulateValidatorsChange
Browse files Browse the repository at this point in the history
- TestHandshakeReplayAll
- TestHandshakeReplaySome
- TestHandshakeReplayOne
- TestHandshakeReplayNone
- TestMockProxyApp
  • Loading branch information
tnasu committed Aug 27, 2021
1 parent 566220f commit 131fc7c
Showing 1 changed file with 0 additions and 10 deletions.
10 changes: 0 additions & 10 deletions consensus/replay_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -532,8 +532,6 @@ func TestSimulateValidatorsChange(t *testing.T) {

// Sync from scratch
func TestHandshakeReplayAll(t *testing.T) {
t.Skip("With decision to skip TestSimulateValidatorsChange, we also skip this test case" +
" that uses the StateDB/BlockDB created in there.")
for _, m := range modes {
testHandshakeReplay(t, config, 0, m, false)
}
Expand All @@ -544,8 +542,6 @@ func TestHandshakeReplayAll(t *testing.T) {

// Sync many, not from scratch
func TestHandshakeReplaySome(t *testing.T) {
t.Skip("With decision to skip TestSimulateValidatorsChange, we also skip this test case" +
" that uses the StateDB/BlockDB created in there.")
for _, m := range modes {
testHandshakeReplay(t, config, 2, m, false)
}
Expand All @@ -556,8 +552,6 @@ func TestHandshakeReplaySome(t *testing.T) {

// Sync from lagging by one
func TestHandshakeReplayOne(t *testing.T) {
t.Skip("With decision to skip TestSimulateValidatorsChange, we also skip this test case" +
" that uses the StateDB/BlockDB created in there.")
for _, m := range modes {
testHandshakeReplay(t, config, numBlocks-1, m, false)
}
Expand All @@ -568,8 +562,6 @@ func TestHandshakeReplayOne(t *testing.T) {

// Sync from caught up
func TestHandshakeReplayNone(t *testing.T) {
t.Skip("With decision to skip TestSimulateValidatorsChange, we also skip this test case" +
" that uses the StateDB/BlockDB created in there.")
for _, m := range modes {
testHandshakeReplay(t, config, numBlocks, m, false)
}
Expand All @@ -580,8 +572,6 @@ func TestHandshakeReplayNone(t *testing.T) {

// Test mockProxyApp should not panic when app return ABCIResponses with some empty ResponseDeliverTx
func TestMockProxyApp(t *testing.T) {
t.Skip("With decision to skip TestSimulateValidatorsChange, we also skip this test case" +
" that uses global variable `sim` initialized in testHandshakeReplay.")
sim.CleanupFunc() // clean the test env created in TestSimulateValidatorsChange
logger := log.TestingLogger()
var validTxs, invalidTxs = 0, 0
Expand Down

0 comments on commit 131fc7c

Please sign in to comment.