allows dash (-) in interest name. makes tests more deterministic

Author: minaorangina

pusher_push_notifications/__init__.py

@@ -8,7 +8,7 @@
 
 SDK_VERSION = '0.9.2'
 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
 
 
@@ -135,7 +135,7 @@ def publish(self, interests, publish_body):
                         interest,
                     )
                     + 'Allowed characters are: ASCII upper/lower-case letters, '
-                    + 'numbers or one of _=@,.:'
+                    + 'numbers or one of _=@,.:-'
                 )
 
         publish_body['interests'] = interests

tests/test_push_notifications.py

@@ -192,7 +192,7 @@ def test_publish_should_succeed_if_100_interests_passed(self):
                 },
             )
             pn_client.publish(
-                interests=[str(el) for el in random.sample(range(1000), 100)],
+                interests=['interest-' + str(i) for i in range(0, 100)],
                 publish_body={
                     'apns': {
                         'aps': {
@@ -207,8 +207,6 @@ def test_publish_should_fail_if_too_many_interests_passed(self):
             'INSTANCE_ID',
             'SECRET_KEY'
         )
-        interests_ints = random.sample(range(1000), 101)
-
         with requests_mock.Mocker() as http_mock:
             http_mock.register_uri(
                 requests_mock.ANY,
@@ -220,7 +218,7 @@ def test_publish_should_fail_if_too_many_interests_passed(self):
             )
             with self.assertRaises(ValueError):
                 pn_client.publish(
-                    interests=[str(el) for el in interests_ints],
+                    interests=['interest-' + str(i) for i in range(0, 101)],
                     publish_body={
                         'apns': {
                             'aps': {