Merge pull request #7 from pusher/semicolon

: -> ; in interests regex

Author: jonathanlloyd

pusher_push_notifications/__init__.py

@@ -8,7 +8,7 @@
 
 SDK_VERSION = '0.10.0'
 INTEREST_MAX_LENGTH = 164
-INTEREST_REGEX = re.compile('^(_|-|=|@|,|\\.|:|[A-Z]|[a-z]|[0-9])*$')
+INTEREST_REGEX = re.compile('^(_|-|=|@|,|\\.|;|[A-Z]|[a-z]|[0-9])*$')
 MAX_NUMBER_OF_INTERESTS = 100
 
 

tests/test_push_notifications.py

@@ -291,6 +291,19 @@ def test_publish_should_fail_if_interest_contains_invalid_chars(self):
             'INSTANCE_ID',
             'SECRET_KEY'
         )
+        with self.assertRaises(ValueError) as e:
+            pn_client.publish(
+                interests=['bad:interest'],
+                publish_body={
+                    'apns': {
+                        'aps': {
+                            'alert': 'Hello World!',
+                        },
+                    },
+                },
+            )
+        self.assertIn('"bad:interest" contains a forbidden character', str(e.exception))
+
         with self.assertRaises(ValueError) as e:
             pn_client.publish(
                 interests=['bad|interest'],