ably-rest 0.7.3 → 0.7.5

data/lib/submodules/ably-ruby/spec/unit/realtime/channels_spec.rb

@@ -2,8 +2,8 @@
 require 'spec_helper'
 
 describe Ably::Realtime::Channels do
-  let(:connection) { instance_double('Ably::Realtime::Connection', on: true) }
-  let(:client) { instance_double('Ably::Realtime::Client', connection: connection) }
+  let(:connection) { instance_double('Ably::Realtime::Connection', unsafe_on: true, on_resume: true) }
+  let(:client) { instance_double('Ably::Realtime::Client', connection: connection, client_id: 'clientId') }
   let(:channel_name) { 'unique' }
   let(:options) { { 'bizarre' => 'value' } }
 
@@ -41,7 +41,7 @@ describe Ably::Realtime::Channels do
   end
 
   context 'destroying channels' do
-    it '#release detatches and then releases the channel resoures' do
+    it '#release detaches and then releases the channel resources' do
       released_channel = subject.get(channel_name, options)
       expect(released_channel).to receive(:detach).and_yield
       subject.release(channel_name)

data/lib/submodules/ably-ruby/spec/unit/realtime/connection_spec.rb

@@ -30,4 +30,34 @@ describe Ably::Realtime::Connection do
 
   it_behaves_like 'an incoming protocol message bus'
   it_behaves_like 'an outgoing protocol message bus'
+
+  describe 'connection resume callbacks', api_private: true do
+    let(:callbacks) { [] }
+
+    describe '#resumed' do
+      it 'triggers callbacks' do
+        subject.on_resume { callbacks << true }
+        subject.resumed
+        expect(callbacks.count).to eql(1)
+      end
+    end
+
+    describe '#on_resume' do
+      it 'registers a callback' do
+        subject.on_resume { callbacks << true }
+        subject.resumed
+        expect(callbacks.count).to eql(1)
+      end
+    end
+
+    describe '#off_resume' do
+      it 'registers a callback' do
+        subject.on_resume { callbacks << true }
+        additional_proc = proc { raise 'This should not be called' }
+        subject.off_resume &additional_proc
+        subject.resumed
+        expect(callbacks.count).to eql(1)
+      end
+    end
+  end
 end

data/lib/submodules/ably-ruby/spec/unit/realtime/presence_spec.rb

@@ -28,7 +28,7 @@ describe Ably::Realtime::Presence do
       Ably::Models::PresenceMessage.new({
         'action' => 0,
         'connection_id' => random_str,
-      }, instance_double('Ably::Models::ProtocolMessage'))
+      }, protocol_message: instance_double('Ably::Models::ProtocolMessage'))
     end
     let(:msgbus) { subject.__incoming_msgbus__ }
 
@@ -47,63 +47,89 @@ describe Ably::Realtime::Presence do
 
   context 'subscriptions' do
     let(:message_history) { Hash.new { |hash, key| hash[key] = 0 } }
-    let(:presence_action) { Ably::Models::PresenceMessage::ACTION.Enter }
-    let(:message) do
-      instance_double('Ably::Models::PresenceMessage', action: presence_action, connection_id: random_str, decode: true, member_key: random_str)
+    let(:enter_action) { Ably::Models::PresenceMessage::ACTION.Enter }
+    let(:enter_message) do
+      instance_double('Ably::Models::PresenceMessage', action: enter_action, connection_id: random_str, decode: true, member_key: random_str)
+    end
+    let(:leave_message) do
+      instance_double('Ably::Models::PresenceMessage', action: Ably::Models::PresenceMessage::ACTION.Leave, connection_id: random_str, decode: true, member_key: random_str)
+    end
+    let(:update_message) do
+      instance_double('Ably::Models::PresenceMessage', action: Ably::Models::PresenceMessage::ACTION.Update, connection_id: random_str, decode: true, member_key: random_str)
     end
 
     context '#subscribe' do
-      before do
-        subject.sync_completed
+      specify 'without a block raises an invalid ArgumentError' do
+        expect { subject.subscribe }.to raise_error ArgumentError
       end
 
-      specify 'to all presence state actions' do
+      specify 'with no action specified subscribes the provided block to all action' do
         subject.subscribe { |message| message_history[:received] += 1}
-        subject.__incoming_msgbus__.publish(:presence, message)
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(1)
       end
 
-      specify 'to specific presence state actions' do
-        subject.subscribe(presence_action) { |message| message_history[:received] += 1 }
+      specify 'with a single action argument subscribes that block to matching actions' do
+        subject.subscribe(enter_action) { |message| message_history[:received] += 1 }
         subject.subscribe(:leave)  { |message| message_history[:received] += 1 }
-        subject.__incoming_msgbus__.publish(:presence, message)
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(1)
       end
-    end
 
-    context '#unsubscribe' do
-      before do
-        subject.sync_completed
+      specify 'with a multiple action arguments subscribes that block to all of those actions' do
+        subject.subscribe(:leave, enter_action) { |message| message_history[:received] += 1 }
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
+        expect(message_history[:received]).to eql(1)
+        subject.__incoming_msgbus__.publish(:presence, leave_message)
+        expect(message_history[:received]).to eql(2)
+
+        # This message should be ignored as subscribed to :leave and :enter
+        subject.__incoming_msgbus__.publish(:presence, update_message)
+        expect(message_history[:received]).to eql(2)
       end
 
+      specify 'with a multiple duplicate action arguments subscribes that block to all of those unique actions once' do
+        subject.subscribe(enter_action, enter_action) { |message| message_history[:received] += 1 }
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
+        expect(message_history[:received]).to eql(1)
+      end
+    end
+
+    context '#unsubscribe' do
       let(:callback) do
         Proc.new { |message| message_history[:received] += 1 }
       end
       before do
-        subject.subscribe(presence_action, &callback)
+        subject.subscribe(enter_action, &callback)
       end
 
-      specify 'to all presence state actions' do
+      specify 'with no action specified unsubscribes that block from all events' do
         subject.unsubscribe &callback
-        subject.__incoming_msgbus__.publish(:presence, message)
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
+        expect(message_history[:received]).to eql(0)
+      end
+
+      specify 'with a single action argument unsubscribes the provided block with the matching action' do
+        subject.unsubscribe enter_action, &callback
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(0)
       end
 
-      specify 'to specific presence state actions' do
-        subject.unsubscribe presence_action, &callback
-        subject.__incoming_msgbus__.publish(:presence, message)
+      specify 'with multiple action arguments unsubscribes each of those matching actions with the provided block' do
+        subject.unsubscribe :update, :leave, enter_action, &callback
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(0)
       end
 
-      specify 'to specific non-matching presence state actions' do
+      specify 'with a non-matching action argument has no effect' do
         subject.unsubscribe :leave, &callback
-        subject.__incoming_msgbus__.publish(:presence, message)
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(1)
       end
 
-      specify 'all callbacks by not providing a callback' do
-        subject.unsubscribe presence_action
-        subject.__incoming_msgbus__.publish(:presence, message)
+      specify 'with no block argument unsubscribes all blocks for the action argument' do
+        subject.unsubscribe enter_action
+        subject.__incoming_msgbus__.publish(:presence, enter_message)
         expect(message_history[:received]).to eql(0)
       end
     end

data/lib/submodules/ably-ruby/spec/unit/realtime/safe_deferrable_spec.rb

@@ -0,0 +1,12 @@
+# encoding: utf-8
+require 'spec_helper'
+require 'shared/safe_deferrable_behaviour'
+require 'ably/realtime'
+
+[Ably::Models::ProtocolMessage, Ably::Models::Message, Ably::Models::PresenceMessage].each do |model_klass|
+  describe model_klass do
+    subject { model_klass.new(action: 1) }
+
+    it_behaves_like 'a safe Deferrable'
+  end
+end

data/spec/spec_helper.rb

@@ -15,6 +15,11 @@ require 'support/private_api_formatter'
 require 'support/protocol_helper'
 require 'support/random_helper'
 
+# Load in all shared behaviours
+Dir.glob(File.expand_path("../lib/submodules/ably-ruby/spec/shared/*.rb", File.dirname(__FILE__))).each do |file|
+  require file
+end
+
 require 'rspec_config'
 
 def require_tests_from(path, options = {})

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ably-rest
 version: !ruby/object:Gem::Version
-  version: 0.7.3
+  version: 0.7.5
 platform: ruby
 authors:
 - Matthew O'Riordan
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-10 00:00:00.000000000 Z
+date: 2015-03-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: faraday
@@ -173,7 +173,6 @@ files:
 - lib/ably-rest/modules/eventmachine_deferrable.rb
 - lib/submodules/ably-ruby/.gitignore
 - lib/submodules/ably-ruby/.rspec
-- lib/submodules/ably-ruby/.ruby-version.old
 - lib/submodules/ably-ruby/.travis.yml
 - lib/submodules/ably-ruby/Gemfile
 - lib/submodules/ably-ruby/LICENSE.txt
@@ -208,8 +207,11 @@ files:
 - lib/submodules/ably-ruby/lib/ably/modules/event_emitter.rb
 - lib/submodules/ably-ruby/lib/ably/modules/event_machine_helpers.rb
 - lib/submodules/ably-ruby/lib/ably/modules/http_helpers.rb
+- lib/submodules/ably-ruby/lib/ably/modules/message_emitter.rb
 - lib/submodules/ably-ruby/lib/ably/modules/message_pack.rb
 - lib/submodules/ably-ruby/lib/ably/modules/model_common.rb
+- lib/submodules/ably-ruby/lib/ably/modules/safe_deferrable.rb
+- lib/submodules/ably-ruby/lib/ably/modules/safe_yield.rb
 - lib/submodules/ably-ruby/lib/ably/modules/state_emitter.rb
 - lib/submodules/ably-ruby/lib/ably/modules/state_machine.rb
 - lib/submodules/ably-ruby/lib/ably/modules/statesman_monkey_patch.rb
@@ -228,6 +230,9 @@ files:
 - lib/submodules/ably-ruby/lib/ably/realtime/connection/websocket_transport.rb
 - lib/submodules/ably-ruby/lib/ably/realtime/models/nil_channel.rb
 - lib/submodules/ably-ruby/lib/ably/realtime/presence.rb
+- lib/submodules/ably-ruby/lib/ably/realtime/presence/members_map.rb
+- lib/submodules/ably-ruby/lib/ably/realtime/presence/presence_manager.rb
+- lib/submodules/ably-ruby/lib/ably/realtime/presence/presence_state_machine.rb
 - lib/submodules/ably-ruby/lib/ably/rest.rb
 - lib/submodules/ably-ruby/lib/ably/rest/channel.rb
 - lib/submodules/ably-ruby/lib/ably/rest/channels.rb
@@ -242,6 +247,7 @@ files:
 - lib/submodules/ably-ruby/lib/ably/rest/presence.rb
 - lib/submodules/ably-ruby/lib/ably/util/crypto.rb
 - lib/submodules/ably-ruby/lib/ably/util/pub_sub.rb
+- lib/submodules/ably-ruby/lib/ably/util/safe_deferrable.rb
 - lib/submodules/ably-ruby/lib/ably/version.rb
 - lib/submodules/ably-ruby/spec/acceptance/realtime/channel_history_spec.rb
 - lib/submodules/ably-ruby/spec/acceptance/realtime/channel_spec.rb
@@ -269,6 +275,7 @@ files:
 - lib/submodules/ably-ruby/spec/shared/client_initializer_behaviour.rb
 - lib/submodules/ably-ruby/spec/shared/model_behaviour.rb
 - lib/submodules/ably-ruby/spec/shared/protocol_msgbus_behaviour.rb
+- lib/submodules/ably-ruby/spec/shared/safe_deferrable_behaviour.rb
 - lib/submodules/ably-ruby/spec/spec_helper.rb
 - lib/submodules/ably-ruby/spec/support/api_helper.rb
 - lib/submodules/ably-ruby/spec/support/event_machine_helper.rb
@@ -304,6 +311,7 @@ files:
 - lib/submodules/ably-ruby/spec/unit/realtime/incoming_message_dispatcher_spec.rb
 - lib/submodules/ably-ruby/spec/unit/realtime/presence_spec.rb
 - lib/submodules/ably-ruby/spec/unit/realtime/realtime_spec.rb
+- lib/submodules/ably-ruby/spec/unit/realtime/safe_deferrable_spec.rb
 - lib/submodules/ably-ruby/spec/unit/realtime/websocket_transport_spec.rb
 - lib/submodules/ably-ruby/spec/unit/rest/channel_spec.rb
 - lib/submodules/ably-ruby/spec/unit/rest/channels_spec.rb
@@ -338,7 +346,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.4.5
+rubygems_version: 2.4.6
 signing_key: 
 specification_version: 4
 summary: A Ruby REST only client library for ably.io, the real-time messaging service

data/lib/submodules/ably-ruby/.ruby-version.old

@@ -1 +0,0 @@
-1.9.3-p547