apn_on_rails 0.4.0 → 0.4.1

data/.gitignore

@@ -13,4 +13,5 @@ build/*
 .svn
 profile
 spec/active_record/test.db
-.bundle
+.bundle
+**/.svn

data/Gemfile

@@ -8,8 +8,9 @@ gem 'configatron'
 # Add dependencies to develop your gem here.
 # Include everything needed to run rake, tests, features, etc.
 group :development do
+  gem 'autotest'
   gem 'sqlite3-ruby'
-  gem "rspec", ">= 2.0.0.beta.19"
+  gem "rspec", ">= 2.0.0"
   gem "bundler", ">= 1.0.0.rc.5"
   gem "jeweler", "~> 1.5.0.pre2"
   gem "rcov", ">= 0"

data/Gemfile.lock

@@ -7,6 +7,7 @@ GEM
     activerecord (2.3.9)
       activesupport (= 2.3.9)
     activesupport (2.3.9)
+    autotest (4.3.2)
     configatron (2.6.4)
       yamler (>= 0.1.0)
     diff-lcs (1.1.2)
@@ -18,16 +19,16 @@ GEM
     rack (1.1.0)
     rake (0.8.7)
     rcov (0.9.9)
-    rspec (2.0.0.rc)
-      rspec-core (= 2.0.0.rc)
-      rspec-expectations (= 2.0.0.rc)
-      rspec-mocks (= 2.0.0.rc)
-    rspec-core (2.0.0.rc)
-    rspec-expectations (2.0.0.rc)
+    rspec (2.0.1)
+      rspec-core (~> 2.0.1)
+      rspec-expectations (~> 2.0.1)
+      rspec-mocks (~> 2.0.1)
+    rspec-core (2.0.1)
+    rspec-expectations (2.0.1)
       diff-lcs (>= 1.1.2)
-    rspec-mocks (2.0.0.rc)
-      rspec-core (= 2.0.0.rc)
-      rspec-expectations (= 2.0.0.rc)
+    rspec-mocks (2.0.1)
+      rspec-core (~> 2.0.1)
+      rspec-expectations (~> 2.0.1)
     sqlite3-ruby (1.3.1)
     yamler (0.1.0)
 
@@ -37,9 +38,10 @@ PLATFORMS
 DEPENDENCIES
   actionpack (~> 2.3.8)
   activerecord (~> 2.3.8)
+  autotest
   bundler (>= 1.0.0.rc.5)
   configatron
   jeweler (~> 1.5.0.pre2)
   rcov
-  rspec (>= 2.0.0.beta.19)
+  rspec (>= 2.0.0)
   sqlite3-ruby

data/README

@@ -4,7 +4,7 @@ APN on Rails is a Ruby on Rails gem that allows you to easily add Apple Push Not
 support to your Rails application.  
 
 It supports: 
-* Multiple iPhone apps managed from the same Rails application
+* Multiple iPhone apps managed from the same Rails application as well as a legacy default "app" with certs stored in config
 * Individual notifications and group notifications
 * Alerts, badges, sounds, and custom properties in notifications
 * Pull notifications
@@ -14,13 +14,16 @@ It supports:
 Multiple iPhone Apps: In previous versions of this gem a single Rails application was set up to 
 manage push notifications for a single iPhone app.  In many cases it is useful to have a single Rails
 app manage push notifications for multiple iPhone apps.  With the addition of an APN::App model, this 
-is now possible.  The certificates are now stored on instances of APN::APP and all devices are associated
-with a particular app.  
+is now possible.  The certificates are now stored on instances of APN::App and devices are intended to be associated
+with a particular app.  For compatibility with existing implementations it is still possible to create devices that 
+are not associated with an APN::App and to send individual notifications to them using the certs stored in the 
+config directory.  
 
 Individual and Group Notifications: Previous versions of this gem treated each notification individually
 and did not provide a built-in way to send a broadcast notification to a group of devices.  Group notifications
 are now built into the gem.  A group notification is associated with a group of devices and shares its 
-contents across the entire group of devices.  
+contents across the entire group of devices.  (Group notifications are only available for groups of devices associated 
+with an APN::App)
 
 Notification Content Areas: Notifications may contain alerts, badges, sounds, and custom properties.
 
@@ -28,7 +31,14 @@ Pull Notifications: This version of the gem supports an alternative notification
 on pulls from client devices and does not interact with the Apple Push Notification servers.  This feature
 may be used entirely independently of the push notification features.  Pull notifications may be
 created for an app.  A client app can query for the most recent pull notification available since a 
-given date to retrieve any notifications waiting for it.  
+given date to retrieve any notifications waiting for it. 
+
+==Version 0.4.1 Notes
+
+* Backwards compatibility.  0.4.0 required a manual upgrade to associate existing and new devices with an APN::App model.  This version allows continued use of devices that are associated with a default "app" that stores its certificates in the config directory.  This ought to allow upgrade to this version without code changes.  
+* Batched finds.  Finds on the APN::Device model that can return large numbers of records have been batched to limit memory impact. 
+* Custom properties migration.  At a pre-0.4.0 version the custom_properties attribute was added to the migration template that created the notifications table.  This introduced a potential problem for gem users who had previously run this migration.  The custom_properties alteration to the apn_notifications table has been moved to its own migration and should work regardless of whether your apn_notifications table already has a custom_properties attribute. 
+* last_registered_at changed to work intuitively.  The last_registered_at attribute of devices was being updated only on creation potentially causing a bug in which a device that opts out of APNs and then opts back in before apn_on_rails received feedback about it might miss a period of APNs that it should receive.  
 
 ==Acknowledgements:
 
@@ -105,9 +115,9 @@ Now, to create the tables you need for APN on Rails, run the following task:
 
 APN on Rails uses the Configatron gem, http://github.com/markbates/configatron/tree/master, 
 to configure itself. (With the change to multi-app support, the certifications are stored in the 
-database rather than in the config directory.  These configurations remain for now.) 
-APN on Rails has the following default configurations that you change as you
-see fit:
+database rather than in the config directory, however, it is still possible to use the default "app" and the certificates
+stored in the config directory.  For this setup, the following configurations apply.) 
+APN on Rails has the following default configurations that you change as you see fit:
 
   # development (delivery):
   configatron.apn.passphrase # => ''
@@ -142,7 +152,7 @@ That way you ensure you have the latest version of the database tables needed.
 ==Example (assuming you have created an app and stored your keys on it):
 
   $ ./script/console
-  >> app = APN::App.find(:first)
+  >> app = APN::App.create(:name => "My App", :apn_dev_cert => "PASTE YOUR DEV CERT HERE", :apn_prod_cert => "PASTE YOUR PROD CERT HERE")
   >> device = APN::Device.create(:token => "XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX",:app_id => app.id)
   >> notification = APN::Notification.new
   >> notification.device = device

data/README.textile

@@ -4,7 +4,7 @@ APN on Rails is a Ruby on Rails gem that allows you to easily add Apple Push Not
 support to your Rails application.  
 
 It supports: 
-* Multiple iPhone apps managed from the same Rails application
+* Multiple iPhone apps managed from the same Rails application as well as a legacy default "app" with certs stored in config
 * Individual notifications and group notifications
 * Alerts, badges, sounds, and custom properties in notifications
 * Pull notifications
@@ -14,13 +14,16 @@ h2. Feature Descriptions
 Multiple iPhone Apps: In previous versions of this gem a single Rails application was set up to 
 manage push notifications for a single iPhone app.  In many cases it is useful to have a single Rails
 app manage push notifications for multiple iPhone apps.  With the addition of an APN::App model, this 
-is now possible.  The certificates are now stored on instances of APN::APP and all devices are associated
-with a particular app.  
+is now possible.  The certificates are now stored on instances of APN::App and all devices are intended to be associated
+with a particular app.  For compatibility with existing implementations it is still possible to create devices that 
+are not associated with an APN::App and to send individual notifications to them using the certs stored in the 
+config directory.
 
 Individual and Group Notifications: Previous versions of this gem treated each notification individually
 and did not provide a built-in way to send a broadcast notification to a group of devices.  Group notifications
 are now built into the gem.  A group notification is associated with a group of devices and shares its 
-contents across the entire group of devices.  
+contents across the entire group of devices. (Group notifications are only available for groups of devices associated 
+with an APN::App) 
 
 Notification Content Areas: Notifications may contain alerts, badges, sounds, and custom properties.
 
@@ -30,6 +33,13 @@ may be used entirely independently of the push notification features.  Pull noti
 created for an app.  A client app can query for the most recent pull notification available since a 
 given date to retrieve any notifications waiting for it.  
 
+h2. Version 0.4.1 Notes
+
+* Backwards compatibility.  0.4.0 required a manual upgrade to associate existing and new devices with an APN::App model.  This version allows continued use of devices that are associated with a default "app" that stores its certificates in the config directory.  This ought to allow upgrade to this version without code changes.  
+* Batched finds.  Finds on the APN::Device model that can return large numbers of records have been batched to limit memory impact. 
+* Custom properties migration.  At a pre-0.4.0 version the custom_properties attribute was added to the migration template that created the notifications table.  This introduced a potential problem for gem users who had previously run this migration.  The custom_properties alteration to the apn_notifications table has been moved to its own migration and should work regardless of whether your apn_notifications table already has a custom_properties attribute. 
+* last_registered_at changed to work intuitively.  The last_registered_at attribute of devices was being updated only on creation potentially causing a bug in which a device that opts out of APNs and then opts back in before apn_on_rails received feedback about it might miss a period of APNs that it should receive.
+
 h2. Acknowledgements:
 
 From Mark Bates: 
@@ -121,7 +131,8 @@ Now, to create the tables you need for APN on Rails, run the following task:
 
 APN on Rails uses the Configatron gem, http://github.com/markbates/configatron/tree/master, 
 to configure itself. (With the change to multi-app support, the certifications are stored in the 
-database rather than in the config directory.  These configurations remain for now.) 
+database rather than in the config directory, however, it is still possible to use the default "app" and the certificates
+stored in the config directory.  For this setup, the following configurations apply.)
 APN on Rails has the following default configurations that you change as you
 see fit:
 
@@ -166,7 +177,7 @@ h2. Example (assuming you have created an app and stored your keys on it):
 
 <pre><code>
   $ ./script/console
-  >> app = APN::App.find(:first)
+  >> app = APN::App.create(:name => "My App", :apn_dev_cert => "PASTE YOUR DEV CERT HERE", :apn_prod_cert => "PASTE YOUR PROD CERT HERE")
   >> device = APN::Device.create(:token => "XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX XXXXXXXX",:app_id => app.id)
   >> notification = APN::Notification.new
   >> notification.device = device

data/VERSION

@@ -1 +1 @@
-0.4.0
+0.4.1

data/apn_on_rails.gemspec

@@ -5,11 +5,11 @@
 
 Gem::Specification.new do |s|
   s.name = %q{apn_on_rails}
-  s.version = "0.3.1"
+  s.version = "0.4.1"
 
   s.required_rubygems_version = Gem::Requirement.new(">= 0") if s.respond_to? :required_rubygems_version=
   s.authors = ["markbates", "Rebecca Nesson"]
-  s.date = %q{2010-10-06}
+  s.date = %q{2010-11-09}
   s.description = %q{APN on Rails is a Ruby on Rails gem that allows you to
 easily add Apple Push Notification (iPhone) support to your Rails application.
 }
@@ -22,6 +22,7 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
   s.files = [
     ".gitignore",
     ".rspec",
+    ".specification",
     "Gemfile",
     "Gemfile.lock",
     "LICENSE",
@@ -30,28 +31,50 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
     "Rakefile",
     "VERSION",
     "apn_on_rails.gemspec",
+    "autotest/discover.rb",
     "generators/apn_migrations_generator.rb",
     "generators/templates/apn_migrations/001_create_apn_devices.rb",
     "generators/templates/apn_migrations/002_create_apn_notifications.rb",
     "generators/templates/apn_migrations/003_alter_apn_devices.rb",
+    "generators/templates/apn_migrations/004_create_apn_apps.rb",
+    "generators/templates/apn_migrations/005_create_groups.rb",
+    "generators/templates/apn_migrations/006_alter_apn_groups.rb",
+    "generators/templates/apn_migrations/007_create_device_groups.rb",
+    "generators/templates/apn_migrations/008_create_apn_group_notifications.rb",
+    "generators/templates/apn_migrations/009_create_pull_notifications.rb",
+    "generators/templates/apn_migrations/010_alter_apn_notifications.rb",
+    "generators/templates/apn_migrations/011_make_device_token_index_nonunique.rb",
     "lib/apn_on_rails.rb",
     "lib/apn_on_rails/apn_on_rails.rb",
+    "lib/apn_on_rails/app/models/apn/app.rb",
     "lib/apn_on_rails/app/models/apn/base.rb",
     "lib/apn_on_rails/app/models/apn/device.rb",
+    "lib/apn_on_rails/app/models/apn/device_grouping.rb",
+    "lib/apn_on_rails/app/models/apn/group.rb",
+    "lib/apn_on_rails/app/models/apn/group_notification.rb",
     "lib/apn_on_rails/app/models/apn/notification.rb",
+    "lib/apn_on_rails/app/models/apn/pull_notification.rb",
     "lib/apn_on_rails/libs/connection.rb",
     "lib/apn_on_rails/libs/feedback.rb",
     "lib/apn_on_rails/tasks/apn.rake",
     "lib/apn_on_rails/tasks/db.rake",
     "lib/apn_on_rails_tasks.rb",
     "spec/active_record/setup_ar.rb",
+    "spec/apn_on_rails/app/models/apn/app_spec.rb",
     "spec/apn_on_rails/app/models/apn/device_spec.rb",
+    "spec/apn_on_rails/app/models/apn/group_notification_spec.rb",
     "spec/apn_on_rails/app/models/apn/notification_spec.rb",
+    "spec/apn_on_rails/app/models/apn/pull_notification_spec.rb",
     "spec/apn_on_rails/libs/connection_spec.rb",
     "spec/apn_on_rails/libs/feedback_spec.rb",
     "spec/extensions/string.rb",
+    "spec/factories/app_factory.rb",
     "spec/factories/device_factory.rb",
+    "spec/factories/device_grouping_factory.rb",
+    "spec/factories/group_factory.rb",
+    "spec/factories/group_notification_factory.rb",
     "spec/factories/notification_factory.rb",
+    "spec/factories/pull_notification_factory.rb",
     "spec/fixtures/hexa.bin",
     "spec/fixtures/message_for_sending.bin",
     "spec/rails_root/config/apple_push_notification_development.pem",
@@ -63,13 +86,21 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
   s.summary = %q{Apple Push Notifications on Rails}
   s.test_files = [
     "spec/active_record/setup_ar.rb",
+    "spec/apn_on_rails/app/models/apn/app_spec.rb",
     "spec/apn_on_rails/app/models/apn/device_spec.rb",
+    "spec/apn_on_rails/app/models/apn/group_notification_spec.rb",
     "spec/apn_on_rails/app/models/apn/notification_spec.rb",
+    "spec/apn_on_rails/app/models/apn/pull_notification_spec.rb",
     "spec/apn_on_rails/libs/connection_spec.rb",
     "spec/apn_on_rails/libs/feedback_spec.rb",
     "spec/extensions/string.rb",
+    "spec/factories/app_factory.rb",
     "spec/factories/device_factory.rb",
+    "spec/factories/device_grouping_factory.rb",
+    "spec/factories/group_factory.rb",
+    "spec/factories/group_notification_factory.rb",
     "spec/factories/notification_factory.rb",
+    "spec/factories/pull_notification_factory.rb",
     "spec/spec_helper.rb"
   ]
 
@@ -79,8 +110,9 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
 
     if Gem::Version.new(Gem::VERSION) >= Gem::Version.new('1.2.0') then
       s.add_runtime_dependency(%q<configatron>, [">= 0"])
+      s.add_development_dependency(%q<autotest>, [">= 0"])
       s.add_development_dependency(%q<sqlite3-ruby>, [">= 0"])
-      s.add_development_dependency(%q<rspec>, [">= 2.0.0.beta.19"])
+      s.add_development_dependency(%q<rspec>, [">= 2.0.0"])
       s.add_development_dependency(%q<bundler>, [">= 1.0.0.rc.5"])
       s.add_development_dependency(%q<jeweler>, ["~> 1.5.0.pre2"])
       s.add_development_dependency(%q<rcov>, [">= 0"])
@@ -88,8 +120,9 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
       s.add_development_dependency(%q<activerecord>, ["~> 2.3.8"])
     else
       s.add_dependency(%q<configatron>, [">= 0"])
+      s.add_dependency(%q<autotest>, [">= 0"])
       s.add_dependency(%q<sqlite3-ruby>, [">= 0"])
-      s.add_dependency(%q<rspec>, [">= 2.0.0.beta.19"])
+      s.add_dependency(%q<rspec>, [">= 2.0.0"])
       s.add_dependency(%q<bundler>, [">= 1.0.0.rc.5"])
       s.add_dependency(%q<jeweler>, ["~> 1.5.0.pre2"])
       s.add_dependency(%q<rcov>, [">= 0"])
@@ -98,8 +131,9 @@ easily add Apple Push Notification (iPhone) support to your Rails application.
     end
   else
     s.add_dependency(%q<configatron>, [">= 0"])
+    s.add_dependency(%q<autotest>, [">= 0"])
     s.add_dependency(%q<sqlite3-ruby>, [">= 0"])
-    s.add_dependency(%q<rspec>, [">= 2.0.0.beta.19"])
+    s.add_dependency(%q<rspec>, [">= 2.0.0"])
     s.add_dependency(%q<bundler>, [">= 1.0.0.rc.5"])
     s.add_dependency(%q<jeweler>, ["~> 1.5.0.pre2"])
     s.add_dependency(%q<rcov>, [">= 0"])

data/autotest/discover.rb

@@ -0,0 +1 @@
+Autotest.add_discovery { "rspec2" }

data/generators/templates/apn_migrations/002_create_apn_notifications.rb

@@ -9,7 +9,6 @@ class CreateApnNotifications < ActiveRecord::Migration # :nodoc:
       t.string :sound
       t.string :alert, :size => 150
       t.integer :badge
-      t.text :custom_properties
       t.datetime :sent_at
       t.timestamps
     end

data/generators/templates/apn_migrations/004_create_apn_apps.rb

@@ -12,7 +12,7 @@ class CreateApnApps < ActiveRecord::Migration # :nodoc:
   end
 
   def self.down
-    drop_table :apps
+    drop_table :apn_apps
     remove_column :apn_devices, :app_id
   end
-end
+end

data/generators/templates/apn_migrations/010_alter_apn_notifications.rb

@@ -0,0 +1,21 @@
+class AlterApnNotifications < ActiveRecord::Migration # :nodoc:
+  
+  module APN # :nodoc:
+    class Notification < ActiveRecord::Base # :nodoc:
+      set_table_name 'apn_notifications'
+    end
+  end
+  
+  def self.up
+    unless APN::Notification.column_names.include?("custom_properties")
+      add_column :apn_notifications, :custom_properties, :text
+    end
+  end
+
+  def self.down
+    if APN::Notification.column_names.include?("custom_properties")
+      remove_column :apn_notifications, :custom_properties
+    end
+  end
+  
+end

data/generators/templates/apn_migrations/011_make_device_token_index_nonunique.rb

@@ -0,0 +1,11 @@
+class MakeDeviceTokenIndexNonunique < ActiveRecord::Migration
+  def self.up
+    remove_index :apn_devices, :column => :token
+    add_index :apn_devices, :token
+  end
+
+  def self.down
+    remove_index :apn_devices, :column => :token
+    add_index :apn_devices, :token, :unique => true
+  end
+end

data/lib/apn_on_rails/app/models/apn/app.rb

@@ -23,15 +23,7 @@ class APN::App < APN::Base
       raise APN::Errors::MissingCertificateError.new
       return
     end
-    unless self.unsent_notifications.nil? || self.unsent_notifications.empty?
-      APN::Connection.open_for_delivery({:cert => self.cert}) do |conn, sock|
-        unsent_notifications.each do |noty|
-          conn.write(noty.message_for_sending)
-          noty.sent_at = Time.now
-          noty.save
-        end
-      end
-    end
+    APN::App.send_notifications_for_cert(self.cert, self.id)
   end
   
   def self.send_notifications
@@ -39,6 +31,32 @@ class APN::App < APN::Base
     apps.each do |app|
       app.send_notifications
     end
+    global_cert = File.read(configatron.apn.cert)
+    if global_cert
+      send_notifications_for_cert(global_cert, nil)
+    end
+  end
+  
+  def self.send_notifications_for_cert(the_cert, app_id)
+    # unless self.unsent_notifications.nil? || self.unsent_notifications.empty?
+      if (app_id == nil)
+        conditions = "app_id is null"
+      else 
+        conditions = ["app_id = ?", app_id]
+      end
+      begin
+        APN::Connection.open_for_delivery({:cert => the_cert}) do |conn, sock|
+          APN::Device.find_each(:conditions => conditions) do |dev|
+            dev.unsent_notifications.each do |noty|
+              conn.write(noty.message_for_sending)
+              noty.sent_at = Time.now
+              noty.save
+            end
+          end
+        end
+      rescue
+      end
+    # end   
   end
   
   def send_group_notifications
@@ -49,8 +67,7 @@ class APN::App < APN::Base
     unless self.unsent_group_notifications.nil? || self.unsent_group_notifications.empty? 
       APN::Connection.open_for_delivery({:cert => self.cert}) do |conn, sock|
         unsent_group_notifications.each do |gnoty|
-          puts "number of devices is #{gnoty.devices.size}"
-          gnoty.devices.each do |device|
+          gnoty.devices.find_each do |device|
             conn.write(gnoty.message_for_sending(device))
           end
           gnoty.sent_at = Time.now
@@ -67,7 +84,7 @@ class APN::App < APN::Base
     end
     unless gnoty.nil?
       APN::Connection.open_for_delivery({:cert => self.cert}) do |conn, sock|
-        gnoty.devices.each do |device|
+        gnoty.devices.find_each do |device|
           conn.write(gnoty.message_for_sending(device))
         end
         gnoty.sent_at = Time.now
@@ -98,11 +115,7 @@ class APN::App < APN::Base
       raise APN::Errors::MissingCertificateError.new
       return
     end
-    APN::Feedback.devices(self.cert).each do |device|
-      if device.last_registered_at < device.feedback_at
-        device.destroy
-      end
-    end
+    APN::App.process_devices_for_cert(self.cert)
   end # process_devices
   
   def self.process_devices
@@ -110,6 +123,22 @@ class APN::App < APN::Base
     apps.each do |app|
       app.process_devices
     end
+    global_cert = File.read(configatron.apn.cert)
+    if global_cert
+      APN::App.process_devices_for_cert(global_cert)
+    end
+  end
+  
+  def self.process_devices_for_cert(the_cert)
+    puts "in APN::App.process_devices_for_cert"
+    APN::Feedback.devices(the_cert).each do |device|
+      if device.last_registered_at < device.feedback_at
+        puts "device #{device.id} -> #{device.last_registered_at} < #{device.feedback_at}"
+        device.destroy
+      else 
+        puts "device #{device.id} -> #{device.last_registered_at} not < #{device.feedback_at}"
+      end
+    end 
   end
     
 end

data/lib/apn_on_rails/app/models/apn/device.rb

@@ -17,7 +17,7 @@ class APN::Device < APN::Base
   validates_uniqueness_of :token, :scope => :app_id
   validates_format_of :token, :with => /^[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}\s[a-z0-9]{8}$/
   
-  before_save :set_last_registered_at
+  before_create :set_last_registered_at
   
   # The <tt>feedback_at</tt> accessor is set when the 
   # device is marked as potentially disconnected from your
@@ -42,9 +42,8 @@ class APN::Device < APN::Base
     [self.token.delete(' ')].pack('H*')
   end
   
-  private
   def set_last_registered_at
-    self.last_registered_at = Time.now if self.last_registered_at.nil?
+    self.last_registered_at = Time.now #if self.last_registered_at.nil?
   end
   
 end

data/spec/apn_on_rails/app/models/apn/app_spec.rb

@@ -10,27 +10,59 @@ describe APN::App do
       device = DeviceFactory.create({:app_id => app.id})
       notifications = [NotificationFactory.create({:device_id => device.id}), 
                        NotificationFactory.create({:device_id => device.id})]
-      notifications.each_with_index do |notify, i|
-        notify.stub(:message_for_sending).and_return("message-#{i}")
-        notify.should_receive(:sent_at=).with(instance_of(Time))
-        notify.should_receive(:save)
-      end
-    
-      APN::App.should_receive(:all).and_return([app])
-      app.should_receive(:unsent_notifications).at_least(:once).and_return(notifications)
+                       
+     notifications.each_with_index do |notify, i|
+       notify.stub(:message_for_sending).and_return("message-#{i}")
+       notify.should_receive(:sent_at=).with(instance_of(Time))
+       notify.should_receive(:save)
+     end
+      
+      APN::App.should_receive(:all).once.and_return([app])                 
       app.should_receive(:cert).twice.and_return(app.apn_dev_cert)
       
+      APN::Device.should_receive(:find_each).twice.and_yield(device)
+      
+      device.should_receive(:unsent_notifications).and_return(notifications,[])
+      
+      
       ssl_mock = mock('ssl_mock')
       ssl_mock.should_receive(:write).with('message-0')
       ssl_mock.should_receive(:write).with('message-1')
-      APN::Connection.should_receive(:open_for_delivery).and_yield(ssl_mock, nil)
-    
+      APN::Connection.should_receive(:open_for_delivery).twice.and_yield(ssl_mock, nil)
       APN::App.send_notifications
     
     end
   
   end
   
+  describe 'send_notifications_not_associated_with_an_app' do 
+    
+    it 'should send unsent notifications that are associated with devices that are not with any app' do 
+      RAILS_ENV = 'staging'
+      device = DeviceFactory.create
+      device.app_id = nil
+      device.save
+      APN::App.all.each { |a| a.destroy }
+      notifications = [NotificationFactory.create({:device_id => device.id}), 
+                       NotificationFactory.create({:device_id => device.id})]
+                   
+       notifications.each_with_index do |notify, i|
+         notify.stub(:message_for_sending).and_return("message-#{i}")
+         notify.should_receive(:sent_at=).with(instance_of(Time))
+         notify.should_receive(:save)
+       end  
+   
+       APN::Device.should_receive(:find_each).and_yield(device)
+       device.should_receive(:unsent_notifications).and_return(notifications)
+  
+       ssl_mock = mock('ssl_mock')
+       ssl_mock.should_receive(:write).with('message-0')
+       ssl_mock.should_receive(:write).with('message-1')
+       APN::Connection.should_receive(:open_for_delivery).and_yield(ssl_mock, nil)
+       APN::App.send_notifications
+    end
+  end               
+  
   describe 'send_group_notifications' do
   
     it 'should send the unsent group notifications' do
@@ -133,7 +165,12 @@ describe APN::App do
       app = AppFactory.create
       devices = [DeviceFactory.create(:app_id => app.id, :last_registered_at => 1.week.ago, :feedback_at => Time.now),
                  DeviceFactory.create(:app_id => app.id, :last_registered_at => 1.week.from_now, :feedback_at => Time.now)]
-      APN::Feedback.should_receive(:devices).and_return(devices)
+      puts "device ids are #{devices[0].id} and #{devices[1].id}"
+      devices[0].last_registered_at = 1.week.ago
+      devices[0].save
+      devices[1].last_registered_at = 1.week.from_now
+      devices[1].save
+      APN::Feedback.should_receive(:devices).twice.and_return(devices)
       APN::App.should_receive(:all).and_return([app])
       app.should_receive(:cert).twice.and_return(app.apn_dev_cert)
       lambda {
@@ -143,6 +180,21 @@ describe APN::App do
     
   end
   
+  describe 'process_devices for global app' do 
+    
+    it 'should destroy devices that have a last_registered_at date that is before the feedback_at date that have no app' do 
+      device = DeviceFactory.create(:app_id => nil, :last_registered_at => 1.week.ago, :feedback_at => Time.now)
+      device.app_id = nil
+      device.last_registered_at = 1.week.ago
+      device.save
+      APN::Feedback.should_receive(:devices).and_return([device])
+      APN::App.should_receive(:all).and_return([])
+      lambda { 
+        APN::App.process_devices
+      }.should change(APN::Device, :count).by(-1)
+    end
+  end
+  
   describe 'nil cert when processing devices' do 
     
     it 'should raise an exception for processing devices for an app with no cert' do

data/spec/apn_on_rails/app/models/apn/device_spec.rb

@@ -40,19 +40,19 @@ describe APN::Device do
     
   end
   
-  describe 'before_save' do
+  describe 'before_create' do
     
-    it 'should set the last_registered_at date to Time.now if nil' do
+    it 'should set the last_registered_at date to Time.now' do
       time = Time.now
       Time.stub(:now).and_return(time)
       device = DeviceFactory.create
       device.last_registered_at.should_not be_nil
       device.last_registered_at.to_s.should == time.to_s
       
-      ago = 1.week.ago
-      device = DeviceFactory.create(:last_registered_at => ago)
-      device.last_registered_at.should_not be_nil
-      device.last_registered_at.to_s.should == ago.to_s
+      # ago = 1.week.ago
+      # device = DeviceFactory.create(:last_registered_at => ago)
+      # device.last_registered_at.should_not be_nil
+      # device.last_registered_at.to_s.should == ago.to_s
     end
     
   end

data/spec/factories/app_factory.rb

@@ -9,9 +9,9 @@ module AppFactory
     end
     
     def create(options = {})
-      device = AppFactory.new(options)
-      device.save
-      return device
+      app = AppFactory.new(options)
+      app.save
+      return app
     end
     
     def random_cert

metadata

@@ -1,13 +1,13 @@
 --- !ruby/object:Gem::Specification 
 name: apn_on_rails
 version: !ruby/object:Gem::Version 
-  hash: 15
+  hash: 13
   prerelease: false
   segments: 
   - 0
   - 4
-  - 0
-  version: 0.4.0
+  - 1
+  version: 0.4.1
 platform: ruby
 authors: 
 - markbates
@@ -16,14 +16,13 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2010-10-07 00:00:00 -04:00
+date: 2010-11-09 00:00:00 -05:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
-  type: :runtime
-  prerelease: false
   name: configatron
-  version_requirements: &id001 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id001 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -32,12 +31,12 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  requirement: *id001
+  type: :runtime
+  version_requirements: *id001
 - !ruby/object:Gem::Dependency 
-  type: :development
+  name: autotest
   prerelease: false
-  name: sqlite3-ruby
-  version_requirements: &id002 !ruby/object:Gem::Requirement 
+  requirement: &id002 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -46,30 +45,42 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  requirement: *id002
-- !ruby/object:Gem::Dependency 
   type: :development
+  version_requirements: *id002
+- !ruby/object:Gem::Dependency 
+  name: sqlite3-ruby
   prerelease: false
+  requirement: &id003 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        hash: 3
+        segments: 
+        - 0
+        version: "0"
+  type: :development
+  version_requirements: *id003
+- !ruby/object:Gem::Dependency 
   name: rspec
-  version_requirements: &id003 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id004 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 62196421
+        hash: 15
         segments: 
         - 2
         - 0
         - 0
-        - beta
-        - 19
-        version: 2.0.0.beta.19
-  requirement: *id003
-- !ruby/object:Gem::Dependency 
+        version: 2.0.0
   type: :development
-  prerelease: false
+  version_requirements: *id004
+- !ruby/object:Gem::Dependency 
   name: bundler
-  version_requirements: &id004 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id005 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -82,12 +93,12 @@ dependencies:
         - rc
         - 5
         version: 1.0.0.rc.5
-  requirement: *id004
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id005
+- !ruby/object:Gem::Dependency 
   name: jeweler
-  version_requirements: &id005 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id006 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -99,12 +110,12 @@ dependencies:
         - 0
         - pre2
         version: 1.5.0.pre2
-  requirement: *id005
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id006
+- !ruby/object:Gem::Dependency 
   name: rcov
-  version_requirements: &id006 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id007 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ">="
@@ -113,12 +124,12 @@ dependencies:
         segments: 
         - 0
         version: "0"
-  requirement: *id006
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id007
+- !ruby/object:Gem::Dependency 
   name: actionpack
-  version_requirements: &id007 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id008 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -129,12 +140,12 @@ dependencies:
         - 3
         - 8
         version: 2.3.8
-  requirement: *id007
-- !ruby/object:Gem::Dependency 
   type: :development
-  prerelease: false
+  version_requirements: *id008
+- !ruby/object:Gem::Dependency 
   name: activerecord
-  version_requirements: &id008 !ruby/object:Gem::Requirement 
+  prerelease: false
+  requirement: &id009 !ruby/object:Gem::Requirement 
     none: false
     requirements: 
     - - ~>
@@ -145,7 +156,8 @@ dependencies:
         - 3
         - 8
         version: 2.3.8
-  requirement: *id008
+  type: :development
+  version_requirements: *id009
 description: |
   APN on Rails is a Ruby on Rails gem that allows you to
   easily add Apple Push Notification (iPhone) support to your Rails application.
@@ -171,6 +183,7 @@ files:
 - Rakefile
 - VERSION
 - apn_on_rails.gemspec
+- autotest/discover.rb
 - generators/apn_migrations_generator.rb
 - generators/templates/apn_migrations/001_create_apn_devices.rb
 - generators/templates/apn_migrations/002_create_apn_notifications.rb
@@ -181,6 +194,8 @@ files:
 - generators/templates/apn_migrations/007_create_device_groups.rb
 - generators/templates/apn_migrations/008_create_apn_group_notifications.rb
 - generators/templates/apn_migrations/009_create_pull_notifications.rb
+- generators/templates/apn_migrations/010_alter_apn_notifications.rb
+- generators/templates/apn_migrations/011_make_device_token_index_nonunique.rb
 - lib/apn_on_rails.rb
 - lib/apn_on_rails/apn_on_rails.rb
 - lib/apn_on_rails/app/models/apn/app.rb