electric_slide 0.3.0 → 0.4.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 018120f529e0824da4b7fb13d5ce713b53c16ec7
-  data.tar.gz: 7172bc885ad750962da497dd0d1afa560f5d5729
+  metadata.gz: 7a85c9a483bad9fb21fb302c8914f74323d5f41b
+  data.tar.gz: 596262ea3479ca13f5031a46fe84210732b1c558
 SHA512:
-  metadata.gz: 8b3f3a9fd91292a2582f60c53f1d27a924b9ecb43d66013140f9c41ade3708fc2f6793a06cb3300bff0a4641116b1a9cef8afdcca84a120e3eff9fdf52043b1f
-  data.tar.gz: 8d9a3b56211a774348a0f7b953308415a69ae662e40355a63782786d9ce525350d9662ab4bec9be55edefb33bdab4fda8ff5d8248ba467125d6161ac2ba3dda6
+  metadata.gz: 606297daca9e364dd4ddfa7daa47578183ff6d359fcb1d4b66daacbba7786d36e4cbd36aa180c6c80fbd75645c390b55ecbb5dcf862df6f98ccb3c7cab844f8e
+  data.tar.gz: 1771312a5bbda69e0b2fe802cb3510ee66cc34c158d36b69ec11efc5a0f960c4a955c3869269c333ec81ec2300ceb325d66b0b40e7750d13c76a32476e622b67

data/.lgtm

@@ -0,0 +1,2 @@
+approvals = 1
+pattern = "(?i):shipit:|:\\+1:|LGTM"

data/CHANGELOG.md

@@ -1,4 +1,10 @@
 # [develop](https://github.com/adhearsion/electric_slide)
+
+# [0.4.0](https://github.com/adhearsion/electric_slide/compare/v0.3.0...v0.4.0) - [2015-12-17](https://rubygems.org/gems/adhearsion/versions/0.4.0)
+  * Change `ElectricSlide::Agent` `#presence=` method name to `#update_presence`. It will also accept one more parameter called `extra_params`, a hash that holds application specific parameters that are passed to the presence change callback.
+  * `ElectricSlide::CallQueue#add_agent` will no longer wait for a connection attempt to complete before returning
+
+# [0.3.0](https://github.com/adhearsion/electric_slide/compare/v0.2.0...v0.3.0) - [2015-12-01](https://rubygems.org/gems/adhearsion/versions/0.3.0)
   * Trigger an agent "presence change" callback when the agent is removed from the queue
   * Bugfix: Fix `NameError` exception when referencing namespaced constant `Adhearsion::Call::ExpiredError` in injected `Adhearsion::Call#active?` method
   * Provide Electric Slide with a way to check if a queue with a given set of attributes is valid/can be instantiated before Electric Slide adds it to the supervision group. The supervisor will crash if its attempt to create the queue raises an exception.

data/MAINTAINERS

@@ -0,0 +1,2 @@
+Ben Langfeld <ben@langfeld.me> (@benlangfeld)
+Ben Klang <bklang@mojolingo.com> (@bklang)

data/lib/electric_slide/agent.rb

@@ -2,7 +2,8 @@
 
 class ElectricSlide
   class Agent
-    attr_accessor :id, :address, :presence, :call, :queue
+    attr_accessor :id, :address, :call, :queue
+    attr_reader :presence
 
     # @param [Hash] opts Agent parameters
     # @option opts [String] :id The Agent's ID
@@ -14,10 +15,10 @@ class ElectricSlide
       @presence = opts[:presence] || :available
     end
 
-    def presence=(new_presence)
+    def update_presence(new_presence, extra_params = {})
       old_presence = @presence
       @presence = new_presence
-      callback :presence_change, queue, @call, new_presence, old_presence
+      callback :presence_change, queue, @call, new_presence, old_presence, extra_params
     end
 
     def callback(type, *args)

data/lib/electric_slide/call_queue.rb

@@ -131,7 +131,7 @@ class ElectricSlide
     def checkout_agent
       agent = @strategy.checkout_agent
       if agent
-        agent.presence = :on_call
+        agent.update_presence(:on_call)
       end
       agent
     end
@@ -179,7 +179,7 @@ class ElectricSlide
       # Fake the presence callback since this is a new agent
       agent.callback :presence_change, self, agent.call, agent.presence, :unavailable
 
-      check_for_connections
+      async.check_for_connections
     end
 
     # Marks an agent as available to take a call. To be called after an agent completes a call
@@ -192,7 +192,7 @@ class ElectricSlide
 
       return false unless get_agent(agent.id)
 
-      agent.presence = new_presence
+      agent.update_presence(new_presence)
       agent.address = address if address
 
       case agent.presence
@@ -216,9 +216,10 @@ class ElectricSlide
 
     # Removes an agent from the queue entirely
     # @param [Agent] agent The {Agent} to be removed from the queue
+    # @param [Hash] extra_params Application specific extra params
     # @return [Agent, Nil] The Agent object if removed, Nil otherwise
-    def remove_agent(agent)
-      agent.presence = :unavailable
+    def remove_agent(agent, extra_params = {})
+      agent.update_presence(:unavailable, extra_params)
       @strategy.delete agent
       @agents.delete agent
       logger.info "Removing agent #{agent} from the queue"

data/lib/electric_slide/version.rb

@@ -1,4 +1,4 @@
 # encoding: utf-8
 class ElectricSlide
-  VERSION = '0.3.0'
+  VERSION = '0.4.0'
 end

data/spec/electric_slide/agent_spec.rb

@@ -44,6 +44,23 @@ describe ElectricSlide::Agent do
     called = false
     ElectricSlide::Agent.on_presence_change { |queue, agent_call, presence| called = true }
     agent = ElectricSlide::Agent.new presence: :unavailable
-    agent.presence = :busy
+    agent.update_presence(:busy)
+
+    expect(called).to be_truthy
+  end
+
+  it 'sends `extra_params` and `old_presence` to the presence change callback' do
+    presence_change_attributes = []
+    ElectricSlide::Agent.on_presence_change do |*attrs|
+      presence_change_attributes = attrs
+    end
+
+    agent = ElectricSlide::Agent.new presence: :unavailable
+    agent.update_presence(:busy, triggered_by: 'auto')
+
+    queue, agent_call, presence, old_presence, extra_params = presence_change_attributes
+
+    expect(old_presence).to eq :unavailable
+    expect(extra_params[:triggered_by]).to eq 'auto'
   end
 end

data/spec/electric_slide/call_queue_spec.rb

@@ -137,7 +137,7 @@ describe ElectricSlide::CallQueue do
               # prevent the agent from being returned to the queue so the queued
               # call isn't grabbed by the agent again, changing queued call state
               # before the example can check it
-              agent.presence = :unavailable
+              agent.update_presence(:unavailable)
             end
 
             it 'unsets the `:agent` call variable on the queued call' do
@@ -264,7 +264,15 @@ describe ElectricSlide::CallQueue do
       }.to change(queue, :checkout_agent).from(nil).to(agent)
     end
 
-    it "connects the agent to waiting queued calls"
+    it "connects the agent to waiting queued calls" do
+      call = Adhearsion::OutboundCall.new
+      queue.enqueue call
+
+      queue.add_agent agent
+      sleep 0.5
+      expect(agent.presence).to eq(:on_call)
+      expect(call[:agent]).to eq(agent)
+    end
 
     context 'when given an agent already in the queue' do
       before do
@@ -352,7 +360,7 @@ describe ElectricSlide::CallQueue do
       called = false
       ElectricSlide::Agent.on_presence_change { |queue, agent_call, presence| called = true }
       queue.remove_agent agent
-      expect(called).to be
+      expect(called).to be_truthy
     end
 
     it 'takes the agent out of the call rotation' do

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: electric_slide
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.4.0
 platform: ruby
 authors:
 - Ben Klang
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-12-01 00:00:00.000000000 Z
+date: 2015-12-17 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: adhearsion
@@ -158,12 +158,14 @@ extensions: []
 extra_rdoc_files: []
 files:
 - ".gitignore"
+- ".lgtm"
 - ".rspec"
 - ".travis.yml"
 - CHANGELOG.md
 - Gemfile
 - Guardfile
 - LICENSE
+- MAINTAINERS
 - README.markdown
 - Rakefile
 - config/ahn_acd.yml
@@ -200,7 +202,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.2.2
+rubygems_version: 2.4.5
 signing_key: 
 specification_version: 2
 summary: Automatic Call Distributor for Adhearsion