tddium_client 0.0.3 → 0.0.4

data/.gitignore

@@ -4,6 +4,5 @@ pkg/*
 nbproject
 .DS_Store
 *.swp
-Gemfile.lock
 coverage/
 .rvmrc

data/Gemfile.lock

@@ -0,0 +1,44 @@
+PATH
+  remote: .
+  specs:
+    tddium_client (0.0.4)
+      httparty
+      json
+
+GEM
+  remote: http://rubygems.org/
+  specs:
+    crack (0.1.8)
+    diff-lcs (1.1.2)
+    fakefs (0.3.1)
+    fakeweb (1.3.0)
+    httparty (0.7.4)
+      crack (= 0.1.8)
+    json (1.5.1)
+    rack (1.2.2)
+    rack-test (0.5.7)
+      rack (>= 1.0)
+    rake (0.8.7)
+    rspec (2.5.0)
+      rspec-core (~> 2.5.0)
+      rspec-expectations (~> 2.5.0)
+      rspec-mocks (~> 2.5.0)
+    rspec-core (2.5.1)
+    rspec-expectations (2.5.0)
+      diff-lcs (~> 1.1.2)
+    rspec-mocks (2.5.0)
+    simplecov (0.4.1)
+      simplecov-html (~> 0.4.3)
+    simplecov-html (0.4.3)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  fakefs
+  fakeweb
+  rack-test
+  rake
+  rspec
+  simplecov
+  tddium_client!

data/Rakefile

@@ -9,3 +9,25 @@ RSpec::Core::RakeTask.new(:spec) do |t|
   t.rspec_opts = "--color"
 end
 task :default => :spec
+
+namespace :spec do
+  RUBY_VERSIONS = ["1.9.2-p180", "1.8.7-p302"]
+  GEMSET = "tddium_client"
+  desc "Runs the specs across Ruby 1.8.7 and 1.9.2"
+  task :xruby do
+    commands = []
+    gemsets = []
+    RUBY_VERSIONS.each do |ruby_version|
+      current_gemset = "ruby-#{ruby_version}@#{GEMSET}"
+      gemsets << current_gemset
+      commands << "rvm use #{ruby_version}" << "gem install bundler" << "bundle" << "rvm gemset create #{GEMSET}" <<
+                  "rvm use #{current_gemset}" << "gem install bundler --no-rdoc --no-ri" << "bundle"
+    end
+    puts ""
+    puts "Attempting to run the specs across ruby #{RUBY_VERSIONS.join(" and ")}..."
+    puts "If you get an error, try running the following commands to get your environment set up:"
+    puts commands.join(" && ")
+    puts ""
+    Kernel.exec("rvm #{gemsets.join(",")} rake spec")
+  end
+end

data/lib/tddium_client.rb

@@ -6,56 +6,94 @@ require 'rubygems'
 require 'httparty'
 require 'json'
 
-class TddiumClient
+module TddiumClient
   API_KEY_HEADER = "X-tddium-api-key"
   API_ERROR_TEXT = "An error occured: "
 
-  attr_accessor :environment
+  class Error < RuntimeError; end
+  class TimeoutError < TddiumClient::Error; end
+  
+  class APIError < TddiumClient::Error
+    attr_accessor :tddium_result
 
-  def initialize(env = :development)
-    self.environment = env
+    def initialize(result)
+      self.tddium_result = result
+    end
   end
 
-  def call_api(method, api_path, params = {}, api_key = nil, retries = 5, &block)
-    headers = { API_KEY_HEADER => api_key } if api_key
-
-    done = false
-    tries = 0
-    while (retries < 0 || tries <= retries) && !done
-      begin
-        http = HTTParty.send(method, tddium_uri(api_path), :body => params, :headers => headers)
-        done = true
-      rescue Timeout::Error
-      ensure
-        tries += 1
-      end
+  class ServerError < TddiumClient::Error
+    attr_accessor :http_code, :http_result
+
+    def initialize(http_code, http_result)
+      self.http_code = http_code
+      self.http_result = http_result
     end
+  end
 
-    raise Timeout::Error if tries > retries
+  class Result
+    attr_accessor :http_code, :http_result, :response
 
-    response = JSON.parse(http.body) rescue {}
+    def initialize(http_code, http_result, response)
+      self.http_code = http_code
+      self.http_result = http_result
+      self.response = response
+    end
 
-    if http.success?
-      (response["status"] == 0 && block_given?) ? yield(response) : message = API_ERROR_TEXT + response["explanation"].to_s
-    else
-      message = API_ERROR_TEXT + http.response.header.msg.to_s
-      message << " #{response["explanation"]}" if response["status"].to_i > 0
+    def success?
+      !response.nil? && response.is_a?(Hash) && response["status"] == 0
     end
-    [response["status"], http.code, message]
   end
 
-  private
+  class Client
+    attr_accessor :environment
 
-  def tddium_uri(path)
-    uri = URI.parse("")
-    uri.host = tddium_config["api"]["host"]
-    uri.port = tddium_config["api"]["port"]
-    uri.scheme = tddium_config["api"]["scheme"]
-    URI.join(uri.to_s, "#{tddium_config["api"]["version"]}/#{path}").to_s
-  end
+    def initialize(env = :development)
+      self.environment = env
+    end
+
+    def call_api(method, api_path, params = {}, api_key = nil, retries = 5)
+      headers = { API_KEY_HEADER => api_key } if api_key
+
+      done = false
+      tries = 0
+      while (retries < 0 || tries <= retries) && !done
+        begin
+          http = HTTParty.send(method, tddium_uri(api_path), :body => params, :headers => headers)
+          done = true
+        rescue Timeout::Error
+        ensure
+          tries += 1
+        end
+      end
+
+      raise TimeoutError if tries > retries
 
-  def tddium_config
-   @tddium_config = YAML.load(File.read(File.join(File.dirname(__FILE__), "..", "config", "environment.yml")))[environment.to_s] unless @tddium_config
-   @tddium_config
+      response = JSON.parse(http.body) rescue {}
+
+      http_message = http.response.header.msg.to_s
+
+      raise ServerError.new(http.code, http_message) unless response.is_a?(Hash) && response.include?("status")
+      
+      result = Result.new(http.code, http.response.header.msg.to_s, response)
+
+      raise APIError.new(result) if !result.success?
+
+      result
+    end
+
+    private
+
+    def tddium_uri(path)
+      uri = URI.parse("")
+      uri.host = tddium_config["api"]["host"]
+      uri.port = tddium_config["api"]["port"]
+      uri.scheme = tddium_config["api"]["scheme"]
+      URI.join(uri.to_s, "#{tddium_config["api"]["version"]}/#{path}").to_s
+    end
+
+    def tddium_config
+     @tddium_config = YAML.load(File.read(File.join(File.dirname(__FILE__), "..", "config", "environment.yml")))[environment.to_s] unless @tddium_config
+     @tddium_config
+    end
   end
 end

data/lib/tddium_client/version.rb

@@ -2,6 +2,6 @@
 Copyright (c) 2011 Solano Labs All Rights Reserved
 =end
 
-module TddiumClientVersion
-  VERSION = "0.0.3"
+module TddiumClient
+  VERSION = "0.0.4"
 end

data/spec/tddium_client_spec.rb

@@ -4,202 +4,229 @@ Copyright (c) 2011 Solano Labs All Rights Reserved
 
 require "spec_helper"
 
-describe TddiumClient do
-  include FakeFS::SpecHelpers
-  include TddiumSpecHelpers
-
-  EXAMPLE_HTTP_METHOD = :post
-  EXAMPLE_TDDIUM_RESOURCE = "suites"
-  EXAMPLE_PARAMS = {"key" => "value"}
-  EXAMPLE_API_KEY = "afb12412bdafe124124asfasfabebafeabwbawf1312342erbfasbb"
-
-  def stub_http_response(method, path, options = {})
-    uri = api_uri(path)
-    FakeWeb.register_uri(method, uri, register_uri_options(options))
-  end
+module TddiumClient
+  describe Result do
+    describe "#success?" do
+      context "with successful params" do
+        it "should be true" do
+          result = TddiumClient::Result.new 200, "OK", {"status" => 0}
+          result.should be_success
+        end
+      end
+
+      context "with unsuccessful params" do
+        it "should handle no response" do
+          result = TddiumClient::Result.new 200, "OK", nil
+          result.should_not be_success
+        end
 
-  def parse_request_params
-    Rack::Utils.parse_nested_query(FakeWeb.last_request.body)
+        it "should handle 5xx" do
+          result = TddiumClient::Result.new 501, "Internal Server Error", {"status" => 1}
+          result.should_not be_success
+        end
+      end
+    end
   end
 
-  let(:tddium_client) { TddiumClient.new }
+  describe Client do
+    include FakeFS::SpecHelpers
+    include TddiumSpecHelpers
 
-  it "should set the default environment to :development" do
-    tddium_client.environment.should == :development
-  end
+    EXAMPLE_HTTP_METHOD = :post
+    EXAMPLE_TDDIUM_RESOURCE = "suites"
+    EXAMPLE_PARAMS = {"key" => "value"}
+    EXAMPLE_API_KEY = "afb12412bdafe124124asfasfabebafeabwbawf1312342erbfasbb"
 
-  describe "#environment" do
-    it "should set the environment" do
-      tddium_client.environment = :production
-      tddium_client.environment.should == :production
+    def stub_http_response(method, path, options = {})
+      uri = api_uri(path)
+      FakeWeb.register_uri(method, uri, register_uri_options(options))
     end
-  end
 
-  describe "#call_api" do
-    before do
-      FakeWeb.clean_registry
-      stub_tddium_client_config
-      stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+    def parse_request_params
+      Rack::Utils.parse_nested_query(FakeWeb.last_request.body)
     end
 
-    context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}')" do
-      it "should make a '#{EXAMPLE_HTTP_METHOD.to_s.upcase}' request to the api" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
-        FakeWeb.last_request.method.downcase.to_sym.should == EXAMPLE_HTTP_METHOD
-      end
+    let(:tddium_client) { TddiumClient::Client.new }
 
-      it "should make a request to the correct resource" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
-        FakeWeb.last_request.path.should =~ /#{EXAMPLE_TDDIUM_RESOURCE}$/
-      end
+    it "should set the default environment to :development" do
+      tddium_client.environment.should == :development
     end
 
-    context "raises an error" do
-      before do
-        HTTParty.stub(EXAMPLE_HTTP_METHOD).and_raise(Timeout::Error)
+    describe "#environment" do
+      it "should set the environment" do
+        tddium_client.environment = :production
+        tddium_client.environment.should == :production
       end
+    end
 
-      it "should retry 5 times by default to contact the API" do
-        HTTParty.should_receive(EXAMPLE_HTTP_METHOD).exactly(6).times
-        expect { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE) }.to raise_error(Timeout::Error)
+    describe "#call_api" do
+      before(:each) do
+        FakeWeb.clean_registry
+        stub_tddium_client_config
+        stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :response => fixture_path("post_suites_201.json"))
       end
 
-      it "should retry as many times as we want to contact the API" do
-        HTTParty.should_receive(EXAMPLE_HTTP_METHOD).exactly(3).times
-        expect { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil, 2) }.to raise_error(Timeout::Error)
+      context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}')" do
+        it "should make a '#{EXAMPLE_HTTP_METHOD.to_s.upcase}' request to the api" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+          FakeWeb.last_request.method.downcase.to_sym.should == EXAMPLE_HTTP_METHOD
+        end
+
+        it "should make a request to the correct resource" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+          FakeWeb.last_request.path.should =~ /#{EXAMPLE_TDDIUM_RESOURCE}$/
+        end
       end
-    end
 
-    context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}', {}, #{EXAMPLE_API_KEY}) # with api_key" do
-      it "should include #{TddiumClient::API_KEY_HEADER}=#{EXAMPLE_API_KEY} in the request headers" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, EXAMPLE_API_KEY)
-        FakeWeb.last_request[TddiumClient::API_KEY_HEADER].should == EXAMPLE_API_KEY
+      context "raises a timeout error" do
+        before do
+          HTTParty.stub(EXAMPLE_HTTP_METHOD).and_raise(Timeout::Error)
+        end
+
+        it "should retry 5 times by default to contact the API" do
+          HTTParty.should_receive(EXAMPLE_HTTP_METHOD).exactly(6).times
+          expect { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE) }.to raise_error(TddiumClient::TimeoutError)
+        end
+
+        it "should retry as many times as we want to contact the API" do
+          HTTParty.should_receive(EXAMPLE_HTTP_METHOD).exactly(3).times
+          expect { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil, 2) }.to raise_error(TddiumClient::TimeoutError)
+        end
       end
-    end
 
-    context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}') # without api_key" do
-      it "should not include #{TddiumClient::API_KEY_HEADER} in the request headers" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {})
-        FakeWeb.last_request[TddiumClient::API_KEY_HEADER].should be_nil
+      context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}', {}, #{EXAMPLE_API_KEY}) # with api_key" do
+        it "should include #{TddiumClient::API_KEY_HEADER}=#{EXAMPLE_API_KEY} in the request headers" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, EXAMPLE_API_KEY)
+          FakeWeb.last_request[TddiumClient::API_KEY_HEADER].should == EXAMPLE_API_KEY
+        end
       end
-    end
 
-    context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}', #{EXAMPLE_PARAMS}) # with params" do
-      it "should include #{EXAMPLE_PARAMS} in the request params" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, EXAMPLE_PARAMS)
-        parse_request_params.should include(EXAMPLE_PARAMS)
+      context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}') # without api_key" do
+        it "should not include #{TddiumClient::API_KEY_HEADER} in the request headers" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {})
+          FakeWeb.last_request[TddiumClient::API_KEY_HEADER].should be_nil
+        end
       end
-    end
 
-    context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}') # without params" do
-      it "should not include any request params" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
-        parse_request_params.should == {}
+      context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}', #{EXAMPLE_PARAMS}) # with params" do
+        it "should include #{EXAMPLE_PARAMS} in the request params" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, EXAMPLE_PARAMS)
+          parse_request_params.should include(EXAMPLE_PARAMS)
+        end
       end
-    end
-    
-    context "results in a successful response" do
-      before do
-        stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :response => fixture_path("post_suites_201.json"))
+
+      context "('#{EXAMPLE_HTTP_METHOD}', '#{EXAMPLE_TDDIUM_RESOURCE}') # without params" do
+        it "should not include any request params" do
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+          parse_request_params.should == {}
+        end
       end
 
-      context "called with a block" do
-        let(:dummy_block) { Proc.new { |response| @parsed_http_response = response } }
+      context "results in a successful response" do
+        before do
+          stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :response => fixture_path("post_suites_201.json"))
+        end
 
         it "should try to contact the api only once" do
           HTTParty.should_receive(EXAMPLE_HTTP_METHOD).exactly(1).times.and_return(mock(HTTParty).as_null_object)
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil, &dummy_block)
+          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil) rescue {}
         end
 
-        it "should parse the JSON response" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil, &dummy_block)
-          @parsed_http_response.should be_a(Hash)
-          @parsed_http_response["status"].should == 0
-          @parsed_http_response["suite"]["id"].should == 19
+        it "should return a TddiumClient::Result" do
+          result = tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil)
+          result.should be_a(TddiumClient::Result)
         end
 
-        it "should return a triple with element[0]==json_status, element[1]==http_status, element[2]==error_message" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil, &dummy_block).should == [0, 201, nil]
+        it "should parse the JSON response" do
+          result = tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, {}, nil)
+          result.http_code.should == 201
+          result.response.should be_a(Hash)
+          result.response["status"].should == 0
+          result.should be_success
+          result.response["suite"]["id"].should == 19
         end
       end
 
-      context "called without a block" do
-        it "should not yield" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+      context "exceptions for an unprocessable response" do
+        before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :status => ["501", "Internal Server Error"]) }
+        it "should raise a TddiumClient::ServerError" do
+          expect do
+            tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+          end.to raise_error do |error|
+            puts error.inspect
+            error.should be_a(TddiumClient::ServerError)
+            error.should respond_to(:http_code)
+            error.http_code.should == 500
+            error.http_message.should =~ /Internal Server Error/
+          end
         end
-      end
-    end
-
-    context "results for an unsuccessful response" do
-      before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :status => ["404", "Not Found"]) }
 
-      shared_examples_for "returning that an error occured" do
-        it "should return that an error occured in the third element" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[2].should =~ /^#{TddiumClient::API_ERROR_TEXT}/
+        context "when no response is present" do
+          before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :status => ["200", "OK"]) }
+          it "should raise a TddiumClient::ServerError" do
+            expect do
+              tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+            end.to raise_error do |error|
+              error.should be_a(TddiumClient::ServerError)
+              error.should respond_to(:http_code)
+              error.http_code.should == 200
+              error.http_message.should =~ /OK/
+            end
+          end
         end
       end
 
-      shared_examples_for("returning the API error") do
-        it "should return the API error message in the third element" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[2].should =~ /\{\:suite_name\=\>\[\"has already been taken\"\]\}$/
+      shared_examples_for "raising an APIError" do
+        # users can set:
+        #
+        # aproc
+        # http_code
+        # http_message
+        it "should raise the right exception" do
+          expect { aproc.call }.to raise_error do |error|
+            error.should be_a(TddiumClient::APIError)
+            error.should respond_to(:tddium_result)
+            error.tddium_result.should be_a(TddiumClient::Result)
+          end
         end
-      end
 
-      shared_examples_for("returning the HTTP status code") do
-        it "should return the HTTP status code in the second element" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[1].should == http_status_code
+        it "should capture the http response line" do
+          expect { aproc.call }.to raise_error do |error|
+            error.tddium_result.http_code.should == http_code if http_code
+            error.tddium_result.http_message.should == http_message if http_message
+          end
         end
       end
 
-      it "should return an array with three elements" do
-        tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE).size.should == 3
-      end
-      
       context "where the http request was successful but API status is not 0" do
         before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :response => fixture_path("post_suites_269_json_status_1.json")) }
-        it_should_behave_like("returning that an error occured")
-        it_should_behave_like("returning the API error")
-
-        it "should return the API error code in the first element" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[0].should == 1
-        end
 
-        it_should_behave_like("returning the HTTP status code") do
-          let(:http_status_code) {269}
+        it_should_behave_like "raising an APIError" do
+          let(:aproc) { Proc.new { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE) } }
+          let(:http_code) {269}
         end
       end
 
-      context "where the http request was unsuccessful" do
-        before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :status => ["501", "Internal Server Error"]) }
-        it_should_behave_like("returning that an error occured")
-
-        it_should_behave_like("returning the HTTP status code") do
-          let(:http_status_code) {501}
-        end
-
-        it "should return the HTTP error message in the third element" do
-          tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[2].should =~ /Internal Server Error/
+      context "exceptions for an error response" do
+        before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :status => ["404", "Not Found"]) }
+        it_should_behave_like "raising an APIError" do
+          let(:aproc) { Proc.new { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE) } }
+          let(:http_code) {404}
         end
 
         context "and an API error is returned" do
           before { stub_http_response(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE, :response => fixture_path("post_suites_409.json")) }
-          it_should_behave_like("returning the API error")
-          it "should return the API error code in the first element" do
-            tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[0].should == 1
+          it_should_behave_like "raising an APIError" do
+            let(:aproc) { Proc.new { tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE) } }
+            let(:http_code) {409}
           end
 
-          it_should_behave_like("returning the HTTP status code") do
-            let(:http_status_code) {409}
-          end
-        end
-
-        context "and no API error is returned" do
-          it "should return a nil API error code in the first element" do
-            tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)[0].should be_nil
-          end
-          
-          it_should_behave_like("returning the HTTP status code") do
-            let(:http_status_code) {501}
+          it "should have an api status" do
+            expect do
+              tddium_client.call_api(EXAMPLE_HTTP_METHOD, EXAMPLE_TDDIUM_RESOURCE)
+            end.to raise_error do |error|
+              error.tddium_result.response[:status].should == 1
+            end
           end
         end
       end

data/tddium_client.gemspec

@@ -8,7 +8,7 @@ require "tddium_client/version"
 
 Gem::Specification.new do |s|
   s.name        = "tddium_client"
-  s.version     = TddiumClientVersion::VERSION
+  s.version     = TddiumClient::VERSION
   s.platform    = Gem::Platform::RUBY
   s.authors     = ["Jay Moorthi"]
   s.email       = ["info@tddium.com"]
@@ -31,4 +31,5 @@ Gem::Specification.new do |s|
   s.add_development_dependency("fakefs")
   s.add_development_dependency("rack-test")
   s.add_development_dependency("simplecov")
+  s.add_development_dependency("rake")
 end

metadata

@@ -1,13 +1,8 @@
 --- !ruby/object:Gem::Specification 
 name: tddium_client
 version: !ruby/object:Gem::Version 
-  hash: 25
-  prerelease: false
-  segments: 
-  - 0
-  - 0
-  - 3
-  version: 0.0.3
+  prerelease: 
+  version: 0.0.4
 platform: ruby
 authors: 
 - Jay Moorthi
@@ -15,7 +10,7 @@ autorequire:
 bindir: bin
 cert_chain: []
 
-date: 2011-03-18 00:00:00 -07:00
+date: 2011-03-25 00:00:00 -07:00
 default_executable: 
 dependencies: 
 - !ruby/object:Gem::Dependency 
@@ -26,9 +21,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :runtime
   version_requirements: *id001
@@ -40,9 +32,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :runtime
   version_requirements: *id002
@@ -54,9 +43,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :development
   version_requirements: *id003
@@ -68,9 +54,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :development
   version_requirements: *id004
@@ -82,9 +65,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :development
   version_requirements: *id005
@@ -96,9 +76,6 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :development
   version_requirements: *id006
@@ -110,12 +87,20 @@ dependencies:
     requirements: 
     - - ">="
       - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
         version: "0"
   type: :development
   version_requirements: *id007
+- !ruby/object:Gem::Dependency 
+  name: rake
+  prerelease: false
+  requirement: &id008 !ruby/object:Gem::Requirement 
+    none: false
+    requirements: 
+    - - ">="
+      - !ruby/object:Gem::Version 
+        version: "0"
+  type: :development
+  version_requirements: *id008
 description: Internal Gem used to call the Tddium API
 email: 
 - info@tddium.com
@@ -128,6 +113,7 @@ extra_rdoc_files: []
 files: 
 - .gitignore
 - Gemfile
+- Gemfile.lock
 - README
 - Rakefile
 - config/environment.yml
@@ -154,23 +140,17 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
       version: "0"
 required_rubygems_version: !ruby/object:Gem::Requirement 
   none: false
   requirements: 
   - - ">="
     - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
       version: "0"
 requirements: []
 
 rubyforge_project: tddium_client
-rubygems_version: 1.3.7
+rubygems_version: 1.5.2
 signing_key: 
 specification_version: 3
 summary: tddium Client Gem