Fix the match rule semantics so that a match rule of "arg0='/'" does not match

object paths in Python3, as per Simon's review comments.

1 files changed, 19 insertions, 0 deletions

diff --git a/test/test-standalone.py b/test/test-standalone.py
index 48cbe09..22e8444 100755
--- a/test/test-standalone.py
+++ b/test/test-standalone.py
@@ -344,6 +344,25 @@ class TestMessageMarshalling(unittest.TestCase):
                                  'should fail')
 
 
+class TestMatching(unittest.TestCase):
+    def setUp(self):
+        from _dbus_bindings import SignalMessage
+        from dbus.connection import SignalMatch
+        self._message = SignalMessage('/', 'a.b', 'c')
+        class FakeConn(object): pass
+        def ignore_cb(*args, **kws): pass
+        self._match = SignalMatch(FakeConn(), None, '/', None, None, 
+                                  ignore_cb, arg0='/')
+
+    def test_string_match(self):
+        self._message.append('/', signature='s')
+        self.assertTrue(self._match.maybe_handle_message(self._message))
+
+    def test_object_path_no_match(self):
+        self._message.append('/', signature='o')
+        self.assertFalse(self._match.maybe_handle_message(self._message))
+
+
 if __name__ == '__main__':
     # Python 2.6 doesn't accept a `verbosity` keyword.
     kwargs = {}