razorpay 1.2.1 → 2.0.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: f1342dbfd634ead754a60f675870429894fdab71
-  data.tar.gz: f3b91327653aa94ac1e19e684223ffc9e7e890c7
+  metadata.gz: dc80169863413c4b51ebcd5dbc7a163a48c35a06
+  data.tar.gz: 73f38b5fafff2e2933c112a54e7d3e418f9440a1
 SHA512:
-  metadata.gz: 977fee36b31a1d7e44d0679eddf5593702387974a46438613383ea9f9588f84c3e9dbda7d6a05f719f22086a206ab5e07918eb474be04822b144cfb329d45bbd
-  data.tar.gz: 3e32025de35f357144a6da06a492e42c9efe20e93ef546cfc55bafb607dc4df6a3d9d13fcbb01f656a6ac6cbeb018d108c8e674b5e5f78956247d334dd871305
+  metadata.gz: 1740e3ece8cbdd0b3af137d861c8fecbeb880688df7f971d6780cd81bfac40a3341625e8f5d62dca1842f6a1a9015598c25737778fc0bd241accb0de8b4e7d52
+  data.tar.gz: 56c6cbba384b3035cc9cb08a5d1ecdf5ca09401e0af7d1335d90e1a37962f455efaa9d5fb318af7f4acab7df690e1a00f29d11ee9b8733cc5f13f7d92733c4e5

data/.travis.yml

@@ -1,6 +1,10 @@
-language: ruby # Auto-detected as well
+language: ruby
 rvm:
-  - "2.0.0"
   - "2.1.10"
   - "2.2.6"
   - "2.3.3"
+  - "2.4.0"
+before_install: gem update --system
+script:
+  - bundle exec rake test
+  - bundle exec rake rubocop

data/CHANGELOG

@@ -2,9 +2,25 @@
 
 Changelog for Razorpay-Ruby SDK.
 
+## Unreleased
+
+## [2.0.0] - 2016-03-02
+### Added
+- Adds `require` for all Razorpay supported entities
+- All entity objects now throw `NoMethodError` instead of `NameError` if the attribute doesn't exist
+- Adds customer edit API
+- Adds card fetch API
+- Adds custom header support
+- Adds constant time signature verification API for payments and webhooks
+- Adds payment capture-without-fetch API
+- Enables warnings for tests
+- Removes circular `require` calls
+- Adds rake test groups
+
 ## [1.2.1] - 2016-12-22
 ### Changed
 - Drops ArgumentError checks for local validation. Rely on server side checks instead.
+
 ### Added
 - Support for customers and invoices API
 - Loads Order class by default.
@@ -30,7 +46,8 @@ Changelog for Razorpay-Ruby SDK.
 ### Added
 - Initial Release
 
-[Unreleased]: https://github.com/razorpay/razorpay-ruby/compare/1.2.1...HEAD
+[Unreleased]: https://github.com/razorpay/razorpay-ruby/compare/2.0.0...HEAD
+[2.0.0]: https://github.com/razorpay/razorpay-ruby/compare/1.2.1...2.0.0
 [1.2.1]: https://github.com/razorpay/razorpay-ruby/compare/1.2.0...1.2.1
 [1.2.0]: https://github.com/razorpay/razorpay-ruby/compare/1.1.0...1.2.0
 [1.1.0]: https://github.com/razorpay/razorpay-ruby/compare/1.0.3...1.1.0

data/README.md

@@ -1,6 +1,6 @@
 # Razorpay Ruby bindings
 
-[![Build Status](https://travis-ci.org/razorpay/razorpay-ruby.svg?branch=master)](https://travis-ci.org/razorpay/razorpay-ruby) [![Gem Version](https://badge.fury.io/rb/razorpay.svg)](http://badge.fury.io/rb/razorpay) [![License](http://img.shields.io/:license-mit-blue.svg)](http://doge.mit-license.org)
+[![Build Status](https://travis-ci.org/razorpay/razorpay-ruby.svg?branch=master)](https://travis-ci.org/razorpay/razorpay-ruby) [![Gem Version](https://badge.fury.io/rb/razorpay.svg)](http://badge.fury.io/rb/razorpay) [![Coverage Status](https://coveralls.io/repos/github/Razorpay/razorpay-ruby/badge.svg?branch=master)](https://coveralls.io/github/Razorpay/razorpay-ruby?branch=master) [![License](http://img.shields.io/:license-mit-blue.svg)](http://doge.mit-license.org)
 
 This is the base ruby gem for interacting with the Razorpay API. This is primarily meant for merchants who wish to perform interactions with the Razorpay API programatically.
 
@@ -30,6 +30,11 @@ Next, you need to setup your key and secret using the following:
 Razorpay.setup("merchant_key_id", "merchant_key_secret")
 ```
 
+You can set customer headers for your requests using the following:
+```rb
+Razorpay.headers = {"CUSTOM_APP_HEADER" => "CUSTOM_VALUE"}
+```
+
 You can find your API keys at <https://dashboard.razorpay.com/#/app/keys>.
 
 If you are using rails, the right place to do this might be `config/initializers/razorpay.rb`.
@@ -37,7 +42,7 @@ If you are using rails, the right place to do this might be `config/initializers
 The most common construct is capturing a payment, which you do via the following:
 
 ```rb
-Razorpay::Payment.fetch("pay-ment_id").capture({amount:500})
+Razorpay::Payment.fetch("payment_id").capture({amount:500})
 # Note that the amount should come from your session, so as to verify
 # that the purchase was correctly done and not tampered
 ```
@@ -46,6 +51,13 @@ You can handle refunds using the following constructs:
 
 ```rb
 Razorpay::Payment.fetch("payment_id").refund({amount:500})
+refunds = Razorpay::Payment.fetch("payment_id").refunds
+```
+
+Refunds can also be handled without fetching payments:
+```rb
+refund = Razorpay::Refund.create(payment_id:"payment_id")
+Razorpay::Refund.fetch(refund.id)
 ```
 
 For other applications (such as fetching payments and refunds),
@@ -70,11 +82,38 @@ puts order.amount
 
 # Fetching payments corresponding to an order
 payments = order.payments
+```
 
+### Verification
+You can use the Utility class to verify the signature received in response to a payment made using Orders API
+```rb
+puts payment_response
+# {
+#   :razorpay_order_id   => "fake_order_id",
+#   :razorpay_payment_id => "fake_payment_id",
+#   :razorpay_signature  => "signature"
+# }
+Razorpay::Utility.verify_payment_signature(payment_response)
+```
+
+### Customers
+```rb
 # Create a customer
 customer = Razorpay::Customer.create email: 'test@razorpay.com', contact: '9876543210'
 puts customer.id #cust_6vRXClWqnLhV14
+```
+
+### Cards
+```rb
+# Fetch a card
+card = Razorpay::Card.fetch('card_7EZLhWkDt05n7V')
+puts card.network #VISA
+```
 
+You can find invoices API documentation at <https://docs.razorpay.com/v1/page/cards>.
+
+### Invoices
+```rb
 # Creating an invoice
 invoice = Razorpay::Invoice.create customer_id: customer.id, amount: 100, currency: 'INR', description: 'Test description', type: 'link'
 ```

data/Rakefile

@@ -1,9 +1,24 @@
 require 'rake/testtask'
+require 'rubocop/rake_task'
 
 Rake::TestTask.new do |t|
   t.libs << 'test'
+  t.warning = true
   t.pattern = 'test/razorpay/test_*.rb'
 end
 
 desc 'Run tests'
-task default: :test
+task default: [:test, :rubocop]
+
+desc 'Run rubocop'
+task :rubocop do
+  RuboCop::RakeTask.new
+end
+
+FileList['test/razorpay/test_*.rb'].each do |file|
+  group = File.basename(file, '.rb').split('_').last.to_sym
+  Rake::TestTask.new(group) do |t|
+    t.libs << 'test'
+    t.test_files = [file]
+  end
+end

data/lib/razorpay.rb

@@ -1,14 +1,25 @@
-require 'razorpay/constants'
-require 'razorpay/payment'
+require 'razorpay/card'
 require 'razorpay/order'
+require 'razorpay/errors'
+require 'razorpay/refund'
+require 'razorpay/invoice'
+require 'razorpay/payment'
+require 'razorpay/utility'
+require 'razorpay/customer'
+require 'razorpay/constants'
+require 'razorpay/collection'
 
 # Base Razorpay module
 module Razorpay
   class << self
-    attr_accessor :auth
+    attr_accessor :auth, :custom_headers
   end
 
   def self.setup(key_id, key_secret)
     self.auth = { username: key_id, password: key_secret }
   end
+
+  def self.headers=(headers = {})
+    self.custom_headers = headers
+  end
 end

data/lib/razorpay/card.rb

@@ -0,0 +1,17 @@
+require 'razorpay/request'
+require 'razorpay/entity'
+
+module Razorpay
+  # Card API allows you to fetch cards
+  # saved with Razorpay
+  # Docs: https://docs.razorpay.com/v1/page/cards
+  class Card < Entity
+    def self.request
+      Razorpay::Request.new('cards')
+    end
+
+    def self.fetch(id)
+      request.fetch id
+    end
+  end
+end

data/lib/razorpay/constants.rb

@@ -2,5 +2,5 @@
 module Razorpay
   BASE_URI = 'https://api.razorpay.com/v1/'.freeze
   TEST_URL = 'https://api.razorpay.com/'.freeze
-  VERSION = '1.2.1'.freeze
+  VERSION = '2.0.0'.freeze
 end

data/lib/razorpay/customer.rb

@@ -2,7 +2,7 @@ require 'razorpay/request'
 require 'razorpay/entity'
 
 module Razorpay
-  # Customer API allows you to create customer IDs on Razorpay
+  # Customer API allows you to create and fetch customers on Razorpay
   class Customer < Entity
     def self.request
       Razorpay::Request.new('customers')
@@ -11,5 +11,17 @@ module Razorpay
     def self.create(options)
       request.create options
     end
+
+    def self.fetch(id)
+      request.fetch id
+    end
+
+    def self.edit(id, options = {})
+      request.put id, options
+    end
+
+    def self.all(options = {})
+      request.all options
+    end
   end
 end

data/lib/razorpay/entity.rb

@@ -13,9 +13,15 @@ module Razorpay
     # the @attributes hash as the source, instead of
     # instance variables
     def method_missing(name)
-      name = name.to_s
-      raise NameError, "No such data member: #{name}" unless @attributes.key? name
-      @attributes[name]
+      if @attributes.key? name.to_s
+        @attributes[name.to_s]
+      else
+        super
+      end
+    end
+
+    def respond_to_missing?(method_name, include_private = false)
+      @attributes.key?(method_name.to_s) || super
     end
 
     # Public: Convert the Entity object to JSON

data/lib/razorpay/payment.rb

@@ -19,19 +19,22 @@ module Razorpay
       request.all options
     end
 
-    def refund(options = {})
-      self.class.request.post "#{id}/refund", options
+    def self.capture(id, options)
+      request.post "#{id}/capture", options
     end
 
-    def refunds
-      # This needs to be a string, not a symbol
-      Refund.new('payment_id' => id)
+    def refund(options = {})
+      self.class.request.post "#{id}/refund", options
     end
 
     def capture(options)
       self.class.request.post "#{id}/capture", options
     end
 
+    def refunds
+      self.class.request.get "#{id}/refunds"
+    end
+
     def method
       method_missing(:method)
     end

data/lib/razorpay/refund.rb

@@ -1,22 +1,23 @@
-require 'razorpay'
 require 'razorpay/request'
 require 'razorpay/entity'
 
 module Razorpay
   # Refund class handles all refund objects
   class Refund < Entity
-    def initialize(data)
-      super
-      @request = Razorpay::Request.new("payments/#{payment_id}/refunds")
+    def self.request
+      Razorpay::Request.new('refunds')
     end
 
-    def all(options = {})
-      # We receive an array of item hashes
-      @request.all options
+    def self.create(options)
+      request.create options
     end
 
-    def fetch(id)
-      @request.fetch id
+    def self.all(options = {})
+      request.all options
+    end
+
+    def self.fetch(id)
+      request.fetch id
     end
   end
 end

data/lib/razorpay/request.rb

@@ -1,4 +1,4 @@
-require 'razorpay'
+require 'razorpay/constants'
 require 'httparty'
 
 module Razorpay
@@ -13,12 +13,16 @@ module Razorpay
     def initialize(entity_name)
       self.class.base_uri(Razorpay::BASE_URI)
       @entity_name = entity_name
+      custom_headers = Razorpay.custom_headers || {}
+      predefined_headers = {
+        'User-Agent' => "Razorpay-Ruby/#{Razorpay::VERSION}"
+      }
+      # Order is important to give precedence to predefined headers
+      headers = custom_headers.merge(predefined_headers)
       @options = {
         basic_auth: Razorpay.auth,
         timeout: 30,
-        headers: {
-          'User-Agent' => "Razorpay-Ruby/#{Razorpay::VERSION}"
-        }
+        headers: headers
       }
     end
 
@@ -34,6 +38,14 @@ module Razorpay
       request :post, "/#{@entity_name}/#{url}", data
     end
 
+    def get(url)
+      request :get, "/#{@entity_name}/#{url}"
+    end
+
+    def put(id, data = {})
+      request :put, "/#{@entity_name}/#{id}", data
+    end
+
     def create(data)
       request :post, "/#{@entity_name}", data
     end
@@ -44,6 +56,8 @@ module Razorpay
         @options[:query] = data
       when :post
         @options[:body] = data
+      when :put
+        @options[:body] = data
       end
       create_instance self.class.send(method, url, @options)
     end
@@ -75,15 +89,15 @@ module Razorpay
 
     def raise_error(error, status)
       # Get the error class name, require it and instantiate an error
-      require "razorpay/errors/#{error['code'].downcase}"
       class_name = error['code'].split('_').map(&:capitalize).join('')
       args = [error['code'], status]
       args.push error['field'] if error.key?('field')
       klass =
         begin
+          require "razorpay/errors/#{error['code'].downcase}"
           Razorpay.const_get(class_name)
         # We got an unknown error, cast it to Error for now
-        rescue NameError
+        rescue NameError, LoadError
           Razorpay::Error
         end
       raise klass.new(*args), error['description']

data/lib/razorpay/utility.rb

@@ -0,0 +1,49 @@
+require 'openssl'
+
+module Razorpay
+  # Helper functions are defined here
+  class Utility
+    def self.verify_payment_signature(attributes)
+      signature = attributes[:razorpay_signature]
+      order_id = attributes[:razorpay_order_id]
+      payment_id = attributes[:razorpay_payment_id]
+
+      data = [order_id, payment_id].join '|'
+
+      verify_signature(signature, data)
+    end
+
+    def self.verify_webhook_signature(signature, body)
+      verify_signature(signature, body)
+    end
+
+    class << self
+      private
+
+      def verify_signature(signature, data)
+        secret = Razorpay.auth[:password]
+
+        expected_signature = OpenSSL::HMAC.hexdigest('SHA256', secret, data)
+
+        verified = secure_compare(expected_signature, signature)
+
+        raise SecurityError, 'Signature verification failed' unless verified
+      end
+
+      def secure_compare(a, b)
+        return false unless a.bytesize == b.bytesize
+
+        l = a.unpack('C*')
+        r = 0
+        i = -1
+
+        b.each_byte do |v|
+          i += 1
+          r |= v ^ l[i]
+        end
+
+        r.zero?
+      end
+    end
+  end
+end

data/razorpay-ruby.gemspec

@@ -18,9 +18,10 @@ Gem::Specification.new do |spec|
   spec.test_files    = spec.files.grep(/^(test|spec|features)/)
   spec.require_paths = ['lib']
 
-  spec.add_development_dependency 'rake', '~> 10.5'
-  spec.add_development_dependency 'minitest', '~> 5.8'
-  spec.add_development_dependency 'webmock', '~> 1.24'
-  spec.add_development_dependency 'simplecov', '~> 0.11'
-  spec.add_dependency 'httparty', '~> 0.13'
+  spec.add_development_dependency 'rake', '~> 12.0'
+  spec.add_development_dependency 'minitest', '~> 5.10'
+  spec.add_development_dependency 'webmock', '~> 2.3'
+  spec.add_development_dependency 'coveralls', '~> 0.8'
+  spec.add_development_dependency 'rubocop', '~> 0.47'
+  spec.add_dependency 'httparty', '~> 0.14'
 end

data/test/fixtures/customer_collection.json

@@ -0,0 +1,22 @@
+{
+  "entity": "collection",
+  "count": 2,
+  "items": [
+    {
+      "id": "cust_6vRXClWqnLhV14",
+      "email": "test@razorpay.com",
+      "contact": "9876543210",
+      "notes": [],
+      "created_at": 1482299274,
+      "entity": "customer"
+    },
+    {
+      "id": "cust_50smJrkZ4Qb0HJ",
+      "email": "test2@razorpay.com",
+      "contact": "9876543211",
+      "notes": [],
+      "created_at": 1482299274,
+      "entity": "customer"
+    }
+  ]
+}

data/test/fixtures/fake_card.json

@@ -0,0 +1,9 @@
+{
+  "id": "card_7EZLhWkDt05n7V",
+  "entity": "card",
+  "name": "Fake customer name",
+  "last4": "4321",
+  "network": "Visa",
+  "type": "debit",
+  "international": false
+}

data/test/fixtures/fake_customer_edited.json

@@ -0,0 +1,8 @@
+{
+  "id": "cust_6vRXClWqnLhV14",
+  "email": "test.edit@razorpay.com",
+  "contact": "9876543210",
+  "notes": [],
+  "created_at": 1482299274,
+  "entity": "customer"
+}

data/test/fixtures/fake_payment_authorized_webhook.json

@@ -0,0 +1,40 @@
+{
+  "event": "payment.authorized",
+  "entity": "event",
+  "contains": [
+    "payment"
+  ],
+  "payload": {
+    "payment": {
+      "entity": {
+        "id": "pay_6koWN7bvxujzxM",
+        "entity": "payment",
+        "amount": 1000000,
+        "currency": "INR",
+        "status": "captured",
+        "order_id": "order_100000000order",
+        "international": false,
+        "method": "card",
+        "amount_refunded": 0,
+        "refund_status": null,
+        "captured": true,
+        "description": "random description",
+        "card_id": "card_6koWNAT6LASUqy",
+        "bank": null,
+        "wallet": null,
+        "vpa": null,
+        "email": "a@b.com",
+        "contact": "+919999999999",
+        "notes": {
+          "merchant_order_id": "random order id"
+        },
+        "fee": 23000,
+        "service_tax": 3000,
+        "error_code": null,
+        "error_description": null,
+        "created_at": 1479978483
+      }
+    }
+  },
+  "created_at": 1400826760
+}

data/test/fixtures/refund_collection.json

@@ -1,5 +1,5 @@
 {
-  "count": 2,
+  "count": 3,
   "entity": "collection",
   "items": [
     {
@@ -17,6 +17,14 @@
       "currency": "INR",
       "payment_id": "fake_payment_id",
       "created_at": 1500826750
+    },
+    {
+      "id": "rfnd_6vRZmJu7EPEMY0",
+      "entity": "refund",
+      "amount": 200,
+      "currency": "INR",
+      "payment_id": "fake_payment_id_2",
+      "created_at": 1500826750
     }
   ]
-}
+}

data/test/fixtures/refund_collection_for_payment.json

@@ -0,0 +1,22 @@
+{
+  "count": 2,
+  "entity": "collection",
+  "items": [
+    {
+      "id": "rfnd_AABBdHIieexn5c",
+      "entity": "refund",
+      "amount": 100,
+      "currency": "INR",
+      "payment_id": "fake_payment_id",
+      "created_at": 1500826750
+    },
+    {
+      "id": "rfnd_19btGlBig6xZ2f",
+      "entity": "refund",
+      "amount": 100,
+      "currency": "INR",
+      "payment_id": "fake_payment_id",
+      "created_at": 1500826750
+    }
+  ]
+}

data/test/razorpay/test_card.rb

@@ -0,0 +1,23 @@
+require 'test_helper'
+
+module Razorpay
+  # Tests for Razorpay::Card
+  class RazorpayCardTest < Minitest::Test
+    def setup
+      @card_id = 'card_7EZLhWkDt05n7V'
+
+      # Any request that ends with cards/card_id
+      stub_get(%r{cards/#{@card_id}$}, 'fake_card')
+    end
+
+    def test_card_should_be_defined
+      refute_nil Razorpay::Card
+    end
+
+    def test_cards_should_be_fetched
+      card = Razorpay::Card.fetch(@card_id)
+      assert_instance_of Razorpay::Card, card, 'card not an instance of Razorpay::Card class'
+      assert_equal @card_id, card.id, 'card IDs do not match'
+    end
+  end
+end

data/test/razorpay/test_customer.rb

@@ -1,6 +1,4 @@
 require 'test_helper'
-require 'razorpay/customer'
-require 'razorpay/collection'
 
 module Razorpay
   # Tests for Razorpay::Customer
@@ -10,6 +8,7 @@ module Razorpay
 
       # Any request that ends with customers/customer_id
       stub_get(%r{customers/#{@customer_id}$}, 'fake_customer')
+      stub_get(/customers$/, 'customer_collection')
     end
 
     def test_customer_should_be_defined
@@ -23,5 +22,24 @@ module Razorpay
       assert_equal 'test@razorpay.com', customer.email
       assert_equal '9876543210', customer.contact
     end
+
+    def test_edit_customer
+      new_email = 'test.edit@razorpay.com'
+      stub_put(%r{customers/#{@customer_id}$}, 'fake_customer_edited', "email=#{new_email}")
+      customer = Razorpay::Customer.edit(@customer_id, "email=#{new_email}")
+      assert_instance_of Razorpay::Customer, customer
+      assert_equal customer.email, new_email
+    end
+
+    def test_fetch_all_customers
+      customers = Razorpay::Customer.all
+      assert_instance_of Razorpay::Collection, customers
+    end
+
+    def test_fetch_specific_customer
+      customer = Razorpay::Customer.fetch(@customer_id)
+      assert_instance_of Razorpay::Customer, customer
+      assert_equal customer.id, @customer_id
+    end
   end
 end

data/test/razorpay/test_entity.rb

@@ -1,5 +1,5 @@
 require 'test_helper'
-require 'razorpay/entity'
+require 'ostruct'
 
 module Razorpay
   # Tests for Razorpay::Entity
@@ -9,6 +9,29 @@ module Razorpay
       @entity = Entity.new(@hash)
     end
 
+    def test_create_instance
+      res = OpenStruct.new(parsed_response: { 'entity' => 'non_existent_entity' })
+      entity = Razorpay::Request.new('dummy').create_instance(res)
+      assert_instance_of Razorpay::Entity, entity
+    end
+
+    def test_raise_error
+      error = { 'code' => 'NON_EXISTENT_ERROR', 'description' => 'Unknown error' }
+      assert_raises(Razorpay::Error) { Razorpay::Request.new('dummy').raise_error(error, 500) }
+    end
+
+    def test_respond_to_missing_method
+      order_id = 'order_50sX9hGHZJvjjI'
+      stub_get(%r{orders/#{order_id}$}, 'fake_order')
+      order = Razorpay::Order.fetch(order_id)
+      assert_equal order.respond_to?(:non_existent_method), false
+
+      payment_id = 'fake_payment_id'
+      stub_get(%r{payments/#{payment_id}$}, 'fake_payment')
+      payment = Razorpay::Payment.fetch(payment_id)
+      assert_equal payment.respond_to?(:method), true
+    end
+
     def test_attribute_get
       assert_equal @hash['a'], @entity.a
     end
@@ -18,7 +41,7 @@ module Razorpay
     end
 
     def test_invalid_attribute_get
-      assert_raises(NameError, 'It must raise a NameError on invalid attribute') { @entity.b }
+      assert_raises(NoMethodError, 'It must raise a NoMethodError on invalid attribute') { @entity.b }
     end
   end
 end

data/test/razorpay/test_errors.rb

@@ -1,6 +1,4 @@
 require 'test_helper'
-require 'razorpay/entity'
-require 'razorpay/errors/bad_request_error'
 
 module Razorpay
   # Tests for Razorpay::Entity

data/test/razorpay/test_invoice.rb

@@ -1,6 +1,4 @@
 require 'test_helper'
-require 'razorpay/invoice'
-require 'razorpay/collection'
 
 module Razorpay
   # Tests for Razorpay::Invoice
@@ -17,8 +15,12 @@ module Razorpay
     end
 
     def test_invoice_should_be_created
-      stub_post(/invoices$/, 'fake_invoice', 'customer_id=cust_6vRXClWqnLhV14&amount=100&currency=INR&description=Test%20description&type=link')
-      invoice = Razorpay::Invoice.create customer_id: 'cust_6vRXClWqnLhV14', amount: 100, currency: 'INR', description: 'Test description', type: 'link'
+      stub_post(/invoices$/, 'fake_invoice', 'customer_id=cust_6vRXClWqnLhV14&'\
+                'amount=100&currency=INR&description=Test%20description&type=link')
+      invoice = Razorpay::Invoice.create customer_id: 'cust_6vRXClWqnLhV14',
+                                         amount: 100, currency: 'INR',
+                                         description: 'Test description',
+                                         type: 'link'
 
       assert_equal 'cust_6vRXClWqnLhV14', invoice.customer_id
       assert_equal 100, invoice.amount

data/test/razorpay/test_order.rb

@@ -1,6 +1,4 @@
 require 'test_helper'
-require 'razorpay/order'
-require 'razorpay/collection'
 
 module Razorpay
   # Tests for Razorpay::Order

data/test/razorpay/test_payment.rb

@@ -1,6 +1,4 @@
 require 'test_helper'
-require 'razorpay/payment'
-require 'razorpay/collection'
 
 module Razorpay
   # Tests for Razorpay::Payment
@@ -60,5 +58,11 @@ module Razorpay
       payment = payment.capture(amount: 5100)
       assert_equal 'captured', payment.status
     end
+
+    def test_payment_capture_without_fetch
+      stub_post(%r{payments/#{@payment_id}/capture$}, 'fake_captured_payment', 'amount=5100')
+      payment = Razorpay::Payment.capture(@payment_id, amount: 5100)
+      assert_equal 'captured', payment.status
+    end
   end
 end

data/test/razorpay/test_razorpay.rb

@@ -1,7 +1,5 @@
 require 'test_helper'
-require 'razorpay'
 require 'webmock'
-require 'test_helper'
 
 module Razorpay
   # Tests for Razorpay
@@ -19,6 +17,19 @@ module Razorpay
       assert_equal auth, Razorpay.auth
     end
 
+    def test_custom_header
+      custom_headers = { 'key' => 'value' }
+      stub_get(/$/, 'hello_response')
+      Razorpay.headers = custom_headers
+      Razorpay::Request.new('dummy').make_test_request
+      user_agent = "Razorpay-Ruby/#{Razorpay::VERSION}"
+      headers = { 'User-Agent' => user_agent, 'Authorization' => 'Basic a2V5X2lkOmtleV9zZWNyZXQ=' }
+      headers = headers.merge(custom_headers)
+      assert_requested :get, 'https://api.razorpay.com/',
+                       headers: headers,
+                       times: 1
+    end
+
     # We make a request to the / endpoint to test SSL support
     def test_sample_request
       WebMock.allow_net_connect!
@@ -33,8 +44,9 @@ module Razorpay
       stub_get(/$/, 'hello_response')
       Razorpay::Request.new('dummy').make_test_request
       user_agent = "Razorpay-Ruby/#{Razorpay::VERSION}"
-      assert_requested :get, 'https://key_id:key_secret@api.razorpay.com/',
-                       headers: { 'User-Agent' => user_agent },
+      headers = { 'User-Agent' => user_agent, 'Authorization' => 'Basic a2V5X2lkOmtleV9zZWNyZXQ=' }
+      assert_requested :get, 'https://api.razorpay.com/',
+                       headers: headers,
                        times: 1
     end
   end

data/test/razorpay/test_refund.rb

@@ -1,31 +1,42 @@
 require 'test_helper'
-require 'razorpay/payment'
-require 'razorpay/refund'
 
 module Razorpay
   # Tests for Razorpay::Refund
   class RazorpayRefundTest < Minitest::Test
     def setup
       @payment_id = 'fake_payment_id'
-      stub_get(%r{payments\/#{Regexp.quote(@payment_id)}$}, 'fake_payment')
+      stub_get(%r{payments/#{@payment_id}$}, 'fake_payment')
+      @refund_id = 'fake_refund_id'
+      stub_get(%r{refunds/#{@refund_id}$}, 'fake_refund')
     end
 
     def test_refund_should_be_defined
       refute_nil Razorpay::Refund
     end
 
+    def test_fetch_all_refunds_for_payment
+      stub_get(%r{payments/#{@payment_id}/refunds$}, 'refund_collection_for_payment')
+      refunds = Razorpay::Payment.fetch(@payment_id).refunds
+      assert_instance_of Razorpay::Collection, refunds
+    end
+
     def test_fetch_all_refunds
-      stub_get(%r{payments/#{@payment_id}/refunds$}, 'refund_collection')
-      refunds = Razorpay::Payment.fetch(@payment_id).refunds.all
+      stub_get(/refunds$/, 'refund_collection')
+      refunds = Razorpay::Refund.all
       assert_instance_of Razorpay::Collection, refunds
     end
 
     def test_fetch_specific_refund
-      refund_id = 'fake_refund_id'
-      stub_get(%r{payments/#{@payment_id}/refunds/#{refund_id}$}, 'fake_refund')
-      refund = Razorpay::Payment.fetch(@payment_id).refunds.fetch('fake_refund_id')
+      refund = Razorpay::Refund.fetch(@refund_id)
+      assert_instance_of Razorpay::Refund, refund
+      assert_equal refund.id, @refund_id
+    end
+
+    def test_create_refund
+      stub_post(/refunds$/, 'fake_refund', "payment_id=#{@payment_id}")
+      refund = Razorpay::Refund.create(payment_id: @payment_id)
       assert_instance_of Razorpay::Refund, refund
-      assert_equal refund.id, refund_id
+      assert_equal refund.payment_id, @payment_id
     end
   end
 end

data/test/razorpay/test_utility.rb

@@ -0,0 +1,35 @@
+require 'test_helper'
+
+module Razorpay
+  # Tests for Razorpay::Utility
+  class RazorpayUtilityTest < Minitest::Test
+    def setup
+      Razorpay.setup('key_id', 'key_secret')
+    end
+
+    def test_payment_signature_verification
+      payment_response = {
+        razorpay_order_id: 'fake_order_id',
+        razorpay_payment_id: 'fake_payment_id',
+        razorpay_signature: 'b2335e3b0801106b84a7faff035df56ecffde06918c9ddd1f0fafbb37a51cc89'
+      }
+      Razorpay::Utility.verify_payment_signature(payment_response)
+
+      payment_response[:razorpay_signature] = '_dummy_signature' * 4
+      assert_raises(SecurityError) do
+        Razorpay::Utility.verify_payment_signature(payment_response)
+      end
+    end
+
+    def test_webhook_signature_verification
+      webhook_body = fixture_file('fake_payment_authorized_webhook')
+      signature = 'd60e67fd884556c045e9be7dad57903e33efc7172c17c6e3ef77db42d2b366e9'
+      Razorpay::Utility.verify_webhook_signature(signature, webhook_body)
+
+      signature = '_dummy_signature' * 4
+      assert_raises(SecurityError) do
+        Razorpay::Utility.verify_webhook_signature(signature, webhook_body)
+      end
+    end
+  end
+end

data/test/test_helper.rb

@@ -1,6 +1,10 @@
+require 'coveralls'
 require 'simplecov'
 require 'minitest/autorun'
 require 'webmock/minitest'
+require 'razorpay'
+
+Coveralls.wear! if ENV['CI']
 
 def fixture_file(filename)
   return '' if filename == ''
@@ -27,3 +31,11 @@ def stub_post(*args)
   url = args[0]
   stub_request(:post, url).with(body: data).to_return(response)
 end
+
+def stub_put(*args)
+  # The last argument is put data
+  data = args.pop
+  response = stub_response(*args)
+  url = args[0]
+  stub_request(:put, url).with(body: data).to_return(response)
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: razorpay
 version: !ruby/object:Gem::Version
-  version: 1.2.1
+  version: 2.0.0
 platform: ruby
 authors:
 - Abhay Rana
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-22 00:00:00.000000000 Z
+date: 2017-03-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rake
@@ -16,70 +16,84 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.5'
+        version: '12.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.5'
+        version: '12.0'
 - !ruby/object:Gem::Dependency
   name: minitest
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.8'
+        version: '5.10'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '5.8'
+        version: '5.10'
 - !ruby/object:Gem::Dependency
   name: webmock
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.24'
+        version: '2.3'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '1.24'
+        version: '2.3'
 - !ruby/object:Gem::Dependency
-  name: simplecov
+  name: coveralls
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.11'
+        version: '0.8'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.11'
+        version: '0.8'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.47'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.47'
 - !ruby/object:Gem::Dependency
   name: httparty
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.13'
+        version: '0.14'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '0.13'
+        version: '0.14'
 description: Official ruby bindings for the Razorpay API
 email:
 - nemo@razorpay.com
@@ -99,6 +113,7 @@ files:
 - Rakefile
 - lib/ca-bundle.crt
 - lib/razorpay.rb
+- lib/razorpay/card.rb
 - lib/razorpay/collection.rb
 - lib/razorpay/constants.rb
 - lib/razorpay/customer.rb
@@ -113,13 +128,18 @@ files:
 - lib/razorpay/payment.rb
 - lib/razorpay/refund.rb
 - lib/razorpay/request.rb
+- lib/razorpay/utility.rb
 - razorpay-ruby.gemspec
 - test/fixtures/bad_request_error.json
+- test/fixtures/customer_collection.json
 - test/fixtures/fake_captured_payment.json
+- test/fixtures/fake_card.json
 - test/fixtures/fake_customer.json
+- test/fixtures/fake_customer_edited.json
 - test/fixtures/fake_invoice.json
 - test/fixtures/fake_order.json
 - test/fixtures/fake_payment.json
+- test/fixtures/fake_payment_authorized_webhook.json
 - test/fixtures/fake_refund.json
 - test/fixtures/hello_response.json
 - test/fixtures/invoice_collection.json
@@ -128,6 +148,8 @@ files:
 - test/fixtures/payment_collection.json
 - test/fixtures/payment_collection_with_one_payment.json
 - test/fixtures/refund_collection.json
+- test/fixtures/refund_collection_for_payment.json
+- test/razorpay/test_card.rb
 - test/razorpay/test_customer.rb
 - test/razorpay/test_entity.rb
 - test/razorpay/test_errors.rb
@@ -136,6 +158,7 @@ files:
 - test/razorpay/test_payment.rb
 - test/razorpay/test_razorpay.rb
 - test/razorpay/test_refund.rb
+- test/razorpay/test_utility.rb
 - test/test_helper.rb
 homepage: https://razorpay.com/
 licenses:
@@ -163,11 +186,15 @@ specification_version: 4
 summary: Razorpay's Ruby API
 test_files:
 - test/fixtures/bad_request_error.json
+- test/fixtures/customer_collection.json
 - test/fixtures/fake_captured_payment.json
+- test/fixtures/fake_card.json
 - test/fixtures/fake_customer.json
+- test/fixtures/fake_customer_edited.json
 - test/fixtures/fake_invoice.json
 - test/fixtures/fake_order.json
 - test/fixtures/fake_payment.json
+- test/fixtures/fake_payment_authorized_webhook.json
 - test/fixtures/fake_refund.json
 - test/fixtures/hello_response.json
 - test/fixtures/invoice_collection.json
@@ -176,6 +203,8 @@ test_files:
 - test/fixtures/payment_collection.json
 - test/fixtures/payment_collection_with_one_payment.json
 - test/fixtures/refund_collection.json
+- test/fixtures/refund_collection_for_payment.json
+- test/razorpay/test_card.rb
 - test/razorpay/test_customer.rb
 - test/razorpay/test_entity.rb
 - test/razorpay/test_errors.rb
@@ -184,4 +213,5 @@ test_files:
 - test/razorpay/test_payment.rb
 - test/razorpay/test_razorpay.rb
 - test/razorpay/test_refund.rb
+- test/razorpay/test_utility.rb
 - test/test_helper.rb