rpush 4.2.0 → 7.0.1

data/spec/unit/client/active_record/apns/notification_spec.rb

@@ -1,329 +1,65 @@
-# encoding: US-ASCII
-
 require "unit_spec_helper"
-require 'unit/notification_shared.rb'
 
 describe Rpush::Client::ActiveRecord::Apns::Notification do
-  it_should_behave_like 'an Notification subclass'
-
-  let(:app) { Rpush::Client::ActiveRecord::Apns::App.create!(name: 'my_app', environment: 'development', certificate: TEST_CERT) }
-  let(:notification_class) { Rpush::Client::ActiveRecord::Apns::Notification }
-  let(:notification) { notification_class.new }
+  subject(:notification) { described_class.new }
 
-  it "should validate the format of the device_token" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(device_token: "{$%^&*()}")
-    expect(notification.valid?).to be_falsey
-    expect(notification.errors[:device_token].include?("is invalid")).to be_truthy
-  end
+  it_behaves_like 'Rpush::Client::Apns::Notification'
+  it_behaves_like 'Rpush::Client::ActiveRecord::Notification'
 
   it "should validate the length of the binary conversion of the notification" do
+    notification = described_class.new
+    notification.app = Rpush::Apns2::App.create(name: 'test', environment: 'development', certificate: TEST_CERT)
     notification.device_token = "a" * 108
-    notification.alert = "way too long!" * 200
-    expect(notification.valid?).to be_falsey
-    expect(notification.errors[:base].include?("APN notification cannot be larger than 2048 bytes. Try condensing your alert and device attributes.")).to be_truthy
-  end
+    notification.alert = ""
 
-  it "should store long alerts" do
-    notification.app = app
-    notification.device_token = "a" * 108
-    notification.alert = "*" * 300
+    notification.alert << "a" until notification.payload.bytesize == 2048
     expect(notification.valid?).to be_truthy
+    expect(notification.errors[:base]).to be_empty
 
-    notification.save!
-    notification.reload
-    expect(notification.alert).to eq("*" * 300)
-  end
-
-  it "should default the sound to nil" do
-    expect(notification.sound).to be_nil
-  end
-
-  it "should default the expiry to 1 day" do
-    expect(notification.expiry).to eq 1.day.to_i
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, "when assigning the device token" do
-  it "should strip spaces from the given string" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(device_token: "o m g")
-    expect(notification.device_token).to eq "omg"
-  end
-
-  it "should strip chevrons from the given string" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(device_token: "<omg>")
-    expect(notification.device_token).to eq "omg"
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, "as_json" do
-  it "should include the alert if present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(alert: "hi mom")
-    expect(notification.as_json["aps"]["alert"]).to eq "hi mom"
-  end
-
-  it "should not include the alert key if the alert is not present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(alert: nil)
-    expect(notification.as_json["aps"].key?("alert")).to be_falsey
-  end
-
-  it "should encode the alert as JSON if it is a Hash" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(alert: { 'body' => "hi mom", 'alert-loc-key' => "View" })
-    expect(notification.as_json["aps"]["alert"]).to eq('body' => "hi mom", 'alert-loc-key' => "View")
-  end
-
-  it "should include the badge if present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(badge: 6)
-    expect(notification.as_json["aps"]["badge"]).to eq 6
-  end
-
-  it "should not include the badge key if the badge is not present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(badge: nil)
-    expect(notification.as_json["aps"].key?("badge")).to be_falsey
-  end
-
-  it "should include the sound if present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(sound: "my_sound.aiff")
-    expect(notification.as_json["aps"]["sound"]).to eq "my_sound.aiff"
-  end
-
-  it "should not include the sound key if the sound is not present" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(sound: nil)
-    expect(notification.as_json["aps"].key?("sound")).to be_falsey
-  end
-
-  it "should encode the sound as JSON if it is a Hash" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new(sound: { 'name' => "my_sound.aiff", 'critical' => 1, 'volume' => 0.5 })
-    expect(notification.as_json["aps"]["sound"]).to eq('name' => "my_sound.aiff", 'critical' => 1, 'volume' => 0.5)
-  end
-
-  it "should include attributes for the device" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new
-    notification.data = { omg: :lol, wtf: :dunno }
-    expect(notification.as_json["omg"]).to eq "lol"
-    expect(notification.as_json["wtf"]).to eq "dunno"
-  end
-
-  it "should allow attributes to include a hash" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new
-    notification.data = { omg: { ilike: :hashes } }
-    expect(notification.as_json["omg"]["ilike"]).to eq "hashes"
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'MDM' do
-  let(:magic) { 'abc123' }
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  before do
-    notification.device_token = "a" * 108
-    notification.id = 1234
-  end
-
-  it 'includes the mdm magic in the payload' do
-    notification.mdm = magic
-    expect(notification.as_json).to eq('mdm' => magic)
-  end
-
-  it 'does not include aps attribute' do
-    notification.alert = "i'm doomed"
-    notification.mdm = magic
-    expect(notification.as_json.key?('aps')).to be_falsey
-  end
-
-  it 'can be converted to binary' do
-    notification.mdm = magic
-    expect(notification.to_binary).to be_present
+    notification.alert << "a"
+    expect(notification.valid?).to be_falsey
+    expect(notification.errors[:base].include?("APN notification cannot be larger than 2048 bytes. Try condensing your alert and device attributes.")).to be_truthy
   end
-end if active_record?
 
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'mutable-content' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  it 'includes mutable-content in the payload' do
-    notification.mutable_content = true
-    expect(notification.as_json['aps']['mutable-content']).to eq 1
-  end
+  describe "multi_json usage" do
+    describe "alert" do
+      subject(:notification) { described_class.new(alert: { a: 1 }, alert_is_json: true) }
 
-  it 'does not include content-available in the payload if not set' do
-    expect(notification.as_json['aps'].key?('mutable-content')).to be_falsey
-  end
+      it "should call MultiJson.load when multi_json version is 1.3.0" do
+        allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.3.0'))
+        expect(MultiJson).to receive(:load).with(any_args)
+        notification.alert
+      end
 
-  it 'does not include mutable-content as a non-aps attribute' do
-    notification.mutable_content = true
-    expect(notification.as_json.key?('mutable-content')).to be_falsey
+      it "should call MultiJson.decode when multi_json version is 1.2.9" do
+        allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.2.9'))
+        expect(MultiJson).to receive(:decode).with(any_args)
+        notification.alert
+      end
+    end
   end
 
-  it 'does not overwrite existing attributes for the device' do
-    notification.data = { hi: :mom }
-    notification.mutable_content = true
-    expect(notification.as_json['aps']['mutable-content']).to eq 1
-    expect(notification.as_json['hi']).to eq 'mom'
+  it "should default the sound to nil" do
+    expect(notification.sound).to be_nil
   end
 
   it 'does not overwrite the mutable-content flag when setting attributes for the device' do
     notification.mutable_content = true
-    notification.data = { hi: :mom }
+    notification.data = { 'hi' => 'mom' }
     expect(notification.as_json['aps']['mutable-content']).to eq 1
     expect(notification.as_json['hi']).to eq 'mom'
   end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'content-available' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  it 'includes content-available in the payload' do
-    notification.content_available = true
-    expect(notification.as_json['aps']['content-available']).to eq 1
-  end
-
-  it 'does not include content-available in the payload if not set' do
-    expect(notification.as_json['aps'].key?('content-available')).to be_falsey
-  end
-
-  it 'does not include content-available as a non-aps attribute' do
-    notification.content_available = true
-    expect(notification.as_json.key?('content-available')).to be_falsey
-  end
-
-  it 'does not overwrite existing attributes for the device' do
-    notification.data = { hi: :mom }
-    notification.content_available = true
-    expect(notification.as_json['aps']['content-available']).to eq 1
-    expect(notification.as_json['hi']).to eq 'mom'
-  end
 
   it 'does not overwrite the content-available flag when setting attributes for the device' do
     notification.content_available = true
-    notification.data = { hi: :mom }
+    notification.data = { 'hi' => 'mom' }
     expect(notification.as_json['aps']['content-available']).to eq 1
     expect(notification.as_json['hi']).to eq 'mom'
   end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'url-args' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  it 'includes url-args in the payload' do
-    notification.url_args = ['url-arg-1']
-    expect(notification.as_json['aps']['url-args']).to eq ['url-arg-1']
-  end
-
-  it 'does not include url-args in the payload if not set' do
-    expect(notification.as_json['aps'].key?('url-args')).to be_falsey
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'category' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  it 'includes category in the payload' do
-    notification.category = 'INVITE_CATEGORY'
-    expect(notification.as_json['aps']['category']).to eq 'INVITE_CATEGORY'
-  end
-
-  it 'does not include category in the payload if not set' do
-    expect(notification.as_json['aps'].key?('category')).to be_falsey
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'to_binary' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  before do
-    notification.device_token = "a" * 108
-    notification.id = 1234
-  end
-
-  it 'uses APNS_PRIORITY_CONSERVE_POWER if content-available is the only key' do
-    notification.alert = nil
-    notification.badge = nil
-    notification.sound = nil
-    notification.content_available = true
-    bytes = notification.to_binary.bytes.to_a[-4..-1]
-    expect(bytes.first).to eq 5 # priority item ID
-    expect(bytes.last).to eq Rpush::Client::ActiveRecord::Apns::Notification::APNS_PRIORITY_CONSERVE_POWER
-  end
-
-  it 'uses APNS_PRIORITY_IMMEDIATE if content-available is not the only key' do
-    notification.alert = "New stuff!"
-    notification.badge = nil
-    notification.sound = nil
-    notification.content_available = true
-    bytes = notification.to_binary.bytes.to_a[-4..-1]
-    expect(bytes.first).to eq 5 # priority item ID
-    expect(bytes.last).to eq Rpush::Client::ActiveRecord::Apns::Notification::APNS_PRIORITY_IMMEDIATE
-  end
-
-  it "should correctly convert the notification to binary" do
-    notification.sound = "1.aiff"
-    notification.badge = 3
-    notification.alert = "Don't panic Mr Mainwaring, don't panic!"
-    notification.data = { hi: :mom }
-    notification.expiry = 86_400 # 1 day
-    notification.priority = Rpush::Client::ActiveRecord::Apns::Notification::APNS_PRIORITY_IMMEDIATE
-    notification.app = Rpush::Client::ActiveRecord::Apns::App.new(name: 'my_app', environment: 'development', certificate: TEST_CERT)
-    now = Time.now
-    allow(Time).to receive_messages(now: now)
-    expect(notification.to_binary).to eq "\x02\x00\x00\x00\xAF\x01\x00 \xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\xAA\x02\x00a{\"aps\":{\"alert\":\"Don't panic Mr Mainwaring, don't panic!\",\"badge\":3,\"sound\":\"1.aiff\"},\"hi\":\"mom\"}\x03\x00\x04\x00\x00\x04\xD2\x04\x00\x04#{[now.to_i + 86_400].pack('N')}\x05\x00\x01\n"
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, "bug #31" do
-  it 'does not confuse a JSON looking string as JSON' do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new
-    notification.alert = "{\"one\":2}"
-    expect(notification.alert).to eq "{\"one\":2}"
-  end
 
   it 'does confuse a JSON looking string as JSON if the alert_is_json attribute is not present' do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new
     allow(notification).to receive_messages(has_attribute?: false)
     notification.alert = "{\"one\":2}"
     expect(notification.alert).to eq('one' => 2)
   end
 end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, "bug #35" do
-  it "should limit payload size to 256 bytes but not the entire packet" do
-    notification = Rpush::Client::ActiveRecord::Apns::Notification.new do |n|
-      n.device_token = "a" * 108
-      n.alert = "a" * 210
-      n.app = Rpush::Client::ActiveRecord::Apns::App.create!(name: 'my_app', environment: 'development', certificate: TEST_CERT)
-    end
-
-    expect(notification.to_binary(for_validation: true).bytesize).to be > 256
-    expect(notification.payload.bytesize).to be < 256
-    expect(notification).to be_valid
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, "multi_json usage" do
-  describe Rpush::Client::ActiveRecord::Apns::Notification, "alert" do
-    it "should call MultiJson.load when multi_json version is 1.3.0" do
-      notification = Rpush::Client::ActiveRecord::Apns::Notification.new(alert: { a: 1 }, alert_is_json: true)
-      allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.3.0'))
-      expect(MultiJson).to receive(:load).with(any_args)
-      notification.alert
-    end
-
-    it "should call MultiJson.decode when multi_json version is 1.2.9" do
-      notification = Rpush::Client::ActiveRecord::Apns::Notification.new(alert: { a: 1 }, alert_is_json: true)
-      allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.2.9'))
-      expect(MultiJson).to receive(:decode).with(any_args)
-      notification.alert
-    end
-  end
-end if active_record?
-
-describe Rpush::Client::ActiveRecord::Apns::Notification, 'thread-id' do
-  let(:notification) { Rpush::Client::ActiveRecord::Apns::Notification.new }
-
-  it 'includes thread-id in the payload' do
-    notification.thread_id = 'THREAD-ID'
-    expect(notification.as_json['aps']['thread-id']).to eq 'THREAD-ID'
-  end
-
-  it 'does not include thread-id in the payload if not set' do
-    expect(notification.as_json['aps']).to_not have_key('thread-id')
-  end
-end if active_record?

data/spec/unit/client/active_record/apns2/app_spec.rb

@@ -0,0 +1,5 @@
+require 'unit_spec_helper'
+
+describe Rpush::Client::ActiveRecord::Apns2::App do
+  it_behaves_like 'Rpush::Client::Apns::App'
+end if active_record?

data/spec/unit/client/active_record/apns2/notification_spec.rb

@@ -0,0 +1,65 @@
+require "unit_spec_helper"
+
+describe Rpush::Client::ActiveRecord::Apns2::Notification do
+  subject(:notification) { described_class.new }
+
+  it_behaves_like 'Rpush::Client::Apns::Notification'
+  it_behaves_like 'Rpush::Client::ActiveRecord::Notification'
+
+  it "should validate the length of the binary conversion of the notification" do
+    notification = described_class.new
+    notification.app = Rpush::Apns2::App.create(name: 'test', environment: 'development', certificate: TEST_CERT)
+    notification.device_token = "a" * 108
+    notification.alert = ""
+
+    notification.alert << "a" until notification.payload.bytesize == 4096
+    expect(notification.valid?).to be_truthy
+    expect(notification.errors[:base]).to be_empty
+
+    notification.alert << "a"
+    expect(notification.valid?).to be_falsey
+    expect(notification.errors[:base].include?("APN notification cannot be larger than 4096 bytes. Try condensing your alert and device attributes.")).to be_truthy
+  end
+
+  describe "multi_json usage" do
+    describe "alert" do
+      subject(:notification) { described_class.new(alert: { a: 1 }, alert_is_json: true) }
+
+      it "should call MultiJson.load when multi_json version is 1.3.0" do
+        allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.3.0'))
+        expect(MultiJson).to receive(:load).with(any_args)
+        notification.alert
+      end
+
+      it "should call MultiJson.decode when multi_json version is 1.2.9" do
+        allow(Gem).to receive(:loaded_specs).and_return('multi_json' => Gem::Specification.new('multi_json', '1.2.9'))
+        expect(MultiJson).to receive(:decode).with(any_args)
+        notification.alert
+      end
+    end
+  end
+
+  it "should default the sound to nil" do
+    expect(notification.sound).to be_nil
+  end
+
+  it 'does not overwrite the mutable-content flag when setting attributes for the device' do
+    notification.mutable_content = true
+    notification.data = { 'hi' => 'mom' }
+    expect(notification.as_json['aps']['mutable-content']).to eq 1
+    expect(notification.as_json['hi']).to eq 'mom'
+  end
+
+  it 'does not overwrite the content-available flag when setting attributes for the device' do
+    notification.content_available = true
+    notification.data = { 'hi' => 'mom' }
+    expect(notification.as_json['aps']['content-available']).to eq 1
+    expect(notification.as_json['hi']).to eq 'mom'
+  end
+
+  it 'does confuse a JSON looking string as JSON if the alert_is_json attribute is not present' do
+    allow(notification).to receive_messages(has_attribute?: false)
+    notification.alert = "{\"one\":2}"
+    expect(notification.alert).to eq('one' => 2)
+  end
+end if active_record?

data/spec/unit/client/active_record/apnsp8/notification_spec.rb

@@ -0,0 +1,28 @@
+require "unit_spec_helper"
+
+describe Rpush::Client::ActiveRecord::Apnsp8::Notification do
+  subject(:notification) { described_class.new }
+
+  it_behaves_like 'Rpush::Client::Apns::Notification'
+  it_behaves_like 'Rpush::Client::ActiveRecord::Notification'
+
+  it "should validate the length of the binary conversion of the notification", :aggregate_failures do
+    notification = described_class.new
+    notification.app = Rpush::Apnsp8::App.create(apn_key: "1",
+                                                 apn_key_id: "2",
+                                                 name: 'test',
+                                                 environment: 'development',
+                                                 team_id: "3",
+                                                 bundle_id: "4")
+    notification.device_token = "a" * 108
+    notification.alert = ""
+
+    notification.alert << "a" until notification.payload.bytesize == 4096
+    expect(notification.valid?).to be_truthy
+    expect(notification.errors[:base]).to be_empty
+
+    notification.alert << "a"
+    expect(notification.valid?).to be_falsey
+    expect(notification.errors[:base].include?("APN notification cannot be larger than 4096 bytes. Try condensing your alert and device attributes.")).to be_truthy
+  end
+end if active_record?

data/spec/unit/client/active_record/app_spec.rb

@@ -1,30 +1,5 @@
 require 'unit_spec_helper'
 
 describe Rpush::Client::ActiveRecord::App do
-  it 'validates the uniqueness of name within type and environment' do
-    Rpush::Client::ActiveRecord::Apns::App.create!(name: 'test', environment: 'production', certificate: TEST_CERT)
-    app = Rpush::Client::ActiveRecord::Apns::App.new(name: 'test', environment: 'production', certificate: TEST_CERT)
-    expect(app.valid?).to eq(false)
-    expect(app.errors[:name]).to eq ['has already been taken']
-
-    app = Rpush::Client::ActiveRecord::Apns::App.new(name: 'test', environment: 'development', certificate: TEST_CERT)
-    expect(app.valid?).to eq(true)
-
-    app = Rpush::Client::ActiveRecord::Gcm::App.new(name: 'test', environment: 'production', auth_key: TEST_CERT)
-    expect(app.valid?).to eq(true)
-  end
-
-  context 'validating certificates' do
-    it 'rescues from certificate error' do
-      app = Rpush::Client::ActiveRecord::Apns::App.new(name: 'test', environment: 'development', certificate: 'bad')
-      expect { app.valid? }.not_to raise_error
-      expect(app.valid?).to eq(false)
-    end
-
-    it 'raises other errors' do
-      app = Rpush::Client::ActiveRecord::Apns::App.new(name: 'test', environment: 'development', certificate: 'bad')
-      allow(OpenSSL::X509::Certificate).to receive(:new).and_raise(NameError, 'simulating no openssl')
-      expect { app.valid? }.to raise_error(NameError)
-    end
-  end
+  it_behaves_like 'Rpush::Client::App'
 end if active_record?

data/spec/unit/client/active_record/gcm/app_spec.rb

@@ -1,4 +1,6 @@
 require 'unit_spec_helper'
 
 describe Rpush::Client::ActiveRecord::Gcm::App do
-end
+  it_behaves_like 'Rpush::Client::Gcm::App'
+  it_behaves_like 'Rpush::Client::ActiveRecord::App'
+end if active_record?

data/spec/unit/client/active_record/gcm/notification_spec.rb

@@ -1,96 +1,14 @@
 require 'unit_spec_helper'
-require 'unit/notification_shared.rb'
 
 describe Rpush::Client::ActiveRecord::Gcm::Notification do
-  it_should_behave_like 'an Notification subclass'
+  it_behaves_like 'Rpush::Client::Gcm::Notification'
+  it_behaves_like 'Rpush::Client::ActiveRecord::Notification'
 
-  let(:app) { Rpush::Client::ActiveRecord::Gcm::App.create!(name: 'test', auth_key: 'abc') }
-  let(:notification_class) { Rpush::Client::ActiveRecord::Gcm::Notification }
-  let(:notification) { notification_class.new }
+  subject(:notification) { described_class.new }
+  let(:app) { Rpush::Gcm::App.create!(name: 'test', auth_key: 'abc') }
 
-  it "has a 'data' payload limit of 4096 bytes" do
-    notification.data = { key: "a" * 4096 }
-    expect(notification.valid?).to be_falsey
-    expect(notification.errors[:base]).to eq ["Notification payload data cannot be larger than 4096 bytes."]
-  end
-
-  it 'limits the number of registration ids to 1000' do
-    notification.registration_ids = ['a'] * (1000 + 1)
-    expect(notification.valid?).to be_falsey
-    expect(notification.errors[:base]).to eq ["Number of registration_ids cannot be larger than 1000."]
-  end
-
-  it 'validates expiry is present if collapse_key is set' do
-    notification.collapse_key = 'test'
-    notification.expiry = nil
-    expect(notification.valid?).to be_falsey
-    expect(notification.errors[:expiry]).to eq ['must be set when using a collapse_key']
-  end
-
-  it 'includes time_to_live in the payload' do
-    notification.expiry = 100
-    expect(notification.as_json['time_to_live']).to eq 100
-  end
-
-  it 'includes content_available in the payload' do
-    notification.content_available = true
-    expect(notification.as_json['content_available']).to eq true
-  end
-
-  it 'includes mutable_content in the payload' do
-    notification.mutable_content = true
-    expect(notification.as_json['mutable_content']).to eq true
-  end
-
-  it 'sets the priority to high when set to high' do
-    notification.priority = 'high'
-    expect(notification.as_json['priority']).to eq 'high'
-  end
-
-  it 'sets the priority to normal when set to normal' do
-    notification.priority = 'normal'
-    expect(notification.as_json['priority']).to eq 'normal'
-  end
-
-  it 'validates the priority is either "normal" or "high"' do
-    notification.priority = 'invalid'
-    expect(notification.errors[:priority]).to eq ['must be one of either "normal" or "high"']
-  end
-
-  it 'excludes the priority if it is not defined' do
-    expect(notification.as_json).not_to have_key 'priority'
-  end
-
-  it 'includes the notification payload if defined' do
-    notification.notification = { key: 'any key is allowed' }
-    expect(notification.as_json).to have_key 'notification'
-  end
-
-  it 'excludes the notification payload if undefined' do
-    expect(notification.as_json).not_to have_key 'notification'
-  end
-
-  it 'includes the dry_run payload if defined' do
-    notification.dry_run = true
+  it 'accepts non-booleans as a truthy value' do
+    notification.dry_run = 'Not a boolean'
     expect(notification.as_json['dry_run']).to eq true
   end
-
-  it 'excludes the dry_run payload if undefined' do
-    expect(notification.as_json).not_to have_key 'dry_run'
-  end
-
-  # In Rails 4.2 this value casts to `false` and thus will not be included in
-  # the payload. This changed to match Ruby's semantics, and will casts to
-  # `true` in Rails 5 and above.
-  if ActiveRecord.version <= Gem::Version.new('5')
-    it 'accepts non-booleans as a falsey value' do
-      notification.dry_run = 'Not a boolean'
-      expect(notification.as_json).not_to have_key 'dry_run'
-    end
-  else
-    it 'accepts non-booleans as a truthy value' do
-      notification.dry_run = 'Not a boolean'
-      expect(notification.as_json['dry_run']).to eq true
-    end
-  end
 end if active_record?

data/spec/unit/client/active_record/notification_spec.rb

@@ -1,20 +1,12 @@
 require 'unit_spec_helper'
 
 describe Rpush::Client::ActiveRecord::Notification do
-  let(:notification) { Rpush::Client::ActiveRecord::Notification.new }
+  it_behaves_like 'Rpush::Client::Notification'
 
-  it 'allows assignment of many registration IDs' do
-    notification.registration_ids = %w(a b)
-    expect(notification.registration_ids).to eq %w(a b)
-  end
-
-  it 'allows assignment of a single registration ID' do
-    notification.registration_ids = 'a'
-    expect(notification.registration_ids).to eq ['a']
-  end
+  subject(:notification) { described_class.new }
 
   it 'saves its parent App if required' do
-    notification.app = Rpush::Client::ActiveRecord::App.new(name: "aname")
+    notification.app = Rpush::App.new(name: "aname")
     expect(notification.app).to be_valid
     expect(notification).to be_valid
   end

data/spec/unit/client/active_record/pushy/app_spec.rb

@@ -1,17 +1,6 @@
 require 'unit_spec_helper'
 
 describe Rpush::Client::ActiveRecord::Pushy::App do
-  describe 'validates' do
-    subject { described_class.new }
-
-    it 'validates presence of name' do
-      is_expected.not_to be_valid
-      expect(subject.errors[:name]).to eq ["can't be blank"]
-    end
-
-    it 'validates presence of api_key' do
-      is_expected.not_to be_valid
-      expect(subject.errors[:api_key]).to eq ["can't be blank"]
-    end
-  end
+  it_behaves_like 'Rpush::Client::Pushy::App'
+  it_behaves_like 'Rpush::Client::ActiveRecord::App'
 end if active_record?