Skip to content

Commit

Permalink
Merge pull request #4095 from tybug/avoid-extraneous-explain
Browse files Browse the repository at this point in the history
Don't run explain phase unnecessarily
  • Loading branch information
tybug authored Aug 29, 2024
2 parents e339c5f + 90f004a commit eaafdfc
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion hypothesis-python/tests/conjecture/common.py
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@
import math
from contextlib import contextmanager

from hypothesis import HealthCheck, assume, settings, strategies as st
from hypothesis import HealthCheck, Phase, assume, settings, strategies as st
from hypothesis.control import current_build_context
from hypothesis.errors import InvalidArgument
from hypothesis.internal.conjecture import engine as engine_module
Expand Down Expand Up @@ -64,6 +64,9 @@ def accept(f):
max_examples=5000,
database=None,
suppress_health_check=list(HealthCheck),
# avoid running the explain phase in shrinker.shrink() in tests
# which don't test the inquisitor.
phases=set(settings.default.phases) - {Phase.explain},
),
)
runner.cached_test_function(start)
Expand Down

0 comments on commit eaafdfc

Please sign in to comment.