solidus_paypal_braintree 0.2.0 → 1.1.0

data/lib/views/backend_v2.4/spree/admin/shared/preference_fields/_hash.html.erb

@@ -0,0 +1,12 @@
+<% label = local_assigns[:label].presence %>
+<% html_options = {class: 'input_hash fullwidth'}.update(local_assigns[:html_options] || {}) %>
+
+<div class="field">
+  <% if local_assigns[:form] %>
+    <%= form.label attribute, label %>
+    <%= form.text_area attribute, html_options %>
+  <% else %>
+    <%= label_tag name, label %>
+    <%= text_area_tag name, value, html_options %>
+  <% end %>
+</div>

data/lib/views/frontend/solidus_paypal_braintree/payments/_payment.html.erb

@@ -0,0 +1,12 @@
+<br />
+<%= payment.source.display_payment_type %>
+
+<% if payment.source.paypal? %>
+  <% if payment.source.respond_to?(:paypal_funding_source) && payment.source.paypal_funding_source.present? %>
+    <br />
+    <%= t('spree.paypal_funding', funding: payment.source.display_paypal_funding_source) %>
+  <% end %>
+<% elsif payment.source.venmo? %>
+  <br />
+  <%= payment.source.source_description %>
+<% end %>

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

@@ -1,90 +1,23 @@
-<% address = current_order.ship_address %>
 <% id = payment_method.id %>
 
-<% content_for :head do %>
-  <script src="https://js.braintreegateway.com/web/3.14.0/js/client.min.js"></script>
-  <script src="https://js.braintreegateway.com/web/3.14.0/js/data-collector.min.js"></script>
-
-  <% if current_store.braintree_configuration.paypal? %>
-    <script src="https://js.braintreegateway.com/web/3.14.0/js/paypal.min.js"></script>
-  <% end %>
-
-  <% if current_store.braintree_configuration.credit_card? %>
-    <script src="https://js.braintreegateway.com/web/3.14.0/js/hosted-fields.min.js"></script>
-  <% end %>
-
-  <% if current_store.braintree_configuration.apple_pay? %>
-    <script src="https://js.braintreegateway.com/web/3.14.0/js/apple-pay.min.js"></script>
-  <% end %>
-
-  <%= javascript_include_tag "solidus_paypal_braintree/checkout" %>
-<% end %>
+<%= render "spree/shared/paypal_braintree_head_scripts" %>
 
 <% if current_store.braintree_configuration.paypal? %>
-  <script src="https://www.paypalobjects.com/api/button.js?"
-          data-merchant="braintree"
-          data-id="paypal-button"
-          data-button="checkout"
-          data-color="blue"
-          data-size="medium"
-          data-shape="pill"
-          data-button_type="button"
-          data-button_disabled="true"
-          ></script>
-
-  <script>
-    var address = {
-      line1: '<%= address.address1 %>',
-      line2: '<%= address.address2 %>',
-      city: '<%= address.city %>',
-      state: '<%= address.state.name %>',
-      postalCode: '<%= address.zipcode %>',
-      countryCode: '<%= address.country.iso %>',
-      phone: '<%= address.phone %>',
-      recipientName: '<%= "#{address.firstname} #{address.lastname}" %>'
-    }
-
-    var paypalOptions = {
-      flow: 'vault',
-      enableShippingAddress: true,
-      shippingAddressOverride: address,
-      shippingAddressEditable: false
-    }
-
-    var button = new SolidusPaypalBraintree.createPaypalButton(document.querySelector("#paypal-button"), paypalOptions);
-    button.initialize();
-  </script>
+  <%= render "spree/shared/paypal_checkout_button" %>
 <% end %>
 
 <% if current_store.braintree_configuration.credit_card? %>
   <fieldset class="braintree-hosted-fields" data-braintree-hosted-fields data-id="<%= id %>">
-    <%= render "spree/shared/braintree_hosted_fields", id: id %>
+    <%= render "spree/shared/braintree_hosted_fields", payment_method: payment_method %>
   </fieldset>
 <% end %>
 
 <% if current_store.braintree_configuration.apple_pay? %>
-  <button id="apple-pay-button" class="apple-pay-button"></button>
-  <script>
-    var applePayButtonElement = document.getElementById('apple-pay-button');
-    var applePayOptions = {
-      paymentMethodId: <%= id %>,
-      storeName: "<%= current_store.name %>",
-      orderEmail: "<%= current_order.email %>",
-      amount: "<%= current_order.total %>",
-      shippingContact: {
-        emailAddress: '<%= current_order.email %>',
-        familyName: '<%= address.firstname %>',
-        givenName: '<%= address.lastname %>',
-        phoneNumber: '<%= address.phone %>',
-        addressLines: ['<%= address.address1 %>','<%= address.address2 %>'],
-        locality: '<%= address.city %>',
-        administrativeArea: '<%= address.state.name %>',
-        postalCode: '<%= address.zipcode %>',
-        country: '<%= address.country.name %>',
-        countryCode: '<%= address.country.iso %>'
-      }
-    };
-    var button = new SolidusPaypalBraintree.createApplePayButton(applePayButtonElement, applePayOptions);
-    button.initialize();
-  </script>
+  <%= render "spree/shared/apple_pay_button", id: id %>
 <% end %>
+
+<% if current_store.braintree_configuration.venmo? %>
+  <%= render "spree/shared/venmo_button", payment_method: payment_method %>
+<% end %>
+
+<%= render "spree/shared/braintree_errors" %>

data/lib/views/frontend/spree/shared/_paypal_checkout_button.html.erb

@@ -0,0 +1,32 @@
+<div id="paypal-button"></div>
+<div data-pp-message data-pp-placement="payment" data-pp-amount="<%= @order.total %>"></div>
+
+<script>
+  var address = <%= sanitize SolidusPaypalBraintree::Address.new(current_order.ship_address).to_json %>
+
+  var paypalOptions = {
+    flow: '<%= SolidusPaypalBraintree::Gateway.first.preferred_paypal_flow %>',
+    amount: '<%= current_order.total %>',
+    currency: '<%= current_order.currency %>',
+    enableShippingAddress: true,
+    venmoFunding: <%= SolidusPaypalBraintree::Gateway.first.preferred_enable_venmo_funding %>,
+    buyerCountry: '<%= SolidusPaypalBraintree::Gateway.first.preferred_force_buyer_country %>',
+    shippingAddressOverride: address,
+    shippingAddressEditable: false,
+    environment: '<%= Rails.env.production? ? "production" : "sandbox" %>',
+    locale: '<%= paypal_button_preference(:paypal_button_locale, store: current_store) %>',
+    style: {
+      color: '<%= paypal_button_preference(:paypal_button_color, store: current_store) %>',
+      shape: '<%= paypal_button_preference(:paypal_button_shape, store: current_store) %>',
+      label: '<%= paypal_button_preference(:paypal_button_label, store: current_store) %>',
+      layout: '<%= paypal_button_preference(:paypal_button_layout, store: current_store) %>',
+      <% if paypal_button_preference(:paypal_button_layout, store: current_store) == "horizontal" %>
+        tagline: '<%= paypal_button_preference(:paypal_button_tagline, store: current_store) %>',
+      <% end %>
+      messaging: '<%= paypal_button_preference(:paypal_button_messaging, store: current_store) %>'
+    }
+  }
+
+  var button = new SolidusPaypalBraintree.createPaypalButton(document.querySelector("#paypal-button"), paypalOptions);
+  button.initialize();
+</script>

data/solidus_paypal_braintree.gemspec

@@ -0,0 +1,43 @@
+# frozen_string_literal: true
+
+$:.push File.expand_path('lib', __dir__)
+require 'solidus_paypal_braintree/version'
+
+Gem::Specification.new do |s|
+  s.name        = 'solidus_paypal_braintree'
+  s.version     = SolidusPaypalBraintree::VERSION
+  s.summary     = 'Officially supported Paypal/Braintree extension'
+  s.description = 'Uses the javascript API for seamless braintree payments'
+  s.license     = 'BSD-3-Clause'
+
+  s.author    = 'Stembolt'
+  s.email     = 'braintree+gemfile@stembolt.com'
+  s.homepage  = 'https://github.com/solidusio/solidus_paypal_braintree'
+
+  s.required_ruby_version = '>= 2.5'
+
+  s.files = Dir.chdir(File.expand_path(__dir__)) do
+    `git ls-files -z`.split("\x0").reject { |f| f.match(%r{^(test|spec|features)/}) }
+  end
+  s.test_files = Dir['spec/**/*']
+  s.bindir = "exe"
+  s.executables = s.files.grep(%r{^exe/}) { |f| File.basename(f) }
+  s.require_paths = ["lib"]
+
+  if s.respond_to?(:metadata)
+    s.metadata["homepage_uri"] = s.homepage if s.homepage
+    s.metadata["source_code_uri"] = s.homepage if s.homepage
+    s.metadata["rubygems_mfa_required"] = 'true'
+  end
+
+  s.add_dependency 'activemerchant', '~> 1.48'
+  s.add_dependency 'braintree', '~> 3.4'
+  s.add_dependency 'solidus_api', ['>= 2.0.0', '< 4']
+  s.add_dependency 'solidus_core', ['>= 2.0.0', '< 4']
+  s.add_dependency 'solidus_support', ['>= 0.8.1', '< 1']
+
+  s.add_development_dependency 'selenium-webdriver'
+  s.add_development_dependency 'solidus_dev_support'
+  s.add_development_dependency 'vcr'
+  s.add_development_dependency 'webmock'
+end

data/spec/controllers/solidus_paypal_braintree/checkouts_controller_spec.rb

@@ -0,0 +1,99 @@
+require 'spec_helper'
+require 'support/order_ready_for_payment'
+
+RSpec.describe SolidusPaypalBraintree::CheckoutsController, type: :controller do
+  routes { SolidusPaypalBraintree::Engine.routes }
+
+  include_context 'when order is ready for payment'
+
+  describe 'PATCH update' do
+    subject(:patch_update) { patch :update, params: params }
+
+    let(:params) do
+      {
+        "state" => "payment",
+        "order" => {
+          "payments_attributes" => [
+            {
+              "payment_method_id" => payment_method.id,
+              "source_attributes" => {
+                "nonce" => "fake-paypal-billing-agreement-nonce",
+                "payment_type" => SolidusPaypalBraintree::Source::PAYPAL
+              }
+            }
+          ],
+          "use_billing" => "1",
+          "use_postmates_shipping" => "0"
+        },
+        "reuse_credit" => "1",
+        "order_bill_address" => "",
+        "reuse_bill_address" => "1"
+      }
+    end
+
+    let!(:payment_method) do
+      create_gateway
+    end
+
+    before do
+      allow(controller).to receive(:try_spree_current_user) { user }
+      allow(controller).to receive(:spree_current_user) { user }
+      allow(controller).to receive(:current_order) { order }
+    end
+
+    context "when a payment is created successfully", vcr: {
+      cassette_name: 'checkout/update',
+      match_requests_on: [:braintree_uri]
+    } do
+      it 'creates a payment' do
+        expect { patch_update }.
+          to change { order.payments.count }.
+          from(0).
+          to(1)
+      end
+
+      it 'creates a payment source' do
+        expect { patch_update }.
+          to change { SolidusPaypalBraintree::Source.count }.
+          from(0).
+          to(1)
+      end
+
+      it 'assigns @order' do
+        patch_update
+        expect(assigns(:order)).to eq order
+      end
+
+      it "is successful" do
+        expect(patch_update).to be_successful
+      end
+
+      it "renders 'ok'" do
+        expect(patch_update.body).to eql("ok")
+      end
+    end
+
+    context "when a payment is not created successfully" do
+      before do
+        allow_any_instance_of(::Spree::Payment).to receive(:save).and_return(false)
+      end
+
+      # No idea why this is the case, but I'm just adding the tests here
+      it "is successful" do
+        expect(patch_update).to be_successful
+      end
+
+      it "renders 'not-ok'" do
+        expect(patch_update.body).to eq('not-ok')
+      end
+
+      it "does not change the number of payments in the system" do
+        expect{ patch_update }.not_to(change{ ::Spree::Payment.count })
+      end
+
+      it "does not change the number of sources in the system" do
+        expect{ patch_update }.not_to(change{ SolidusPaypalBraintree::Source.count })
+      end
+    end
+  end
+end

data/spec/controllers/solidus_paypal_braintree/client_tokens_controller_spec.rb

@@ -0,0 +1,55 @@
+require 'spec_helper'
+
+describe SolidusPaypalBraintree::ClientTokensController do
+  routes { SolidusPaypalBraintree::Engine.routes }
+
+  cassette_options = { cassette_name: "braintree/token" }
+  describe "POST create", vcr: cassette_options do
+    let!(:gateway) { create_gateway }
+    let(:user) { create(:user) }
+    let(:json) { JSON.parse(response.body) }
+
+    before { user.generate_spree_api_key! }
+
+    context 'without a payment method id' do
+      subject(:response) do
+        post :create, params: { token: user.spree_api_key }
+      end
+
+      it "returns a client token", aggregate_failures: true do
+        expect(response).to have_http_status(:success)
+        expect(response.content_type).to include 'application/json'
+        expect(json["client_token"]).to be_present
+        expect(json["client_token"]).to be_a String
+        expect(json["payment_method_id"]).to eq gateway.id
+      end
+
+      context "when there's two gateway's for different stores" do
+        let!(:store1) { create(:store, code: 'store_1') }
+        let!(:store2) { create(:store, code: 'store_2') }
+        let!(:gateway_for_store1) { create_gateway.tap{ |gw| store1.payment_methods << gw } }
+        let!(:gateway_for_store2) { create_gateway.tap{ |gw| store2.payment_methods << gw } }
+
+        it "returns the correct gateway for store1" do
+          allow_any_instance_of(described_class).to receive(:current_store).and_return store1
+          expect(json["payment_method_id"]).to eq gateway_for_store1.id
+        end
+
+        it "returns the correct gateway for store2" do
+          allow_any_instance_of(described_class).to receive(:current_store).and_return store2
+          expect(json["payment_method_id"]).to eq gateway_for_store2.id
+        end
+      end
+    end
+
+    context 'with a payment method id' do
+      subject(:response) do
+        post :create, params: { token: user.spree_api_key, payment_method_id: gateway.id }
+      end
+
+      it 'uses the selected gateway' do
+        expect(json["payment_method_id"]).to eq gateway.id
+      end
+    end
+  end
+end

data/spec/controllers/solidus_paypal_braintree/configurations_controller_spec.rb

@@ -0,0 +1,73 @@
+require 'spec_helper'
+
+describe SolidusPaypalBraintree::ConfigurationsController, type: :controller do
+  routes { SolidusPaypalBraintree::Engine.routes }
+
+  let!(:store_1) { create :store }
+  let!(:store_2) { create :store }
+
+  stub_authorization!
+
+  describe "GET #list" do
+    subject { get :list }
+
+    it "assigns all store's configurations as @configurations" do
+      subject
+      expect(assigns(:configurations)).
+        to eq [store_1.braintree_configuration, store_2.braintree_configuration]
+    end
+
+    it "renders the correct view" do
+      expect(subject).to render_template :list
+    end
+  end
+
+  describe "POST #update" do
+    subject { post :update, params: configurations_params }
+
+    let(:paypal_button_color) { 'blue' }
+    let(:configurations_params) do
+      {
+        configurations: {
+          configuration_fields: {
+            store_1.braintree_configuration.id.to_s => { paypal: true, apple_pay: true },
+            store_2.braintree_configuration.id.to_s => {
+              paypal: true,
+              apple_pay: false,
+              preferred_paypal_button_color: paypal_button_color
+            }
+          }
+        }
+      }
+    end
+
+    context "with valid parameters" do
+      it "updates the configuration" do
+        expect { subject }.to change { store_1.braintree_configuration.reload.paypal }.
+          from(false).to(true)
+      end
+
+      it "displays a success message to the user" do
+        subject
+        expect(flash[:success]).to eq "Successfully updated Braintree configurations."
+      end
+
+      it "displays all configurations" do
+        expect(subject).to redirect_to action: :list
+      end
+    end
+
+    context "with invalid parameters" do
+      let(:paypal_button_color) { 'invalid-color' }
+
+      it "displays an error message to the user" do
+        subject
+        expect(flash[:error]).to eq "An error occurred while updating Braintree configurations."
+      end
+
+      it "returns the user to the edit page" do
+        expect(subject).to redirect_to action: :list
+      end
+    end
+  end
+end

data/spec/controllers/solidus_paypal_braintree/transactions_controller_spec.rb

@@ -0,0 +1,183 @@
+require 'spec_helper'
+
+RSpec.describe SolidusPaypalBraintree::TransactionsController, type: :controller do
+  routes { SolidusPaypalBraintree::Engine.routes }
+
+  let!(:country) { create :country }
+  let(:line_item) { create :line_item, price: 50 }
+  let(:order) do
+    Spree::Order.create!(
+      line_items: [line_item],
+      email: 'test@example.com',
+      bill_address: create(:address, country: country),
+      ship_address: create(:address, country: country),
+      user: create(:user)
+    )
+  end
+
+  let(:payment_method) { create_gateway }
+
+  before do
+    allow(controller).to receive(:spree_current_user) { order.user }
+    allow(controller).to receive(:current_order) { order }
+    create :shipping_method, cost: 5
+    create :state, abbr: "WA", country: country
+  end
+
+  cassette_options = {
+    cassette_name: "transactions_controller/create",
+    match_requests_on: [:braintree_uri]
+  }
+  describe "POST create", vcr: cassette_options do
+    subject(:post_create) { post :create, params: params }
+
+    let!(:country) { create :country, iso: 'US' }
+
+    let(:params) do
+      {
+        transaction: {
+          nonce: "fake-valid-nonce",
+          payment_type: SolidusPaypalBraintree::Source::PAYPAL,
+          phone: "1112223333",
+          email: "batman@example.com",
+          address_attributes: {
+            name: "Wade Wilson",
+            address_line_1: "123 Fake Street",
+            city: "Seattle",
+            zip: "98101",
+            state_code: "WA",
+            country_code: "US"
+          }
+        },
+        payment_method_id: payment_method.id
+      }
+    end
+
+    before do
+      create :state, abbr: "WA", country: country
+    end
+
+    context "when import has invalid address" do
+      before { params[:transaction][:address_attributes][:city] = nil }
+
+      it "raises a validation error" do
+        expect { post_create }.to raise_error(
+          SolidusPaypalBraintree::TransactionsController::InvalidImportError,
+          "Import invalid: " \
+          "Address is invalid, " \
+          "Address city can't be blank"
+        )
+      end
+    end
+
+    context "when the transaction is valid", vcr: {
+      cassette_name: 'transaction/import/valid',
+      match_requests_on: [:braintree_uri]
+    } do
+      it "imports the payment" do
+        expect { post_create }.to change { order.payments.count }.by(1)
+        expect(order.payments.first.amount).to eq 55
+      end
+
+      context "when no end state is provided" do
+        it "advances the order to confirm" do
+          post_create
+          expect(order).to be_confirm
+        end
+      end
+
+      context "when end state provided is delivery" do
+        let(:params) { super().merge(state: 'delivery') }
+
+        it "advances the order to delivery" do
+          post_create
+          expect(order).to be_delivery
+        end
+      end
+
+      context "with provided address" do
+        it "creates a new address" do
+          # Creating the order also creates 3 addresses, we want to make sure
+          # the transaction import only creates 1 new one
+          order
+          expect { post_create }.to change { Spree::Address.count }.by(1)
+          expect(Spree::Address.last.address1).to eq "123 Fake Street"
+        end
+      end
+
+      context "without country ISO" do
+        before do
+          params[:transaction][:address_attributes][:country_code] = ""
+          params[:transaction][:address_attributes][:country_name] = "United States"
+        end
+
+        it "creates a new address, looking up the ISO by country name" do
+          order
+          expect { post_create }.to change { Spree::Address.count }.by(1)
+          expect(Spree::Address.last.country.iso).to eq "US"
+        end
+      end
+
+      context "without transaction address" do
+        before { params[:transaction].delete(:address_attributes) }
+
+        it "does not create a new address" do
+          order
+          expect { post_create }.not_to(change { Spree::Address.count })
+        end
+      end
+
+      context "when format is HTML" do
+        context "when import! leaves the order in confirm" do
+          it "redirects the user to the confirm page" do
+            expect(post_create).to redirect_to spree.checkout_state_path("confirm")
+          end
+        end
+
+        context "when import! completes the order" do
+          before { allow(order).to receive(:complete?).and_return(true) }
+
+          it "displays the order to the user" do
+            expect(post_create).to redirect_to spree.order_path(order)
+          end
+        end
+      end
+
+      context "when format is JSON" do
+        before { params[:format] = :json }
+
+        it "has a successful response" do
+          post_create
+          expect(response).to be_successful
+        end
+      end
+    end
+
+    context "when the transaction is invalid" do
+      before { params[:transaction][:email] = nil }
+
+      context "when format is HTML" do
+        it "raises an error including the validation messages" do
+          expect { post_create }.to raise_error(
+            SolidusPaypalBraintree::TransactionsController::InvalidImportError,
+            "Import invalid: Email can't be blank"
+          )
+        end
+      end
+
+      context "when format is JSON" do
+        before { params[:format] = :json }
+
+        it "has a failed status" do
+          post_create
+          expect(response.status).to eq 422
+        end
+
+        it "returns the errors as JSON" do
+          post_create
+          expect(JSON.parse(response.body)["errors"]["email"]).to eq ["can't be blank"]
+        end
+      end
+    end
+  end
+end

data/spec/features/backend/configuration_spec.rb

@@ -0,0 +1,23 @@
+require 'spec_helper'
+
+RSpec.describe "viewing the configuration interface" do
+  stub_authorization!
+
+  # Regression to ensure this page still renders on old versions of solidus
+  it "doesn't raise any errors due to unavailable route helpers" do
+    visit "/solidus_paypal_braintree/configurations/list"
+    expect(page).to have_content("Braintree Configurations")
+  end
+
+  # Regression to ensure this page renders on Solidus 2.4
+  it "doesn't raise any errors due to unavailable preference field partial" do
+    Rails.application.config.spree.payment_methods << SolidusPaypalBraintree::Gateway
+    Spree::PaymentMethod.create!(
+      type: 'SolidusPaypalBraintree::Gateway',
+      name: 'Braintree Payments'
+    )
+    visit '/admin/payment_methods'
+    page.find('a[title="Edit"]').click
+    expect(page).to have_field 'Name', with: 'Braintree Payments'
+  end
+end