punchblock 2.2.1 → 2.2.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 7e22367a7afc3824dff77a61e77d6b0008208f7b
-  data.tar.gz: b4b52ffd999987e0bb67a4a3da052f3b1547e4d3
+  metadata.gz: 60e6ce85c4eb99dac8e18f5b06f868e13562fa64
+  data.tar.gz: 70ce395958873bbb5078caf5bd9ba517015d4e35
 SHA512:
-  metadata.gz: f212f2fcbba8b23ed7a88ac16dd8a9785cefaef44818f9d52c35d2155514169fae63ee797268482b0b79b68bf1e01a7c011f59baac6f444eb5f9cc122eced8c4
-  data.tar.gz: 48846a2da3e64093ea3fef764ff5f33357f393496f13e70dd767c65fd6040b443fdb44b134b7ae02c9b2be25a2fcf538b71cd91d5e59b220f22c62b148748002
+  metadata.gz: deb55ad4aa2888d889f33b55c769d567b837849475fd594ba9e8a07b3628d43880ccb25ef8e9ea1d05889481b4c423b994ce15cad5fe015791baa3525b3f11a5
+  data.tar.gz: 758ccbfed733614dbede05e2dd94525103085c7b5a0b6cf5e0bccb6e89f024e93a802716ae862157c6d87dd3d9a5a97d18444fe0673b471ddfc1520598a42a3c

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # [develop](https://github.com/adhearsion/punchblock)
 
+# [v2.2.2](https://github.com/adhearsion/punchblock/compare/v2.2.1...v2.2.2) - [2014-01-21](https://rubygems.org/gems/punchblock/versions/2.2.2)
+  * Bugfix: Delay sending Answered event on call until AsyncAGI is invoked
+
 # [v2.2.1](https://github.com/adhearsion/punchblock/compare/v2.2.0...v2.2.1) - [2014-01-17](https://rubygems.org/gems/punchblock/versions/2.2.1)
   * Bugfix: `Punchblock::Event::Complete::Reason` is now a `Punchblock::Event`
 

data/lib/punchblock/translator/asterisk/call.rb

@@ -118,13 +118,15 @@ module Punchblock
             if component = component_with_id(ami_event['CommandID'])
               component.handle_ami_event ami_event
             end
+
+            if @answered == false && ami_event['SubEvent'] == 'Start'
+              @answered = true
+              send_pb_event Event::Answered.new
+            end
           when 'Newstate'
             case ami_event['ChannelState']
             when '5'
               send_pb_event Event::Ringing.new
-            when '6'
-              @answered = true
-              send_pb_event Event::Answered.new
             end
           when 'OriginateResponse'
             if ami_event['Response'] == 'Failure' && ami_event['Uniqueid'] == '<null>'

data/lib/punchblock/version.rb

@@ -1,5 +1,5 @@
 # encoding: utf-8
 
 module Punchblock
-  VERSION = "2.2.1"
+  VERSION = "2.2.2"
 end

data/spec/punchblock/translator/asterisk/call_spec.rb

@@ -498,6 +498,42 @@ module Punchblock
               component.should_receive(:handle_ami_event).once.with ami_event
               subject.process_ami_event ami_event
             end
+
+            it 'should not send an answered event' do
+              translator.should_receive(:handle_pb_event).with(kind_of(Punchblock::Event::Answered)).never
+              subject.process_ami_event ami_event
+            end
+          end
+
+          context 'with an AsyncAGI Start event' do
+            let(:ami_event) do
+              RubyAMI::Event.new "AsyncAGI",
+                "SubEvent"  => "Start",
+                "Channel"   => "SIP/1234-00000000",
+                "Env"       => "agi_request%3A%20async%0Aagi_channel%3A%20SIP%2Fuserb-00000006%0Aagi_language%3A%20en%0Aagi_type%3A%20SIP%0Aagi_uniqueid%3A%201390303636.6%0Aagi_version%3A%2011.7.0%0Aagi_callerid%3A%20userb%0Aagi_calleridname%3A%20User%20B%0Aagi_callingpres%3A%200%0Aagi_callingani2%3A%200%0Aagi_callington%3A%200%0Aagi_callingtns%3A%200%0Aagi_dnid%3A%20unknown%0Aagi_rdnis%3A%20unknown%0Aagi_context%3A%20adhearsion-redirect%0Aagi_extension%3A%201%0Aagi_priority%3A%201%0Aagi_enhanced%3A%200.0%0Aagi_accountcode%3A%20%0Aagi_threadid%3A%20139696536876800%0A%0A"
+            end
+
+            it 'should send an answered event' do
+              expected_answered = Punchblock::Event::Answered.new
+              expected_answered.target_call_id = subject.id
+              translator.should_receive(:handle_pb_event).with expected_answered
+              subject.process_ami_event ami_event
+            end
+
+            it '#answered? should be true' do
+              subject.process_ami_event ami_event
+              subject.answered?.should be_true
+            end
+
+            context "for a second time" do
+              it 'should only send one answered event' do
+                expected_answered = Punchblock::Event::Answered.new
+                expected_answered.target_call_id = subject.id
+                translator.should_receive(:handle_pb_event).with(expected_answered).once
+                subject.process_ami_event ami_event
+                subject.process_ami_event ami_event
+              end
+            end
           end
 
           context 'with a Newstate event' do
@@ -530,23 +566,6 @@ module Punchblock
                 subject.answered?.should be_false
               end
             end
-
-            context 'up' do
-              let(:channel_state)       { '6' }
-              let(:channel_state_desc)  { 'Up' }
-
-              it 'should send a ringing event' do
-                expected_answered = Punchblock::Event::Answered.new
-                expected_answered.target_call_id = subject.id
-                translator.should_receive(:handle_pb_event).with expected_answered
-                subject.process_ami_event ami_event
-              end
-
-              it '#answered? should be true' do
-                subject.process_ami_event ami_event
-                subject.answered?.should be_true
-              end
-            end
           end
 
           context 'with an OriginateResponse event' do

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: punchblock
 version: !ruby/object:Gem::Version
-  version: 2.2.1
+  version: 2.2.2
 platform: ruby
 authors:
 - Jason Goecke
@@ -10,7 +10,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-01-17 00:00:00.000000000 Z
+date: 2014-01-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: nokogiri