rpush 4.1.1 → 5.3.0

data/spec/unit/daemon/webpush/delivery_spec.rb

@@ -0,0 +1,142 @@
+require 'unit_spec_helper'
+
+describe Rpush::Daemon::Webpush::Delivery do
+  let(:app) { Rpush::Webpush::App.create!(name: 'MyApp', vapid_keypair: VAPID_KEYPAIR) }
+
+  # Push subscription information as received from a client browser when the
+  # user subscribed to push notifications.
+  let(:device_reg) {
+    { endpoint: 'https://webpush-provider.example.org/push/some-id',
+      keys: {'auth' => 'DgN9EBia1o057BdhCOGURA', 'p256dh' => 'BAtxJ--7vHq9IVm8utUB3peJ4lpxRqk1rukCIkVJOomS83QkCnrQ4EyYQsSaCRgy_c8XPytgXxuyAvRJdnTPK4A'} }
+  }
+
+  let(:data) { { message: 'some message' } }
+  let(:notification) { Rpush::Webpush::Notification.create!(app: app, registration_ids: [device_reg], data: data) }
+  let(:batch) { instance_double('Rpush::Daemon::Batch', notification_processed: nil) }
+  let(:response) { instance_double('Net::HTTPResponse', code: response_code, header: response_header, body: response_body) }
+  let(:response_code) { 201 }
+  let(:response_header) { {} }
+  let(:response_body) { nil }
+  let(:http) { instance_double('Net::HTTP::Persistent', request: response) }
+  let(:logger) { instance_double('Rpush::Logger', error: nil, info: nil, warn: nil, internal_logger: nil) }
+  let(:now) { Time.parse('2020-10-13 00:00:00 UTC') }
+
+  before do
+    allow(Rpush).to receive_messages(logger: logger)
+    allow(Time).to receive_messages(now: now)
+  end
+
+  subject(:delivery) { described_class.new(app, http, notification, batch) }
+
+  describe '#perform' do
+    shared_examples 'process notification' do
+      it 'invoke batch.notification_processed' do
+        subject.perform rescue nil
+        expect(batch).to have_received(:notification_processed)
+      end
+    end
+
+    context 'when response code is 201' do
+      before do
+        allow(batch).to receive(:mark_delivered)
+        Rpush::Daemon.store = Rpush::Daemon::Store.const_get(Rpush.config.client.to_s.camelcase).new
+      end
+
+      it 'marks the notification as delivered' do
+        delivery.perform
+        expect(batch).to have_received(:mark_delivered).with(notification)
+      end
+
+      it_behaves_like 'process notification'
+    end
+
+    shared_examples 'retry delivery' do |response_code:|
+      let(:response_code) { response_code }
+
+      shared_examples 'logs' do |deliver_after:|
+        let(:expected_log_message) do
+          "[MyApp] Webpush endpoint responded with a #{response_code} error. Notification #{notification.id} will be retried after #{deliver_after} (retry 1)."
+        end
+
+        it 'logs that the notification will be retried' do
+          delivery.perform
+          expect(logger).to have_received(:info).with(expected_log_message)
+        end
+      end
+
+      context 'when Retry-After header is present' do
+        let(:response_header) { { 'retry-after' => 10 } }
+
+        before do
+          allow(delivery).to receive(:mark_retryable) do
+            notification.deliver_after = now + 10.seconds
+            notification.retries += 1
+          end
+        end
+
+        it 'retry the notification' do
+          delivery.perform
+          expect(delivery).to have_received(:mark_retryable).with(notification, now + 10.seconds)
+        end
+
+        it_behaves_like 'logs', deliver_after: '2020-10-13 00:00:10'
+        it_behaves_like 'process notification'
+      end
+
+      context 'when Retry-After header is not present' do
+        before do
+          allow(delivery).to receive(:mark_retryable_exponential) do
+            notification.deliver_after = now + 2.seconds
+            notification.retries = 1
+          end
+        end
+
+        it 'retry the notification' do
+          delivery.perform
+          expect(delivery).to have_received(:mark_retryable_exponential).with(notification)
+        end
+
+        it_behaves_like 'logs', deliver_after: '2020-10-13 00:00:02'
+        it_behaves_like 'process notification'
+      end
+    end
+
+    it_behaves_like 'retry delivery', response_code: 429
+    it_behaves_like 'retry delivery', response_code: 500
+    it_behaves_like 'retry delivery', response_code: 502
+    it_behaves_like 'retry delivery', response_code: 503
+    it_behaves_like 'retry delivery', response_code: 504
+
+    context 'when delivery failed' do
+      let(:response_code) { 400 }
+      let(:fail_message) { 'that was a bad request' }
+      before do
+        allow(response).to receive(:body) { fail_message }
+        allow(batch).to receive(:mark_failed)
+      end
+
+      it 'marks the notification as failed' do
+        expect { delivery.perform }.to raise_error(Rpush::DeliveryError)
+        expected_message = "Unable to deliver notification #{notification.id}, " \
+                           "received error 400 (Bad Request: #{fail_message})"
+        expect(batch).to have_received(:mark_failed).with(notification, 400, expected_message)
+      end
+
+      it_behaves_like 'process notification'
+    end
+
+    context 'when SocketError raised' do
+      before do
+        allow(http).to receive(:request) { raise SocketError }
+        allow(delivery).to receive(:mark_retryable)
+      end
+
+      it 'retry delivery after 10 seconds' do
+        expect { delivery.perform }.to raise_error(SocketError)
+        expect(delivery).to have_received(:mark_retryable).with(notification, now + 10.seconds, SocketError)
+      end
+
+      it_behaves_like 'process notification'
+    end
+  end
+end

data/spec/unit/daemon/wns/post_request_spec.rb

@@ -30,6 +30,8 @@ describe Rpush::Daemon::Wns::PostRequest do
       expect(request.body).to include('<toast>')
       expect(request.body).to include('MyApp')
       expect(request.body).to include('Example notification')
+      # no priority header
+      expect { request.fetch('X-WNS-PRIORITY') }.to raise_error(KeyError)
     end
 
     context 'with launch' do
@@ -55,6 +57,29 @@ describe Rpush::Daemon::Wns::PostRequest do
       end
     end
 
+    context 'with priority' do
+      let(:notification) do
+        Rpush::Wns::Notification.create!(
+          app: app,
+          data: {
+            title: "MyApp",
+            body: "Example notification"
+          },
+          uri: "http://some.example/",
+          priority: 1
+        )
+      end
+
+      it 'creates a request characteristic for toast notification with priority' do
+        request = Rpush::Daemon::Wns::PostRequest.create(notification, 'token')
+        expect(request['X-WNS-Type']).to eq('wns/toast')
+        expect(request['Content-Type']).to eq('text/xml')
+        expect(request['X-WNS-PRIORITY']).to eq('1')
+        expect(request.body).to include('MyApp')
+        expect(request.body).to include('Example notification')
+      end
+    end
+
     context 'with sound' do
       let(:notification) do
         Rpush::Wns::Notification.create!(
@@ -95,6 +120,25 @@ describe Rpush::Daemon::Wns::PostRequest do
       expect(request['Content-Type']).to eq('application/octet-stream')
       expect(request.body).to eq("{\"foo\":\"foo\",\"bar\":\"bar\"}")
     end
+
+    context 'with priority' do
+      let(:notification) do
+        Rpush::Wns::RawNotification.create!(
+          app: app,
+          data: { foo: 'foo', bar: 'bar' },
+          uri: "http://some.example/",
+          priority: 3
+        )
+      end
+
+      it 'creates a request characteristic for raw notification with priority' do
+        request = Rpush::Daemon::Wns::PostRequest.create(notification, 'token')
+        expect(request['X-WNS-Type']).to eq('wns/raw')
+        expect(request['X-WNS-PRIORITY']).to eq('3')
+        expect(request['Content-Type']).to eq('application/octet-stream')
+        expect(request.body).to eq("{\"foo\":\"foo\",\"bar\":\"bar\"}")
+      end
+    end
   end
 
   context 'BadgeNotification' do
@@ -113,5 +157,25 @@ describe Rpush::Daemon::Wns::PostRequest do
       expect(request.body).to include('<badge')
       expect(request.body).to include('42')
     end
+
+    context 'with priority' do
+      let(:notification) do
+        Rpush::Wns::BadgeNotification.create!(
+          app: app,
+          uri: "http://some.example/",
+          badge: 42,
+          priority: 4
+        )
+      end
+
+      it 'creates a request characteristic for badge notification with priority' do
+        request = Rpush::Daemon::Wns::PostRequest.create(notification, 'token')
+        expect(request['X-WNS-Type']).to eq('wns/badge')
+        expect(request['X-WNS-PRIORITY']).to eq('4')
+        expect(request['Content-Type']).to eq('text/xml')
+        expect(request.body).to include('<badge')
+        expect(request.body).to include('42')
+      end
+    end
   end
 end

data/spec/unit_spec_helper.rb

@@ -1,6 +1,9 @@
 require 'spec_helper'
 require 'rails'
 
+# load all shared example files
+Dir["./spec/unit/**/shared/**/*.rb"].sort.each { |f| require f }
+
 def unit_example?(metadata)
   metadata[:file_path] =~ %r{spec/unit}
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: rpush
 version: !ruby/object:Gem::Version
-  version: 4.1.1
+  version: 5.3.0
 platform: ruby
 authors:
 - Ian Leitch
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-05-13 00:00:00.000000000 Z
+date: 2021-01-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: multi_json
@@ -44,14 +44,20 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.18'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.18.3
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.14'
+        version: '0.18'
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.18.3
 - !ruby/object:Gem::Dependency
   name: jwt
   requirement: !ruby/object:Gem::Requirement
@@ -72,14 +78,14 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '5.0'
 - !ruby/object:Gem::Dependency
   name: thor
   requirement: !ruby/object:Gem::Requirement
@@ -128,6 +134,20 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: webpush
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.0'
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
@@ -392,6 +412,7 @@ files:
 - lib/generators/templates/rpush_3_3_1_updates.rb
 - lib/generators/templates/rpush_4_1_0_updates.rb
 - lib/generators/templates/rpush_4_1_1_updates.rb
+- lib/generators/templates/rpush_4_2_0_updates.rb
 - lib/rpush.rb
 - lib/rpush/apns_feedback.rb
 - lib/rpush/cli.rb
@@ -400,9 +421,9 @@ files:
 - lib/rpush/client/active_model/adm/data_validator.rb
 - lib/rpush/client/active_model/adm/notification.rb
 - lib/rpush/client/active_model/apns/app.rb
-- lib/rpush/client/active_model/apns/binary_notification_validator.rb
 - lib/rpush/client/active_model/apns/device_token_format_validator.rb
 - lib/rpush/client/active_model/apns/notification.rb
+- lib/rpush/client/active_model/apns/notification_payload_size_validator.rb
 - lib/rpush/client/active_model/apns2/app.rb
 - lib/rpush/client/active_model/apns2/notification.rb
 - lib/rpush/client/active_model/apnsp8/app.rb
@@ -416,6 +437,8 @@ files:
 - lib/rpush/client/active_model/pushy/notification.rb
 - lib/rpush/client/active_model/pushy/time_to_live_validator.rb
 - lib/rpush/client/active_model/registration_ids_count_validator.rb
+- lib/rpush/client/active_model/webpush/app.rb
+- lib/rpush/client/active_model/webpush/notification.rb
 - lib/rpush/client/active_model/wns/app.rb
 - lib/rpush/client/active_model/wns/notification.rb
 - lib/rpush/client/active_model/wpns/app.rb
@@ -423,6 +446,7 @@ files:
 - lib/rpush/client/active_record.rb
 - lib/rpush/client/active_record/adm/app.rb
 - lib/rpush/client/active_record/adm/notification.rb
+- lib/rpush/client/active_record/apns/active_record_serializable_notification.rb
 - lib/rpush/client/active_record/apns/app.rb
 - lib/rpush/client/active_record/apns/feedback.rb
 - lib/rpush/client/active_record/apns/notification.rb
@@ -436,6 +460,8 @@ files:
 - lib/rpush/client/active_record/notification.rb
 - lib/rpush/client/active_record/pushy/app.rb
 - lib/rpush/client/active_record/pushy/notification.rb
+- lib/rpush/client/active_record/webpush/app.rb
+- lib/rpush/client/active_record/webpush/notification.rb
 - lib/rpush/client/active_record/wns/app.rb
 - lib/rpush/client/active_record/wns/badge_notification.rb
 - lib/rpush/client/active_record/wns/notification.rb
@@ -458,6 +484,8 @@ files:
 - lib/rpush/client/redis/notification.rb
 - lib/rpush/client/redis/pushy/app.rb
 - lib/rpush/client/redis/pushy/notification.rb
+- lib/rpush/client/redis/webpush/app.rb
+- lib/rpush/client/redis/webpush/notification.rb
 - lib/rpush/client/redis/wns/app.rb
 - lib/rpush/client/redis/wns/badge_notification.rb
 - lib/rpush/client/redis/wns/notification.rb
@@ -512,6 +540,8 @@ files:
 - lib/rpush/daemon/string_helpers.rb
 - lib/rpush/daemon/synchronizer.rb
 - lib/rpush/daemon/tcp_connection.rb
+- lib/rpush/daemon/webpush.rb
+- lib/rpush/daemon/webpush/delivery.rb
 - lib/rpush/daemon/wns.rb
 - lib/rpush/daemon/wns/badge_request.rb
 - lib/rpush/daemon/wns/delivery.rb
@@ -538,11 +568,13 @@ files:
 - spec/functional/apns_spec.rb
 - spec/functional/cli_spec.rb
 - spec/functional/embed_spec.rb
+- spec/functional/gcm_priority_spec.rb
 - spec/functional/gcm_spec.rb
 - spec/functional/new_app_spec.rb
 - spec/functional/pushy_spec.rb
 - spec/functional/retry_spec.rb
 - spec/functional/synchronization_spec.rb
+- spec/functional/webpush_spec.rb
 - spec/functional/wpns_spec.rb
 - spec/functional_spec_helper.rb
 - spec/spec_helper.rb
@@ -559,16 +591,60 @@ files:
 - spec/unit/client/active_record/apns/app_spec.rb
 - spec/unit/client/active_record/apns/feedback_spec.rb
 - spec/unit/client/active_record/apns/notification_spec.rb
+- spec/unit/client/active_record/apns2/app_spec.rb
+- spec/unit/client/active_record/apns2/notification_spec.rb
+- spec/unit/client/active_record/apnsp8/notification_spec.rb
 - spec/unit/client/active_record/app_spec.rb
 - spec/unit/client/active_record/gcm/app_spec.rb
 - spec/unit/client/active_record/gcm/notification_spec.rb
 - spec/unit/client/active_record/notification_spec.rb
 - spec/unit/client/active_record/pushy/app_spec.rb
 - spec/unit/client/active_record/pushy/notification_spec.rb
+- spec/unit/client/active_record/shared/app.rb
+- spec/unit/client/active_record/shared/notification.rb
+- spec/unit/client/active_record/webpush/app_spec.rb
+- spec/unit/client/active_record/webpush/notification_spec.rb
 - spec/unit/client/active_record/wns/badge_notification_spec.rb
 - spec/unit/client/active_record/wns/raw_notification_spec.rb
 - spec/unit/client/active_record/wpns/app_spec.rb
 - spec/unit/client/active_record/wpns/notification_spec.rb
+- spec/unit/client/redis/adm/app_spec.rb
+- spec/unit/client/redis/adm/notification_spec.rb
+- spec/unit/client/redis/apns/app_spec.rb
+- spec/unit/client/redis/apns/feedback_spec.rb
+- spec/unit/client/redis/apns/notification_spec.rb
+- spec/unit/client/redis/apns2/app_spec.rb
+- spec/unit/client/redis/apns2/notification_spec.rb
+- spec/unit/client/redis/apnsp8/notification_spec.rb
+- spec/unit/client/redis/app_spec.rb
+- spec/unit/client/redis/gcm/app_spec.rb
+- spec/unit/client/redis/gcm/notification_spec.rb
+- spec/unit/client/redis/notification_spec.rb
+- spec/unit/client/redis/pushy/app_spec.rb
+- spec/unit/client/redis/pushy/notification_spec.rb
+- spec/unit/client/redis/webpush/app_spec.rb
+- spec/unit/client/redis/webpush/notification_spec.rb
+- spec/unit/client/redis/wns/badge_notification_spec.rb
+- spec/unit/client/redis/wns/raw_notification_spec.rb
+- spec/unit/client/redis/wpns/app_spec.rb
+- spec/unit/client/redis/wpns/notification_spec.rb
+- spec/unit/client/shared/adm/app.rb
+- spec/unit/client/shared/adm/notification.rb
+- spec/unit/client/shared/apns/app.rb
+- spec/unit/client/shared/apns/feedback.rb
+- spec/unit/client/shared/apns/notification.rb
+- spec/unit/client/shared/app.rb
+- spec/unit/client/shared/gcm/app.rb
+- spec/unit/client/shared/gcm/notification.rb
+- spec/unit/client/shared/notification.rb
+- spec/unit/client/shared/pushy/app.rb
+- spec/unit/client/shared/pushy/notification.rb
+- spec/unit/client/shared/webpush/app.rb
+- spec/unit/client/shared/webpush/notification.rb
+- spec/unit/client/shared/wns/badge_notification.rb
+- spec/unit/client/shared/wns/raw_notification.rb
+- spec/unit/client/shared/wpns/app.rb
+- spec/unit/client/shared/wpns/notification.rb
 - spec/unit/configuration_spec.rb
 - spec/unit/daemon/adm/delivery_spec.rb
 - spec/unit/daemon/apns/certificate_expired_error_spec.rb
@@ -587,11 +663,13 @@ files:
 - spec/unit/daemon/pushy/delivery_spec.rb
 - spec/unit/daemon/retryable_error_spec.rb
 - spec/unit/daemon/service_config_methods_spec.rb
+- spec/unit/daemon/shared/store.rb
 - spec/unit/daemon/signal_handler_spec.rb
 - spec/unit/daemon/store/active_record/reconnectable_spec.rb
 - spec/unit/daemon/store/active_record_spec.rb
 - spec/unit/daemon/store/redis_spec.rb
 - spec/unit/daemon/tcp_connection_spec.rb
+- spec/unit/daemon/webpush/delivery_spec.rb
 - spec/unit/daemon/wns/delivery_spec.rb
 - spec/unit/daemon/wns/post_request_spec.rb
 - spec/unit/daemon/wpns/delivery_spec.rb
@@ -600,7 +678,6 @@ files:
 - spec/unit/deprecation_spec.rb
 - spec/unit/embed_spec.rb
 - spec/unit/logger_spec.rb
-- spec/unit/notification_shared.rb
 - spec/unit/plugin_spec.rb
 - spec/unit/push_spec.rb
 - spec/unit/reflectable_spec.rb
@@ -610,7 +687,10 @@ files:
 homepage: https://github.com/rpush/rpush
 licenses:
 - MIT
-metadata: {}
+metadata:
+  bug_tracker_uri: https://github.com/rpush/rpush/issues
+  changelog_uri: https://github.com/rpush/rpush/blob/master/CHANGELOG.md
+  source_code_uri: https://github.com/rpush/rpush
 post_install_message: |
   When upgrading, don't forget to run `bundle exec rpush init` to get all the latest migrations.
 
@@ -630,8 +710,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.0.3
-signing_key: 
+rubygems_version: 3.1.4
+signing_key:
 specification_version: 4
 summary: The push notification service for Ruby.
 test_files:
@@ -641,11 +721,13 @@ test_files:
 - spec/functional/apns_spec.rb
 - spec/functional/cli_spec.rb
 - spec/functional/embed_spec.rb
+- spec/functional/gcm_priority_spec.rb
 - spec/functional/gcm_spec.rb
 - spec/functional/new_app_spec.rb
 - spec/functional/pushy_spec.rb
 - spec/functional/retry_spec.rb
 - spec/functional/synchronization_spec.rb
+- spec/functional/webpush_spec.rb
 - spec/functional/wpns_spec.rb
 - spec/functional_spec_helper.rb
 - spec/spec_helper.rb
@@ -662,16 +744,60 @@ test_files:
 - spec/unit/client/active_record/apns/app_spec.rb
 - spec/unit/client/active_record/apns/feedback_spec.rb
 - spec/unit/client/active_record/apns/notification_spec.rb
+- spec/unit/client/active_record/apns2/app_spec.rb
+- spec/unit/client/active_record/apns2/notification_spec.rb
+- spec/unit/client/active_record/apnsp8/notification_spec.rb
 - spec/unit/client/active_record/app_spec.rb
 - spec/unit/client/active_record/gcm/app_spec.rb
 - spec/unit/client/active_record/gcm/notification_spec.rb
 - spec/unit/client/active_record/notification_spec.rb
 - spec/unit/client/active_record/pushy/app_spec.rb
 - spec/unit/client/active_record/pushy/notification_spec.rb
+- spec/unit/client/active_record/shared/app.rb
+- spec/unit/client/active_record/shared/notification.rb
+- spec/unit/client/active_record/webpush/app_spec.rb
+- spec/unit/client/active_record/webpush/notification_spec.rb
 - spec/unit/client/active_record/wns/badge_notification_spec.rb
 - spec/unit/client/active_record/wns/raw_notification_spec.rb
 - spec/unit/client/active_record/wpns/app_spec.rb
 - spec/unit/client/active_record/wpns/notification_spec.rb
+- spec/unit/client/redis/adm/app_spec.rb
+- spec/unit/client/redis/adm/notification_spec.rb
+- spec/unit/client/redis/apns/app_spec.rb
+- spec/unit/client/redis/apns/feedback_spec.rb
+- spec/unit/client/redis/apns/notification_spec.rb
+- spec/unit/client/redis/apns2/app_spec.rb
+- spec/unit/client/redis/apns2/notification_spec.rb
+- spec/unit/client/redis/apnsp8/notification_spec.rb
+- spec/unit/client/redis/app_spec.rb
+- spec/unit/client/redis/gcm/app_spec.rb
+- spec/unit/client/redis/gcm/notification_spec.rb
+- spec/unit/client/redis/notification_spec.rb
+- spec/unit/client/redis/pushy/app_spec.rb
+- spec/unit/client/redis/pushy/notification_spec.rb
+- spec/unit/client/redis/webpush/app_spec.rb
+- spec/unit/client/redis/webpush/notification_spec.rb
+- spec/unit/client/redis/wns/badge_notification_spec.rb
+- spec/unit/client/redis/wns/raw_notification_spec.rb
+- spec/unit/client/redis/wpns/app_spec.rb
+- spec/unit/client/redis/wpns/notification_spec.rb
+- spec/unit/client/shared/adm/app.rb
+- spec/unit/client/shared/adm/notification.rb
+- spec/unit/client/shared/apns/app.rb
+- spec/unit/client/shared/apns/feedback.rb
+- spec/unit/client/shared/apns/notification.rb
+- spec/unit/client/shared/app.rb
+- spec/unit/client/shared/gcm/app.rb
+- spec/unit/client/shared/gcm/notification.rb
+- spec/unit/client/shared/notification.rb
+- spec/unit/client/shared/pushy/app.rb
+- spec/unit/client/shared/pushy/notification.rb
+- spec/unit/client/shared/webpush/app.rb
+- spec/unit/client/shared/webpush/notification.rb
+- spec/unit/client/shared/wns/badge_notification.rb
+- spec/unit/client/shared/wns/raw_notification.rb
+- spec/unit/client/shared/wpns/app.rb
+- spec/unit/client/shared/wpns/notification.rb
 - spec/unit/configuration_spec.rb
 - spec/unit/daemon/adm/delivery_spec.rb
 - spec/unit/daemon/apns/certificate_expired_error_spec.rb
@@ -690,11 +816,13 @@ test_files:
 - spec/unit/daemon/pushy/delivery_spec.rb
 - spec/unit/daemon/retryable_error_spec.rb
 - spec/unit/daemon/service_config_methods_spec.rb
+- spec/unit/daemon/shared/store.rb
 - spec/unit/daemon/signal_handler_spec.rb
 - spec/unit/daemon/store/active_record/reconnectable_spec.rb
 - spec/unit/daemon/store/active_record_spec.rb
 - spec/unit/daemon/store/redis_spec.rb
 - spec/unit/daemon/tcp_connection_spec.rb
+- spec/unit/daemon/webpush/delivery_spec.rb
 - spec/unit/daemon/wns/delivery_spec.rb
 - spec/unit/daemon/wns/post_request_spec.rb
 - spec/unit/daemon/wpns/delivery_spec.rb
@@ -703,7 +831,6 @@ test_files:
 - spec/unit/deprecation_spec.rb
 - spec/unit/embed_spec.rb
 - spec/unit/logger_spec.rb
-- spec/unit/notification_shared.rb
 - spec/unit/plugin_spec.rb
 - spec/unit/push_spec.rb
 - spec/unit/reflectable_spec.rb