spree_braintree_vzero 3.5.1

data/spec/models/gateway/braintree_vzero_dropin_ui_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Spree::Gateway::BraintreeVzeroDropInUi, :vcr do
+  subject { create(:vzero_dropin_ui_gateway) }
+
+  describe 'after_save' do
+    let(:hosted_fields_gateway) { create(:vzero_hosted_fields_gateway) }
+
+    it 'activation should disable Hosted Fields Gateways' do
+      subject.update_column(:active, false)
+      expect(hosted_fields_gateway.active?).to be true
+      subject.update(active: true)
+      expect(hosted_fields_gateway.reload.active?).to be false
+    end
+
+    it 'deactivation should not disable Hosted Fields Gateways' do
+      subject
+      expect(hosted_fields_gateway.active?).to be true
+      subject.update(active: false)
+      expect(hosted_fields_gateway.reload.active?).to be true
+    end
+
+    it 'creation of activated should disable Hosted Fields Gateways' do
+      expect(hosted_fields_gateway.active?).to be true
+      subject
+      expect(hosted_fields_gateway.reload.active?).to be false
+    end
+
+    it 'creation of deactivated should not disable Hosted Fields Gateways' do
+      expect(hosted_fields_gateway.active?).to be true
+      create(:vzero_dropin_ui_gateway, active: false)
+      expect(hosted_fields_gateway.reload.active?).to be true
+    end
+  end
+end

data/spec/models/gateway/braintree_vzero_hosted_fields_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Spree::Gateway::BraintreeVzeroHostedFields, :vcr do
+  subject { create(:vzero_hosted_fields_gateway) }
+
+  describe 'after_save' do
+    let(:dropin_ui_gateway) { create(:vzero_dropin_ui_gateway) }
+
+    it 'activation should disable DropIn UI Gateways' do
+      subject.update_column(:active, false)
+      expect(dropin_ui_gateway.active?).to be true
+      subject.update(active: true)
+      expect(dropin_ui_gateway.reload.active?).to be false
+    end
+
+    it 'deactivation should not disable DropIn UI Gateways' do
+      subject
+      expect(dropin_ui_gateway.active?).to be true
+      subject.update(active: false)
+      expect(dropin_ui_gateway.reload.active?).to be true
+    end
+
+    it 'creation of activated should disable DropIn UI Gateways' do
+      expect(dropin_ui_gateway.active?).to be true
+      subject
+      expect(dropin_ui_gateway.reload.active?).to be false
+    end
+
+    it 'creation of deactivated should not disable DropIn UI Gateways' do
+      expect(dropin_ui_gateway.active?).to be true
+      create(:vzero_dropin_ui_gateway, active: false)
+      expect(dropin_ui_gateway.reload.active?).to be true
+    end
+  end
+end

data/spec/models/spree/order_spec.rb

@@ -0,0 +1,186 @@
+require 'spec_helper'
+
+describe Spree::Order, :vcr do
+  let(:gateway) { create(:vzero_gateway, auto_capture: true) }
+  let(:payment) { create(:braintree_vzero_payment, payment_method: gateway) }
+  let(:payment_source) { payment.payment_source }
+  let(:order) { OrderWalkthrough.up_to(:payment) }
+  let(:add_payment_to_order!) { order.payments << payment }
+
+  describe 'complete with braintree vzero standard payment' do
+    let!(:complete_order!) do
+      add_payment_to_order!
+      2.times { order.next! }
+    end
+
+    context 'with auto_capture' do
+      it 'should complete payment' do
+        expect(payment.reload.state).to eq 'completed'
+      end
+
+      it "should update payment's response_code" do
+        expect(payment.reload.response_code).not_to be_blank
+      end
+
+      it "should update payment's source state and transaction_id" do
+        expect(payment.reload.source.state).to eq 'submitted_for_settlement'
+        expect(payment.reload.source.transaction_id).not_to be_blank
+      end
+    end
+
+    context 'without auto_capture' do
+      let(:gateway) { create(:vzero_gateway, auto_capture: false) }
+
+      it 'should pend payment' do
+        expect(payment.reload.state).to eq 'pending'
+      end
+
+      it "should update payment's response_code" do
+        expect(payment.reload.response_code).not_to be_blank
+      end
+
+      it "should update payment's source state and transaction_id" do
+        expect(payment.reload.source.state).to eq 'authorized'
+        expect(payment.reload.source.transaction_id).not_to be_blank
+      end
+    end
+  end
+
+  describe 'checkout steps' do
+    it 'should not include confirmation step by default' do
+      expect(order.checkout_steps).not_to include 'confirm'
+    end
+
+    it 'should include payment step by default' do
+      expect(order.checkout_steps).to include 'payment'
+    end
+
+    context 'with braintree dropin payment' do
+      before { add_payment_to_order! }
+
+      it 'should include confirmation step' do
+        order.update(state: 'payment')
+        expect(order.confirmation_required?).to be true
+      end
+
+      it 'should include payment step' do
+        expect(order.checkout_steps).to include 'payment'
+      end
+    end
+
+    context 'with braintree paypal express payment' do
+      let(:order) { OrderWalkthrough.up_to(:delivery) }
+      let(:gateway) { create(:vzero_paypal_gateway, auto_capture: true) }
+      before { add_payment_to_order! }
+
+      it 'should not include confirmation step when payed by paypal express from cart' do
+        order.update(state: 'delivery')
+        expect(order.confirmation_required?).to be false
+      end
+
+      it 'should include confirmation step when payed by paypal express from payment step' do
+        order.update(state: 'payment')
+        expect(order.confirmation_required?).to be true
+      end
+
+      it 'should not include payment step' do
+        expect(order.checkout_steps).not_to include 'payment'
+      end
+    end
+  end
+
+  describe 'addresses managment' do
+    let(:country) { create(:country, name: 'Poland', iso: 'PL') }
+    let(:state) { create(:state, name: 'Mazowieckie', abbr: 'MZ', country: country) }
+
+    context 'paypal addresses' do
+      let(:type) { 'ship_address' }
+      let(:attr) { address_hash.except(:full_name, :country, :state).keys.map(&:to_s) }
+      let(:address_hash) do
+        {
+          zipcode: '02-796',
+          city: 'Warsaw',
+          address1: 'gder 1/7',
+          address2: '13b',
+          phone: '408-391-8922',
+          full_name: 'Test ree ree Acd',
+          country: 'PL',
+          state: 'MZ'
+        }.with_indifferent_access
+      end
+
+      let(:prepare_data) do
+        order
+        state
+      end
+
+      let(:save_paypal_address!) do
+        prepare_data
+        expect(order.ship_address.attributes.slice(*attr)).not_to match address_hash.slice(*attr)
+        order.save_paypal_address(type, address_hash.dup)
+      end
+
+      shared_examples 'comparable' do
+        it 'should save basic address data without changes' do
+          expect(order.reload.ship_address.attributes.slice(*attr)).to match address_hash.slice(*attr)
+        end
+
+        it 'should find and set proper country' do
+          expect(order.reload.ship_address.country.iso).to eq address_hash[:country]
+        end
+
+        it 'should split name in proper way' do
+          name_array = address_hash[:full_name].split(' ')
+          last = name_array.slice!(-1)
+          first = name_array.join(' ')
+          expect(order.reload.ship_address.firstname).to eq first
+          expect(order.reload.ship_address.lastname).to eq last
+        end
+      end
+
+      context 'with state as abbreviation' do
+        before do
+          save_paypal_address!
+        end
+
+        include_examples 'comparable'
+
+        it 'should find and set proper state' do
+          expect(order.reload.ship_address.state.abbr).to eq address_hash[:state]
+        end
+      end
+
+      context 'with state as name' do
+        before do
+          address_hash[:state] = 'MAZOWIECKIE'
+          save_paypal_address!
+        end
+
+        include_examples 'comparable'
+
+        it 'should find and set proper state' do
+          expect(order.reload.ship_address.state.name).to eq address_hash[:state].capitalize
+        end
+      end
+
+      it 'should not include fields with text "undefined"' do
+        address_hash[:address2] = 'undefined'
+        save_paypal_address!
+        expect(order.reload.ship_address.address2).to eq nil
+      end
+
+      context 'billing address from shipping_address' do
+        it 'should be set when empty' do
+          order.update_column(:bill_address_id, nil)
+          order.set_billing_address
+          expect(order.reload.bill_address.attributes.slice(*attr)).to match order.ship_address.attributes.slice(*attr)
+        end
+
+        it 'should not be set when exists' do
+          order.set_billing_address
+          expect(order.reload.bill_address.attributes.slice(*attr)).not_to match address_hash.slice(*attr)
+        end
+      end
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,100 @@
+# 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
+
+# CodeClimate coverage
+require "simplecov"
+SimpleCov.start
+
+# 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'
+require 'spree/testing_support/order_walkthrough'
+
+# Requires factories defined in lib/spree_braintree_vzero/factories.rb
+require 'spree_braintree_vzero/factories'
+
+require 'pry'
+
+RSpec.configure do |config|
+  config.include FactoryBot::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
+
+  # == Requests support
+  #
+  # Adds convenient methods to request Spree's controllers
+  # spree_get :index
+  config.include Spree::TestingSupport::ControllerRequests, type: :controller
+
+  # == 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/spec/support/vcr.rb

@@ -0,0 +1,10 @@
+require 'vcr'
+
+VCR.configure do |config|
+  config.cassette_library_dir = 'spec/vcr'
+  config.hook_into :webmock
+  config.ignore_localhost = true
+  config.configure_rspec_metadata!
+  config.default_cassette_options = { match_requests_on: [:method, :host], record: :new_episodes }
+  config.ignore_hosts 'codeclimate.com'
+end

data/spec/vcr/Log_entries/with_a_failed_log_entry/shows_a_failed_attempt.yml

@@ -0,0 +1,91 @@
+---
+http_interactions:
+- request:
+    method: post
+    uri: https://5ffsvj9qxt38xptf:50ea069d312d35fbf779a66b6c86b897@api.sandbox.braintreegateway.com/merchants/zt9ssvxzppgzdjmn/transactions
+    body:
+      encoding: UTF-8
+      string: |
+        <?xml version="1.0" encoding="UTF-8"?>
+        <transaction>
+          <channel>SpreeCommerceInc_Cart_Braintree</channel>
+          <customer>
+          </customer>
+          <payment-method-nonce>fake-invalid-nonce</payment-method-nonce>
+          <amount>29.99</amount>
+          <order-id>R774803962</order-id>
+          <descriptor>
+            <name></name>
+          </descriptor>
+          <options>
+            <submit-for-settlement type="boolean">false</submit-for-settlement>
+            <add-billing-address-to-payment-method type="boolean">false</add-billing-address-to-payment-method>
+            <three-d-secure>
+              <required nil="true"/>
+            </three-d-secure>
+            <store-in-vault type="boolean">false</store-in-vault>
+          </options>
+          <type>sale</type>
+        </transaction>
+    headers:
+      Accept-Encoding:
+      - gzip
+      Accept:
+      - application/xml
+      User-Agent:
+      - Braintree Ruby Gem 2.73.0
+      X-Apiversion:
+      - '4'
+      Content-Type:
+      - application/xml
+  response:
+    status:
+      code: 422
+      message: Unprocessable Entity
+    headers:
+      Date:
+      - Sun, 09 Apr 2017 10:52:22 GMT
+      Content-Type:
+      - application/xml; charset=utf-8
+      Transfer-Encoding:
+      - chunked
+      X-Frame-Options:
+      - SAMEORIGIN
+      - SAMEORIGIN
+      X-Xss-Protection:
+      - 1; mode=block
+      X-Content-Type-Options:
+      - nosniff
+      X-Authentication:
+      - basic_auth
+      X-User:
+      - vzw4dwqpkk82wnfg
+      Vary:
+      - Accept-Encoding
+      Content-Encoding:
+      - gzip
+      Cache-Control:
+      - no-cache
+      X-Request-Id:
+      - 680372b6-1653-4628-ba65-446bbcb07235
+      X-Runtime:
+      - '0.157729'
+      Strict-Transport-Security:
+      - max-age=31536000; includeSubDomains
+    body:
+      encoding: ASCII-8BIT
+      string: !binary |-
+        H4sIAGYS6lgAA6RUTW/bMAy991cYvituu/XDgKICLTBg1209B4zNNkIlSqPk
+        rPn3kx1/xK07DOhN5CMpvkdK8u7VmmyPHLSjdX6xOs8zpMrVmp7X+eOvb+I2
+        v1NnErwWyOxYMAbvKKA6yzLZuUJ7HI0sHjyuc2CGQ170UGSgAFVMlxw9y+ED
+        NqCTnTypKVTlxdX1lSy68ykIMbLeNhH7euFgt87kysPBIsWNxbhz9YYcVSiL
+        MXpWw2II8IzqkV7I/aHMcYavXjPW2VKZlSyGjKntYtZ3bw4CFW9kOIWlBwYb
+        /qFXtQMiNOqnZ8QHZy1yhd+p2jwAx809g6aYkCROHzgmNiG6FJ2RNus8coPD
+        XLprO2biyEx0zNQTvKDQtAeje5csFgOHKmBdQ1FdlquyTPIerQF0XCMLXasf
+        Nzdfb8+/lNeXshidQ1SNoWLt4+nUJYHFxbaL9+HS+VavcJIemq3VUTyltQ0Y
+        o8GWQWJnQiK0DE7JUNdiq41JL0Gkc9r7IKITcx2GYv8XPBWPuzQqUaebq4bn
+        a8j4u+mWboF3t0QfpMo0ZW7HJvbQmInm3DsK+EYt2b4bFcCklO740cpOe/qZ
+        B5MUW/hR/gIAAP//AwBjnbFpjgQAAA==
+    http_version: 
+  recorded_at: Sun, 09 Apr 2017 10:52:22 GMT
+recorded_with: VCR 3.0.3