Skip to content

Commit 3b80752

Browse files
authored
core/state: fix the flaky TestSizeTracker (#32993)
1 parent 6608a2a commit 3b80752

File tree

1 file changed

+14
-7
lines changed

1 file changed

+14
-7
lines changed

core/state/state_sizer_test.go

Lines changed: 14 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -94,6 +94,14 @@ func TestSizeTracker(t *testing.T) {
9494
}
9595
baselineRoot := currentRoot
9696

97+
// Close and reopen the trie database so all async flushes triggered by the
98+
// baseline commits are written before we measure the baseline snapshot.
99+
if err := tdb.Close(); err != nil {
100+
t.Fatalf("Failed to close triedb before baseline measurement: %v", err)
101+
}
102+
tdb = triedb.NewDatabase(db, &triedb.Config{PathDB: pathdb.Defaults})
103+
sdb = NewDatabase(tdb, nil)
104+
97105
// Wait for snapshot completion
98106
for !tdb.SnapshotCompleted() {
99107
time.Sleep(100 * time.Millisecond)
@@ -215,13 +223,12 @@ func TestSizeTracker(t *testing.T) {
215223
if actualStats.ContractCodeBytes != expectedStats.ContractCodeBytes {
216224
t.Errorf("Contract code bytes mismatch: expected %d, got %d", expectedStats.ContractCodeBytes, actualStats.ContractCodeBytes)
217225
}
218-
// TODO: failed on github actions, need to investigate
219-
// if actualStats.AccountTrienodes != expectedStats.AccountTrienodes {
220-
// t.Errorf("Account trie nodes mismatch: expected %d, got %d", expectedStats.AccountTrienodes, actualStats.AccountTrienodes)
221-
// }
222-
// if actualStats.AccountTrienodeBytes != expectedStats.AccountTrienodeBytes {
223-
// t.Errorf("Account trie node bytes mismatch: expected %d, got %d", expectedStats.AccountTrienodeBytes, actualStats.AccountTrienodeBytes)
224-
// }
226+
if actualStats.AccountTrienodes != expectedStats.AccountTrienodes {
227+
t.Errorf("Account trie nodes mismatch: expected %d, got %d", expectedStats.AccountTrienodes, actualStats.AccountTrienodes)
228+
}
229+
if actualStats.AccountTrienodeBytes != expectedStats.AccountTrienodeBytes {
230+
t.Errorf("Account trie node bytes mismatch: expected %d, got %d", expectedStats.AccountTrienodeBytes, actualStats.AccountTrienodeBytes)
231+
}
225232
if actualStats.StorageTrienodes != expectedStats.StorageTrienodes {
226233
t.Errorf("Storage trie nodes mismatch: expected %d, got %d", expectedStats.StorageTrienodes, actualStats.StorageTrienodes)
227234
}

0 commit comments

Comments
 (0)