Skip to content

Commit

Permalink
Merge pull request #1068 from scrtlabs/lior-change-recursion-limit-to-10
Browse files Browse the repository at this point in the history
Fix another recursion test
  • Loading branch information
liorbond authored Aug 10, 2022
2 parents bfb28f4 + 2bb5974 commit f86d6d9
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 5 deletions.
8 changes: 4 additions & 4 deletions x/compute/internal/keeper/recurse_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -306,14 +306,14 @@ func TestLimitRecursiveQueryGas(t *testing.T) {
expectOOM: false,
expectRecursionLimit: false,
},
"recursion 9, lots of work": {
"recursion 11, lots of work": {
gasLimit: 4_000_000,
msg: Recurse{
Depth: 9,
Depth: 11,
Work: 2000,
},
expectQueriesFromContract: 9,
expectedGas: GasWork2k + 9*(GasWork2k+GasReturnHashed),
expectQueriesFromContract: 11,
expectedGas: GasWork2k + 11*(GasWork2k+GasReturnHashed),
expectOutOfGas: false,
expectOOM: false,
expectRecursionLimit: true,
Expand Down
1 change: 0 additions & 1 deletion x/compute/internal/keeper/secret_contracts_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -3668,7 +3668,6 @@ func TestWasmMsgStructure(t *testing.T) {
require.Empty(t, err)
} else {
require.NotEmpty(t, err)
fmt.Printf("LIORRRR %+v", err)
require.Contains(t, fmt.Sprintf("%+v", err), test.expectedError)
}
})
Expand Down

0 comments on commit f86d6d9

Please sign in to comment.