angus-remote 0.0.1 → 0.0.2

data/spec/angus/remote/response/builder_spec.rb

@@ -0,0 +1,69 @@
+require 'spec_helper'
+
+require 'angus/remote/response/builder'
+
+require 'angus/remote/remote_response'
+
+describe Angus::Remote::Response::Builder do
+
+  subject(:builder) { Angus::Remote::Response::Builder }
+
+  let(:raw_response) { { 'user' => {} } }
+  let(:response_class) { builder.build_response_class('Get user') }
+  let(:response) { Angus::Remote::RemoteResponse.new() }
+  let(:email_field) { double(:email_field, :name => :email, :required => true, :type => 'string',
+                             :elements_type => nil) }
+  let(:representations_hash) { { 'user' => double(:user_rep, :fields => [email_field]) } }
+  let(:glossary_terms_hash) { {} }
+  let(:element) { double(:element, :name => 'user', :required => true, :type => 'user') }
+
+  describe '.build_response_method' do
+
+    it 'adds the method to the response class' do
+      builder.build_response_method(raw_response, response_class, response, representations_hash,
+                                    glossary_terms_hash, element)
+
+      response_class.new.should respond_to(:user)
+    end
+
+  end
+
+  describe '.build_from_representation' do
+
+    context 'hash_value is nil' do
+      it 'should return nil' do
+        Angus::Remote::Response::Builder.build_from_representation(
+          nil,
+          double(:type),
+          double(:representations),
+          double(:glossary_terms_hash)
+        ).should be_nil
+      end
+    end
+  end
+
+  describe '.build_response_class' do
+    let(:operation_name) { 'foo' }
+
+    it 'should return a client class' do
+      response_class = Angus::Remote::Response::Builder.build_response_class(operation_name)
+
+      response = response_class.new
+
+      response.should be_kind_of(Angus::Remote::RemoteResponse)
+    end
+
+    it 'the response_class#name should include the operation\'s name' do
+      response_class = Angus::Remote::Response::Builder.build_response_class(operation_name)
+
+      response_class.name.should include(operation_name)
+    end
+
+    it 'the response_class#to_s should include the operation\'s name' do
+      response_class = Angus::Remote::Response::Builder.build_response_class(operation_name)
+
+      response_class.to_s.should include(operation_name)
+    end
+  end
+
+end

data/spec/angus/remote/service_directory_spec.rb

@@ -0,0 +1,76 @@
+require 'spec_helper'
+
+require 'angus/remote/service_directory'
+
+describe Angus::Remote::ServiceDirectory do
+
+  subject(:service_directory) { Angus::Remote::ServiceDirectory }
+
+  let(:code_name) { 'vpos' }
+  let(:version) { '0.1' }
+  let(:doc_url) { 'some_url/doc' }
+  let(:api_url) { 'some_url/api' }
+
+  before do
+    Angus::Remote::ServiceDirectory.stub(:service_configuration => { "v#{version}" => {
+      'doc_url' => doc_url, 'api_url' => api_url }
+    })
+  end
+
+  describe '.get_service_definition' do
+
+    let(:service_definition){ double(:service_definition)}
+
+    context 'when a file url' do
+      let(:doc_url) { 'file://path/to/doc' }
+
+      before do
+        Angus::SDoc::DefinitionsReader.stub(:service_definition => service_definition)
+      end
+
+      it 'builds the service definition from the path' do
+        Angus::SDoc::DefinitionsReader.should_receive(
+          :service_definition
+        ).with('path/to/doc')
+
+        service_directory.get_service_definition(code_name, version)
+      end
+
+      it 'returns the service definition' do
+        service_directory.get_service_definition(code_name, version).should eq(service_definition)
+      end
+    end
+
+    context 'when a remote url' do
+      let(:doc_url) { 'some_url/doc' }
+      let(:definition_hash) { {} }
+
+      before do
+        service_directory.stub(:fetch_remote_service_definition => definition_hash)
+        Angus::SDoc::DefinitionsReader.stub(:build_service_definition => service_definition)
+      end
+
+      it 'gets the definition hash from the remote service' do
+        service_directory.should_receive(:fetch_remote_service_definition).with(
+          doc_url
+        ).and_return(definition_hash)
+
+        service_directory.get_service_definition(code_name, version)
+      end
+
+      it 'builds the service definition from the definition hash' do
+        Angus::SDoc::DefinitionsReader.should_receive(
+          :build_service_definition
+        ).with(definition_hash)
+
+        service_directory.get_service_definition(code_name, version)
+      end
+
+      it 'returns the service definition' do
+        service_directory.get_service_definition(code_name, version).should eq(service_definition)
+      end
+    end
+
+  end
+
+end

data/spec/angus/remote/utils_spec.rb

@@ -0,0 +1,204 @@
+require 'json'
+
+require 'fakefs/spec_helpers'
+
+require 'angus/remote/utils'
+
+describe Angus::Remote::Utils do
+
+  include FakeFS::SpecHelpers
+
+  subject(:utils) { Angus::Remote::Utils }
+
+  describe '.build_request' do
+
+    let(:request) { utils.build_request(:get, '/listing', :q => 'spec') }
+
+    it 'returns a Net::HTTPRequest object' do
+      request.should be_kind_of(Net::HTTPRequest)
+    end
+
+    context 'when not encoding as json' do
+      subject { utils.build_request(:get, '/listing', {:q => 'spec'}, false) }
+
+      describe 'the built request' do
+
+        its(:method) { should eq('GET') }
+        its(:path) { should eq('/listing?q=spec') }
+
+      end
+    end
+
+    context 'when encoding as json' do
+      subject { utils.build_request(:get, '/listing', {:q => 'spec'}, true) }
+
+      describe 'the built request' do
+
+        its(:method) { should eq('GET') }
+        its(:path) { should eq('/listing') }
+        its(:body) { should eq(JSON({:q => 'spec'})) }
+
+      end
+    end
+
+  end
+
+
+  describe '.build_base_request' do
+
+    let(:path) { '/' }
+
+    context 'when invalid http method' do
+      it 'raises MethodArgumentError' do
+        expect {
+          utils.build_base_request(:invalid, path)
+        }.to raise_error(Angus::Remote::MethodArgumentError)
+      end
+    end
+
+    shared_examples 'a client builder' do |method, multipart, kind_of|
+      context "when #{method}, multipart = #{multipart}" do
+        it "returns a kind_of #{kind_of}" do
+          res = utils.build_base_request(method, path, multipart)
+
+          res.should be_a(kind_of)
+        end
+      end
+    end
+
+    it_behaves_like 'a client builder', :get,     false,  Net::HTTP::Get
+    it_behaves_like 'a client builder', :post,    false,  Net::HTTP::Post
+    it_behaves_like 'a client builder', :post,    true,   Http::MultipartMethods::Post
+    it_behaves_like 'a client builder', :put,     false,  Net::HTTP::Put
+    it_behaves_like 'a client builder', :put,     true,   Http::MultipartMethods::Put
+    it_behaves_like 'a client builder', :delete,  false,  Net::HTTP::Delete
+  end
+
+  describe '.severe_error_response?' do
+    shared_examples 'a status checker' do |code|
+      it "is true when code = #{code}" do
+        response = double(:response, :code => code)
+        utils.severe_error_response?(response).should be
+      end
+    end
+
+    it_behaves_like 'a status checker', 500
+    it_behaves_like 'a status checker', 501
+    it_behaves_like 'a status checker', 503
+  end
+
+  describe '.build_path' do
+    let(:path) { '/users/:user_id/profile/:profile_id' }
+
+    it 'buils a path using the given params' do
+      path_params = [4201, 2]
+
+      res = utils.build_path(path, path_params)
+
+      res.should eq('/users/4201/profile/2')
+    end
+
+    it 'raises a PathArgumentError when received more args than needed' do
+      path_params = [:more, :args, :than, :needed]
+
+      expect {
+        utils.build_path(path, path_params)
+      }.to raise_error(Angus::Remote::PathArgumentError)
+    end
+
+    it 'raises a PathArgumentError when received less args than needed' do
+      path_params = [:less]
+
+      expect {
+        utils.build_path(path, path_params)
+      }.to raise_error(Angus::Remote::PathArgumentError)
+    end
+  end
+
+  describe '.build_normal_request' do
+
+    subject { utils.build_normal_request(:get, '/listing', {:q => 'spec'}) }
+
+    shared_examples 'a method without body' do |method|
+      describe 'the built request' do
+        subject { utils.build_normal_request(method, '/listing', {:q => 'spec'}) }
+
+        its(:method) { should eq(method.upcase) }
+        its(:path) { should eq('/listing?q=spec') }
+        its(:body) { should be_nil }
+
+      end
+    end
+
+    shared_examples 'a method with body' do |method|
+      describe 'the built request' do
+        subject { utils.build_normal_request(method, '/listing', {:q => 'spec'}) }
+
+        its(:method) { should eq(method.upcase) }
+        its(:path) { should eq('/listing') }
+        its(:body) { should eq('q=spec') }
+
+      end
+    end
+
+    it_behaves_like 'a method without body', 'get'
+    it_behaves_like 'a method without body', 'delete'
+    it_behaves_like 'a method with body', 'post'
+    it_behaves_like 'a method with body', 'put'
+
+    context 'a multipart request' do
+      let(:file_name) { 'some_file.txt' }
+
+      let(:file) { File.new(file_name) }
+      let(:tempfile) { Tempfile.new('tempfile', temp_dir) }
+      let(:temp_dir) { '/tmp' }
+      let(:some_upload_io) { UploadIO.new(file, 'application/octet-stream') }
+
+      let(:request) { utils.build_normal_request('post', '/files', {:file => some_upload_io}) }
+
+      before do
+        Dir.mkdir('/tmp')
+
+        File.new(file_name, 'w')
+      end
+
+      describe 'the built request' do
+        subject { request }
+
+        it 'is of class Http::MultipartMethods::MultipartBase' do
+          should be_kind_of(Http::MultipartMethods::MultipartBase)
+        end
+
+        its(:method) { should eq('POST') }
+        its(:path) { should eq('/files') }
+        its(:body) { should be_nil }
+
+      end
+    end
+  end
+
+  describe '.build_json_request' do
+
+    let(:method) { :post }
+    let(:path)   { '/' }
+    let(:params) { [0, 1, 2] }
+
+    let(:request) do
+      utils.build_json_request(method, path, params)
+    end
+
+    describe 'the returned request' do
+
+      it 'its Content-Type header = application/json' do
+        request['Content-Type'].should eq('application/json')
+      end
+
+      it 'its body = JSON encoded params' do
+        request.body.should eq(JSON(params))
+      end
+
+    end
+
+  end
+
+end

metadata

@@ -1,86 +1,246 @@
 --- !ruby/object:Gem::Specification
 name: angus-remote
 version: !ruby/object:Gem::Version
-  prerelease:
-  version: 0.0.1
+  version: 0.0.2
+  prerelease: 
 platform: ruby
 authors:
 - Adrian Gomez
-autorequire:
+- Gianfranco Zas
+autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-10-10 00:00:00.000000000 Z
+date: 2013-10-31 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
-  name: bundler
-  version_requirements: !ruby/object:Gem::Requirement
+  name: angus-sdoc
+  requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '0.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.0'
+- !ruby/object:Gem::Dependency
+  name: persistent_http
   requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.0'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: '1.0'
+- !ruby/object:Gem::Dependency
+  name: multipart-post
+  requirement: !ruby/object:Gem::Requirement
     none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.2'
+  type: :runtime
   prerelease: false
-  type: :development
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.2'
 - !ruby/object:Gem::Dependency
   name: rake
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ! '>='
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ! '>='
       - !ruby/object:Gem::Version
         version: '0'
+- !ruby/object:Gem::Dependency
+  name: fakefs
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.4'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.4'
+- !ruby/object:Gem::Dependency
+  name: rspec
   requirement: !ruby/object:Gem::Requirement
+    none: false
     requirements:
-    - - '>='
+    - - ~>
       - !ruby/object:Gem::Version
-        version: '0'
+        version: '2.14'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
     none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '2.14'
+- !ruby/object:Gem::Dependency
+  name: simplecov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.7'
+  type: :development
   prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.7'
+- !ruby/object:Gem::Dependency
+  name: simplecov-rcov
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.2'
   type: :development
-description: angus-remote
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.2'
+- !ruby/object:Gem::Dependency
+  name: simplecov-rcov-text
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '0.0'
+- !ruby/object:Gem::Dependency
+  name: ci_reporter
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.9'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.9'
+description: Provides support code for making requests and building responses
 email:
-- adrian.gomez@moove-it.com
+- angus@moove-it.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- .gitignore
-- Gemfile
-- LICENSE.txt
-- README.md
-- Rakefile
-- angus-remote.gemspec
-- lib/angus/remote.rb
+- lib/angus-remote.rb
+- lib/angus/unmarshalling.rb
+- lib/angus/remote/representation.rb
 - lib/angus/remote/version.rb
-homepage: ''
+- lib/angus/remote/http/query_params.rb
+- lib/angus/remote/http/multipart_methods/multipart_put.rb
+- lib/angus/remote/http/multipart_methods/multipart_base.rb
+- lib/angus/remote/http/multipart_methods/multipart_post.rb
+- lib/angus/remote/http/multipart.rb
+- lib/angus/remote/message.rb
+- lib/angus/remote/builder.rb
+- lib/angus/remote/proxy_client_utils.rb
+- lib/angus/remote/response/builder.rb
+- lib/angus/remote/response/serializer.rb
+- lib/angus/remote/response/hash.rb
+- lib/angus/remote/service_directory.rb
+- lib/angus/remote/remote_response.rb
+- lib/angus/remote/exceptions.rb
+- lib/angus/remote/client.rb
+- lib/angus/remote/proxy_client.rb
+- lib/angus/remote/utils.rb
+- spec/angus/remote/service_directory_spec.rb
+- spec/angus/remote/proxy_client_utils_spec.rb
+- spec/angus/remote/http/query_params_spec.rb
+- spec/angus/remote/http/multipart_spec.rb
+- spec/angus/remote/http/multipart_methods/multipart_base_spec.rb
+- spec/angus/remote/utils_spec.rb
+- spec/angus/remote/response/builder_spec.rb
+- spec/angus/remote/client_spec.rb
+- spec/angus/remote/builder_spec.rb
+homepage: http://mooveit.github.io/angus
 licenses:
 - MIT
-post_install_message:
+post_install_message: 
 rdoc_options: []
 require_paths:
 - lib
 required_ruby_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-  none: false
 required_rubygems_version: !ruby/object:Gem::Requirement
+  none: false
   requirements:
-  - - '>='
+  - - ! '>='
     - !ruby/object:Gem::Version
       version: '0'
-  none: false
 requirements: []
-rubyforge_project:
-rubygems_version: 1.8.24
-signing_key:
+rubyforge_project: 
+rubygems_version: 1.8.25
+signing_key: 
 specification_version: 3
-summary: angust-remote
-test_files: []
+summary: Client for building service objects.
+test_files:
+- spec/angus/remote/service_directory_spec.rb
+- spec/angus/remote/proxy_client_utils_spec.rb
+- spec/angus/remote/http/query_params_spec.rb
+- spec/angus/remote/http/multipart_spec.rb
+- spec/angus/remote/http/multipart_methods/multipart_base_spec.rb
+- spec/angus/remote/utils_spec.rb
+- spec/angus/remote/response/builder_spec.rb
+- spec/angus/remote/client_spec.rb
+- spec/angus/remote/builder_spec.rb