feat(experimental): remove 'expect' override

Scope of expect_override was too narrow and does not solve the typing issue adequately
this reverts #676

Author: aleneum

Changelog.md

@@ -5,7 +5,6 @@
 - Bug #610: Decorate models appropriately when `HierarchicalMachine` is passed to `add_state` (thanks @e0lithic)
 - Bug #647: Let `may_<trigger>` check all parallel states in processing order (thanks @spearsear)
 - Bug: `HSM.is_state` works with parallel states now
-- Experimental feature: Use the `transitions.experimental.typing.expect_override` decorator to mark methods as safe for overriding `Machine._checked_assignment` will now override _existing_ model methods (e.g. existing trigger like `melt` or convenience functions like `is_<state>`) if they have a `expect_override` attribute set to `True`. 
 
 ## 0.9.1 (May 2024)
 

tests/test_experimental.py

@@ -2,45 +2,22 @@
 from unittest.mock import MagicMock
 
 from transitions import Machine
-from transitions.experimental.decoration import expect_override
 
 
 class TestExperimental(TestCase):
 
     def setUp(self) -> None:
-        self.machine_cls = Machine
-        return super().setUp()
 
-    def test_override_decorator(self):
-        b_mock = MagicMock()
-        c_mock = MagicMock()
 
         class Model:
 
-            @expect_override
-            def is_A(self) -> bool:
                 raise RuntimeError("Should be overridden")
 
-            def is_B(self) -> bool:
-                b_mock()
-                return False
 
-            @expect_override
             def is_C(self) -> bool:
-                c_mock()
-                return False
 
         model = Model()
-        machine = self.machine_cls(model, states=["A", "B"], initial="A")
         self.assertTrue(model.is_A())
-        self.assertTrue(model.to_B())
-        self.assertFalse(model.is_B())  # not overridden with convenience function
-        self.assertTrue(b_mock.called)
-        self.assertFalse(model.is_C())  # not overridden yet
-        self.assertTrue(c_mock.called)
         machine.add_state("C")
         self.assertFalse(model.is_C())  # now it is!
-        self.assertEqual(1, c_mock.call_count)   # call_count is not increased
-        self.assertTrue(model.to_C())
         self.assertTrue(model.is_C())
-        self.assertEqual(1, c_mock.call_count)