Skip to content

Commit 0905660

Browse files
authored
core/state: fix inconsistent verkle test error messages (#29753)
1 parent 41abab9 commit 0905660

File tree

1 file changed

+10
-10
lines changed

1 file changed

+10
-10
lines changed

core/state/access_events_test.go

Lines changed: 10 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -41,8 +41,8 @@ func TestAccountHeaderGas(t *testing.T) {
4141

4242
// Check cold read cost
4343
gas := ae.VersionGas(testAddr, false)
44-
if gas != params.WitnessBranchReadCost+params.WitnessChunkReadCost {
45-
t.Fatalf("incorrect gas computed, got %d, want %d", gas, params.WitnessBranchReadCost+params.WitnessChunkReadCost)
44+
if want := params.WitnessBranchReadCost + params.WitnessChunkReadCost; gas != want {
45+
t.Fatalf("incorrect gas computed, got %d, want %d", gas, want)
4646
}
4747

4848
// Check warm read cost
@@ -71,8 +71,8 @@ func TestAccountHeaderGas(t *testing.T) {
7171

7272
// Check cold write cost
7373
gas = ae.VersionGas(testAddr, true)
74-
if gas != params.WitnessBranchWriteCost+params.WitnessChunkWriteCost {
75-
t.Fatalf("incorrect gas computed, got %d, want %d", gas, params.WitnessBranchReadCost+params.WitnessBranchWriteCost)
74+
if want := params.WitnessBranchWriteCost + params.WitnessChunkWriteCost; gas != want {
75+
t.Fatalf("incorrect gas computed, got %d, want %d", gas, want)
7676
}
7777

7878
// Check warm write cost
@@ -83,8 +83,8 @@ func TestAccountHeaderGas(t *testing.T) {
8383

8484
// Check a write without a read charges both read and write costs
8585
gas = ae.BalanceGas(testAddr2, true)
86-
if gas != params.WitnessBranchReadCost+params.WitnessBranchWriteCost+params.WitnessChunkWriteCost+params.WitnessChunkReadCost {
87-
t.Fatalf("incorrect gas computed, got %d, want %d", gas, params.WitnessBranchReadCost+params.WitnessBranchWriteCost+params.WitnessChunkWriteCost+params.WitnessChunkReadCost)
86+
if want := params.WitnessBranchReadCost + params.WitnessBranchWriteCost + params.WitnessChunkWriteCost + params.WitnessChunkReadCost; gas != want {
87+
t.Fatalf("incorrect gas computed, got %d, want %d", gas, want)
8888
}
8989

9090
// Check that a write followed by a read charges nothing
@@ -113,8 +113,8 @@ func TestContractCreateInitGas(t *testing.T) {
113113

114114
// Check cold read cost, without a value
115115
gas := ae.ContractCreateInitGas(testAddr, false)
116-
if gas != params.WitnessBranchWriteCost+params.WitnessBranchReadCost+params.WitnessChunkWriteCost*2+params.WitnessChunkReadCost*2 {
117-
t.Fatalf("incorrect gas computed, got %d, want %d", gas, params.WitnessBranchWriteCost+params.WitnessBranchReadCost+params.WitnessChunkWriteCost*3)
116+
if want := params.WitnessBranchWriteCost + params.WitnessBranchReadCost + params.WitnessChunkWriteCost*2 + params.WitnessChunkReadCost*2; gas != want {
117+
t.Fatalf("incorrect gas computed, got %d, want %d", gas, want)
118118
}
119119

120120
// Check warm read cost
@@ -131,8 +131,8 @@ func TestMessageCallGas(t *testing.T) {
131131

132132
// Check cold read cost, without a value
133133
gas := ae.MessageCallGas(testAddr)
134-
if gas != params.WitnessBranchReadCost+params.WitnessChunkReadCost*2 {
135-
t.Fatalf("incorrect gas computed, got %d, want %d", gas, params.WitnessBranchReadCost+params.WitnessChunkReadCost*2)
134+
if want := params.WitnessBranchReadCost + params.WitnessChunkReadCost*2; gas != want {
135+
t.Fatalf("incorrect gas computed, got %d, want %d", gas, want)
136136
}
137137

138138
// Check that reading the version and code size of the same account does not incur the branch read cost

0 commit comments

Comments
 (0)