Merge pull request #1152 from ProjectOpenSea/less-criteria-fuzzing

fuzz on criteria items less frequently for performance
ProjectOpenSea · Apr 14, 2023 · 3caa72c · 3caa72c
2 parents e9f8a25 + 12de5fa
commit 3caa72c
Showing 1 changed file with 11 additions and 3 deletions.
diff --git a/test/foundry/new/helpers/FuzzGenerators.sol b/test/foundry/new/helpers/FuzzGenerators.sol
@@ -247,8 +247,12 @@ library TestStateGenerator {
  OfferItemSpace[] memory offer = new OfferItemSpace[](len);
  for (uint256 i; i < len; ++i) {
  offer[i] = OfferItemSpace({
- itemType: ItemType(context.randEnum(0, 5)),
- tokenIndex: TokenIndex(context.randEnum(0, 1)),
+ itemType: ItemType(
+ context.randRange(0, 10) != 0
+ ? context.randEnum(0, 3)
+  : context.randEnum(4, 5)
+ ),
+ tokenIndex: TokenIndex(context.randEnum(0, 2)),
  criteria: Criteria(context.randEnum(0, 1)),
  amount: Amount(context.randEnum(0, 2))
  });
@@ -294,7 +298,11 @@ library TestStateGenerator {
  if (!isBasic) {
  for (uint256 i; i < len; ++i) {
  consideration[i] = ConsiderationItemSpace({
- itemType: ItemType(context.randEnum(0, 5)),
+ itemType: ItemType(
+ context.randRange(0, 10) != 0
+ ? context.randEnum(0, 3)
+  : context.randEnum(4, 5)
+ ),
  tokenIndex: TokenIndex(context.randEnum(0, 2)),
  criteria: Criteria(context.randEnum(0, 1)),
  amount: atLeastOne