remove payload alidation for the client lib, this is done by the server

also remove upper bound check on the number of interests

Author: topliceanu

README.md

@@ -262,7 +262,7 @@ The native notifications API is hosted at `nativepush-cluster1.pusher.com` and o
 
 You can send pushes by using the `notify` method, either globally or on the instance. The method takes two parameters:
 
-- `interests`: An Array of strings which represents the interests your devices are subscribed to. These are akin to channels in the DDN with less of an epehemeral nature. Note that currently, you can only send to _one_ interest.
+- `interests`: An Array of strings which represents the interests your devices are subscribed to. These are akin to channels in the DDN with less of an epehemeral nature. Note that currently, you can only publish to, at most, _ten_ interests.
 - `data`: The content of the notification represented by a Hash. You must supply either the `gcm` or `apns` key. For a detailed list of the acceptable keys, take a look at the [iOS](https://pusher.com/docs/push_notifications/ios/server) and [Android](https://pusher.com/docs/push_notifications/android/server) docs.
 
 Example:

lib/pusher/native_notification/client.rb

@@ -42,8 +42,13 @@ def notify(interests, data = {})
       # @return [String]
       def payload(interests, data)
         interests = Array(interests).map(&:to_s)
-        raise Pusher::Error, "Too many interests provided" if interests.length > 1
-        data = deep_symbolize_keys!(data).merge(interests: interests)
+
+        raise Pusher::Error, "Interests array must not be empty" if interests.length == 0
+
+        data = deep_symbolize_keys!(data)
+
+        data.merge!(interests: interests)
+
         MultiJson.encode(data)
       end
 

spec/client_spec.rb

@@ -550,7 +550,7 @@
         expect(@client.notification_host).to eq(@client.notification_client.host)
       end
 
-      it "should raise an error if more than one interest is provided" do
+      it "should raise an error if no interest is provided" do
         payload = {
           gcm: {
             notification: {
@@ -560,7 +560,7 @@
           }
         }
 
-        expect { @client.notify(["test1", "test2"], payload) }.to raise_error(Pusher::Error)
+        expect { @client.notify([], payload) }.to raise_error(Pusher::Error)
       end
 
       it "should send a request to the notifications endpoint" do