Merge pull request #151 from pusher/remove-push-notif

remove native push notifications

Author: Jon Elverkilde

lib/pusher.rb

@@ -66,4 +66,3 @@ def default_client
 require 'pusher/request'
 require 'pusher/resource'
 require 'pusher/webhook'
-require 'pusher/native_notification/client'

lib/pusher/client.rb

@@ -330,24 +330,6 @@ def trigger_batch_async(*events)
       post_async('/batch_events', trigger_batch_params(events.flatten))
     end
 
-    def notification_client
-      @notification_client ||=
-        NativeNotification::Client.new(@app_id, @notification_host, @notification_scheme, self)
-    end
-
-
-    # Send a push notification
-    #
-    # POST /apps/[app_id]/notifications
-    #
-    # @param interests [Array] An array of interests
-    # @param message [String] Message to send
-    # @param options [Hash] Additional platform specific options
-    #
-    # @return [Hash]
-    def notify(interests, data = {})
-      notification_client.notify(interests, data)
-    end
 
     # Generate the expected response for an authentication endpoint.
     # See http://pusher.com/docs/authenticating_users for details.

lib/pusher/native_notification/client.rb

@@ -697,70 +697,6 @@
         end
       end
     end
-
-    describe "native notifications" do
-      before :each do
-        @client.app_id = "20"
-        @client.key = "testytest"
-        @client.secret = "mysupersecretkey"
-      end
-
-      it "should configure a native notification client using the pusher client object" do
-        expect(@client.notification_client).to_not be(nil)
-      end
-
-      it "should use the default host if not provided" do
-        expect(@client.notification_host).to eq("nativepush-cluster1.pusher.com")
-      end
-
-      it "should use a newly provided host" do
-        @client.notification_host = "test.com"
-        expect(@client.notification_host).to eq("test.com")
-      end
-
-      it "should set the native notification client host to the same one" do
-        expect(@client.notification_host).to eq(@client.notification_client.host)
-      end
-
-      it "should raise an error if no interest is provided" do
-        payload = {
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon",
-            }
-          }
-        }
-
-        expect { @client.notify([], payload) }.to raise_error(Pusher::Error)
-      end
-
-      it "should send a request to the notifications endpoint" do
-        notification_host_regexp = %r{nativepush-cluster1.pusher.com}
-        payload = {
-          interests: ["test"],
-          gcm: {
-            notification: {
-              title: "Hello",
-              icon: "icon",
-            }
-          }
-        }
-
-        stub_request(
-          :post,
-          notification_host_regexp,
-        ).with(
-          body: MultiJson.encode(payload)
-        ).to_return({
-          :status => 200,
-          :body => MultiJson.encode({ :foo => "bar" })
-        })
-
-        res = @client.notify(["test"], payload)
-        expect(res).to eq({foo: "bar"})
-      end
-    end
   end
 
   describe 'configuring cluster' do