punchblock 0.10.0 → 0.11.0

data/spec/punchblock/translator/asterisk/component/input_spec.rb

@@ -9,16 +9,16 @@ module Punchblock
         describe Input do
           let(:connection) do
             mock_connection_with_event_handler do |event|
-              command.add_event event
+              original_command.add_event event
             end
           end
           let(:media_engine)    { nil }
           let(:translator)      { Punchblock::Translator::Asterisk.new mock('AMI'), connection, media_engine }
           let(:call)            { Punchblock::Translator::Asterisk::Call.new 'foo', translator }
-          let(:command_options) { {} }
+          let(:original_command_options) { {} }
 
-          let :command do
-            Punchblock::Component::Input.new command_options
+          let :original_command do
+            Punchblock::Component::Input.new original_command_options
           end
 
           let :grammar do
@@ -37,10 +37,10 @@ module Punchblock
             end
           end
 
-          subject { Input.new command, call }
+          subject { Input.new original_command, call }
 
           describe '#execute' do
-            before { command.request! }
+            before { original_command.request! }
 
             it "calls answer_if_not_answered on the call" do
               call.expects :answer_if_not_answered
@@ -57,10 +57,10 @@ module Punchblock
             context 'with a media engine of :asterisk' do
               let(:media_engine) { :asterisk }
 
-              let(:command_opts) { {} }
+              let(:original_command_opts) { {} }
 
-              let :command_options do
-                { :mode => :dtmf, :grammar => { :value => grammar } }.merge(command_opts)
+              let :original_command_options do
+                { :mode => :dtmf, :grammar => { :value => grammar } }.merge(original_command_opts)
               end
 
               def ami_event_for_dtmf(digit, position)
@@ -76,7 +76,7 @@ module Punchblock
                 call.process_ami_event ami_event_for_dtmf(digit, :end)
               end
 
-              let(:reason) { command.complete_event(5).reason }
+              let(:reason) { original_command.complete_event(5).reason }
 
               describe "receiving DTMF events" do
                 before do
@@ -92,11 +92,11 @@ module Punchblock
 
                   let :expected_event do
                     Punchblock::Component::Input::Complete::Success.new :mode => :dtmf,
-                                                                        :confidence => 1,
-                                                                        :utterance => '12',
-                                                                        :interpretation => 'dtmf-1 dtmf-2',
-                                                                        :component_id => subject.id,
-                                                                        :call_id => call.id
+                      :confidence => 1,
+                      :utterance => '12',
+                      :interpretation => 'dtmf-1 dtmf-2',
+                      :component_id => subject.id,
+                      :call_id => call.id
                   end
 
                   it "should send a success complete event with the relevant data" do
@@ -123,40 +123,40 @@ module Punchblock
 
               describe 'grammar' do
                 context 'unset' do
-                  let(:command_opts) { { :grammar => nil } }
+                  let(:original_command_opts) { { :grammar => nil } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A grammar document is required.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A grammar document is required.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
 
               describe 'mode' do
                 context 'unset' do
-                  let(:command_opts) { { :mode => nil } }
+                  let(:original_command_opts) { { :mode => nil } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
 
                 context 'any' do
-                  let(:command_opts) { { :mode => :any } }
+                  let(:original_command_opts) { { :mode => :any } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
 
                 context 'speech' do
-                  let(:command_opts) { { :mode => :speech } }
+                  let(:original_command_opts) { { :mode => :speech } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A mode value other than DTMF is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -171,7 +171,7 @@ module Punchblock
 
               describe 'initial-timeout' do
                 context 'a positive number' do
-                  let(:command_opts) { { :initial_timeout => 1000 } }
+                  let(:original_command_opts) { { :initial_timeout => 1000 } }
 
                   it "should not cause a NoInput if first input is received in time" do
                     subject.execute
@@ -191,7 +191,7 @@ module Punchblock
                 end
 
                 context '-1' do
-                  let(:command_opts) { { :initial_timeout => -1 } }
+                  let(:original_command_opts) { { :initial_timeout => -1 } }
 
                   it "should not start a timer" do
                     subject.wrapped_object.expects(:begin_initial_timer).never
@@ -200,7 +200,7 @@ module Punchblock
                 end
 
                 context 'unset' do
-                  let(:command_opts) { { :initial_timeout => nil } }
+                  let(:original_command_opts) { { :initial_timeout => nil } }
 
                   it "should not start a timer" do
                     subject.wrapped_object.expects(:begin_initial_timer).never
@@ -209,19 +209,19 @@ module Punchblock
                 end
 
                 context 'a negative number other than -1' do
-                  let(:command_opts) { { :initial_timeout => -1000 } }
+                  let(:original_command_opts) { { :initial_timeout => -1000 } }
 
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An initial timeout value that is negative (and not -1) is invalid.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An initial timeout value that is negative (and not -1) is invalid.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
 
               describe 'inter-digit-timeout' do
                 context 'a positive number' do
-                  let(:command_opts) { { :inter_digit_timeout => 1000 } }
+                  let(:original_command_opts) { { :inter_digit_timeout => 1000 } }
 
                   it "should not prevent a Match if input is received in time" do
                     subject.execute
@@ -243,7 +243,7 @@ module Punchblock
                 end
 
                 context '-1' do
-                  let(:command_opts) { { :inter_digit_timeout => -1 } }
+                  let(:original_command_opts) { { :inter_digit_timeout => -1 } }
 
                   it "should not start a timer" do
                     subject.wrapped_object.expects(:begin_inter_digit_timer).never
@@ -252,7 +252,7 @@ module Punchblock
                 end
 
                 context 'unset' do
-                  let(:command_opts) { { :inter_digit_timeout => nil } }
+                  let(:original_command_opts) { { :inter_digit_timeout => nil } }
 
                   it "should not start a timer" do
                     subject.wrapped_object.expects(:begin_inter_digit_timer).never
@@ -261,12 +261,12 @@ module Punchblock
                 end
 
                 context 'a negative number other than -1' do
-                  let(:command_opts) { { :inter_digit_timeout => -1000 } }
+                  let(:original_command_opts) { { :inter_digit_timeout => -1000 } }
 
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An inter-digit timeout value that is negative (and not -1) is invalid.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An inter-digit timeout value that is negative (and not -1) is invalid.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -284,6 +284,37 @@ module Punchblock
               end
             end
           end
+
+          describe "#execute_command" do
+            context "with a command it does not understand" do
+              let(:command) { Punchblock::Component::Output::Pause.new }
+
+              before { command.request! }
+
+              it "returns a ProtocolError response" do
+                subject.execute_command command
+                command.response(0.1).should be_a ProtocolError
+              end
+            end
+
+            context "with a Stop command" do
+              let(:command) { Punchblock::Component::Stop.new }
+              let(:reason) { original_command.complete_event(5).reason }
+
+              before do
+                command.request!
+                original_command.request!
+                original_command.execute!
+              end
+
+              it "sets the command response to true" do
+                subject.execute_command command
+                command.response(0.1).should be == true
+                reason.should be_a Punchblock::Event::Complete::Stop
+              end
+            end
+          end
+
         end
       end
     end

data/spec/punchblock/translator/asterisk/component/output_spec.rb

@@ -9,14 +9,14 @@ module Punchblock
         describe Output do
           let(:connection) do
             mock_connection_with_event_handler do |event|
-              command.add_event event
+              original_command.add_event event
             end
           end
           let(:media_engine)  { nil }
           let(:translator)    { Punchblock::Translator::Asterisk.new mock('AMI'), connection, media_engine }
           let(:mock_call)     { Punchblock::Translator::Asterisk::Call.new 'foo', translator }
 
-          let :command do
+          let :original_command do
             Punchblock::Component::Output.new command_options
           end
 
@@ -30,10 +30,10 @@ module Punchblock
             { :ssml => ssml_doc }
           end
 
-          subject { Output.new command, mock_call }
+          subject { Output.new original_command, mock_call }
 
           describe '#execute' do
-            before { command.request! }
+            before { original_command.request! }
 
             it "calls answer_if_not_answered on the call" do
               mock_call.expects :answer_if_not_answered
@@ -75,7 +75,7 @@ module Punchblock
                   block.call Punchblock::Component::Asterisk::AGI::Command::Complete::Success.new(:code => 200, :result => 1)
                 end
                 subject.execute
-                command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
+                original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
               end
 
               describe 'interrupt_on' do
@@ -107,8 +107,8 @@ module Punchblock
                   let(:command_opts) { { :interrupt_on => :speech } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An interrupt-on value of speech is unsupported.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An interrupt-on value of speech is unsupported.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -168,7 +168,7 @@ module Punchblock
                   block.call Punchblock::Component::Asterisk::AGI::Command::Complete::Success.new(:code => 200, :result => 1)
                 end
                 subject.execute
-                command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
+                original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
               end
 
               describe 'ssml' do
@@ -176,8 +176,8 @@ module Punchblock
                   let(:command_opts) { { :ssml => nil } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An SSML document is required.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An SSML document is required.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -195,8 +195,8 @@ module Punchblock
                   let(:command_opts) { { :start_offset => 10 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A start_offset value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A start_offset value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -214,8 +214,8 @@ module Punchblock
                   let(:command_opts) { { :start_paused => true } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A start_paused value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A start_paused value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -233,8 +233,8 @@ module Punchblock
                   let(:command_opts) { { :repeat_interval => 10 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A repeat_interval value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A repeat_interval value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -252,8 +252,8 @@ module Punchblock
                   let(:command_opts) { { :repeat_times => 2 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A repeat_times value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A repeat_times value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -271,8 +271,8 @@ module Punchblock
                   let(:command_opts) { { :max_time => 30 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A max_time value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A max_time value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -324,8 +324,8 @@ module Punchblock
                   let(:command_opts) { { :interrupt_on => :speech } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An interrupt-on value of speech is unsupported.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An interrupt-on value of speech is unsupported.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -356,7 +356,7 @@ module Punchblock
                 { :ssml => ssml_doc }.merge(command_opts)
               end
 
-              let :command do
+              let :original_command do
                 Punchblock::Component::Output.new command_options
               end
 
@@ -365,8 +365,8 @@ module Punchblock
                   let(:command_opts) { { :ssml => nil } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An SSML document is required.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An SSML document is required.'
+                    original_command.response(0.1).should be == error
                   end
                 end
 
@@ -388,7 +388,29 @@ module Punchblock
                       block.call Punchblock::Component::Asterisk::AGI::Command::Complete::Success.new(:code => 200, :result => 1)
                     end
                     subject.execute
-                    command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
+                  end
+                end
+
+                context 'with a single text node without spaces' do
+                  let(:audio_filename) { 'tt-monkeys' }
+                  let :command_options do
+                    {
+                      :ssml => RubySpeech::SSML.draw { string audio_filename }
+                    }
+                  end
+
+                  it 'should playback the audio file using STREAM FILE' do
+                    expect_stream_file_with_options
+                    subject.execute
+                  end
+
+                  it 'should send a complete event when the file finishes playback' do
+                    def mock_call.send_agi_action!(*args, &block)
+                      block.call Punchblock::Component::Asterisk::AGI::Command::Complete::Success.new(:code => 200, :result => 1)
+                    end
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Success
                   end
                 end
 
@@ -423,7 +445,7 @@ module Punchblock
                     def mock_call.send_agi_action!(*args, &block)
                       block.call Punchblock::Component::Asterisk::AGI::Command::Complete::Success.new(:code => 200, :result => 1)
                     end
-                    command.expects(:add_event).once.with do |e|
+                    original_command.expects(:add_event).once.with do |e|
                       e.reason.should be_a Punchblock::Component::Output::Complete::Success
                     end
                     subject.execute
@@ -434,15 +456,15 @@ module Punchblock
                   let :command_options do
                     {
                       :ssml => RubySpeech::SSML.draw do
-                        string "FooBar"
+                        string "Foo Bar"
                       end
                     }
                   end
 
                   it "should return an unrenderable document error" do
                     subject.execute
-                    error = ProtocolError.new 'unrenderable document error', 'The provided document could not be rendered.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'unrenderable document error', 'The provided document could not be rendered.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -460,8 +482,8 @@ module Punchblock
                   let(:command_opts) { { :start_offset => 10 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A start_offset value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A start_offset value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -479,8 +501,8 @@ module Punchblock
                   let(:command_opts) { { :start_paused => true } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A start_paused value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A start_paused value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -498,8 +520,8 @@ module Punchblock
                   let(:command_opts) { { :repeat_interval => 10 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A repeat_interval value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A repeat_interval value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -517,8 +539,8 @@ module Punchblock
                   let(:command_opts) { { :repeat_times => 2 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A repeat_times value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A repeat_times value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -536,8 +558,8 @@ module Punchblock
                   let(:command_opts) { { :max_time => 30 } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A max_time value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A max_time value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -555,8 +577,8 @@ module Punchblock
                   let(:command_opts) { { :voice => 'alison' } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'A voice value is unsupported on Asterisk.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'A voice value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
@@ -590,13 +612,65 @@ module Punchblock
                   let(:command_opts) { { :interrupt_on => :speech } }
                   it "should return an error and not execute any actions" do
                     subject.execute
-                    error = ProtocolError.new 'option error', 'An interrupt-on value of speech is unsupported.'
-                    command.response(0.1).should be == error
+                    error = ProtocolError.new.setup 'option error', 'An interrupt-on value of speech is unsupported.'
+                    original_command.response(0.1).should be == error
                   end
                 end
               end
             end
           end
+
+          describe "#execute_command" do
+            context "with a command it does not understand" do
+              let(:command) { Punchblock::Component::Output::Pause.new }
+
+              before { command.request! }
+              it "returns a ProtocolError response" do
+                subject.execute_command command
+                command.response(0.1).should be_a ProtocolError
+              end
+            end
+
+            context "with a Stop command" do
+              let(:command) { Punchblock::Component::Stop.new }
+              let(:reason) { original_command.complete_event(5).reason }
+              let(:channel) { "SIP/1234-00000000" }
+              let :ami_event do
+                RubyAMI::Event.new('AsyncAGI').tap do |e|
+                  e['SubEvent'] = "Start"
+                  e['Channel']  = channel
+                  e['Env']      = "agi_request%3A%20async%0Aagi_channel%3A%20SIP%2F1234-00000000%0Aagi_language%3A%20en%0Aagi_type%3A%20SIP%0Aagi_uniqueid%3A%201320835995.0%0Aagi_version%3A%201.8.4.1%0Aagi_callerid%3A%205678%0Aagi_calleridname%3A%20Jane%20Smith%0Aagi_callingpres%3A%200%0Aagi_callingani2%3A%200%0Aagi_callington%3A%200%0Aagi_callingtns%3A%200%0Aagi_dnid%3A%201000%0Aagi_rdnis%3A%20unknown%0Aagi_context%3A%20default%0Aagi_extension%3A%201000%0Aagi_priority%3A%201%0Aagi_enhanced%3A%200.0%0Aagi_accountcode%3A%20%0Aagi_threadid%3A%204366221312%0A%0A"
+                end
+              end
+
+              before do
+                command.request!
+                original_command.request!
+                original_command.execute!
+              end
+
+              it "sets the command response to true" do
+                mock_call.expects(:redirect_back!)
+                subject.execute_command command
+                command.response(0.1).should be == true
+              end
+
+              it "sends the correct complete event" do
+                mock_call.expects(:redirect_back!)
+                subject.execute_command command
+                original_command.should_not be_complete
+                mock_call.process_ami_event! ami_event
+                reason.should be_a Punchblock::Event::Complete::Stop
+                original_command.should be_complete
+              end
+
+              it "redirects the call by unjoining it" do
+                mock_call.expects(:redirect_back!).with(nil)
+                subject.execute_command command
+              end
+            end
+          end
+
         end
       end
     end

data/spec/punchblock/translator/asterisk/component/stop_by_redirect_spec.rb

@@ -0,0 +1,57 @@
+# encoding: utf-8
+
+require 'spec_helper'
+
+module Punchblock
+  module Translator
+    class Asterisk
+      module Component
+        describe StopByRedirect do
+
+          class MockComponent < Component
+            include StopByRedirect
+          end
+
+          let(:mock_call) { mock('Call') }
+          subject { MockComponent.new Hash.new, mock_call }
+
+          describe "#execute_command" do
+            context "with a command it does not understand" do
+              let(:command) { Punchblock::Component::Output::Pause.new }
+
+              before { command.request! }
+              it "returns a ProtocolError response" do
+                mock_call.expects(:id)
+                subject.execute_command command
+                command.response(0.1).should be_a ProtocolError
+              end
+            end
+
+            context "with a Stop command" do
+              let(:command) { Punchblock::Component::Stop.new }
+
+              before do
+                command.request!
+              end
+
+              it "sets the command response to true" do
+                mock_call.expects(:redirect_back!)
+                mock_call.expects(:register_handler).with do |type, *guards|
+                  type.should be == :ami
+                  guards.should have(2).guards
+                  guards[0].should be_a Proc
+                  guards[1].should be == {:name => 'AsyncAGI'}
+                end
+
+                subject.execute_command command
+                command.response(0.1).should be == true
+              end
+
+            end
+          end
+
+        end
+      end
+    end
+  end
+end

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

@@ -83,7 +83,7 @@ module Punchblock
 
               it 'sends an error in response to the command' do
                 subject.execute_command component_command
-                component_command.response.should be == ProtocolError.new('command-not-acceptable', "Did not understand command for component #{subject.id}", call.id, subject.id)
+                component_command.response.should be == ProtocolError.new.setup('command-not-acceptable', "Did not understand command for component #{subject.id}", call.id, subject.id)
               end
             end
           end