spree_gateway 3.7.2 → 3.9.0

data/lib/views/frontend/spree/checkout/payment/_stripe_elements.html.erb

@@ -103,7 +103,8 @@
       }
     });
   };
-  
+
 </script>
 
 <%= render 'spree/checkout/payment/stripe_additional_info' %>
+

data/spec/factories/check_factory.rb

@@ -0,0 +1,10 @@
+FactoryBot.define do
+  factory :check, class: Spree::Check do
+    account_holder_name { 'John Doe' }
+    account_holder_type { 'Individual' }
+    account_type { 'checking' }
+    routing_number { '110000000' }
+    account_number { '000123456789' }
+    association(:payment_method, factory: :credit_card_payment_method)
+  end
+end

data/spec/features/admin/stripe_elements_payment_spec.rb

@@ -0,0 +1,109 @@
+require 'spec_helper'
+
+describe 'Admin Panel Stripe elements payment', type: :feature, :js => true do
+  stub_authorization!
+
+  let!(:country)         { create(:country, :states_required => true) }
+  let!(:state)           { create(:state, :country => country) }
+  let!(:shipping_method) { create(:shipping_method) }
+  let!(:stock_location)  { create(:stock_location) }
+  let!(:mug)             { create(:product, :name => 'RoR Mug') }
+  let!(:zone)            { create(:zone) }
+  let!(:stripe_elements_payment_method) do
+    Spree::Gateway::StripeElementsGateway.create!(
+      :name => 'Stripe Element',
+      :preferred_secret_key => 'sk_test_VCZnDv3GLU15TRvn8i2EsaAN',
+      :preferred_publishable_key => 'pk_test_Cuf0PNtiAkkMpTVC2gwYDMIg'
+    )
+  end
+  
+  let!(:order) { OrderWalkthrough.up_to(:payment) }
+  before { visit spree.new_admin_order_payment_path(order.number) }
+
+  it 'can process a valid payment' do
+    fill_in_stripe_payment
+    wait_for { !page.has_current_path?(spree.admin_order_payments_path(order.number)) }
+    
+    expect(page.body).to have_content('Payment has been successfully created!')
+    expect(page).to have_current_path spree.admin_order_payments_path(order.number)
+  end
+  
+  if Spree.version.to_f >= 4.1
+    it 'shows an error with an invalid card name' do
+      fill_in_stripe_payment(true)
+
+      expect(page).to have_content("Credit card Name can't be blank")
+      expect(page).to have_current_path spree.admin_order_payments_path(order.number)
+    end
+  else
+    it 'can proces valid payment with invalid card name' do
+      fill_in_stripe_payment(true)
+      wait_for { !page.has_current_path?(spree.admin_order_payments_path(order.number)) }
+
+      expect(page.body).to have_content('Payment has been successfully created!')
+      expect(page).to have_current_path spree.admin_order_payments_path(order.number)
+    end
+  end
+
+  it 'shows an error with an invalid card number' do
+    fill_in_stripe_payment(false, true)
+    
+    expect(page).to have_content('The card number is not a valid credit card number.')
+    expect(page).to have_current_path spree.new_admin_order_payment_path(order.number)
+  end
+
+  it 'shows an error with an invalid card code' do
+    fill_in_stripe_payment(false, false, true)
+
+    expect(page).to have_content("Your card's security code is invalid.")
+    expect(page).to have_current_path spree.new_admin_order_payment_path(order.number)
+  end
+
+  it 'shows an error with an invalid card expiration' do
+    fill_in_stripe_payment(false, false, false, true)
+
+    if Spree.version.to_f >= 4.1
+      expect(page).to have_content('Credit card Month is not a number')
+      expect(page).to have_content('Credit card Year is not a number')
+      expect(page).to have_current_path spree.admin_order_payments_path(order.number)
+    else
+      expect(page).to have_content("Your card's expiration year is invalid.")
+      expect(page).to have_current_path spree.new_admin_order_payment_path(order.number)
+    end
+  end
+
+  def fill_in_stripe_payment(invalid_name = false, invalid_number = false, invalid_code = false, invalid_expiration = false)
+    fill_in 'Name *', with: invalid_name ? '' : 'Stripe Elements Gateway Payment'
+    fill_in_card_number(invalid_number)
+    fill_in_cvc(invalid_code)
+    fill_in_card_expiration(invalid_expiration)
+
+    click_button 'Update'
+  end
+
+  def fill_in_card_number(invalid_number)
+    number = invalid_number ? '123' : '4242 4242 4242 4242'
+    fill_in_field('Card Number *', '#card_number1', number)
+  end
+
+  def fill_in_card_expiration(invalid_expiration)
+    valid_expiry = Spree.version.to_f >= 4.2 ? "01/#{Time.current.year + 1}" : "01 / #{Time.current.year + 1}"
+    invalid_expiry = Spree.version.to_f >= 4.2 ? '01/' : '01 / '
+
+    card_expiry = invalid_expiration ? invalid_expiry : valid_expiry
+    fill_in_field('Expiration *', '#card_expiry1', card_expiry)
+  end
+
+  def fill_in_cvc(invalid_code)
+    value = invalid_code ? '1' : '123'
+    label = Spree.version.to_f >= 4.2 ? 'Card Varification Code (CVC) *' : 'Card Code *'
+
+    fill_in label, with: value
+  end
+
+  def fill_in_field(field_name, field_id, number)
+    until page.find(field_id).value == number
+      fill_in field_name, with: number
+    end
+  end
+end

data/spec/features/stripe_checkout_spec.rb

@@ -91,7 +91,10 @@ describe "Stripe checkout", type: :feature do
     expect(page).to have_css('.has-error #card_code.error')
   end
 
+  # this scenario will not occur on Spree 4.2 due to swapping jquery.payment to cleave
+  # see https://github.com/spree/spree/pull/10363
   it "shows an error with invalid expiry month field", :js => true do
+    skip if Spree.version.to_f >= 4.2 
     fill_in 'card_number', with: '4242 4242 4242 4242'
     fill_in 'card_expiry', :with => "00 / #{Time.now.year + 1}"
     fill_in 'card_code', with: '123'

data/spec/features/stripe_elements_3ds_checkout_spec.rb

@@ -0,0 +1,105 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'Stripe Elements 3ds checkout', type: :feature, js: true do
+  let!(:product) { create(:product, name: 'RoR Mug') }
+  let!(:stripe_payment_method) do
+    Spree::Gateway::StripeElementsGateway.create!(
+      name: 'Stripe',
+      preferred_secret_key: 'sk_test_VCZnDv3GLU15TRvn8i2EsaAN',
+      preferred_publishable_key: 'pk_test_Cuf0PNtiAkkMpTVC2gwYDMIg',
+      preferred_intents: preferred_intents
+    )
+  end
+
+  before do
+    user = create(:user)
+    order = OrderWalkthrough.up_to(:confirm)
+    expect(order).to receive(:confirmation_required?).and_return(true).at_least(:once)
+
+    order.reload
+    order.user = user
+    payment = order.payments.first
+    payment.source = create(:credit_card, number: card_number)
+    payment.save!
+
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(current_order: order)
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(try_spree_current_user: user)
+    allow_any_instance_of(Spree::CheckoutController).to receive_messages(skip_state_validation?: true)
+    allow_any_instance_of(Spree::OrdersController).to receive_messages(try_spree_current_user: user)
+
+    add_to_cart(product)
+    click_link 'checkout'
+    click_button 'Place Order'
+  end
+
+  describe 'when intents are disabled' do
+    let(:preferred_intents) { false }
+
+    context 'and credit card does not require 3ds authentication' do
+      let(:card_number) { '4242424242424242' }
+
+      it 'should place order without 3ds authentication' do
+        expect(page).to have_content('Order placed successfully')
+        order = Spree::Order.complete.last
+        expect(page.current_url).to include("/orders/#{order.number}")
+        expect(page).to have_content(order.number)
+      end
+    end
+
+    context 'and credit card does require 3ds authentication' do
+      let(:card_number) { '4000000000003220' }
+
+      it 'should not place the order' do
+        expect(page).to have_content('Your card was declined. This transaction requires authentication.')
+        expect(Spree::Order.complete.last).to be_nil
+      end
+    end
+  end
+
+  describe 'when intents are enabled' do
+    let(:preferred_intents) { true }
+
+    context 'and credit card does not require 3ds authentication' do
+      let(:card_number) { '4242424242424242' }
+
+      it 'should successfully place order without 3ds authentication' do
+        expect(page).to have_content('Order placed successfully')
+        order = Spree::Order.complete.last
+        expect(page.current_url).to include("/orders/#{order.number}")
+        expect(page).to have_content(order.number)
+      end
+    end
+
+    context 'when credit card does require 3ds authentication' do
+      let(:card_number) { '4000000000003220' }
+
+      context 'and authentication is successful' do
+        it 'should place order after 3ds authentication' do
+          within_stripe_3ds_popup do
+            click_button('Complete')
+          end
+
+          expect(page).to have_content('Order placed successfully')
+          order = Spree::Order.complete.last
+          expect(page.current_url).to include("/orders/#{order.number}")
+          expect(page).to have_content(order.number)
+        end
+      end
+
+      context 'and authentication is unsuccessful' do
+        it 'should not place order after 3ds authentication' do
+          within_stripe_3ds_popup do
+            click_button('Fail')
+          end
+
+          expect(page).to_not have_content('Order placed successfully')
+          expect(page).to have_content('We are unable to authenticate your payment method.')
+          expect(page).to have_content('Please choose a different payment method and try again.')
+          expect(Spree::Order.complete.last).to be_nil
+        end
+      end
+    end
+  end
+end

data/spec/models/gateway/stripe_ach_gateway_spec.rb

@@ -0,0 +1,186 @@
+require 'spec_helper'
+
+describe Spree::Gateway::StripeAchGateway do
+  let(:secret_key) { 'key' }
+  let(:email) { 'customer@example.com' }
+  let(:source) { Spree::Check.new }
+  let(:payment) {
+    double('Spree::Payment',
+           source: source,
+           order: double('Spree::Order',
+                         email: email,
+                         bill_address: bill_address,
+                         user: double('Spree::User',
+                                      email: email)
+           )
+    )
+  }
+  let(:provider) do
+    double('provider').tap do |p|
+      p.stub(:purchase)
+      p.stub(:authorize)
+      p.stub(:capture)
+      p.stub(:verify)
+    end
+  end
+
+  before do
+    subject.preferences = { secret_key: secret_key }
+    subject.stub(:options_for_purchase_or_auth).and_return(['money','check','opts'])
+    subject.stub(:provider).and_return provider
+  end
+
+  describe '#create_profile' do
+    before do
+      payment.source.stub(:update!)
+    end
+
+    context 'with an order that has a bill address' do
+      let(:bill_address) {
+        double('Spree::Address',
+               address1: '123 Happy Road',
+               address2: 'Apt 303',
+               city: 'Suzarac',
+               zipcode: '95671',
+               state: double('Spree::State', name: 'Oregon'),
+               country: double('Spree::Country', name: 'United States')
+        )
+      }
+
+      it 'stores the bill address with the provider' do
+        subject.provider.should_receive(:store).with(payment.source, {
+            email: email,
+            login: secret_key,
+
+            address: {
+                address1: '123 Happy Road',
+                address2: 'Apt 303',
+                city: 'Suzarac',
+                zip: '95671',
+                state: 'Oregon',
+                country: 'United States'
+            }
+        }).and_return double.as_null_object
+
+        subject.create_profile payment
+      end
+    end
+
+    context 'with an order that does not have a bill address' do
+      let(:bill_address) { nil }
+
+      it 'does not store a bill address with the provider' do
+        subject.provider.should_receive(:store).with(payment.source, {
+            email: email,
+            login: secret_key
+        }).and_return double.as_null_object
+
+        subject.create_profile payment
+      end
+
+    end
+
+    context 'with a check represents payment_profile' do
+      let(:source) { Spree::Check.new(gateway_payment_profile_id: 'tok_profileid') }
+      let(:bill_address) { nil }
+
+      it 'stores the profile_id as a check' do
+        subject.provider.should_receive(:store).with(source.gateway_payment_profile_id, anything).and_return double.as_null_object
+
+        subject.create_profile payment
+      end
+    end
+  end
+
+  context 'purchasing' do
+    after do
+      subject.purchase(19.99, 'check', {})
+    end
+
+    it 'send the payment to the provider' do
+      provider.should_receive(:purchase).with('money', 'check', 'opts')
+    end
+  end
+
+  context 'authorizing' do
+    after do
+      subject.authorize(19.99, 'check', {})
+    end
+
+    it 'send the authorization to the provider' do
+      provider.should_receive(:authorize).with('money', 'check', 'opts')
+    end
+  end
+
+  context 'verifying' do
+    after do
+      subject.verify(source, {})
+    end
+
+    it 'send the verify to the provider' do
+      provider.should_receive(:verify).with(source, anything)
+    end
+  end
+
+  context 'capturing' do
+
+    after do
+      subject.capture(1234, 'response_code', {})
+    end
+
+    it 'convert the amount to cents' do
+      provider.should_receive(:capture).with(1234, anything, anything)
+    end
+
+    it 'use the response code as the authorization' do
+      provider.should_receive(:capture).with(anything, 'response_code', anything)
+    end
+  end
+
+  context 'capture with payment class' do
+    let(:gateway) do
+      gateway = described_class.new(active: true)
+      gateway.set_preference :secret_key, secret_key
+      gateway.stub(:options_for_purchase_or_auth).and_return(['money', 'check', 'opts'])
+      gateway.stub(:provider).and_return provider
+      gateway.stub source_required: true
+      gateway
+    end
+
+    let(:order) { Spree::Order.create }
+
+    let(:check) do
+      # mock_model(Spree::Check, :gateway_customer_profile_id => 'cus_abcde',
+      # :imported => false)
+      create :check, gateway_customer_profile_id: 'cus_abcde', imported: false
+    end
+
+    let(:payment) do
+      payment = Spree::Payment.new
+      payment.source = check
+      payment.order = order
+      payment.payment_method = gateway
+      payment.amount = 98.55
+      payment.state = 'pending'
+      payment.response_code = '12345'
+      payment
+    end
+
+    after do
+      payment.capture!
+    end
+
+    let!(:success_response) do
+      double('success_response',
+             success?: true,
+             authorization: '123',
+             avs_result: { 'code' => 'avs-code' },
+             cvv_result: { 'code' => 'cvv-code', 'message' => 'CVV Result' },
+             params: {})
+    end
+
+    it 'gets correct amount' do
+      provider.should_receive(:capture).with(9855, '12345', anything).and_return(success_response)
+    end
+  end
+end

data/spec/models/gateway/stripe_gateway_spec.rb

@@ -182,6 +182,7 @@ describe Spree::Gateway::StripeGateway do
 
     let!(:success_response) do
       double('success_response', :success? => true,
+                               :params => {},
                                :authorization => '123',
                                :avs_result => { 'code' => 'avs-code' },
                                :cvv_result => { 'code' => 'cvv-code', 'message' => "CVV Result"})

data/spec/spec_helper.rb

@@ -1,67 +1,10 @@
-require 'simplecov'
-SimpleCov.start 'rails'
+# Configure Rails Environment
+ENV['RAILS_ENV'] = 'test'
 
-ENV["RAILS_ENV"] = "test"
+require File.expand_path('../dummy/config/environment.rb', __FILE__)
 
-require File.expand_path("../dummy/config/environment.rb",  __FILE__)
+require 'spree_dev_tools/rspec/spec_helper'
 
-require 'rspec/rails'
-require 'rspec/active_model/mocks'
-require 'capybara/rspec'
-require 'capybara/rails'
-require 'capybara-screenshot/rspec'
-require "selenium-webdriver"
-require 'webdrivers'
-require 'database_cleaner'
-require 'ffaker'
-require 'rspec/active_model/mocks'
-require 'pry'
-
-Dir[File.join(File.dirname(__FILE__), "support", "**", "*.rb")].each { |f| require f }
-
-require 'spree/testing_support/factories'
-require 'spree/testing_support/order_walkthrough'
-require 'spree/testing_support/preferences'
-require 'spree/testing_support/capybara_ext'
-
-FactoryBot.find_definitions
-
-RSpec.configure do |config|
-  config.infer_spec_type_from_file_location!
-  config.mock_with :rspec do |mock|
-    mock.syntax = [:should, :expect]
-  end
-  config.raise_errors_for_deprecations!
-  config.use_transactional_fixtures = false
-  #config.filter_run focus: true
-  #config.filter_run_excluding slow: true
-
-  config.include FactoryBot::Syntax::Methods
-  config.include Spree::TestingSupport::Preferences
-
-  config.before :suite do
-    DatabaseCleaner.strategy = :transaction
-    DatabaseCleaner.clean_with :truncation
-  end
-
-  config.before do
-    DatabaseCleaner.strategy = RSpec.current_example.metadata[:js] ? :truncation : :transaction
-    DatabaseCleaner.start
-    reset_spree_preferences
-    create(:store)
-  end
-
-  config.after do
-    DatabaseCleaner.clean
-  end
-
-  Capybara.register_driver :chrome do |app|
-    Capybara::Selenium::Driver.new app,
-      browser: :chrome,
-      options: Selenium::WebDriver::Chrome::Options.new(
-        args: %w[no-sandbox disable-dev-shm-usage disable-popup-blocking headless disable-gpu window-size=1920,1080 --enable-features=NetworkService,NetworkServiceInProcess --disable-features=VizDisplayCompositor],
-        log_level: :error
-      )
-  end
-  Capybara.javascript_driver = :chrome
-end
+# Requires supporting ruby files with custom matchers and macros, etc,
+# in spec/support/ and its subdirectories.
+Dir[File.join(File.dirname(__FILE__), 'support/**/*.rb')].sort.each { |f| require f }