punchblock 2.1.1 → 2.2.0

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

@@ -81,7 +81,6 @@ module Punchblock
                 it 'should send the component node false' do
                   subject.execute
                   original_command.response(1).should be_false
-                  subject.should_not be_alive
                 end
 
                 context "which is 'No such channel'" do
@@ -90,7 +89,6 @@ module Punchblock
                   it "should return an :item_not_found error for the command" do
                     subject.execute
                     original_command.response(0.5).should be == ProtocolError.new.setup(:item_not_found, "Could not find a call with ID #{mock_call.id}", mock_call.id)
-                    subject.should_not be_alive
                   end
                 end
 
@@ -100,7 +98,6 @@ module Punchblock
                   it "should return an :item_not_found error for the command" do
                     subject.execute
                     original_command.response(0.5).should be == ProtocolError.new.setup(:item_not_found, "Could not find a call with ID #{mock_call.id}", mock_call.id)
-                    subject.should_not be_alive
                   end
                 end
               end

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

@@ -58,7 +58,7 @@ module Punchblock
               context 'for a non-causal action' do
                 it 'should send a complete event to the component node' do
                   ami_client.should_receive(:send_action).once.and_return response
-                  subject.wrapped_object.should_receive(:send_complete_event).once.with expected_complete_reason
+                  subject.should_receive(:send_complete_event).once.with expected_complete_reason
                   subject.execute
                 end
               end

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

@@ -207,7 +207,7 @@ module Punchblock
               end
 
               it "sets the command response to true" do
-                call.async.should_receive(:redirect_back).once
+                call.should_receive(:redirect_back).once
                 subject.execute_command command
                 command.response(0.1).should be == true
               end
@@ -219,7 +219,7 @@ module Punchblock
                   source_uri: subject.id,
                   target_call_id: call.id
 
-                call.async.should_receive(:redirect_back)
+                call.should_receive(:redirect_back)
                 subject.execute_command command
                 original_command.should_not be_complete
                 call.process_ami_event ami_event

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

@@ -96,7 +96,7 @@ module Punchblock
                 end
 
                 it "should not process further dtmf events" do
-                  subject.async.should_receive(:process_dtmf).never
+                  subject.should_receive(:process_dtmf).never
                   send_ami_events_for_dtmf 3
                 end
               end
@@ -266,7 +266,7 @@ module Punchblock
                   end
 
                   it "should not process further dtmf events" do
-                    subject.async.should_receive(:process_dtmf).never
+                    subject.should_receive(:process_dtmf).never
                     send_ami_events_for_dtmf 3
                   end
                 end
@@ -331,7 +331,7 @@ module Punchblock
                 let(:original_command_opts) { { :initial_timeout => -1 } }
 
                 it "should not start a timer" do
-                  subject.wrapped_object.should_receive(:begin_initial_timer).never
+                  subject.should_receive(:begin_initial_timer).never
                   subject.execute
                 end
               end
@@ -340,7 +340,7 @@ module Punchblock
                 let(:original_command_opts) { { :initial_timeout => nil } }
 
                 it "should not start a timer" do
-                  subject.wrapped_object.should_receive(:begin_initial_timer).never
+                  subject.should_receive(:begin_initial_timer).never
                   subject.execute
                 end
               end
@@ -450,7 +450,7 @@ module Punchblock
                 let(:original_command_opts) { { :inter_digit_timeout => -1 } }
 
                 it "should not start a timer" do
-                  subject.wrapped_object.should_receive(:begin_inter_digit_timer).never
+                  subject.should_receive(:begin_inter_digit_timer).never
                   subject.execute
                 end
               end
@@ -459,7 +459,7 @@ module Punchblock
                 let(:original_command_opts) { { :inter_digit_timeout => nil } }
 
                 it "should not start a timer" do
-                  subject.wrapped_object.should_receive(:begin_inter_digit_timer).never
+                  subject.should_receive(:begin_inter_digit_timer).never
                   subject.execute
                 end
               end

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

@@ -623,13 +623,13 @@ module Punchblock
               end
 
               it "sets the command response to true" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 command.response(0.1).should be == true
               end
 
               it "sends the correct complete event" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 original_command.should_not be_complete
                 mock_call.process_ami_event ami_event
@@ -638,7 +638,7 @@ module Punchblock
               end
 
               it "redirects the call by unjoining it" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
               end
             end

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

@@ -133,22 +133,20 @@ module Punchblock
             context 'with multiple inline documents' do
               let(:output_command_options) { { render_documents: [{value: ssml_doc}, {value: ssml_doc}] } }
 
-              it "should return a ref and execute SynthAndRecog" do
-                param = [[ssml_doc.to_doc.to_s, ssml_doc.to_doc.to_s].join(','), grammar.to_doc].map { |o| "\"#{o.to_s.squish.gsub('"', '\"')}\"" }.push('uer=1&b=1').join(',')
-                mock_call.should_receive(:execute_agi_command).once.with('EXEC SynthAndRecog', param).and_return code: 200, result: 1
+              it "should return an error and not execute any actions" do
                 subject.execute
-                original_command.response(0.1).should be_a Ref
+                error = ProtocolError.new.setup 'option error', 'Only one document is allowed.'
+                original_command.response(0.1).should be == error
               end
             end
 
             context 'with multiple documents by URI' do
               let(:output_command_options) { { render_documents: [{url: 'http://example.com/doc1.ssml'}, {url: 'http://example.com/doc2.ssml'}] } }
 
-              it "should return a ref and execute SynthAndRecog" do
-                param = [['http://example.com/doc1.ssml', 'http://example.com/doc2.ssml'].join(','), grammar.to_doc].map { |o| "\"#{o.to_s.squish.gsub('"', '\"')}\"" }.push('uer=1&b=1').join(',')
-                mock_call.should_receive(:execute_agi_command).once.with('EXEC SynthAndRecog', param).and_return code: 200, result: 1
+              it "should return an error and not execute any actions" do
                 subject.execute
-                original_command.response(0.1).should be_a Ref
+                error = ProtocolError.new.setup 'option error', 'Only one document is allowed.'
+                original_command.response(0.1).should be == error
               end
             end
 
@@ -617,13 +615,13 @@ module Punchblock
               end
 
               it "sets the command response to true" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 command.response(0.1).should be == true
               end
 
               it "sends the correct complete event" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 original_command.should_not be_complete
                 mock_call.process_ami_event ami_event
@@ -632,7 +630,7 @@ module Punchblock
               end
 
               it "redirects the call by unjoining it" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
               end
             end

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

@@ -36,6 +36,13 @@ module Punchblock
             mock_call.stub(:answered?).and_return(value)
           end
 
+          def expect_mrcpsynth_with_options(options)
+            mock_call.should_receive(:execute_agi_command).once.with do |*args|
+              args[0].should be == 'EXEC MRCPSynth'
+              args[1].should match options
+            end.and_return code: 200, result: 1
+          end
+
           describe '#execute' do
             before { original_command.request! }
 
@@ -167,6 +174,25 @@ module Punchblock
                 end
 
               end
+
+              describe "with multiple documents" do
+                let :first_ssml_doc do
+                  RubySpeech::SSML.draw do
+                    audio :src => audio_filename
+                  end
+                end
+                let :second_ssml_doc do
+                  RubySpeech::SSML.draw do
+                    say_as(:interpret_as => :cardinal) { 'FOO' }
+                  end
+                end
+                let(:command_opts) { { render_documents: [{value: first_ssml_doc}, {value: second_ssml_doc}] } }
+
+                it "executes Swift with a concatenated version of the documents" do
+                  mock_call.should_receive(:execute_agi_command).once.with 'EXEC Swift', ssml_with_options
+                  subject.execute
+                end
+              end
             end
 
             context 'with a renderer of :unimrcp' do
@@ -190,13 +216,6 @@ module Punchblock
               let(:synthstatus) { 'OK' }
               before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return synthstatus }
 
-              def expect_mrcpsynth_with_options(options)
-                mock_call.should_receive(:execute_agi_command).once.with do |*args|
-                  args[0].should be == 'EXEC MRCPSynth'
-                  args[1].should match options
-                end.and_return code: 200, result: 1
-              end
-
               before { expect_answered }
 
               it "should execute MRCPSynth" do
@@ -265,10 +284,30 @@ module Punchblock
 
                 context 'with multiple documents' do
                   let(:command_opts) { { :render_documents => [{:value => ssml_doc}, {:value => ssml_doc}] } }
-                  it "should return an error and not execute any actions" do
+
+                  it "should execute MRCPSynth once with each document" do
+                    param = ["\"#{ssml_doc.to_s.squish.gsub('"', '\"')}\"", ''].join(',')
+                    mock_call.should_receive(:execute_agi_command).once.with('EXEC MRCPSynth', param).and_return code: 200, result: 1
+                    mock_call.should_receive(:execute_agi_command).once.with('EXEC MRCPSynth', param).and_return code: 200, result: 1
                     subject.execute
-                    error = ProtocolError.new.setup 'option error', 'Only a single document is supported.'
-                    original_command.response(0.1).should be == error
+                  end
+
+                  it 'should not execute further output after a stop command' do
+                    mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                      sleep 0.5
+                    end
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    Celluloid::Future.new { subject.execute }
+                    sleep 0.2
+                    mock_call.should_receive(:redirect_back).ordered
+                    stop_command = Punchblock::Component::Stop.new
+                    stop_command.request!
+                    subject.execute_command stop_command
+                    latch.wait(2).should be_true
                   end
                 end
               end
@@ -341,10 +380,40 @@ module Punchblock
 
                 context 'set' do
                   let(:command_opts) { { :repeat_times => 2 } }
-                  it "should return an error and not execute any actions" do
+
+                  it "should render the specified number of times" do
+                    2.times { expect_mrcpsynth_with_options(//) }
                     subject.execute
-                    error = ProtocolError.new.setup 'option error', 'A repeat_times value is unsupported on Asterisk.'
-                    original_command.response(0.1).should be == error
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+
+                  context 'to 0' do
+                    let(:command_opts) { { :repeat_times => 0 } }
+
+                    it "should render 10,000 the specified number of times" do
+                      expect_answered
+                      1000.times { expect_mrcpsynth_with_options(//) }
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+                  end
+
+                  it 'should not execute further output after a stop command' do
+                    mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                      sleep 0.2
+                    end
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    Celluloid::Future.new { subject.execute }
+                    sleep 0.1
+                    mock_call.should_receive(:redirect_back).ordered
+                    stop_command = Punchblock::Component::Stop.new
+                    stop_command.request!
+                    subject.execute_command stop_command
+                    latch.wait(2).should be_true
                   end
                 end
               end
@@ -655,6 +724,49 @@ module Punchblock
                       original_command.response(0.1).should be == error
                     end
                   end
+
+                  context 'with multiple documents' do
+                    let(:command_opts) { { render_documents: [{value: ssml_doc}, {value: ssml_doc}] } }
+
+                    it "should render each document in turn using a Playback per document" do
+                      expect_answered
+                      2.times { expect_playback }
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    it 'should not execute further output after a stop command' do
+                      expect_answered
+                      mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                        sleep 0.2
+                      end
+                      latch = CountDownLatch.new 1
+                      original_command.should_receive(:add_event).once.with do |e|
+                        e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                        latch.countdown!
+                      end
+                      Celluloid::Future.new { subject.execute }
+                      sleep 0.1
+                      mock_call.should_receive(:redirect_back).ordered
+                      stop_command = Punchblock::Component::Stop.new
+                      stop_command.request!
+                      subject.execute_command stop_command
+                      latch.wait(2).should be_true
+                    end
+
+                    context "when the PLAYBACKSTATUS variable is set to 'FAILED'" do
+                      let(:playbackstatus) { 'FAILED' }
+
+                      it "should terminate playback and send an error complete event" do
+                        expect_answered
+                        mock_call.should_receive(:execute_agi_command).once.and_return code: 200, result: 1
+                        subject.execute
+                        complete_reason = original_command.complete_event(0.1).reason
+                        complete_reason.should be_a Punchblock::Event::Complete::Error
+                        complete_reason.details.should == "Terminated due to playback error"
+                      end
+                    end
+                  end
                 end
 
                 describe 'start-offset' do
@@ -729,10 +841,42 @@ module Punchblock
 
                   context 'set' do
                     let(:command_opts) { { :repeat_times => 2 } }
-                    it "should return an error and not execute any actions" do
+
+                    it "should render the specified number of times" do
+                      expect_answered
+                      2.times { expect_playback }
                       subject.execute
-                      error = ProtocolError.new.setup 'option error', 'A repeat_times value is unsupported on Asterisk.'
-                      original_command.response(0.1).should be == error
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    context 'to 0' do
+                      let(:command_opts) { { :repeat_times => 0 } }
+
+                      it "should render 10,000 the specified number of times" do
+                        expect_answered
+                        1000.times { expect_playback }
+                        subject.execute
+                        original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      end
+                    end
+
+                    it 'should not execute further output after a stop command' do
+                      expect_answered
+                      mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                        sleep 0.2
+                      end
+                      latch = CountDownLatch.new 1
+                      original_command.should_receive(:add_event).once.with do |e|
+                        e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                        latch.countdown!
+                      end
+                      Celluloid::Future.new { subject.execute }
+                      sleep 0.1
+                      mock_call.should_receive(:redirect_back).ordered
+                      stop_command = Punchblock::Component::Stop.new
+                      stop_command.request!
+                      subject.execute_command stop_command
+                      latch.wait(2).should be_true
                     end
                   end
                 end
@@ -804,7 +948,7 @@ module Punchblock
                     it "does not redirect the call" do
                       expect_answered
                       expect_playback
-                      mock_call.async.should_receive(:redirect_back).never
+                      mock_call.should_receive(:redirect_back).never
                       subject.execute
                       original_command.response(0.1).should be_a Ref
                       send_ami_events_for_dtmf 1
@@ -817,24 +961,24 @@ module Punchblock
                     before do
                       expect_answered
                       mock_call.should_receive(:execute_agi_command).once.with('EXEC Playback', audio_filename)
-                      subject.wrapped_object.should_receive(:send_finish).and_return nil
+                      subject.should_receive(:send_finish).and_return nil
                     end
 
                     context "when a DTMF digit is received" do
                       it "sends the correct complete event" do
-                        mock_call.async.should_receive :redirect_back
+                        mock_call.should_receive :redirect_back
                         subject.execute
                         original_command.response(0.1).should be_a Ref
                         original_command.should_not be_complete
                         send_ami_events_for_dtmf 1
-                        mock_call.async.process_ami_event ami_event
+                        mock_call.process_ami_event ami_event
                         sleep 0.2
                         original_command.should be_complete
                         reason.should be_a Punchblock::Component::Output::Complete::Finish
                       end
 
                       it "redirects the call back to async AGI" do
-                        mock_call.async.should_receive(:redirect_back).once
+                        mock_call.should_receive(:redirect_back).once
                         subject.execute
                         original_command.response(0.1).should be_a Ref
                         send_ami_events_for_dtmf 1
@@ -848,24 +992,24 @@ module Punchblock
                     before do
                       expect_answered
                       mock_call.should_receive(:execute_agi_command).once.with('EXEC Playback', audio_filename)
-                      subject.wrapped_object.should_receive(:send_finish).and_return nil
+                      subject.should_receive(:send_finish).and_return nil
                     end
 
                     context "when a DTMF digit is received" do
                       it "sends the correct complete event" do
-                        mock_call.async.should_receive :redirect_back
+                        mock_call.should_receive :redirect_back
                         subject.execute
                         original_command.response(0.1).should be_a Ref
                         original_command.should_not be_complete
                         send_ami_events_for_dtmf 1
-                        mock_call.async.process_ami_event ami_event
+                        mock_call.process_ami_event ami_event
                         sleep 0.2
                         original_command.should be_complete
                         reason.should be_a Punchblock::Component::Output::Complete::Finish
                       end
 
                       it "redirects the call back to async AGI" do
-                        mock_call.async.should_receive(:redirect_back).once
+                        mock_call.should_receive(:redirect_back).once
                         subject.execute
                         original_command.response(0.1).should be_a Ref
                         send_ami_events_for_dtmf 1
@@ -884,6 +1028,736 @@ module Punchblock
                 end
               end
             end
+
+            context "with a renderer of :native_or_unimrcp" do
+              def expect_playback(filename = audio_filename)
+                mock_call.should_receive(:execute_agi_command).ordered.once.with('EXEC Playback', filename).and_return code: 200
+              end
+
+              def expect_playback_noanswer
+                mock_call.should_receive(:execute_agi_command).once.with('EXEC Playback', audio_filename + ',noanswer').and_return code: 200
+              end
+
+              def expect_mrcpsynth(doc = ssml_doc)
+                mock_call.should_receive(:execute_agi_command).ordered.once.with('EXEC MRCPSynth', ["\"#{doc.to_s.squish.gsub('"', '\"')}\"", ''].join(',')).and_return code: 200, result: 1
+              end
+
+              let(:audio_filename) { 'tt-monkeys' }
+
+              let :ssml_doc do
+                RubySpeech::SSML.draw do
+                  audio :src => audio_filename do
+                    string "Foobar"
+                  end
+                end
+              end
+
+              let(:command_opts) { {} }
+
+              let :command_options do
+                { :render_document => {:value => ssml_doc}, renderer: :native_or_unimrcp }.merge(command_opts)
+              end
+
+              let :original_command do
+                Punchblock::Component::Output.new command_options
+              end
+
+              let(:playbackstatus) { 'SUCCESS' }
+              before { mock_call.stub(:channel_var).with('PLAYBACKSTATUS').and_return playbackstatus }
+
+              describe 'ssml' do
+                context 'unset' do
+                  let(:ssml_doc) { nil }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    error = ProtocolError.new.setup 'option error', 'An SSML document is required.'
+                    original_command.response(0.1).should be == error
+                  end
+                end
+
+                context 'with a single audio SSML node' do
+                  let(:audio_filename) { 'tt-monkeys' }
+                  let :ssml_doc do
+                    RubySpeech::SSML.draw language: 'pt-BR' do
+                      audio :src => audio_filename do
+                        voice name: 'frank' do
+                          string "Hello world"
+                        end
+                      end
+                    end
+                  end
+
+                  it 'should playback the audio file using Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+
+                  it 'should send a complete event when the file finishes playback' do
+                    def mock_call.answered?
+                      true
+                    end
+                    expect_playback
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+
+                  context "when the audio filename is prefixed by file://" do
+                    let(:audio_filename) { 'file://tt-monkeys' }
+
+                    it 'should playback the audio file using Playback' do
+                      expect_answered
+                      expect_playback 'tt-monkeys'
+                      subject.execute
+                    end
+                  end
+
+                  context "when the audio filename has an extension" do
+                    let(:audio_filename) { 'tt-monkeys.wav' }
+
+                    it 'should playback the audio file using Playback' do
+                      expect_answered
+                      expect_playback 'tt-monkeys'
+                      subject.execute
+                    end
+
+                    context "when there are other dots in the filename" do
+                      let(:audio_filename) { 'blue.tt-monkeys.wav' }
+
+                      it 'should playback the audio file using Playback' do
+                        expect_answered
+                        expect_playback 'blue.tt-monkeys'
+                        subject.execute
+                      end
+                    end
+                  end
+
+                  context "when we get a RubyAMI Error" do
+                    it "should send an error complete event" do
+                      expect_answered
+                      error = RubyAMI::Error.new.tap { |e| e.message = 'FooBar' }
+                      mock_call.should_receive(:execute_agi_command).and_raise error
+                      subject.execute
+                      complete_reason = original_command.complete_event(0.1).reason
+                      complete_reason.should be_a Punchblock::Event::Complete::Error
+                      complete_reason.details.should == "Terminated due to AMI error 'FooBar'"
+                    end
+                  end
+
+                  context "when the channel is gone" do
+                    it "should send an error complete event" do
+                      expect_answered
+                      error = ChannelGoneError.new 'FooBar'
+                      mock_call.should_receive(:execute_agi_command).and_raise error
+                      subject.execute
+                      complete_reason = original_command.complete_event(0.1).reason
+                      complete_reason.should be_a Punchblock::Event::Complete::Hangup
+                    end
+                  end
+
+                  context "when the PLAYBACKSTATUS variable is set to 'FAILED'" do
+                    let(:playbackstatus) { 'FAILED' }
+
+                    let(:synthstatus) { 'SUCCESS' }
+                    before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return synthstatus }
+
+                    let :fallback_doc do
+                      RubySpeech::SSML.draw language: 'pt-BR' do
+                        voice name: 'frank' do
+                          string "Hello world"
+                        end
+                      end
+                    end
+
+                    it "should attempt to render the children of the audio tag via MRCP and then send a complete event" do
+                      expect_answered
+                      expect_playback
+                      expect_mrcpsynth fallback_doc
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    context "and the SYNTHSTATUS variable is set to 'ERROR'" do
+                      let(:synthstatus) { 'ERROR' }
+
+                      it "should send an error complete event" do
+                        expect_answered
+                        expect_playback
+                        expect_mrcpsynth fallback_doc
+                        subject.execute
+                        complete_reason = original_command.complete_event(0.1).reason
+                        complete_reason.should be_a Punchblock::Event::Complete::Error
+                        complete_reason.details.should == "Terminated due to UniMRCP error"
+                      end
+                    end
+                  end
+                end
+
+                context 'with a single text node without spaces' do
+                  let(:audio_filename) { 'tt-monkeys' }
+                  let :ssml_doc do
+                    RubySpeech::SSML.draw { string audio_filename }
+                  end
+
+                  it 'should playback the audio file using Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+
+                  it 'should send a complete event when the file finishes playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+
+                  context "when we get a RubyAMI Error" do
+                    it "should send an error complete event" do
+                      expect_answered
+                      error = RubyAMI::Error.new.tap { |e| e.message = 'FooBar' }
+                      mock_call.should_receive(:execute_agi_command).and_raise error
+                      subject.execute
+                      complete_reason = original_command.complete_event(0.1).reason
+                      complete_reason.should be_a Punchblock::Event::Complete::Error
+                      complete_reason.details.should == "Terminated due to AMI error 'FooBar'"
+                    end
+                  end
+
+                  context "with early media playback" do
+                    it "should play the file with Playback" do
+                      expect_answered false
+                      expect_playback_noanswer
+                      mock_call.should_receive(:send_progress)
+                      subject.execute
+                    end
+
+                    context "with interrupt_on set to something that is not nil" do
+                      let(:audio_filename) { 'tt-monkeys' }
+                      let :command_options do
+                        {
+                          :render_document => {
+                            :value => RubySpeech::SSML.draw { string audio_filename },
+                          },
+                          :interrupt_on => :any
+                        }
+                      end
+                      it "should return an error when the output is interruptible and it is early media" do
+                        expect_answered false
+                        error = ProtocolError.new.setup 'option error', 'Interrupt digits are not allowed with early media.'
+                        subject.execute
+                        original_command.response(0.1).should be == error
+                      end
+                    end
+                  end
+                end
+
+                context 'with multiple audio SSML nodes' do
+                  let(:audio_filename1) { 'foo' }
+                  let(:audio_filename2) { 'bar' }
+                  let(:audio_filename3) { 'baz' }
+                  let :ssml_doc do
+                    RubySpeech::SSML.draw do
+                      audio :src => audio_filename1 do
+                        string "Fallback 1"
+                      end
+                      audio :src => audio_filename2 do
+                        string "Fallback 2"
+                      end
+                      audio :src => audio_filename3 do
+                        string "Fallback 3"
+                      end
+                    end
+                  end
+
+                  it 'should playback all audio files using Playback' do
+                    latch = CountDownLatch.new 2
+                    expect_playback audio_filename1
+                    expect_playback audio_filename2
+                    expect_playback audio_filename3
+                    expect_answered
+                    subject.execute
+                    latch.wait 2
+                    sleep 2
+                  end
+
+                  it 'should send a complete event after the final file has finished playback' do
+                    expect_answered
+                    expect_playback audio_filename1
+                    expect_playback audio_filename2
+                    expect_playback audio_filename3
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    subject.execute
+                    latch.wait(2).should be_true
+                  end
+
+                  it 'should not execute further output after a stop command' do
+                    expect_answered
+                    mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                      sleep 0.2
+                    end
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    Celluloid::Future.new { subject.execute }
+                    sleep 0.1
+                    mock_call.should_receive(:redirect_back).ordered
+                    stop_command = Punchblock::Component::Stop.new
+                    stop_command.request!
+                    subject.execute_command stop_command
+                    latch.wait(2).should be_true
+                  end
+
+                  context "when the PLAYBACKSTATUS variable is set to 'FAILED'" do
+                    let(:synthstatus) { 'SUCCESS' }
+                    before { mock_call.stub(:channel_var).with('PLAYBACKSTATUS').and_return 'SUCCESS', 'FAILED', 'SUCCESS' }
+                    before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return synthstatus }
+
+                    let :fallback_doc do
+                      RubySpeech::SSML.draw do
+                        string "Fallback 2"
+                      end
+                    end
+
+                    it "should attempt to render the document via MRCP and then send a complete event" do
+                      expect_answered
+                      expect_playback audio_filename1
+                      expect_playback audio_filename2
+                      expect_mrcpsynth fallback_doc
+                      expect_playback audio_filename3
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    context "and the SYNTHSTATUS variable is set to 'ERROR'" do
+                      let(:synthstatus) { 'ERROR' }
+
+                      it "should terminate playback and send an error complete event" do
+                        expect_answered
+                        expect_playback audio_filename1
+                        expect_playback audio_filename2
+                        expect_mrcpsynth fallback_doc
+                        subject.execute
+                        complete_reason = original_command.complete_event(0.1).reason
+                        complete_reason.should be_a Punchblock::Event::Complete::Error
+                        complete_reason.details.should == "Terminated due to UniMRCP error"
+                      end
+                    end
+                  end
+                end
+
+                context "with an SSML document containing top-level elements other than <audio/>" do
+                  let :ssml_doc do
+                    RubySpeech::SSML.draw do
+                      voice name: 'Paul' do
+                        string "Foo Bar"
+                      end
+                    end
+                  end
+
+                  before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return 'SUCCESS' }
+
+                  it "should attempt to render the document via MRCP and then send a complete event" do
+                    expect_answered
+                    expect_mrcpsynth ssml_doc
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+                end
+
+                context "with mixed TTS and audio tags" do
+                  let :ssml_doc do
+                    RubySpeech::SSML.draw do
+                      voice name: 'Paul' do
+                        string "Foo Bar"
+                      end
+                      audio src: 'tt-monkeys'
+                      voice name: 'Frank' do
+                        string "Doo Dah"
+                      end
+                      string 'tt-weasels'
+                    end
+                  end
+
+                  let :first_doc do
+                    RubySpeech::SSML.draw do
+                      voice name: 'Paul' do
+                        string "Foo Bar"
+                      end
+                    end
+                  end
+
+                  let :second_doc do
+                    RubySpeech::SSML.draw do
+                      voice name: 'Frank' do
+                        string "Doo Dah"
+                      end
+                    end
+                  end
+
+                  before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return 'SUCCESS' }
+
+                  it "should attempt to render the document via MRCP and then send a complete event" do
+                    expect_answered
+                    expect_mrcpsynth first_doc
+                    expect_playback 'tt-monkeys'
+                    expect_mrcpsynth second_doc
+                    expect_playback 'tt-weasels'
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+                end
+
+                context 'with multiple documents' do
+                  let :second_ssml_doc do
+                    RubySpeech::SSML.draw do
+                      audio :src => 'two.wav' do
+                        string "Bazzz"
+                      end
+                    end
+                  end
+
+                  let :third_ssml_doc do
+                    RubySpeech::SSML.draw do
+                      audio :src => 'three.wav' do
+                        string "Barrrr"
+                      end
+                    end
+                  end
+
+                  let(:command_opts) { { render_documents: [{value: ssml_doc}, {value: second_ssml_doc}, {value: third_ssml_doc}] } }
+
+                  it "should render each document in turn using a Playback per document" do
+                    expect_answered
+                    expect_playback
+                    expect_playback 'two'
+                    expect_playback 'three'
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+
+                  it 'should not execute further output after a stop command' do
+                    expect_answered
+                    mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                      sleep 0.2
+                    end
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    Celluloid::Future.new { subject.execute }
+                    sleep 0.1
+                    mock_call.should_receive(:redirect_back).ordered
+                    stop_command = Punchblock::Component::Stop.new
+                    stop_command.request!
+                    subject.execute_command stop_command
+                    latch.wait(2).should be_true
+                  end
+
+                  context "when the PLAYBACKSTATUS variable is set to 'FAILED'" do
+                    let(:synthstatus) { 'SUCCESS' }
+                    before { mock_call.stub(:channel_var).with('PLAYBACKSTATUS').and_return 'SUCCESS', 'FAILED', 'SUCCESS' }
+                    before { mock_call.stub(:channel_var).with('SYNTHSTATUS').and_return synthstatus }
+
+                    let :fallback_doc do
+                      RubySpeech::SSML.draw do
+                        string "Bazzz"
+                      end
+                    end
+
+                    it "should attempt to render the document via MRCP and then send a complete event" do
+                      expect_answered
+                      expect_playback
+                      expect_playback 'two'
+                      expect_mrcpsynth fallback_doc
+                      expect_playback 'three'
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    context "and the SYNTHSTATUS variable is set to 'ERROR'" do
+                      let(:synthstatus) { 'ERROR' }
+
+                      it "should terminate playback and send an error complete event" do
+                        expect_answered
+                        expect_playback
+                        expect_playback 'two'
+                        expect_mrcpsynth fallback_doc
+                        subject.execute
+                        complete_reason = original_command.complete_event(0.1).reason
+                        complete_reason.should be_a Punchblock::Event::Complete::Error
+                        complete_reason.details.should == "Terminated due to UniMRCP error"
+                      end
+                    end
+                  end
+                end
+              end
+
+              describe 'start-offset' do
+                context 'unset' do
+                  let(:command_opts) { { :start_offset => nil } }
+                  it 'should not pass any options to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'set' do
+                  let(:command_opts) { { :start_offset => 10 } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    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
+
+              describe 'start-paused' do
+                context 'false' do
+                  let(:command_opts) { { :start_paused => false } }
+                  it 'should not pass any options to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'true' do
+                  let(:command_opts) { { :start_paused => true } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    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
+
+              describe 'repeat-interval' do
+                context 'unset' do
+                  let(:command_opts) { { :repeat_interval => nil } }
+                  it 'should not pass any options to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'set' do
+                  let(:command_opts) { { :repeat_interval => 10 } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    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
+
+              describe 'repeat-times' do
+                context 'unset' do
+                  let(:command_opts) { { :repeat_times => nil } }
+                  it 'should not pass any options to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'set' do
+                  let(:command_opts) { { :repeat_times => 2 } }
+
+                  it "should render the specified number of times" do
+                    expect_answered
+                    2.times { expect_playback }
+                    subject.execute
+                    original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                  end
+
+                  context 'to 0' do
+                    let(:command_opts) { { :repeat_times => 0 } }
+
+                    it "should render 10,000 the specified number of times" do
+                      expect_answered
+                      1000.times { expect_playback }
+                      subject.execute
+                      original_command.complete_event(0.1).reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+                  end
+
+                  it 'should not execute further output after a stop command' do
+                    expect_answered
+                    mock_call.should_receive(:execute_agi_command).once.ordered.and_return do
+                      sleep 0.2
+                    end
+                    latch = CountDownLatch.new 1
+                    original_command.should_receive(:add_event).once.with do |e|
+                      e.reason.should be_a Punchblock::Component::Output::Complete::Finish
+                      latch.countdown!
+                    end
+                    Celluloid::Future.new { subject.execute }
+                    sleep 0.1
+                    mock_call.should_receive(:redirect_back).ordered
+                    stop_command = Punchblock::Component::Stop.new
+                    stop_command.request!
+                    subject.execute_command stop_command
+                    latch.wait(2).should be_true
+                  end
+                end
+              end
+
+              describe 'max-time' do
+                context 'unset' do
+                  let(:command_opts) { { :max_time => nil } }
+                  it 'should not pass any options to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'set' do
+                  let(:command_opts) { { :max_time => 30 } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    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
+
+              describe 'voice' do
+                context 'unset' do
+                  let(:command_opts) { { :voice => nil } }
+                  it 'should not pass the v option to Playback' do
+                    expect_answered
+                    expect_playback
+                    subject.execute
+                  end
+                end
+
+                context 'set' do
+                  let(:command_opts) { { :voice => 'alison' } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    error = ProtocolError.new.setup 'option error', 'A voice value is unsupported on Asterisk.'
+                    original_command.response(0.1).should be == error
+                  end
+                end
+              end
+
+              describe 'interrupt_on' do
+                def ami_event_for_dtmf(digit, position)
+                  RubyAMI::Event.new 'DTMF',
+                    'Digit' => digit.to_s,
+                    'Start' => position == :start ? 'Yes' : 'No',
+                    'End'   => position == :end ? 'Yes' : 'No'
+                end
+
+                def send_ami_events_for_dtmf(digit)
+                  mock_call.process_ami_event ami_event_for_dtmf(digit, :start)
+                  mock_call.process_ami_event ami_event_for_dtmf(digit, :end)
+                end
+
+                let(:reason) { original_command.complete_event(5).reason }
+                let(:channel) { "SIP/1234-00000000" }
+                let :ami_event do
+                  RubyAMI::Event.new 'AsyncAGI',
+                    'SubEvent'  => "Start",
+                    'Channel'   => channel,
+                    '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
+
+                context "set to nil" do
+                  let(:command_opts) { { :interrupt_on => nil } }
+                  it "does not redirect the call" do
+                    expect_answered
+                    expect_playback
+                    mock_call.should_receive(:redirect_back).never
+                    subject.execute
+                    original_command.response(0.1).should be_a Ref
+                    send_ami_events_for_dtmf 1
+                  end
+                end
+
+                context "set to :any" do
+                  let(:command_opts) { { :interrupt_on => :any } }
+
+                  before do
+                    expect_answered
+                    mock_call.should_receive(:execute_agi_command).once.with('EXEC Playback', audio_filename)
+                    subject.should_receive(:send_finish).and_return nil
+                  end
+
+                  context "when a DTMF digit is received" do
+                    it "sends the correct complete event" do
+                      mock_call.should_receive :redirect_back
+                      subject.execute
+                      original_command.response(0.1).should be_a Ref
+                      original_command.should_not be_complete
+                      send_ami_events_for_dtmf 1
+                      mock_call.process_ami_event ami_event
+                      sleep 0.2
+                      original_command.should be_complete
+                      reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    it "redirects the call back to async AGI" do
+                      mock_call.should_receive(:redirect_back).once
+                      subject.execute
+                      original_command.response(0.1).should be_a Ref
+                      send_ami_events_for_dtmf 1
+                    end
+                  end
+                end
+
+                context "set to :dtmf" do
+                  let(:command_opts) { { :interrupt_on => :dtmf } }
+
+                  before do
+                    expect_answered
+                    mock_call.should_receive(:execute_agi_command).once.with('EXEC Playback', audio_filename)
+                    subject.should_receive(:send_finish).and_return nil
+                  end
+
+                  context "when a DTMF digit is received" do
+                    it "sends the correct complete event" do
+                      mock_call.should_receive :redirect_back
+                      subject.execute
+                      original_command.response(0.1).should be_a Ref
+                      original_command.should_not be_complete
+                      send_ami_events_for_dtmf 1
+                      mock_call.process_ami_event ami_event
+                      sleep 0.2
+                      original_command.should be_complete
+                      reason.should be_a Punchblock::Component::Output::Complete::Finish
+                    end
+
+                    it "redirects the call back to async AGI" do
+                      mock_call.should_receive(:redirect_back).once
+                      subject.execute
+                      original_command.response(0.1).should be_a Ref
+                      send_ami_events_for_dtmf 1
+                    end
+                  end
+                end
+
+                context "set to :voice" do
+                  let(:command_opts) { { :interrupt_on => :voice } }
+                  it "should return an error and not execute any actions" do
+                    subject.execute
+                    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
@@ -915,13 +1789,13 @@ module Punchblock
               end
 
               it "sets the command response to true" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 command.response(0.1).should be == true
               end
 
               it "sends the correct complete event" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
                 original_command.should_not be_complete
                 mock_call.process_ami_event ami_event
@@ -930,7 +1804,7 @@ module Punchblock
               end
 
               it "redirects the call by unjoining it" do
-                mock_call.async.should_receive(:redirect_back)
+                mock_call.should_receive(:redirect_back)
                 subject.execute_command command
               end
             end