notifiable-mpns-nverinaud 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 1c1e7908ec3517ced66df745e0ed88695a05b5fb
-  data.tar.gz: 26950e645cac7a24d83d5b7b9f5838e824c71ad2
+  metadata.gz: 0603e386bffe7f2ec29874223e43307dc6f17380
+  data.tar.gz: a00a3a89015b8ce06a2d2d5d9bf904455390c996
 SHA512:
-  metadata.gz: 927a201220490f0ae3f3e29960c8300178fa3e23db91c596d1d829beeddfe7dc81341c1da5b18c88be8e8bf1b3b2ad677d4e347e41a555ef5e2d326616220315
-  data.tar.gz: 7e5d8c4caf8e4ca5b1ce475ddd9069499d14ed8d81bed007a8e2b861343ef755a63c2d038e9284ff3da7bbc6eff7378e09659cb88e64c8915b204c7693750667
+  metadata.gz: 5306484e69d8450bcbac18ad82afacb6174ed1277aa1db9b1eb93024681b139902ae0145991761650704cf6cb57e579e04f2336153179c8d19d2cbca1ee7d335
+  data.tar.gz: 47860a6cf6381af0ffa069f9f46f25b15c9ee5c7bfe04c205cc21fd94bf02fe1e538484c77f0cf304206f0d98ceba943246a13d2ff7255691a68feaa0e77c565

data/lib/notifiable/mpns/nverinaud/single_notifier.rb

@@ -12,16 +12,19 @@ module Notifiable
             data = {title: n.title, content: n.message, params: n.params}    
             
             response = MicrosoftPushNotificationService.send_notification device_token.token, :toast, data
+            response_code = response.code.to_i
             
-            case response.code.to_i
-            when 200              
-              processed(n, device_token)
+            case response_code
+            when 200
             when 404
               Rails.logger.info "De-registering device token: #{device_token.id}"
               device_token.update_attribute('is_valid', false)
             else
               Rails.logger.error "Error sending notification: #{response.code}"              
             end
+            
+            processed(n, device_token, response_code)
+            
       		end
   		end
     end

data/lib/notifiable/mpns/nverinaud/version.rb

@@ -1,7 +1,7 @@
 module Notifiable
   module Mpns
     module Nverinaud
-      VERSION = "0.3.0"
+      VERSION = "0.4.0"
     end
   end
 end

data/notifiable-mpns-nverinaud.gemspec

@@ -18,7 +18,7 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ["lib"]
 
-  spec.add_dependency "notifiable-rails", ">=0.5.0"
+  spec.add_dependency "notifiable-rails", ">=0.6.0"
   spec.add_dependency "ruby-mpns", "~> 1.2.1"
  
   spec.add_development_dependency "bundler", "~> 1.3"

data/spec/single_notifier_spec.rb

@@ -14,7 +14,9 @@ describe Notifiable::Mpns::Nverinaud::SingleNotifier do
     m.send_notification(n, d)
     m.close
     
-    Notifiable::NotificationDeviceToken.count.should == 1
+    Notifiable::NotificationStatus.count.should == 1
+    Notifiable::NotificationStatus.first.status.should == 200
+    
     a_request(:post, d.token)
       .with(:body => "<?xml version=\"1.0\" encoding=\"UTF-8\"?><wp:Notification xmlns:wp=\"WPNotification\"><wp:Toast><wp:Text1>A title</wp:Text1><wp:Text2></wp:Text2><wp:Param></wp:Param></wp:Toast></wp:Notification>")
       .should have_been_made.once
@@ -27,7 +29,9 @@ describe Notifiable::Mpns::Nverinaud::SingleNotifier do
     m.send_notification(n, d)
     m.close
     
-    Notifiable::NotificationDeviceToken.count.should == 1
+    Notifiable::NotificationStatus.count.should == 1
+    Notifiable::NotificationStatus.first.status.should == 200
+    
     a_request(:post, d.token)
       .with(:body => "<?xml version=\"1.0\" encoding=\"UTF-8\"?><wp:Notification xmlns:wp=\"WPNotification\"><wp:Toast><wp:Text1></wp:Text1><wp:Text2>A message</wp:Text2><wp:Param></wp:Param></wp:Toast></wp:Notification>")
       .should have_been_made.once
@@ -41,7 +45,7 @@ describe Notifiable::Mpns::Nverinaud::SingleNotifier do
     m.send_notification(n, d)
     m.close
     
-    Notifiable::NotificationDeviceToken.count.should == 1
+    Notifiable::NotificationStatus.count.should == 1
     a_request(:post, d.token)
       .with(:body => "<?xml version=\"1.0\" encoding=\"UTF-8\"?><wp:Notification xmlns:wp=\"WPNotification\"><wp:Toast><wp:Text1></wp:Text1><wp:Text2></wp:Text2><wp:Param>?an_object_id=123456</wp:Param></wp:Toast></wp:Notification>")
       .should have_been_made.once
@@ -55,7 +59,8 @@ describe Notifiable::Mpns::Nverinaud::SingleNotifier do
     m.send_notification(n, d)
     m.close
     
-    Notifiable::NotificationDeviceToken.count.should == 0
+    Notifiable::NotificationStatus.count.should == 1
+    Notifiable::NotificationStatus.first.status.should == 404
     Notifiable::DeviceToken.first.is_valid.should == false
   end
   

data/spec/support/db/migrate/20131229104039_create_notifiable_notification_statuses.rb

@@ -0,0 +1,11 @@
+class CreateNotifiableNotificationStatuses < ActiveRecord::Migration
+  
+  def change
+    create_table :notifiable_notification_statuses do |t|
+      t.references :notification
+      t.references :device_token
+      t.integer :status
+    end
+  end
+
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: notifiable-mpns-nverinaud
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Matt Brooke-Smith
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-02-11 00:00:00.000000000 Z
+date: 2014-02-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: notifiable-rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.6.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 0.5.0
+        version: 0.6.0
 - !ruby/object:Gem::Dependency
   name: ruby-mpns
   requirement: !ruby/object:Gem::Requirement
@@ -172,7 +172,7 @@ files:
 - spec/spec_helper.rb
 - spec/support/db/migrate/20131228225139_create_notifiable_device_tokens.rb
 - spec/support/db/migrate/20131228225140_create_notifiable_notifications.rb
-- spec/support/db/migrate/20131229104039_create_notifiable_notification_device_tokens.rb
+- spec/support/db/migrate/20131229104039_create_notifiable_notification_statuses.rb
 homepage: http://www.futureworkshops.com
 licenses:
 - Apache 2.0
@@ -202,4 +202,4 @@ test_files:
 - spec/spec_helper.rb
 - spec/support/db/migrate/20131228225139_create_notifiable_device_tokens.rb
 - spec/support/db/migrate/20131228225140_create_notifiable_notifications.rb
-- spec/support/db/migrate/20131229104039_create_notifiable_notification_device_tokens.rb
+- spec/support/db/migrate/20131229104039_create_notifiable_notification_statuses.rb

data/spec/support/db/migrate/20131229104039_create_notifiable_notification_device_tokens.rb

@@ -1,11 +0,0 @@
-class CreateNotifiableNotificationDeviceTokens < ActiveRecord::Migration
-  
-  def change
-    create_table :notifiable_notification_device_tokens do |t|
-      t.references :notification
-      t.references :device_token
-      t.timestamps
-    end
-  end
-
-end