Add regression test for #26

1 files changed, 29 insertions, 1 deletions

diff --git a/mpv-test.py b/mpv-test.py
index 62eb8fd..eb3c13f 100755
--- a/mpv-test.py
+++ b/mpv-test.py
@@ -253,7 +253,6 @@ class RegressionTests(unittest.TestCase):
         `unobserve_property`.
         """
         handler = mock.Mock()
-        handler.observed_mpv_properties = []
 
         m = mpv.MPV()
         m.observe_property('loop', handler)
@@ -270,6 +269,35 @@ class RegressionTests(unittest.TestCase):
         finally:
             m.terminate()
 
+    def test_instance_method_property_observer(self):
+        """
+        Ensure that bound method objects can be used as property observers.
+        See issue #26
+        """
+        handler = mock.Mock()
+        m = mpv.MPV()
+
+        class T(object):
+            def t(self, *args, **kw):
+                handler(*args, **kw)
+        t =  T()
+
+        m.loop = 'inf'
+
+        m.observe_property('loop', t.t)
+
+        m.loop = 'no'
+        self.assertEqual(m.loop, 'no')
+        m.loop = 'inf'
+        self.assertEqual(m.loop, 'inf')
+
+        time.sleep(0.02)
+        m.unobserve_property('loop', t.t)
+
+        m.loop = 'no'
+        m.loop = 'inf'
+        m.terminate() # needed for synchronization of event thread
+        handler.assert_has_calls([mock.call('loop', 'no'), mock.call('loop', 'inf')])
 
 
 if __name__ == '__main__':