Skip to content

Commit

Permalink
Merge pull request #3856 from jobh/move-test
Browse files Browse the repository at this point in the history
Move slow stateful test from cover to nocover
  • Loading branch information
Zac-HD committed Jan 24, 2024
2 parents fc6939a + 7d489e5 commit 349c726
Show file tree
Hide file tree
Showing 2 changed files with 66 additions and 59 deletions.
58 changes: 0 additions & 58 deletions hypothesis-python/tests/cover/test_stateful.py
Original file line number Diff line number Diff line change
Expand Up @@ -1082,64 +1082,6 @@ def mostly_fails(self, d):
assert "v1 = state.init_data(value=v1)" not in err.value.__notes__


def test_multiple_precondition_bug():
# See https://github.com/HypothesisWorks/hypothesis/issues/2861
class MultiplePreconditionMachine(RuleBasedStateMachine):
@rule(x=integers())
def good_method(self, x):
pass

@precondition(lambda self: True)
@precondition(lambda self: False)
@rule(x=integers())
def bad_method_a(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@precondition(lambda self: False)
@precondition(lambda self: True)
@rule(x=integers())
def bad_method_b(self, x):
raise AssertionError("This rule might be skipped for the wrong reason.")

@precondition(lambda self: True)
@rule(x=integers())
@precondition(lambda self: False)
def bad_method_c(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@rule(x=integers())
@precondition(lambda self: True)
@precondition(lambda self: False)
def bad_method_d(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@precondition(lambda self: True)
@precondition(lambda self: False)
@invariant()
def bad_invariant_a(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@precondition(lambda self: False)
@precondition(lambda self: True)
@invariant()
def bad_invariant_b(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@precondition(lambda self: True)
@invariant()
@precondition(lambda self: False)
def bad_invariant_c(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@invariant()
@precondition(lambda self: True)
@precondition(lambda self: False)
def bad_invariant_d(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

run_state_machine_as_test(MultiplePreconditionMachine)


class TrickyInitMachine(RuleBasedStateMachine):
@initialize()
def init_a(self):
Expand Down
67 changes: 66 additions & 1 deletion hypothesis-python/tests/nocover/test_stateful.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,14 @@
import pytest

from hypothesis import settings as Settings
from hypothesis.stateful import Bundle, RuleBasedStateMachine, precondition, rule
from hypothesis.stateful import (
Bundle,
RuleBasedStateMachine,
invariant,
precondition,
rule,
run_state_machine_as_test,
)
from hypothesis.strategies import booleans, integers, lists

Leaf = namedtuple("Leaf", ("label",))
Expand Down Expand Up @@ -201,3 +208,61 @@ def step(self):

class TestMyStatefulMachine(MyStatefulMachine.TestCase):
settings = Settings(derandomize=True, stateful_step_count=5)


def test_multiple_precondition_bug():
# See https://github.com/HypothesisWorks/hypothesis/issues/2861
class MultiplePreconditionMachine(RuleBasedStateMachine):
@rule(x=integers())
def good_method(self, x):
pass

@precondition(lambda self: True)
@precondition(lambda self: False)
@rule(x=integers())
def bad_method_a(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@precondition(lambda self: False)
@precondition(lambda self: True)
@rule(x=integers())
def bad_method_b(self, x):
raise AssertionError("This rule might be skipped for the wrong reason.")

@precondition(lambda self: True)
@rule(x=integers())
@precondition(lambda self: False)
def bad_method_c(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@rule(x=integers())
@precondition(lambda self: True)
@precondition(lambda self: False)
def bad_method_d(self, x):
raise AssertionError("This rule runs, even though it shouldn't.")

@precondition(lambda self: True)
@precondition(lambda self: False)
@invariant()
def bad_invariant_a(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@precondition(lambda self: False)
@precondition(lambda self: True)
@invariant()
def bad_invariant_b(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@precondition(lambda self: True)
@invariant()
@precondition(lambda self: False)
def bad_invariant_c(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

@invariant()
@precondition(lambda self: True)
@precondition(lambda self: False)
def bad_invariant_d(self):
raise AssertionError("This invariant runs, even though it shouldn't.")

run_state_machine_as_test(MultiplePreconditionMachine)

0 comments on commit 349c726

Please sign in to comment.