spree_komoju 0.0.2 → 0.0.3

data/spec/models/spree/gateway/komoju_bank_transfer_spec.rb

@@ -0,0 +1,59 @@
+require "spec_helper"
+
+describe Spree::Gateway::KomojuBankTransfer, type: :model do
+  subject { described_class.new }
+
+  describe "#authorize" do
+    let(:money) { 1000.0 }
+    let(:source) do
+      Spree::BankTransfer.create!(
+        email: "email",
+        given_name: "given_name",
+        family_name: "family_name",
+        given_name_kana: "given_name_kana",
+        family_name_kana: "family_name_kana"
+      )
+    end
+    let(:options) { { login: "api_key", shipping: 100.0, tax: 200.0, subtotal: 800.0, discount: 100.0,
+                      currency: currency } }
+    let(:details) do
+      {
+        type: "bank_transfer",
+        phone: nil,
+        email: "email",
+        given_name: "given_name",
+        family_name: "family_name",
+        given_name_kana: "given_name_kana",
+        family_name_kana: "family_name_kana"
+      }
+    end
+    let(:params) { { "payment_deadline" => Time.now.iso8601.to_s, "payment_details" => { "order_id" => "order_id", "bank_transfer" => "bank_transfer" } } }
+    let(:response) { double(ActiveMerchant::Billing::Response, params: params) }
+
+    before do
+      allow_any_instance_of(Spree::Gateway::KomojuBankTransfer).to receive(:options) { options }
+    end
+
+    context "with currency is USD" do
+      let(:currency) { "USD" }
+
+      it "calls ActiveMerchant::Billing::KomojuGateway#purchase with original options" do
+        expect_any_instance_of(ActiveMerchant::Billing::KomojuGateway).to receive(:purchase).with(800.0, details, options) { response }
+
+        subject.authorize(money, source, options)
+      end
+    end
+
+    context "with currency is JPY" do
+      let(:currency) { "JPY" }
+
+      it "calls ActiveMerchant::Billing::KomojuGateway#purchase with options converted from cents to dollars" do
+        options_converted_to_dollars = { login: "api_key", shipping: 1.0, tax: 2.0, subtotal: 8.0, discount: 1.0,
+                                         currency: currency }
+        expect_any_instance_of(ActiveMerchant::Billing::KomojuGateway).to receive(:purchase).with(998.0, details, options_converted_to_dollars) { response }
+
+        subject.authorize(money, source, options)
+      end
+    end
+  end
+end

data/spec/models/spree/gateway/komoju_credit_card_spec.rb

@@ -0,0 +1,117 @@
+require "spec_helper"
+
+describe Spree::Gateway::KomojuCreditCard, type: :model do
+  subject { described_class.new }
+  let(:komoju_gateway) { double(ActiveMerchant::Billing::KomojuGateway, purchase: response) }
+  let(:response) { double(ActiveMerchant::Billing::Response) }
+  before do
+    allow(ActiveMerchant::Billing::KomojuGateway).to receive(:new) { komoju_gateway }
+  end
+
+  describe "#purchase" do
+    let(:money) { 1000.0 }
+    let(:source) do
+      double("credit card", gateway_payment_profile_id: payment_profile_id,
+                            gateway_customer_profile_id: nil,
+                            to_active_merchant: details)
+    end
+    let(:currency) { "JPY" }
+    let(:details) { double("payment details") }
+    let(:options) do
+      {
+        email: "foo@example.com",
+        login: "api_key",
+        shipping: 100.0,
+        tax: 200.0,
+        subtotal: 800.0,
+        discount: 100.0,
+        currency: currency,
+        billing_address: { phone: "09011112222" }
+      }
+    end
+
+    context "with no payment profile id" do
+      let(:payment_profile_id) { nil }
+
+      it "calls provider#purchase with original options" do
+        expect(komoju_gateway).to receive(:purchase).with(998.0, details, options) { response }
+        expect(subject.purchase(money, source, options)).to eq(response)
+      end
+    end
+
+    context "with a payment profile id" do
+      let(:payment_profile_id) { "tok_123" }
+
+      it "calls provider#purchase with profile id" do
+        expect(komoju_gateway).to receive(:purchase).with(998.0, "tok_123", options) { response }
+        expect(subject.purchase(money, source, options)).to eq(response)
+      end
+    end
+  end
+
+  describe "#create_profile" do
+    let(:payment) { double("payment", source: source) }
+    let(:details) { double("payment details") }
+
+    context "source has credit card and no gateway_payment_profile_id" do
+      let(:source) { double("credit card", gateway_payment_profile_id: nil,
+                                           number: "4111111111111111",
+                                           to_active_merchant: details) }
+      before do
+        allow(response).to receive(:params).and_return("id" => "tok_123")
+        expect(komoju_gateway).to receive(:store).with(details, hash_including()) { response }
+      end
+
+      context "response is success" do
+        before do
+          allow(response).to receive(:success?).and_return(true)
+          allow(source).to receive(:update_attributes!)
+        end
+
+        it "stores token" do
+          subject.create_profile(payment)
+        end
+
+        it "updates source with payment profile id" do
+          allow(komoju_gateway).to receive(:store).with(details, hash_including()) { response }
+          expect(source).to receive(:update_attributes!).with(gateway_payment_profile_id: "tok_123")
+          subject.create_profile(payment)
+        end
+      end
+
+      context "response is failure" do
+        before do
+          allow(response).to receive(:success?).and_return(false)
+          allow(response).to receive(:message).and_return("error message")
+        end
+
+        it "calls gateway_error on payment" do
+          expect(payment).to receive(:gateway_error).with("error message")
+          subject.create_profile(payment)
+        end
+      end
+    end
+
+    context "source has no number" do
+      let(:source) { double("credit card", gateway_payment_profile_id: nil,
+                                           number: nil,
+                                           to_active_merchant: details) }
+
+      it "does not try to store the payment details" do
+        expect(komoju_gateway).not_to receive(:store)
+        subject.create_profile(payment)
+      end
+    end
+
+    context "source already has a payment profile id" do
+      let(:source) { double("credit card", gateway_payment_profile_id: "tok_123",
+                                           number: nil,
+                                           to_active_merchant: details) }
+
+      it "does not try to store the payment details" do
+        expect(komoju_gateway).not_to receive(:store)
+        subject.create_profile(payment)
+      end
+    end
+  end
+end

data/spec/models/spree/gateway/komoju_konbini_spec.rb

@@ -0,0 +1,41 @@
+require "spec_helper"
+
+describe Spree::Gateway::KomojuKonbini, type: :model do
+  subject { described_class.new }
+
+  describe "#authorize" do
+    let(:money) { 1000.0 }
+    let(:source) { Spree::Konbini.create!(convenience: "lawson") }
+    let(:options) { { email: "foo@example.com", login: "api_key", shipping: 100.0, tax: 200.0, subtotal: 800.0, discount: 100.0,
+                      currency: currency, billing_address: { phone: "09011112222" } } }
+    let(:details) { { type: "konbini", store: "lawson", phone: "09011112222", email: "foo@example.com"  } }
+    let(:params) { { "payment_deadline" => Time.now.iso8601.to_s, "payment_details" => { "receipt" => "receipt", "instructions_url" => "instructions_url", "confirmation_code" => "confirmation_code"} } }
+    let(:response) { double(ActiveMerchant::Billing::Response, params: params, success?: true) }
+
+    before do
+      allow_any_instance_of(Spree::Gateway::KomojuKonbini).to receive(:options) { options }
+    end
+
+    context "with currency is USD" do
+      let(:currency) { "USD" }
+
+      it "calls ActiveMerchant::Billing::KomojuGateway#purchase with original options" do
+        expect_any_instance_of(ActiveMerchant::Billing::KomojuGateway).to receive(:purchase).with(800.0, details, options) { response }
+
+        subject.authorize(money, source, options)
+      end
+    end
+
+    context "with currency is JPY" do
+      let(:currency) { "JPY" }
+
+      it "calls ActiveMerchant::Billing::KomojuGateway#purchase with options converted from cents to dollars" do
+        options_converted_to_dollars = { email: "foo@example.com", login: "api_key", shipping: 1.0, tax: 2.0, subtotal: 8.0, discount: 1.0,
+                                         currency: currency, billing_address: { phone: "09011112222" } }
+        expect_any_instance_of(ActiveMerchant::Billing::KomojuGateway).to receive(:purchase).with(998.0, details, options_converted_to_dollars) { response }
+
+        subject.authorize(money, source, options)
+      end
+    end
+  end
+end

data/spec/models/spree/komoju_gateway_spec.rb

@@ -0,0 +1,50 @@
+require "spec_helper"
+
+describe Spree::KomojuGateway, type: :model do
+  subject { described_class.new }
+
+  describe "#provider_class" do
+    it { expect(subject.provider_class).to eq ActiveMerchant::Billing::KomojuGateway }
+  end
+
+  describe "#options" do
+    it "returns options" do
+      api_key = double("api_key")
+      allow(subject).to receive(:preferred_api_key) { api_key }
+      allow(subject).to receive(:preferred_test) { true }
+
+      expect(subject.options).to eq ({ api_key: nil, test: true, server: "test", test_mode: true, login: api_key })
+    end
+  end
+
+  describe "#auto_capture?" do
+    it { expect(subject.auto_capture?).to be_falsy }
+  end
+
+  describe "#supports?" do
+    it "returns true" do
+      source = double("source")
+      expect(subject.supports?(source)).to be_truthy
+    end
+  end
+
+  describe "#change_options_to_dollar" do
+    it "changes cents to dollar" do
+      options = { shipping: 100, tax: 100, subtotal: 100, discount: 100 }
+
+      expect(subject.change_options_to_dollar(options)).to eq ({ shipping: 1, tax: 1, subtotal: 1, discount: 1 })
+    end
+  end
+
+  describe "#gateway_type" do
+    it { expect(subject.gateway_type).to eq "gateway" }
+  end
+
+  describe "#method_type" do
+    it { expect(subject.method_type).to eq "komoju_gateway" }
+  end
+
+  describe "#payment_source_class" do
+    it { expect(subject.payment_source_class).to eq Spree::Gateway }
+  end
+end

data/spec/models/spree/konbini_spec.rb

@@ -0,0 +1,69 @@
+require "spec_helper"
+
+describe Spree::Konbini, type: :model do
+  subject { described_class.new }
+
+  describe "#actions" do
+    it { expect(subject.actions).to eq ["capture", "void"] }
+  end
+
+  describe "#can_capture?" do
+    let(:payment) { double(Spree::Payment, state: state) }
+
+    before do
+      allow(subject).to receive(:payment) { payment }
+    end
+
+    context "when payment state is not checkout or pending" do
+      let(:state) { "void" }
+
+      it { expect(subject.can_capture?(payment)).to be_falsy }
+    end
+
+    context "when payment state is pending" do
+      let(:payment) { double(Spree::Payment, state: state, source: source) }
+      let(:state) { "pending" }
+      let(:source) { double(Spree::Konbini, expires_at: expires_at) }
+
+      before do
+        allow(subject).to receive(:payment) { payment }
+      end
+
+      context "when expires_at is tomorrow" do
+        let(:expires_at) { Date.tomorrow }
+
+        it { expect(subject.can_capture?(payment)).to be_truthy }
+      end
+
+      context "when expires_at is yesterday" do
+        let(:expires_at) { Date.yesterday}
+
+        it { expect(subject.can_capture?(payment)).to be_falsy }
+      end
+    end
+  end
+
+  describe "#can_void?" do
+    let(:payment) { double(Spree::Payment, state: state) }
+
+    before do
+      allow(subject).to receive(:payment) { payment }
+    end
+
+    context "when payment state is void" do
+      let(:state) { "void" }
+
+      it { expect(subject.can_void?(payment)).to be_falsy }
+    end
+
+    context "when payment state is pending" do
+      let(:state) { "pending" }
+
+      it { expect(subject.can_void?(payment)).to be_truthy }
+    end
+  end
+
+  describe "#instructions_partial_path" do
+    it { expect(subject.instructions_partial_path).to eq "spree/orders/konbini" }
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,87 @@
+# Run Coverage report
+require 'simplecov'
+SimpleCov.start do
+  add_filter 'spec/dummy'
+  add_group 'Controllers', 'app/controllers'
+  add_group 'Helpers', 'app/helpers'
+  add_group 'Mailers', 'app/mailers'
+  add_group 'Models', 'app/models'
+  add_group 'Views', 'app/views'
+  add_group 'Libraries', 'lib'
+end
+
+# Configure Rails Environment
+ENV['RAILS_ENV'] = 'test'
+
+require File.expand_path('../dummy/config/environment.rb',  __FILE__)
+
+require 'rspec/rails'
+require 'database_cleaner'
+require 'ffaker'
+
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.join(File.dirname(__FILE__), 'support/**/*.rb')].each { |f| require f }
+
+# Requires factories and other useful helpers defined in spree_core.
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/capybara_ext'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/factories'
+require 'spree/testing_support/url_helpers'
+
+# Requires factories defined in lib/spree_komoju/factories.rb
+require 'spree_komoju/factories'
+
+RSpec.configure do |config|
+  config.include FactoryGirl::Syntax::Methods
+
+  # Infer an example group's spec type from the file location.
+  config.infer_spec_type_from_file_location!
+
+  # == URL Helpers
+  #
+  # Allows access to Spree's routes in specs:
+  #
+  # visit spree.admin_path
+  # current_path.should eql(spree.products_path)
+  config.include Spree::TestingSupport::UrlHelpers
+
+  # == Mock Framework
+  #
+  # If you prefer to use mocha, flexmock or RR, uncomment the appropriate line:
+  #
+  # config.mock_with :mocha
+  # config.mock_with :flexmock
+  # config.mock_with :rr
+  config.mock_with :rspec
+  config.color = true
+
+  # Remove this line if you're not using ActiveRecord or ActiveRecord fixtures
+  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+
+  # Capybara javascript drivers require transactional fixtures set to false, and we use DatabaseCleaner
+  # to cleanup after each test instead.  Without transactional fixtures set to false the records created
+  # to setup a test will be unavailable to the browser, which runs under a separate server instance.
+  config.use_transactional_fixtures = false
+
+  # Ensure Suite is set to use transactions for speed.
+  config.before :suite do
+    DatabaseCleaner.strategy = :transaction
+    DatabaseCleaner.clean_with :truncation
+  end
+
+  # Before each spec check if it is a Javascript test and switch between using database transactions or not where necessary.
+  config.before :each do
+    DatabaseCleaner.strategy = RSpec.current_example.metadata[:js] ? :truncation : :transaction
+    DatabaseCleaner.start
+  end
+
+  # After each spec clean the database.
+  config.after :each do
+    DatabaseCleaner.clean
+  end
+
+  config.fail_fast = ENV['FAIL_FAST'] || false
+  config.order = "random"
+end

data/spree_komoju.gemspec

@@ -0,0 +1,32 @@
+# encoding: UTF-8
+Gem::Specification.new do |s|
+  s.platform    = Gem::Platform::RUBY
+  s.name        = 'spree_komoju'
+  s.version     = '0.0.3'
+  s.summary     = 'Spree Komoju Payment Gateway'
+  s.description = 'Spree Payment gateway for Komoju payment gateway'
+  s.authors     = ['Masahiro Saito', 'Chris Salzberg']
+  s.required_ruby_version = '>= 2.0.0'
+
+  # s.author    = 'You'
+  # s.email     = 'you@example.com'
+  # s.homepage  = 'http://www.spreecommerce.com'
+
+  s.files       = `git ls-files`.split("\n")
+  s.test_files  = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.require_path = 'lib'
+  s.requirements << 'none'
+
+  s.add_dependency 'spree_core', '~> 3.0.0'
+
+  s.add_development_dependency 'capybara', '~> 2.4'
+  s.add_development_dependency 'coffee-rails'
+  s.add_development_dependency 'database_cleaner'
+  s.add_development_dependency 'factory_girl', '~> 4.5'
+  s.add_development_dependency 'ffaker'
+  s.add_development_dependency 'rspec-rails',  '~> 3.1'
+  s.add_development_dependency 'sass-rails', '~> 5.0.0.beta1'
+  s.add_development_dependency 'selenium-webdriver'
+  s.add_development_dependency 'simplecov'
+  s.add_development_dependency 'sqlite3'
+end