return a useful message from RecordInvalid

fixes zendesk#111

Author: steved

lib/zendesk_api/error.rb

@@ -14,6 +14,10 @@ def initialize(response)
  @errors = {}
  end
  end
+
+ def to_s
+ "#{self.class.name}: #{@errors.to_s}"
+ end
  end
 
  class NetworkError < ClientError; end

spec/core/middleware/response/raise_error_spec.rb

@@ -24,8 +24,11 @@
  end
 
  context "status errors" do
+ let(:body) { "" }
+
  before(:each) do
- stub_request(:any, /.*/).to_return(:status => status)
+ stub_request(:any, /.*/).to_return(:status => status, :body => body,
+ :headers => { :content_type => "application/json" })
  end
 
  context "with status = 404" do
@@ -50,6 +53,21 @@
  it "should raise RecordInvalid" do
  expect { client.connection.get "/non_existent" }.to raise_error(ZendeskAPI::Error::RecordInvalid)
  end
+
+ context "with a body" do
+ let(:body) { MultiJson.dump(:details => "hello") }
+
+ it "should return RecordInvalid with proper message" do
+ begin
+ client.connection.get "/non_existent"
+ rescue ZendeskAPI::Error::RecordInvalid => e
+ e.errors.should == "hello"
+ e.to_s.should == "ZendeskAPI::Error::RecordInvalid: hello"
+ else
+ fail # didn't raise an error
+ end
+ end
+ end
  end
 
  context "with status = 200" do