Merge pull request #107 from laitlhadj/master

Add include_external_user_ids notification parameter

Author: norkunas

src/Resolver/NotificationResolver.php

@@ -60,6 +60,8 @@ public function resolve(array $data)
             ->setAllowedTypes('include_ios_tokens', 'array')
             ->setDefined('include_android_reg_ids')
             ->setAllowedTypes('include_android_reg_ids', 'array')
+            ->setDefined('include_external_user_ids')
+            ->setAllowedTypes('include_external_user_ids', 'array')
             ->setDefined('include_wp_uris')
             ->setAllowedTypes('include_wp_uris', 'array')
             ->setDefined('include_wp_wns_uris')

tests/OneSignal/Tests/Resolver/NotificationResolverTest.php

@@ -44,6 +44,7 @@ public function testResolveWithValidValues()
             'include_player_ids' => ['value'],
             'include_ios_tokens' => ['value'],
             'include_android_reg_ids' => ['value'],
+            'include_external_user_ids' => ['value'],
             'include_wp_uris' => ['value'],
             'include_wp_wns_uris' => ['value'],
             'include_amazon_reg_ids' => ['value'],
@@ -139,6 +140,7 @@ public function wrongValueTypesProvider()
             [['include_player_ids' => 'wrongType']],
             [['include_ios_tokens' => 'wrongType']],
             [['include_android_reg_ids' => 666]],
+            [['include_external_user_ids' => 666]],
             [['include_wp_uris' => 666]],
             [['include_wp_wns_uris' => 666]],
             [['include_amazon_reg_ids' => 666]],