ruby_ami 1.3.3 → 1.3.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 2781ad29f7dec234704d7bd3c50181eb1832316f
-  data.tar.gz: 1c1a3ff6f02c2978af62d08e8132aa853de468a2
+  metadata.gz: 87d233c2b2c4e712089b0f2ce6c1f6ef75b877cf
+  data.tar.gz: f04dff4a31a2b235babc6c8ab3c0ae5561574841
 SHA512:
-  metadata.gz: b454fac0e1c011871b49be9543d8919c598e73bd077716824f752d9d723b935c466915429f5d71d31153317dac79af142015b3d8d7102212c91bc4426dba7201
-  data.tar.gz: f877bba7372958c350deb29478a5b80a4cac49b75f5a222602a9b555ff98d99cca4acf054ce6228cbffd4b0c51916fb32732a6e158d07e7149fd7389053801a4
+  metadata.gz: d462a5e08948fdd719b76929d60da79983bafa4d4b9bc9155bc787a0937ec3ba930fffd91886f344a40944410186ddd6078b2b732e0d1c3356066a3786f23feb
+  data.tar.gz: 7e2a487f30a366182462c0301960d864dadfa17235721038d1e261062c79646fde4d16f1c08ce4ff4ad52bdccf648cf2c59b4aa0ab5a74c394d39c8c90fdc973

data/CHANGELOG.md

@@ -1,5 +1,8 @@
 # [develop](https://github.com/adhearsion/ruby_ami)
 
+# [1.3.4](https://github.com/adhearsion/ruby_ami/compare/v1.3.3...v1.3.4) - [2013-04-25](https://rubygems.org/gems/ruby_ami/versions/1.3.4)
+  * Bugfix: Handle AGI 5xx responses
+
 # [1.3.3](https://github.com/adhearsion/ruby_ami/compare/v1.3.2...v1.3.3) - [2013-04-09](https://rubygems.org/gems/ruby_ami/versions/1.3.3)
   * Bugfix: DBGet actions are now not terminated specially
 

data/lib/ruby_ami/agi_result_parser.rb

@@ -4,7 +4,7 @@ module RubyAMI
   class AGIResultParser
     attr_reader :code, :result, :data
 
-    FORMAT          = /^(?<code>\d{3}) result=(?<result>-?\d*) ?(?<data>\(?.*\)?)?$/.freeze
+    FORMAT          = /^(?<code>\d{3})( result=(?<result>-?\d*))? ?(?<data>\(?.*\)?)?$/.freeze
     DATA_KV_FORMAT  = /(?<key>[\w\d]+)=(?<value>[\w\d]*)/.freeze
     DATA_CLEANER    = /(^\()|(\)$)/.freeze
 
@@ -31,7 +31,7 @@ module RubyAMI
 
     def parse
       @code = match[:code].to_i
-      @result = match[:result].to_i
+      @result = match[:result] ? match[:result].to_i : nil
       @data = match[:data] ? match[:data].gsub(DATA_CLEANER, '').freeze : nil
     end
 

data/lib/ruby_ami/version.rb

@@ -1,4 +1,4 @@
 # encoding: utf-8
 module RubyAMI
-  VERSION = "1.3.3"
+  VERSION = "1.3.4"
 end

data/spec/ruby_ami/agi_result_parser_spec.rb

@@ -48,5 +48,14 @@ module RubyAMI
       its(:data)      { should == 'foo=bar' }
       its(:data_hash) { should == {'foo' => 'bar'} }
     end
+
+    context 'with a 5xx error' do
+      let(:result_string) { "510%20Invalid%20or%20unknown%20command%0A" }
+
+      its(:code)      { should == 510 }
+      its(:result)    { should be_nil }
+      its(:data)      { should == 'Invalid or unknown command' }
+      its(:data_hash) { should be_nil }
+    end
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: ruby_ami
 version: !ruby/object:Gem::Version
-  version: 1.3.3
+  version: 1.3.4
 platform: ruby
 authors:
 - Ben Langfeld
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-04-09 00:00:00.000000000 Z
+date: 2013-04-25 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: celluloid-io