api-client 2.0.0.rc1 → 2.0.0.rc2

data/.gitignore

@@ -15,3 +15,5 @@ pkg
 rdoc
 spec/reports
 tmp
+.*~
+*~

data/CHANGELOG.md

@@ -1,20 +1,26 @@
 # CHANGELOG
 
+## v2.0.0.rc1
+
+* Add Support for global url.
+* Add Support for instance requests.
+* Changed behavior of request methods to only accepts the resource id if needed..
+
 ## v1.10.0
 
-* Add Support for api responses with collections
+* Add Support for api responses with collections.
 
 ## v1.9.1
 
-* Fix for Net::Http on NotFound Exception
+* Fix for Net::Http on NotFound Exception.
 
 ## v1.9.0
 
-* Added Requested Url to NotFound Exception message
+* Added Requested Url to NotFound Exception message.
 
 ## v1.8.2
 
-* Fixed Query String Bug (It was been sent empty)
+* Fixed Query String Bug (It was been sent empty).
 
 ## v1.8.1
 
@@ -105,4 +111,4 @@
 ## v0.0.1
 
 * get method added.
-* handler for response code 404 added.
+* handler for response code 404 added.

data/Gemfile

@@ -1,4 +1,4 @@
 source 'https://rubygems.org'
 
 # Specify your gem's dependencies in api-client.gemspec
-gemspec
+gemspec

data/Rakefile

@@ -1,8 +1,8 @@
 #!/usr/bin/env rake
-require "bundler/gem_tasks"
-require "rspec/core/rake_task"
+require 'bundler/gem_tasks'
+require 'rspec/core/rake_task'
 
 RSpec::Core::RakeTask.new
 
-desc "Default Task"
+desc 'Default Task'
 task :default => [ :spec ]

data/api-client.gemspec

@@ -2,7 +2,7 @@
 require File.expand_path('../lib/api-client/version', __FILE__)
 
 Gem::Specification.new do |gem|
-  gem.name        = "api-client"
+  gem.name        = 'api-client'
   gem.version     = ApiClient::VERSION
   gem.authors     = %q{Paulo Henrique Lopes Ribeiro}
   gem.email       = %q{plribeiro3000@gmail.com}
@@ -13,12 +13,12 @@ Gem::Specification.new do |gem|
   gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   gem.require_paths = %w(lib)
 
-  gem.add_development_dependency "rake"
-  gem.add_development_dependency "webmock"
-  gem.add_development_dependency "rspec"
-  gem.add_development_dependency "yard"
-  gem.add_development_dependency "coveralls"
+  gem.add_development_dependency 'rake'
+  gem.add_development_dependency 'webmock'
+  gem.add_development_dependency 'rspec'
+  gem.add_development_dependency 'yard'
+  gem.add_development_dependency 'coveralls'
 
-  gem.add_runtime_dependency "activemodel"
-  gem.add_runtime_dependency "json_pure"
-end
+  gem.add_runtime_dependency 'activemodel'
+  gem.add_runtime_dependency 'json_pure'
+end

data/examples/config/initializers/api-client.rb

@@ -1,5 +1,5 @@
 # Global Configuration
 # Api path
 ApiClient.configure do |config|
-  config.path = "http://api.example.com"
+  config.path = 'http://api.example.com'
 end

data/examples/controllers/application_controller.rb

@@ -15,11 +15,11 @@ class ApplicationController < ActionController::Base
 
   def not_found
     ApplicationMailer.send_not_found_notice.deliver
-    render :file => "/public/404.html", :status => 404
+    render :file => '/public/404.html', :status => 404
   end
 
   def generic(exception)
     ApplicationMailer.send_generic_error(exception).deliver
-    render :file => "/public/404.html", :status => 404
+    render :file => '/public/404.html', :status => 404
   end
-end
+end

data/examples/controllers/user_controller.rb

@@ -15,7 +15,7 @@ class UserController < ApplicationController
 
   # It will hit http://api.example.com/users with a post request
   def create
-    @user = User.post("", :user => params[:user])
+    @user = User.post('', :user => params[:user])
     respond_with(@user)
   end
 
@@ -27,7 +27,7 @@ class UserController < ApplicationController
 
   # It will hit http://api.example.com/users/1 with a patch request
   def update
-    @user = User.patch("", { :id => params[:id], :user => params[:user] })
+    @user = User.patch(params[:id], { :user => params[:user] })
     respond_with(@user)
   end
 

data/examples/models/admin.rb

@@ -1,6 +1,6 @@
 class Admin < ApiClient::Base
-  self.remote_object = "user"
-  self.association = { :groups => "Group" }
+  self.remote_object = 'user'
+  self.association = { :groups => 'Group' }
 
   # Any of this fields can be called to manage rails form.
   attr_accessor :id, :email, :password, :password_confirmation

data/examples/models/group.rb

@@ -1,3 +1,3 @@
 class Group < ApiClient::Base
-  self.associations = { :members => "User", :owner => "Admin" }
+  self.associations = { :members => 'User', :owner => 'Admin' }
 end

data/examples/models/user.rb

@@ -1,5 +1,5 @@
 class User < ApiClient::Base
-  self.association = { :groups => "Group" }
+  self.association = { :groups => 'Group' }
 
   # Any of this fields can be called to manage rails form.
   attr_accessor :id, :email, :password, :password_confirmation

data/gemfiles/Gemfile.typhoeus

@@ -1,3 +1,5 @@
 source :rubygems
 
-gemspec :path => '../'
+gem 'typhoeus'
+
+gemspec :path => '../'

data/lib/api-client.rb

@@ -1,4 +1,4 @@
-require "api-client/version"
+require 'api-client/version'
 
 # High Level Namespace of the library ApiClient.
 module ApiClient

data/lib/api-client/base.rb

@@ -1,4 +1,4 @@
-require "active_model"
+require 'active_model'
 
 module ApiClient
   # ApiClient::Base provides a way to make easy api requests as well as making possible to use it inside rails.
@@ -27,7 +27,7 @@ module ApiClient
     # @return [Base] the object initialized.
     def initialize(attributes = {})
       @errors = ApiClient::Errors.new(self)
-      attributes.each do |name, value|
+      remove_root(attributes).each do |name, value|
         send("#{name.to_s}=", value)
       end
     end
@@ -43,7 +43,7 @@ module ApiClient
     #
     # @return [String] the api path for this object.
     def self.path
-      return self.to_s.gsub("::", "/").downcase.pluralize unless @path
+      return self.to_s.gsub('::', '/').downcase.pluralize unless @path
       @path
     end
 
@@ -51,7 +51,7 @@ module ApiClient
     #
     # @param [String] path string.
     def self.path=(path)
-      path = path[1, path.size - 1] if path[0, 1] == "/"
+      path = path[1, path.size - 1] if path[0, 1] == '/'
       @path = path
     end
 
@@ -76,7 +76,12 @@ module ApiClient
       associations.each do |association, class_name|
         class_eval <<-EVAL
           def #{association.to_s}=(attributes = {})
-            return @#{association.to_s} = attributes.map { |attr| #{class_name.constantize}.new(attr) } if attributes.instance_of?(Array)
+            if attributes.instance_of?(Array)
+              return @#{association.to_s} = attributes.map { |attr|
+                attr = remove_root(attr)
+                #{class_name.constantize}.new(attr)
+              }
+            end
             @#{association.to_s} = #{class_name.constantize}.new(attributes)
           end
           def #{association.to_s}
@@ -99,18 +104,18 @@ module ApiClient
       super
     end
 
-    # Return an array with all instance variables setted through attr_acessor.
+    # Return an array with all instance variables setted through attr_accessor.
     #
     # @return [Array] instance variables.
     def self.attributes
       @attributes
     end
 
-    # Return an array with all instance variables setted through attr_acessor and its currently values.
+    # Return an array with all instance variables setted through attr_accessor and its currently values.
     #
     # @return [Hash] instance variables and its values.
     def attributes
-      self.class.instance_variable_get("@attributes").inject({}) { |hash, attribute| hash.merge(attribute.to_sym => self.send("#{attribute}")) }
+      self.class.instance_variable_get('@attributes').inject({}) { |hash, attribute| hash.merge(attribute.to_sym => self.send("#{attribute}")) }
     end
 
     alias_method :to_hash, :attributes
@@ -126,9 +131,17 @@ module ApiClient
 
     # Set the hash of errors, making keys symbolic.
     #
-    # @param [Hash] errors of the object.
+    # @param [Hash] errs errors of the object.
     def errors=(errs = {})
       errors.add_errors(Hash[errs.map{|(key,value)| [key.to_sym,value]}])
     end
+
+    protected
+
+    def remove_root(attributes = {})
+      attributes = attributes[self.class.remote_object.to_sym] if attributes.key?(self.class.remote_object.to_sym)
+      attributes = attributes[self.class.remote_object.to_s] if attributes.key?(self.class.remote_object.to_s)
+      attributes
+    end
   end
-end
+end

data/lib/api-client/class_methods.rb

@@ -3,7 +3,7 @@ module ApiClient
   module ClassMethods
     # Initialize an object based on a hash of attributes.
     #
-    # @param [Hash] attributes hash of attributes.
+    # @param [Hash] params hash of attributes.
     # @return [Base] the object initialized.
     def build(params)
       if params.key?(remote_object)
@@ -26,5 +26,15 @@ module ApiClient
       params = ApiClient::Parser.response(response, url)
       build(params)
     end
+
+    # Overwrite respond_to? default behavior
+    #
+    # @param [Symbol] method_name the name of the method.
+    # @param [Boolean] include_private if it does work to private methods as well.
+    # @return [Boolean] if it responds to the method or not.
+    def respond_to_missing?(method_name, include_private = false)
+      return true if ApiClient::Dispatcher.respond_to?(method_name)
+      super
+    end
   end
 end

data/lib/api-client/collection.rb

@@ -6,8 +6,8 @@ class ApiClient::Collection
 
   # Initialize a collection of given objects
   #
-  # @param [Class] The class to instantiate the objects.
-  # @param [String] The url to get the data.
+  # @param [Class] klass The class to instantiate the objects.
+  # @param [String] resource_path The url to get the data.
   # @return [Collection] the collection of objects.
   def initialize(klass, resource_path)
     url = "#{ApiClient.config.path}#{resource_path}"

data/lib/api-client/configuration.rb

@@ -11,10 +11,10 @@ module ApiClient
 
     # Set the api url.
     #
-    # @param [String] api url.
+    # @param [String] path api url.
     def path=(path)
-      path = "#{path}/" unless path[path.size - 1, 1] == "/"
+      path = "#{path}/" unless path[path.size - 1, 1] == '/'
       @path = path
     end
   end
-end
+end

data/lib/api-client/dispatcher.rb

@@ -10,4 +10,15 @@ module ApiClient::Dispatcher
       NetHttp.send(method, *args)
     end
   end
-end
+
+  # Overwrite respond_to? default behavior
+  #
+  # @param [Symbol] method_name the name of the method.
+  # @param [Boolean] include_private if it does work to private methods as well.
+  # @return [Boolean] if it responds to the method or not.
+  def respond_to_missing?(method_name, include_private = false)
+    return true if Typhoeus.respond_to?(method_name)
+    return true if NetHttp.respond_to?(method_name)
+    super
+  end
+end

data/lib/api-client/dispatcher/net-http.rb

@@ -1,5 +1,5 @@
-require "net/http"
-require "api-client/net/http" unless Net::HTTP.new("").respond_to?(:patch)
+require 'net/http'
+require 'api-client/net/http' unless Net::HTTP.new('').respond_to?(:patch)
 
 # ApiClient::Dispatcher provides methods to make requests using the native ruby library 'net/http'
 module ApiClient::Dispatcher::NetHttp

data/lib/api-client/errors.rb

@@ -1,9 +1,10 @@
-require "active_model"
+require 'active_model'
 
 # ApiClient::Errors provide extra functionality to ActiveModel::Errors.
 class ApiClient::Errors < ActiveModel::Errors
-  # Add serveral errors from a hash to the object.
+  # Add several errors from a hash to the object.
   #
+  # @param [Hash] errors The hash with errors to add.
   # @return [ApiClient::Errors] The Error object.
   def add_errors(errors = {})
     errors.each_pair do |key, value|
@@ -21,6 +22,7 @@ class ApiClient::Errors < ActiveModel::Errors
   #   person = Person.create(address: '123 First St.')
   #   person.errors.unique_messages
   #   # => { :name => "is too short (minimum is 5 characters) and can't be blank", :address => nil, :email => "can't be blank" }
+  # @return [Hash] A hash with all the errors joined by attribute.
   def unique_messages
     errors = {}
     map { |attribute, messages| errors[attribute] = unique_message(attribute) }
@@ -37,8 +39,10 @@ class ApiClient::Errors < ActiveModel::Errors
   #   person = Person.create(address: '123 First St.')
   #   person.errors.unique_message(:name) # => "is too short (minimum is 5 characters) and can't be blank"
   #   person.errors.unique_message(:address) # => nil
+  # @param [String] attribute The attribute to check for joined error messages.
+  # @return [String] A string with all errors from the given attribute joined.
   def unique_message(attribute)
     return '' if messages[attribute].blank?
     [messages[attribute]].flatten.to_sentence
   end
-end
+end

data/lib/api-client/exceptions.rb

@@ -9,4 +9,4 @@ module ApiClient::Exceptions
   autoload :ServiceUnavailable, 'api-client/exceptions/service_unavailable'
   autoload :ConnectionRefused, 'api-client/exceptions/connection_refused'
   autoload :NotConfigured, 'api-client/exceptions/not_configured'
-end
+end

data/lib/api-client/exceptions/bad_gateway.rb

@@ -5,6 +5,6 @@ class ApiClient::Exceptions::BadGateway < ApiClient::Exceptions::Generic
   #
   # @return [BadGateway] a new exception.
   def self.initialize
-    super("Bad Gateway!")
+    super('Bad Gateway!')
   end
 end

data/lib/api-client/exceptions/connection_refused.rb

@@ -4,6 +4,6 @@ class ApiClient::Exceptions::ConnectionRefused < ApiClient::Exceptions::Generic
   #
   # @return [ConnectionRefused] a new exception.
   def self.initialize
-    super("Connection Refused!")
+    super('Connection Refused!')
   end
 end

data/lib/api-client/exceptions/forbidden.rb

@@ -5,6 +5,6 @@ class ApiClient::Exceptions::Forbidden < ApiClient::Exceptions::Generic
   #
   # @return [Forbidden] a new exception.
   def self.initialize
-    super("Forbidden!")
+    super('Forbidden!')
   end
 end

data/lib/api-client/exceptions/generic.rb

@@ -6,6 +6,6 @@ class ApiClient::Exceptions::Generic < StandardError
   #
   # @return [Generic] a new exception.
   def self.initialize
-    super("An Error Occurred!")
+    super('An Error Occurred!')
   end
 end

data/lib/api-client/exceptions/internal_server_error.rb

@@ -5,6 +5,6 @@ class ApiClient::Exceptions::InternalServerError < ApiClient::Exceptions::Generi
   #
   # @return [InternalServerError] a new exception.
   def self.initialize
-    super("Internal Server Errorr!")
+    super('Internal Server Error!')
   end
 end

data/lib/api-client/exceptions/not_configured.rb

@@ -4,6 +4,6 @@ class ApiClient::Exceptions::NotConfigured < StandardError
   #
   # @return [NotConfigured] a new exception.
   def self.initialize
-    super("The api path is not properly configured!")
+    super('The api path is not properly configured!')
   end
-end
+end

data/lib/api-client/exceptions/service_unavailable.rb

@@ -5,6 +5,6 @@ class ApiClient::Exceptions::ServiceUnavailable < ApiClient::Exceptions::Generic
   #
   # @return [ServiceUnavailable] a new exception.
   def self.initialize
-    super("Service Unavailable!")
+    super('Service Unavailable!')
   end
 end

data/lib/api-client/exceptions/unauthorized.rb

@@ -5,6 +5,6 @@ class ApiClient::Exceptions::Unauthorized < ApiClient::Exceptions::Generic
   #
   # @return [Unauthorized] a new exception.
   def self.initialize
-    super("Authentication Required!")
+    super('Authentication Required!')
   end
 end

data/lib/api-client/instance_methods.rb

@@ -3,7 +3,7 @@ module ApiClient
   module InstanceMethods
     # Update an object based on a hash of attributes.
     #
-    # @param [Hash] attributes hash of attributes.
+    # @param [Hash] params hash of attributes.
     # @return [Base] the updated object.
     def update(params)
       if params.key?(self.class.remote_object)
@@ -33,5 +33,15 @@ module ApiClient
       params = ApiClient::Parser.response(response, url)
       update(params)
     end
+
+    # Overwrite respond_to? default behavior
+    #
+    # @param [Symbol] method_name the name of the method.
+    # @param [Boolean] include_private if it does work to private methods as well.
+    # @return [Boolean] if it responds to the method or not.
+    def respond_to_missing?(method_name, include_private = false)
+      return true if ApiClient::Dispatcher.respond_to?(method_name)
+      super
+    end
   end
 end

data/lib/api-client/parser.rb

@@ -5,6 +5,7 @@ module ApiClient::Parser
   # Parse the JSON response.
   #
   # @param [HTTP] response HTTP object for the request.
+  # @param [String] url The url of the requisition.
   # @return [Hash] the body parsed.
   def self.response(response, url)
     raise_exception(response, url)
@@ -29,4 +30,4 @@ module ApiClient::Parser
       else return
     end
   end
-end
+end

data/lib/api-client/version.rb

@@ -1,5 +1,5 @@
 # High Level Namespace of the library ApiClient.
 module ApiClient
   # Version of the library.
-  VERSION = "2.0.0.rc1"
-end
+  VERSION = '2.0.0.rc2'
+end

data/spec/api-client/base_spec.rb

@@ -4,167 +4,191 @@ describe ApiClient::Base do
   let(:user) { User.new }
   let(:collection) { Object.new }
 
-  describe "#initialize" do
+  describe '.initialize' do
     context "with a hash {:a => 'a', :b => 'b'}" do
       before :each do
-        @user = User.new({:a => "a", :b => "b"})
+        @user = User.new({:a => 'a', :b => 'b'})
       end
 
-      it "should set #a" do
-        @user.a.should == "a"
+      it 'should set #a' do
+        @user.a.should == 'a'
       end
 
-      it "should set #b" do
-        @user.b.should == "b"
+      it 'should set #b' do
+        @user.b.should == 'b'
       end
 
-      it "should initialize errors" do
+      it 'should initialize errors' do
         @user.errors.should be_an_instance_of(ApiClient::Errors)
       end
     end
+
+    context "with a hash with string keys {'a' => 'a', 'b' => 'b' }" do
+      before :each do
+        @user = User.new({ 'user' => { 'a' => 'a', "b" => "b" } })
+      end
+
+      it 'should initialize attributes' do
+        @user.attributes.should == { :a => 'a', :b => 'b' }
+      end
+    end
+
+    context 'with a root node' do
+      before :each do
+        @user = User.new({ :user => { :a => 'a', :b => 'b' } })
+      end
+
+      it 'should initialize attributes' do
+        @user.attributes.should == { :a => 'a', :b => 'b' }
+      end
+    end
   end
 
-  describe "#path" do
-    describe "when not configured" do
-      it "should return a name based on the class name" do
-        User.path.should == "users"
+  describe '.path' do
+    describe 'when not configured' do
+      it 'should return a name based on the class name' do
+        User.path.should == 'users'
       end
     end
 
-    describe "when properly configured" do
+    describe 'when properly configured' do
       before :each do
-        User.path = "admins"
+        User.path = 'admins'
       end
 
-      it "should return the path value" do
-        User.path.should == "admins"
+      it 'should return the path value' do
+        User.path.should == 'admins'
       end
     end
   end
 
-  describe "#path=" do
+  describe '.path=' do
     describe "with a string without '/'" do
       before :each do
-        User.path = "users"
+        User.path = 'users'
       end
 
-      it "should set it as passed" do
-        User.path.should == "users"
+      it 'should set it as passed' do
+        User.path.should == 'users'
       end
     end
 
     describe "with a string with '/'" do
       before :each do
-        User.path = "/users"
+        User.path = '/users'
       end
 
       it "should set it without the '/'" do
-        User.path.should == "users"
+        User.path.should == 'users'
       end
     end
   end
 
-  describe "#remote_object" do
-    context "on a class without remote object specification" do
-      it "should return the class name" do
-        User.remote_object.should == "user"
+  describe '.remote_object' do
+    context 'on a class without remote object specification' do
+      it 'should return the class name' do
+        User.remote_object.should == 'user'
       end
     end
 
-    context "on a class with remote object specification" do
-      it "should return the class name" do
-        Admin.remote_object.should == "user"
+    context 'on a class with remote object specification' do
+      it 'should return the class name' do
+        Admin.remote_object.should == 'user'
       end
     end
   end
 
-  describe "#remote_object=" do
-    it "should set the remote object name" do
-      Admin.remote_object.should == "user"
+  describe '.remote_object=' do
+    it 'should set the remote object name' do
+      Admin.remote_object.should == 'user'
     end
   end
 
-  describe "#persisted?" do
-    it "should return false" do
+  describe '#persisted?' do
+    it 'should return false' do
       User.new.should_not be_persisted
     end
   end
 
-  describe "#associations" do
+  describe '.associations=' do
     before :each do
-      @group = Group.new(:members => [{:a => "a"}], :owner => {:b => "b"})
+      @group = Group.new(:members => [ :user => {:a => 'a'}], :owner => {:b => 'b'})
+    end
+
+    it 'should instantiate a collection of members' do
+      @group.members.should be_an_instance_of(Array)
     end
 
-    it "should instantiate all members" do
+    it 'should instantiate the member' do
       @group.members.first.should be_an_instance_of(User)
     end
 
-    it "should instantiate the owner" do
+    it 'should instantiate the owner' do
       @group.owner.should be_an_instance_of(Admin)
     end
   end
 
-  describe "#attributes on the class" do
-    it "should return an array of attributes" do
+  describe '.attributes' do
+    it 'should return an array of attributes' do
       User.attributes.should == [:a, :b]
     end
   end
 
-  describe "#attributes on the object" do
-    it "should return a hash with the attributes and currently values" do
-      User.new.attributes.should == {:a => nil, :b => nil}
+  describe '#attributes' do
+    it 'should return a hash with the attributes and currently values' do
+      User.new.attributes.should == { :a => nil, :b => nil }
     end
   end
 
-  describe "#collection" do
+  describe '.collection' do
     before :each do
-      ApiClient::Collection.stub(:new).with(User, "users").and_return(collection)
+      ApiClient::Collection.stub(:new).with(User, 'users').and_return(collection)
       collection.stub(:collection => [ user, user ])
     end
 
-    it "should return a collection of objects" do
+    it 'should return a collection of objects' do
       User.collection.should == [ user, user ]
     end
   end
 
-  describe "#errors=" do
+  describe '#errors=' do
     before :each do
-      @user = User.new(:errors => { "a" => "message", "b" => "message" })
+      @user = User.new(:errors => { 'a' => 'message', 'b' => 'message' })
     end
 
-    it "should set @errors with symbolic keys and messages as an array" do
+    it 'should set @errors with symbolic keys and messages as an array' do
       @user.errors.messages.should == { :a => %w(message), :b => %w(message) }
     end
   end
 
-  describe "requests" do
-    context "on the class" do
+  describe 'requests' do
+    context 'on the class' do
       before :each do
-        stub_request(:any, "http://api.example.com/users").to_return(:body => {"a" => "b"}.to_json)
+        stub_request(:any, 'http://api.example.com/users').to_return(:body => {'a' => 'b'}.to_json)
       end
 
-      it "should return a new instance" do
+      it 'should return a new instance' do
         User.get.should be_an_instance_of(User)
       end
 
-      it "should set the response on the instance" do
-        User.get.response.should == {"a" => "b"}
+      it 'should set the response on the instance' do
+        User.get.response.should == { 'a' => 'b' }
       end
     end
 
-    context "on an object" do
+    context 'on an object' do
       before :each do
-        stub_request(:any, "http://api.example.com/users").to_return(:body => {"a" => "b"}.to_json)
+        stub_request(:any, 'http://api.example.com/users').to_return(:body => {'a' => 'b'}.to_json)
         @user = User.new
       end
 
-      it "should return a new instance" do
+      it 'should return a new instance' do
         @user.get.should be_an_instance_of(User)
       end
 
-      it "should set the response on the instance" do
-        @user.get.response.should == {"a" => "b"}
+      it 'should set the response on the instance' do
+        @user.get.response.should == {'a' => 'b'}
       end
     end
   end
-end
+end

data/spec/api-client/collection_spec.rb

@@ -4,17 +4,17 @@ describe ApiClient::Collection do
   let(:user) { User.new }
 
   before :each do
-    stub_request(:get, "http://api.example.com/users").to_return(:body => [ {"a" => "b"}, {"a" => "b2"} ].to_json)
+    stub_request(:get, 'http://api.example.com/users').to_return(:body => [ { 'a' => 'b' }, { 'a' => 'b2' } ].to_json)
     User.stub(:new => user)
-    @collection = ApiClient::Collection.new(User, "users")
+    @collection = ApiClient::Collection.new(User, 'users')
   end
 
-  it "should include enumerable module" do
+  it 'should include enumerable module' do
     @collection.should respond_to(:first)
   end
 
-  describe "#initialize" do
-    it "Should initialize a collection of Objects" do
+  describe '.initialize' do
+    it 'Should initialize a collection of Objects' do
       @collection.collection.should == [user, user]
     end
   end

data/spec/api-client/configuration_spec.rb

@@ -1,51 +1,51 @@
 require 'spec_helper'
 
 describe ApiClient::Configuration do
-  describe "#path" do
-    describe "when not configured" do
+  describe '#path' do
+    describe 'when not configured' do
       before :each do
         ApiClient.configure do |config|
-          config.path = ""
+          config.path = ''
         end
       end
 
-      it "should raise an error" do
+      it 'should raise an error' do
         lambda { ApiClient.config.path }.should raise_error(ApiClient::Exceptions::NotConfigured)
       end
     end
 
-    describe "when properly configured" do
+    describe 'when properly configured' do
       before :each do
         ApiClient.configure do |config|
-          config.path = "http://api.example.com"
+          config.path = 'http://api.example.com'
         end
       end
 
-      it "should return the path value" do
+      it 'should return the path value' do
         ApiClient.config.path.should_not be_nil
       end
     end
   end
 
-  describe "#path=" do
+  describe '#path=' do
     describe "with a string without '/'" do
       before :each do
-        ApiClient.config.path = "http://api.example.com"
+        ApiClient.config.path = 'http://api.example.com'
       end
 
       it "should set it with a '/'" do
-        ApiClient.config.path.should == "http://api.example.com/"
+        ApiClient.config.path.should == 'http://api.example.com/'
       end
     end
 
     describe "with a string with '/'" do
       before :each do
-        ApiClient.config.path = "http://api.example.com/"
+        ApiClient.config.path = 'http://api.example.com/'
       end
 
       it "should set it as passed" do
-        ApiClient.config.path.should == "http://api.example.com/"
+        ApiClient.config.path.should == 'http://api.example.com/'
       end
     end
   end
-end
+end

data/spec/api-client/dispatcher_spec.rb

@@ -2,36 +2,36 @@ require 'spec_helper'
 
 describe ApiClient::Dispatcher do
   before :each do
-    stub_request(:any, "http://api.example.com/user/5").to_return(:body => "asd")
+    stub_request(:any, 'http://api.example.com/user/5').to_return(:body => 'asd')
   end
 
-  describe "#get" do
-    it "should return the request" do
-      ApiClient::Dispatcher.get("http://api.example.com/user/5", {}).body.should == ("asd")
+  describe '.get' do
+    it 'should return the request' do
+      ApiClient::Dispatcher.get('http://api.example.com/user/5', {}).body.should == ('asd')
     end
   end
 
-  describe "#post" do
-    it "should return the request" do
-      ApiClient::Dispatcher.post("http://api.example.com/user/5", {}, {}).body.should == ("asd")
+  describe '.post' do
+    it 'should return the request' do
+      ApiClient::Dispatcher.post('http://api.example.com/user/5', {}, {}).body.should == ('asd')
     end
   end
 
-  describe "#put" do
-    it "should return the request" do
-      ApiClient::Dispatcher.put("http://api.example.com/user/5", {}, {}).body.should == ("asd")
+  describe '.put' do
+    it 'should return the request' do
+      ApiClient::Dispatcher.put('http://api.example.com/user/5', {}, {}).body.should == ('asd')
     end
   end
 
-  describe "#patch" do
-    it "should return the request" do
-      ApiClient::Dispatcher.patch("http://api.example.com/user/5", {}, {}).body.should == ("asd")
+  describe '.patch' do
+    it 'should return the request' do
+      ApiClient::Dispatcher.patch('http://api.example.com/user/5', {}, {}).body.should == ('asd')
     end
   end if ApiClient::Dispatcher.respond_to?(:patch)
 
-  describe "#delete" do
-    it "should return the request" do
-      ApiClient::Dispatcher.delete("http://api.example.com/user/5", {}).body.should == ("asd")
+  describe '.delete' do
+    it 'should return the request' do
+      ApiClient::Dispatcher.delete('http://api.example.com/user/5', {}).body.should == ('asd')
     end
   end
 end

data/spec/api-client/errors_spec.rb

@@ -1,35 +1,35 @@
 require 'spec_helper'
 
 describe ApiClient::Errors do
-  describe "#add_errors" do
+  describe '#add_errors' do
     before :each do
-      @errors = ApiClient::Errors.new(self).add_errors({:a => 'b'})
+      @errors = ApiClient::Errors.new(self).add_errors({ :a => 'b' })
     end
 
-    it "should initialize a new object" do
+    it 'should initialize a new object' do
       @errors[:a].should == 'b'
     end
   end
 
-  describe "#unique_messages" do
+  describe '#unique_messages' do
     before :each do
       @user = User.new
       @user.valid?
     end
 
-    it "should set a unique message for each attribute" do
+    it 'should set a unique message for each attribute' do
       @user.errors.unique_messages.should == { :a => "can't be blank and is not included in the list" }
     end
   end
 
-  describe "#unique_message" do
+  describe '#unique_message' do
     before :each do
       @user = User.new
       @user.valid?
     end
 
-    it "should return a unique message for the given attribute" do
+    it 'should return a unique message for the given attribute' do
       @user.errors.unique_message(:a).should == "can't be blank and is not included in the list"
     end
   end
-end
+end

data/spec/api-client/parser_spec.rb

@@ -1,95 +1,95 @@
 require 'spec_helper'
 
 describe ApiClient::Parser do
-  describe "#response" do
-    context "with a valid json response" do
+  describe '#response' do
+    context 'with a valid json response' do
       before :each do
-        stub_request(:post, "http://api.example.com/user/5").to_return(:body => {:base => { :a => :b } }.to_json, :status => "201")
+        stub_request(:post, 'http://api.example.com/user/5').to_return(:body => {:base => { :a => :b } }.to_json, :status => '201')
         @response = ApiClient::Dispatcher.post('http://api.example.com/user/5', {}, {})
       end
 
-      it "should return the response code and the body parsed" do
-        ApiClient::Parser.response(@response, 'http://api.example.com/user/5').should == { "base" => { "a" => "b" } }
+      it 'should return the response code and the body parsed' do
+        ApiClient::Parser.response(@response, 'http://api.example.com/user/5').should == { 'base' => { 'a' => 'b' } }
       end
     end
 
-    context "with a invalid json response" do
+    context 'with a invalid json response' do
       before :each do
-        stub_request(:post, "http://api.example.com/user/5").to_return(:body => "wrong", :status => "201")
+        stub_request(:post, 'http://api.example.com/user/5').to_return(:body => 'wrong', :status => '201')
         @response = ApiClient::Dispatcher.post('http://api.example.com/user/5', {}, {})
       end
 
-      it "should return the response code and an empty hash" do
+      it 'should return the response code and an empty hash' do
         ApiClient::Parser.response(@response, 'http://api.example.com/user/5').should == {}
       end
     end
 
-    context "with a response code of" do
-      context "401" do
+    context 'with a response code of' do
+      context '401' do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 401)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 401)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a Unauthorized exception" do
+        it 'should raise a Unauthorized exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::Unauthorized)
         end
       end
 
-      context "403" do
+      context '403' do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 403)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 403)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a Forbidden exception" do
+        it 'should raise a Forbidden exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::Forbidden)
         end
       end
 
-      context "404" do
+      context '404' do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 404)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 404)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a NotFound exception" do
+        it 'should raise a NotFound exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::NotFound, "http://api.example.com/user/5")
         end
       end
 
-      context "500" do
+      context '500' do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 500)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 500)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a InternalServerError exception" do
+        it 'should raise a InternalServerError exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::InternalServerError)
         end
       end
 
-      context "502" do
+      context '502' do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 502)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 502)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a BadGateway exception" do
+        it 'should raise a BadGateway exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::BadGateway)
         end
       end
 
       context "503" do
         before :each do
-          stub_request(:get, "http://api.example.com/user/5").to_return(:status => 503)
+          stub_request(:get, 'http://api.example.com/user/5').to_return(:status => 503)
           @response = ApiClient::Dispatcher.get('http://api.example.com/user/5')
         end
 
-        it "should raise a ServiceUnavailable exception" do
+        it 'should raise a ServiceUnavailable exception' do
           lambda { ApiClient::Parser.response(@response, 'http://api.example.com/user/5') }.should raise_error(ApiClient::Exceptions::ServiceUnavailable)
         end
       end
     end
   end
-end
+end

data/spec/spec_helper.rb

@@ -1,11 +1,11 @@
-require "rspec"
-require "webmock/rspec"
-require "json"
-require "coveralls"
+require 'rspec'
+require 'webmock/rspec'
+require 'json'
+require 'coveralls'
 
 Coveralls.wear!
 
-require File.dirname(__FILE__) + "/../lib/api-client"
+require "#{File.dirname(__FILE__)}/../lib/api-client"
 
 class User < ApiClient::Base
   attr_accessor :a, :b
@@ -15,19 +15,19 @@ class User < ApiClient::Base
 end
 
 class Admin < ApiClient::Base
-  self.remote_object = "user"
+  self.remote_object = 'user'
 
   attr_accessor :a, :b
 end
 
 class Post < ApiClient::Base
-  self.association = { :writer => "User" }
+  self.association = { :writer => 'User' }
 
   attr_accessor :a
 end
 
 class Group < ApiClient::Base
-  self.associations = { :members => "User", :owner => "Admin" }
+  self.associations = { :members => 'User', :owner => 'Admin' }
 end
 
 ApiClient.configure do |config|

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: api-client
 version: !ruby/object:Gem::Version
-  version: 2.0.0.rc1
+  version: 2.0.0.rc2
   prerelease: 6
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-05 00:00:00.000000000 Z
+date: 2013-06-09 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -192,7 +192,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 3711980256019663374
+      hash: -2780574729308827888
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -205,5 +205,20 @@ rubygems_version: 1.8.25
 signing_key: 
 specification_version: 3
 summary: Client to make Api calls
-test_files: []
+test_files:
+- examples/config/initializers/api-client.rb
+- examples/controllers/application_controller.rb
+- examples/controllers/user_controller.rb
+- examples/models/admin.rb
+- examples/models/group.rb
+- examples/models/user.rb
+- gemfiles/Gemfile.net_http
+- gemfiles/Gemfile.typhoeus
+- spec/api-client/base_spec.rb
+- spec/api-client/collection_spec.rb
+- spec/api-client/configuration_spec.rb
+- spec/api-client/dispatcher_spec.rb
+- spec/api-client/errors_spec.rb
+- spec/api-client/parser_spec.rb
+- spec/spec_helper.rb
 has_rdoc: