Skip to content

Commit 5da71bf

Browse files
authored
Use slither for valuegeneration tests (#567)
* Update slither for valuegeneration tests * use slither for deployment order test
1 parent be3c8a3 commit 5da71bf

File tree

1 file changed

+3
-5
lines changed

1 file changed

+3
-5
lines changed

Diff for: fuzzing/fuzzer_test.go

+3-5
Original file line numberDiff line numberDiff line change
@@ -750,15 +750,13 @@ func TestValueGenerationGenerateAllTypes(t *testing.T) {
750750

751751
// TestValueGenerationSolving runs a series of tests to test the value generator can solve expected problems.
752752
func TestValueGenerationSolving(t *testing.T) {
753-
// TODO: match_ints_xy is slower than match_uints_xy in the value generator because AST doesn't retain negative
754-
// numbers, improve our logic to solve it faster, then re-enable this.
755753
filePaths := []string{
756754
"testdata/contracts/value_generation/match_addr_contract.sol",
757755
"testdata/contracts/value_generation/match_addr_exact.sol",
758756
"testdata/contracts/value_generation/match_addr_sender.sol",
759757
"testdata/contracts/value_generation/match_string_exact.sol",
760758
"testdata/contracts/value_generation/match_structs_xy.sol",
761-
//"testdata/contracts/value_generation/match_ints_xy.sol",
759+
"testdata/contracts/value_generation/match_ints_xy.sol",
762760
"testdata/contracts/value_generation/match_uints_xy.sol",
763761
"testdata/contracts/value_generation/match_payable_xy.sol",
764762
}
@@ -769,7 +767,7 @@ func TestValueGenerationSolving(t *testing.T) {
769767
config.Fuzzing.TargetContracts = []string{"TestContract"}
770768
config.Fuzzing.Testing.AssertionTesting.Enabled = false
771769
config.Fuzzing.Testing.OptimizationTesting.Enabled = false
772-
config.Slither.UseSlither = false
770+
config.Slither.UseSlither = true
773771
},
774772
method: func(f *fuzzerTestContext) {
775773
// Start the fuzzer
@@ -990,7 +988,7 @@ func TestDeploymentOrderWithCoverage(t *testing.T) {
990988
config.Fuzzing.TargetContracts = []string{"InheritedFirstContract", "InheritedSecondContract"}
991989
config.Fuzzing.Testing.AssertionTesting.Enabled = false
992990
config.Fuzzing.Testing.OptimizationTesting.Enabled = false
993-
config.Slither.UseSlither = false
991+
config.Slither.UseSlither = true
994992
},
995993
method: func(f *fuzzerTestContext) {
996994
// Setup checks for event emissions

0 commit comments

Comments
 (0)