paysio 1.0.5 → 1.0.6

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    paysio (1.0.4)
+    paysio (1.0.5)
       activemodel
       activesupport
       oj (> 2)

data/README.rdoc

@@ -30,8 +30,29 @@
   charge.refund
 
 
-== Actions: redirect to charge after create (e.g. in Ruby on Rails)
+== Sample: redirect to charge after create (e.g. in Ruby on Rails)
   def create
     charge = Paysio::Charge.create(amount: 100, payment_system_id: 't_123', description: 'test charge')
     redirect_to charge.redirect
   end
+
+== Sample: handle webhook events
+  def webhook
+    Paysio.api_key = 'HZClZur5OW3BYimWSydQNsArbph2L7IRo0ql8HK'
+    event = Paysio::Event.new(params)
+  end
+
+== Sample: handle webhook events, with getting event from api
+  def webhook
+    Paysio.api_key = 'HZClZur5OW3BYimWSydQNsArbph2L7IRo0ql8HK'
+    event = Paysio::Event.retrieve(params['id'])
+    if event.data.object == 'charge':
+      if event.type == 'charge.success'
+        pass #logic for charge success
+      elsif event.type == 'charge.failure'
+        pass #logic for charge failure
+      elsif event.type == 'charge.refund'
+        pass #logic for charge refund
+      end
+    end
+  end

data/VERSION

@@ -1 +1 @@
-1.0.5
+1.0.6

data/lib/paysio/client.rb

@@ -3,7 +3,7 @@ module Paysio
     class << self
       def request(method, path, params = {}, headers = {})
         unless Paysio.api_key
-          raise Paysio::Exceptions::NotAuthorized, "Please specify Paysio.api_key"
+          raise Paysio::Errors::Unauthorized, "Please specify Paysio.api_key"
         end
         headers = {
           :user_agent => "Pays.io RubyClient/#{Paysio::VERSION}",
@@ -23,18 +23,34 @@ module Paysio
         else
           opts[:payload] = stringify_params(params)
         end
-        response = self.execute(opts)
+        response = execute(opts)
         body, code = response.body, response.code
         begin
           resp = Paysio::JSON.decode(body)
         rescue Oj::ParseError
-          raise Paysio::Exceptions::APIError.new("Invalid response object from API: #{body.inspect} (#{code})", code, body)
+          raise Paysio::Exceptions::InternalError.new("Invalid response object from API: #{body.inspect} (#{code})", code, body)
         end
         Paysio::Resource.build_from(resp, response.headers[:location])
       end
 
       def execute(opts)
         RestClient::Request.execute(opts)
+        puts "test"
+      rescue RuntimeError => e
+        case e.http_code.to_i
+        when 400
+          raise Paysio::Errors::BadRequest, e.http_body
+        when 401
+          raise Paysio::Errors::Unauthorized, e.http_body
+        when 403
+          raise Paysio::Errors::Forbidden, e.http_body
+        when 404
+          raise Paysio::Errors::NotFound, e.http_body
+        when 500
+          raise Paysio::Errors::InternalError, e.http_body
+        else
+          raise e
+        end
       end
 
       def uri_escape(key)

data/lib/paysio/errors.rb

@@ -0,0 +1,9 @@
+module Paysio
+  module Errors
+    class Unauthorized < Exception; end;
+    class InternalError < Exception; end;
+    class Forbidden < Exception; end;
+    class BadRequest < Exception; end;
+    class NotFound < Exception; end;
+  end
+end

data/lib/paysio/resource.rb

@@ -5,9 +5,11 @@ module Paysio
 
     attr_accessor :redirect
 
-    def initialize(id = nil)
+    def initialize(values = {})
       @values = {}
-      @id = id
+      @id = values['id'] if values['id']
+      refresh_from(values)
+      self.class.define_attribute_methods(attributes.keys)
     end
 
     def refresh_from(values)
@@ -87,9 +89,7 @@ module Paysio
           else
             klass = Paysio::Resource
           end
-          resource = klass.new(resp['id'])
-          resource.refresh_from(resp)
-          resource.class.define_attribute_methods(resource.attributes.keys)
+          resource = klass.new(resp)
           resource.redirect = redirect_url
           resource
         else

data/lib/paysio/version.rb

@@ -1,3 +1,3 @@
 module Paysio
-  VERSION = '1.0.5'
+  VERSION = '1.0.6'
 end

data/lib/paysio.rb

@@ -9,6 +9,7 @@ require 'oj'
 require 'paysio/client'
 require 'paysio/json'
 require 'paysio/resource'
+require 'paysio/version'
 
 # actions
 require 'paysio/actions/list'
@@ -18,7 +19,7 @@ require 'paysio/actions/destroy'
 require 'paysio/actions/find'
 
 # exceptions
-require 'paysio/exceptions'
+require 'paysio/errors'
 
 # resources
 require 'paysio/resources/list'
@@ -40,4 +41,4 @@ module Paysio
   def self.api_url(path = '')
     "https://#{api_key}@#{api_base}/#{api_version}#{path}"
   end
-end
+end

data/spec/requests/base_spec.rb

@@ -1,4 +1,6 @@
 require 'spec_helper'
+require 'rest_client'
+require 'ostruct'
 describe "API Bindings" do
 
   before :all do
@@ -7,11 +9,54 @@ describe "API Bindings" do
 
   it "should not fetch from network while creating a new Resource" do
     @client.expects(:get).never
-    Paysio::Charge.new("someid")
+    Paysio::Charge.new(id: "someid")
   end
 
   it "should add redirect url to resource from location" do
     @client.expects(:get).once.returns(test_response(test_charge, 200, 'http://example.com'))
     Paysio::Charge.retrieve("someid").redirect == 'http://example.com'
   end
+
+  describe "exception handler" do
+
+    before :all do
+      @client = Paysio::Client
+    end
+
+    it "should raise Paysio::Errors::BadRequest on 400" do
+      exception = RestClient::Exception.new(OpenStruct.new(code: 400, body: 'message'))
+      RestClient::Request.stubs(:execute).raises(exception)
+      expect { @client.execute!({}) }.to raise_error(Paysio::Errors::BadRequest)
+    end
+
+    it "should raise Paysio::Errors::Unauthorized on 401" do
+      exception = RestClient::Exception.new(OpenStruct.new(code: 401, body: 'message'))
+      RestClient::Request.stubs(:execute).raises(exception)
+      expect { @client.execute!({}) }.to raise_error(Paysio::Errors::Unauthorized)
+    end
+
+    it "should raise Paysio::Errors::Forbidden on 403" do
+      exception = RestClient::Exception.new(OpenStruct.new(code: 403, body: 'message'))
+      RestClient::Request.stubs(:execute).raises(exception)
+      expect { @client.execute!({}) }.to raise_error(Paysio::Errors::Forbidden)
+    end
+
+    it "should raise Paysio::Errors::NotFound on 404" do
+      exception = RestClient::Exception.new(OpenStruct.new(code: 404, body: 'message'))
+      RestClient::Request.stubs(:execute).raises(exception)
+      expect { @client.execute!({}) }.to raise_error(Paysio::Errors::NotFound)
+    end
+
+    it "should raise Paysio::Errors::InternalError on 500" do
+      exception = RestClient::Exception.new(OpenStruct.new(code: 500, body: 'message'))
+      RestClient::Request.stubs(:execute).raises(exception)
+      expect { @client.execute!({}) }.to raise_error(Paysio::Errors::InternalError)
+    end
+
+    it "should raise Paysio::Errors::Unauthorized if api_key is not provided" do
+      Paysio.api_key = nil
+      RestClient::Request.stubs(:execute).returns('test')
+      expect { @client.request('get', 'test') }.to raise_error(Paysio::Errors::Unauthorized)
+    end
+  end
 end

data/spec/support/paysio.rb

@@ -1,19 +1,21 @@
 module Paysio
   class Client
-    @mock_rest_client = nil
-
-    def self.mock_rest_client(mock_client)
-      @mock_rest_client = mock_client
-    end
+    class << self
+      @mock_rest_client = nil
+      def mock_rest_client(mock_client)
+        @mock_rest_client = mock_client
+      end
 
-    def self.execute(opts)
-      get_params = (opts[:headers] || {})[:params]
-      post_params = opts[:payload]
-      case opts[:method]
-      when :get then @mock_rest_client.get opts[:url], get_params, post_params
-      when :post then @mock_rest_client.post opts[:url], get_params, post_params
-      when :put then @mock_rest_client.put opts[:url], get_params, post_params
-      when :delete then @mock_rest_client.delete opts[:url], get_params, post_params
+      alias_method :execute!, :execute
+      def execute(opts)
+        get_params = (opts[:headers] || {})[:params]
+        post_params = opts[:payload]
+        case opts[:method]
+        when :get then @mock_rest_client.get opts[:url], get_params, post_params
+        when :post then @mock_rest_client.post opts[:url], get_params, post_params
+        when :put then @mock_rest_client.put opts[:url], get_params, post_params
+        when :delete then @mock_rest_client.delete opts[:url], get_params, post_params
+        end
       end
     end
   end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: paysio
 version: !ruby/object:Gem::Version
-  version: 1.0.5
+  version: 1.0.6
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-03-20 00:00:00.000000000 Z
+date: 2013-03-30 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rest-client
@@ -149,7 +149,7 @@ files:
 - lib/paysio/actions/list.rb
 - lib/paysio/actions/update.rb
 - lib/paysio/client.rb
-- lib/paysio/exceptions.rb
+- lib/paysio/errors.rb
 - lib/paysio/json.rb
 - lib/paysio/resource.rb
 - lib/paysio/resources/charge.rb

data/lib/paysio/exceptions.rb

@@ -1,6 +0,0 @@
-module Paysio
-  module Exceptions
-    class NotAuthorized < Exception; end;
-    class APIError < Exception; end;
-  end
-end