add test for regexp escape, escape the address passed ot the dispatcher first to not get confused if characters with special meanings were part of the osc address

Author: attwad

pythonosc/dispatcher.py

@@ -35,10 +35,11 @@ def handlers_for_address(self, address_pattern):
     # '?' in the OSC Address Pattern matches any single character.
     # Let's consider numbers and _ "characters" too here, it's not said
     # explicitly in the specification but it sounds good.
-    pattern = address_pattern.replace('?', '\\w?')
+    address_pattern = re.escape(address_pattern)
+    pattern = address_pattern.replace('\\?', '\\w?')
     # '*' in the OSC Address Pattern matches any sequence of zero or more
     # characters.
-    pattern = pattern.replace('*', '[\\w|\\\\+]*')
+    pattern = pattern.replace('\\*', '[\w|\+]*')
     # The rest of the syntax in the specification is like the re module so
     # we're fine.
     pattern = pattern + '$'

pythonosc/test/test_dispatcher.py

@@ -81,18 +81,13 @@ def test_match_address_contains_plus_as_character(self):
     self.sortAndAssertSequenceEqual(
         [(1, [])], self.dispatcher.handlers_for_address("/foo*/bar*/*"))
 
-  def aatest_meh(self):
-    ok = object()
-    not_ok = object()
-    self.dispatcher.map('/a+b', ok)
-    self.dispatcher.map('/aaab', not_ok)
+  def test_call_correct_dispatcher_on_star(self):
+    self.dispatcher.map('/a+b', 1)
+    self.dispatcher.map('/aaab', 2)
     self.sortAndAssertSequenceEqual(
-        [(1, not_ok)], self.dispatcher.handlers_for_address('/aaab'))
-    #self.sortAndAssertSequenceEqual(
-    #    [ok], self.dispatcher.handlers_for_address('/a+b'))
-
-    # in the client
-    client.send('/a+b') # should trigger only_correct_dispatcher only!
+        [(2, [])], self.dispatcher.handlers_for_address('/aaab'))
+    self.sortAndAssertSequenceEqual(
+        [(1, [])], self.dispatcher.handlers_for_address('/a+b'))
 
 if __name__ == "__main__":
   unittest.main()