ba_spree_bank_transfer 1.0.1 → 1.0.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: a4b4b4ca1493ac7590b0921b3c1aed73dab92a2b66fea5142a9f3949390c9168
-  data.tar.gz: 6f74e44240418283b5f37bda7fb19fd0c1504e5ba3b446f68293d3d2c06112e9
+  metadata.gz: 24d026e1322d07f87561c83fff4932ca80e1d200fcff44e218a3a6046308360f
+  data.tar.gz: 32bbd18fe9f3c53a9892c0256487712d3ed8da22ad99c927316387299a71ca7e
 SHA512:
-  metadata.gz: 76b4b642a71c0fb3336fd4213eb63c0f00643d08c8d946cb47e3304ce16933d011baef18d3ad07026a2bd752afc6ef54d636cab7ff7143b7c831d90cba63befc
-  data.tar.gz: 6826521f78af5c619c48f116049ecc9d453e2acd2f5ee45743042e2b0ee6124c8421b91cf3a14518dcef22fb9af80bb8584841e30e6e5f7fa5ef622b00ce9cc8
+  metadata.gz: f4bc060da05a75ae66c630d29861670e3d980be26b295ce6e7209d03063c9b8cc40a22e4662b8c615bf5fe595eef248aec277650687afdd6ddc0d6d881e681cd
+  data.tar.gz: 5d9f332e55d49ecd1166c81cb475e976ea74050ac4f7a7a116ce54df9f1c068c118e94e743ed32c747a873919bed91bda2d22b3cf7625e24b9b97440945a35c4

data/lib/spree_bank_transfer/version.rb

@@ -1,3 +1,3 @@
 module SpreeBankTransfer
-  VERSION = "1.0.1"
+  VERSION = "1.0.2"
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ba_spree_bank_transfer
 version: !ruby/object:Gem::Version
-  version: 1.0.1
+  version: 1.0.2
 platform: ruby
 authors:
 - be agile Co., Ltd.
@@ -87,65 +87,35 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".gitignore"
-- ".rspec"
-- ".ruby-version"
-- ".travis.yml"
 - Gemfile
 - LICENSE
 - README.md
 - Rakefile
-- Versionfile
-- app/assets/images/payment_details.png
 - app/assets/images/spree_bank_transfer/payment_details.png
-- app/assets/javascripts/spree/backend/spree_bank_transfer.js
-- app/assets/javascripts/spree/frontend/spree_bank_transfer.js.erb
-- app/assets/stylesheets/spree/backend/spree_bank_transfer.css
-- app/assets/stylesheets/spree/frontend/spree_bank_transfer.css
 - app/controllers/spree/admin/banks_controller.rb
-- app/controllers/spree/content_controller_decorator.rb
 - app/controllers/spree/pages_controller_decorator.rb
 - app/controllers/spree/payments_controller.rb
-- app/helpers/.DS_Store
-- app/helpers/spree/.DS_Store
 - app/helpers/spree/admin/banks_helper.rb
 - app/helpers/spree/admin/base_bank_helper_decorator.rb
-- app/models/.DS_Store
-- app/models/spree/.DS_Store
 - app/models/spree/bank.rb
-- app/models/spree/order_decorator.rb
 - app/models/spree/payment/bank_updater.rb
-- app/models/spree/payment_decorator.rb
 - app/models/spree/payment_method/bank_transfer.rb
 - app/models/spree_bank_transfer/order_decorator.rb
 - app/models/spree_bank_transfer/payment_decorator.rb
-- app/overrides/add_bank_details_form_to_order_show.rb
-- app/overrides/add_bank_transfer_instructions_link_to_payment.rb
-- app/overrides/add_payment_reference_details.rb
-- app/overrides/filter_results_by_transaction_reference_no.rb
 - app/overrides/spree/add_bank_info_link_to_complete_page.rb
 - app/overrides/spree/add_bank_info_link_to_confirm_html_mail.rb
 - app/overrides/spree/add_bank_transfer_menu.rb
 - app/overrides/spree/admin/payments/_payment/add_payment_reference_details.html.erb.deface
 - app/overrides/spree/shared/_order_details/add_bank_details_form_to_order_show.html.erb.deface
-- app/views/.DS_Store
 - app/views/spree/admin/banks/_form.html.erb
 - app/views/spree/admin/banks/edit.html.erb
 - app/views/spree/admin/banks/index.html.erb
 - app/views/spree/admin/banks/new.html.erb
-- app/views/spree/admin/banks/toggle_activation.js.erb
 - app/views/spree/admin/banks/toggle_activation.turbo_stream.erb
-- app/views/spree/admin/payments/source_forms/_banktransfer.html.erb
-- app/views/spree/admin/payments/source_views/_banktransfer.html.erb
-- app/views/spree/checkout/.DS_Store
 - app/views/spree/checkout/payment/_banktransfer.html.erb
-- app/views/spree/content/bank_transfer.html.erb
-- app/views/spree/orders/_bank_transfer_form.html.erb
 - app/views/spree/pages/bank_transfer.html.erb
 - app/views/spree_bank_transfer/admin/banks/_bank_row.html.erb
 - app/views/spree_bank_transfer/orders/_bank_transfer_form.html.erb
-- ba_spree_bank_transfer.gemspec
-- bin/rails
 - config/initializers/admin_navigation.rb
 - config/locales/en.yml
 - config/locales/it.yml
@@ -160,21 +130,10 @@ files:
 - db/migrate/20140530055151_add_column_deposited_on_to_spree_payments.rb
 - db/migrate/20140612131656_add_col_additional_details_to_banks.rb
 - lib/ba_spree_bank_transfer.rb
-- lib/generators/spree_bank_transfer/.DS_Store
 - lib/generators/spree_bank_transfer/install/install_generator.rb
-- lib/payment_details.rb
 - lib/spree_bank_transfer.rb
 - lib/spree_bank_transfer/engine.rb
 - lib/spree_bank_transfer/version.rb
-- spec/controllers/spree/admin/banks_controller_spec.rb
-- spec/controllers/spree/content_controller_spec.rb
-- spec/controllers/spree/payments_controller_spec.rb
-- spec/lib/payment_details_spec.rb
-- spec/models/spree/bank_spec.rb
-- spec/models/spree/payment_method/bank_transfer_spec.rb
-- spec/models/spree/payment_spec.rb
-- spec/models/spree/user_spec.rb
-- spec/spec_helper.rb
 homepage: https://github.com/be-agile/ba_spree_bank_transfer
 licenses:
 - AGPL-3.0-or-later
@@ -199,13 +158,4 @@ rubygems_version: 3.4.9
 signing_key:
 specification_version: 4
 summary: Spree extension to create bank transfer payment method. (be agile fork)
-test_files:
-- spec/controllers/spree/admin/banks_controller_spec.rb
-- spec/controllers/spree/content_controller_spec.rb
-- spec/controllers/spree/payments_controller_spec.rb
-- spec/lib/payment_details_spec.rb
-- spec/models/spree/bank_spec.rb
-- spec/models/spree/payment_method/bank_transfer_spec.rb
-- spec/models/spree/payment_spec.rb
-- spec/models/spree/user_spec.rb
-- spec/spec_helper.rb
+test_files: []

data/.gitignore

@@ -1,15 +0,0 @@
-\#*
-*~
-.#*
-.DS_Store
-.idea
-.project
-.sass-cache
-coverage
-Gemfile.lock
-tmp
-nbproject
-pkg
-*.swp
-spec/dummy
-*.gem

data/.rspec

@@ -1 +0,0 @@
---color

data/.ruby-version

@@ -1 +0,0 @@
-3.1.4

data/.travis.yml

@@ -1,6 +0,0 @@
-language: ruby
-rvm:
-  - ruby-2.2.2
-script:
-  - bundle exec rake test_app
-  - bundle exec rspec .

data/Versionfile

@@ -1,7 +0,0 @@
-'4.1.x' => { branch: '4-1-stable' }
-'3.0.x' => { branch: '3-0-stable' }
-'2.4.x' => { branch: '2-4-stable' }
-'2.3.x' => { branch: '2-3-stable' }
-'2.2.x' => { branch: '2-2-stable' }
-'2.1.x' => { branch: '2-1-stable' }
-'2.0.x' => { branch: '2-0-stable' }

data/app/assets/javascripts/spree/frontend/spree_bank_transfer.js.erb

@@ -1,37 +0,0 @@
-//= require jquery-ui/widgets/datepicker
-$(document).ready(function() {
-  $('#banktransfer_instructions').on("click", function(event) {
-    newwindow = window.open($(this).attr('href'), 'bank transfer', 'left=20,top=20,width=650,height=750,toolbar=0,resizable=0,scrollbars=1');
-    if (window.focus) {
-      newwindow.focus();
-    }
-    return false;
-  });
-
-  var options = $.extend(
-    {}, // empty object
-    $.datepicker.regional['<%= I18n.default_locale %>'], // I18n locale
-    {dateFormat: '<%= Spree.t(:js_format, scope: "date_picker", default: "yy/mm/dd") %>',
-     firstDay: '<%= Spree.t(:first_day, scope: "date_picker", default: "0") %>'}
-  );
-  $.datepicker.setDefaults(options);
-  $('[name="payment[deposited_on]"]').datepicker();
-
-  $('.add_bt_details').on("click", function() {
-    $(this).hide();
-    $('.bt_payment_form').show();
-  });
-
-  $('.bt_payment_form input[type="submit"]').on("click", function(event) {
-    var empty_fields = $('.required_field').filter(function() { return !$(this).val().trim().length }), empty_fields_names = []
-    if ($(empty_fields).length) {
-      $(empty_fields).each(function() {
-        empty_fields_names.push($(this).attr('field-name'));
-      });
-
-      alert(empty_fields_names.join(', ') + " can't be blank");
-      event.preventDefault();
-      return false;
-    }
-  });
-});

data/app/assets/stylesheets/spree/frontend/spree_bank_transfer.css

@@ -1,9 +0,0 @@
-/*
- *= require jquery-ui
- *= require jquery-ui/datepicker
-*/
-.ui-datepicker {
-  width: 18em; /* fix calendar width */
-  padding: .2em .2em 0;
-}
-.hidden { display: none; }

data/app/controllers/spree/content_controller_decorator.rb

@@ -1,8 +0,0 @@
-module Spree
-  module ContentControllerDecorator
-    def bank_transfer
-    end
-  end
-end
-
-Spree::ContentController.prepend(Spree::ContentControllerDecorator)

data/app/models/spree/order_decorator.rb

@@ -1,3 +0,0 @@
-Spree::Order.class_eval do
-  self.whitelisted_ransackable_associations += %w(payments)
-end

data/app/models/spree/payment_decorator.rb

@@ -1,40 +0,0 @@
-# frozen_string_literal: true
-
-module Spree
-  module PaymentDecorator
-    def self.prepended(base)
-      base.class_eval do
-        attr_accessor :validate_bank_details
-
-        has_one_attached :receipt
-        belongs_to :order
-        validates :bank_name, :deposited_on, :receipt, presence: true, if: :validate_bank_details
-        after_save :update_order_state, if: :validate_bank_details
-        before_save :update_source_type, if: :validate_bank_details
-
-        scope :from_bank_transfer, lambda {
-                                     joins(:payment_method).where(spree_payment_methods: { type: 'Spree::PaymentMethod::BankTransfer' })
-                                   }
-
-        self.whitelisted_ransackable_attributes = %w[bank_name state]
-      end
-    end
-
-    def details_submitted?
-      bank_name?
-    end
-
-    private
-
-    def update_source_type
-      self.source_type = 'Spree::PaymentMethod::BankTransfer'
-    end
-
-    def update_order_state
-      order.update(payment_state: :paid)
-    end
-  end
-end
-
-# Register the decorator
-Spree::Payment.prepend(Spree::PaymentDecorator)

data/app/overrides/add_bank_details_form_to_order_show.rb

@@ -1,6 +0,0 @@
-Deface::Override.new(
-  virtual_path: 'spree/orders/show',
-  name: 'add_bank_details_form_to_order_show',
-  insert_bottom: ".payment-info",
-  partial: "spree/orders/bank_transfer_form"
-)

data/app/overrides/add_bank_transfer_instructions_link_to_payment.rb

@@ -1,10 +0,0 @@
-Deface::Override.new(
-  virtual_path: 'spree/checkout/_payment',
-  name: 'add_bank_transfer_instructions_link_to_payment',
-  insert_after: "#payment-method-fields",
-  text: %q{
-    <% if @order.available_payment_methods.any? { |pm| pm.type == 'Spree::PaymentMethod::BankTransfer' } %>
-      <p><%= link_to Spree.t(:how_bank_transfer_works), bank_transfer_instructions_path, id: 'banktransfer_instructions', target: "_blank" %></p>
-    <% end %>
-  }
-)

data/app/overrides/add_payment_reference_details.rb

@@ -1,32 +0,0 @@
-Deface::Override.new(
-  virtual_path: 'spree/admin/payments/_list',
-  name: 'add_payment_reference_details',
-  insert_after: "#payments",
-  text: %q{
-    <% if bank_transfer_payment = @order.payments.from_bank_transfer.first %>
-      <fieldset class="no-border-bottom" >
-        <legend align="center"><%= Spree.t(:payment_reference_details) %></legend>
-        <table class="index table">
-          <thead>
-            <tr data-hook="payments_header" class="table-active">
-              <th><%= Spree.t(:deposited_on) %></th>
-              <th><%= Spree.t(:bank_name) %></th>
-              <th><%= Spree.t(:receipt) %></th>
-            </tr>
-          </thead>
-          <tbody>
-            <tr>
-              <td class='align-center'><%= bank_transfer_payment.deposited_on %></td>
-              <td class='align-center'><%= bank_transfer_payment.bank_name %></td>
-              <td class='align-center' width="450px">
-                <% if bank_transfer_payment.receipt.attached? %>
-                  <%= image_tag main_app.url_for(bank_transfer_payment.receipt), class: "w-100" %>
-                <% end %>
-              </td>
-            </tr>
-          </tbody>
-        </table>
-      </fieldset>
-    <% end %>
-  }
-)

data/app/overrides/filter_results_by_transaction_reference_no.rb

@@ -1,17 +0,0 @@
-Deface::Override.new(
-  virtual_path: 'spree/admin/orders/index',
-  name: 'filter_results_by_transaction_reference_no',
-  insert_before: ".form-actions",
-  text: %q{
-    <div class="row">
-      <div class="col-md-4">
-        <div class="form-group">
-          <%= label_tag nil, Spree.t(:payment_state) %>
-          <%= f.select :payments_state_eq, Spree::Payment.state_machines[:state].states.collect {|s| [s.name, s.value]},
-                {include_blank: true},
-                {class: 'select2 form-control'} %>
-        </div>
-      </div>
-    </div>
-  }
-)

data/app/views/spree/admin/banks/toggle_activation.js.erb

@@ -1,10 +0,0 @@
-<% if @success %>
-  $('#active-status-<%= @bank.id %>').text('<%= bank_status(@bank) %>')
-  <% action_class = @bank.active ? "btn-danger" : "btn-success" %>
-  <% btn_class = "btn " + action_class + " btn-sm" %>
-  <% status = @bank.active ? "active" : "inactive" %>
-  <% bank_id = "bank-" + @bank[:id].to_s %>
-  $('#toggle-status-<%= @bank.id %>').html('<%= link_to_with_icon(action_text_to_toggle_bank_status(@bank), action_to_toggle_bank_status(@bank), toggle_activation_admin_bank_path(@bank), {class: btn_class, id: bank_id, rel: status, remote: true, method: :put}) %>')
-<% else %>
-  alert('<%= @bank.errors.full_messages.to_sentence %>')
-<% end %>

data/app/views/spree/content/bank_transfer.html.erb

@@ -1,38 +0,0 @@
-<div class="container mt-4 pb-3">
-  <h1 class="text-center h2">วิธีการชำระเงินโดยวิธี "โอนเงินผ่านธนาคาร"</h3>
-  <h4><strong>ขั้นตอนการแจ้งชำระเงิน:</strong></h4>
-  <ol>
-    <li>ชำระเงินเต็มจำนวนตามคำสั่งซื้อ ตามบัญชีธนาคารข้างล่าง
-      <% if (banks = Spree::Bank.active).any? %>
-        <div class="w-75">
-          <table class="table table-bordered">
-            <tr class="table-active">
-              <th><%= Spree.t(:bank_name) %></th>
-              <th><%= Spree.t(:account_no) %></th>
-              <th><%= Spree.t(:additional_details) %></th>
-            </tr>
-            <% banks.each do |bank| %>
-              <tr>
-                <td><%= bank.name %></td>
-                <td><%= bank.account_no %></td>
-                <td><%= bank.additional_details %></td>
-              </tr>
-            <% end %>
-          </table>
-        </div>
-      <% end %>
-    </li>
-    <li>แนบหลักฐานการโอนเงินผ่านระบบ
-      <ol>
-        <li>ไปยัง "บัญชีของฉัน" page.</li>
-        <li>เลือก คำสั่งซื้อ</li>
-        <li>คลิก "แนบหลักฐานการชำระเงิน"</li>
-        <li>กรอกข้อมูล และกด ยืนยัน</li>
-      </ol>
-    </li>
-  </ol>
-
-  <div class="w-100">
-    <%= image_tag 'payment_details.png', class: "w-75 img-thumbnail" %>
-  </div>
-</div>

data/app/views/spree/orders/_bank_transfer_form.html.erb

@@ -1,40 +0,0 @@
-<% bank_transfer_payment = @order.payments.from_bank_transfer.first %>
-<% if bank_transfer_payment.present? %>
-  <% unless bank_transfer_payment.details_submitted? %>
-    <br>
-    <%= link_to Spree.t(:add_details), "javascript:void(0);", class: 'button add_bt_details' %>
-    <%= form_for bank_transfer_payment, html: { class: "bt_payment_form", style: "#{bank_transfer_payment.details_submitted? ? '' : 'display:none;'}" } do |f| %>
-      <div class="row mx-0 my-3">
-        <div class="form-group col-12 px-0">
-          <%= f.label Spree.t(:deposited_on), class: 'mb-0' %>
-          <%= f.text_field :deposited_on, disabled: bank_transfer_payment.details_submitted?, class: 'required_field form-control form-control-sm' %>
-        </div>
-        <br>
-        <div class="form-group col-12 px-0">
-          <%= f.label Spree.t(:bank_name), class: 'mb-0' %>
-          <%= f.select :bank_name, options_for_select(Spree::Bank.pluck(:name)), {}, {disabled: bank_transfer_payment.details_submitted?, class: 'required_field form-control form-control-sm'} %>
-        </div>
-        <br>
-        <div class="form-group col-12 px-0">
-          <%= f.label Spree.t(:receipt), class: 'mb-0' %>
-          <%= f.file_field :receipt, disabled: bank_transfer_payment.details_submitted?, class: 'required_field form-control-file' %>
-        </div>
-        <br>
-        <div class="col-12 px-0">
-          <% unless bank_transfer_payment.details_submitted? %>
-            <%= f.submit "Submit", confirm: Spree.t(:confirm_details_submit), class: "btn btn-success btn-block btn-sm" %>
-          <% end %>
-        </div>
-      </div>
-    <% end %>
-  <% else %>
-    <div class="my-3">
-      <p class="text-success font-weight-bold mb-0">
-        <i class="fas fa-check-circle"></i> <%= Spree.t(:bank_transfer_completed) %>
-      </p>
-      <p class="mb-0"><%= Spree.t(:deposited_on) %>: <%= bank_transfer_payment.deposited_on.strftime("%d/%m/%Y") %></p>
-      <p class="mb-0"><%= Spree.t(:bank_name) %>: <%= bank_transfer_payment.bank_name %></p>
-      <a href="<%= main_app.url_for(bank_transfer_payment.receipt) %>" target="_blank"><%= Spree.t(:view_receipt) %></a>
-    </div>
-  <% end %>
-<% end %>

data/ba_spree_bank_transfer.gemspec

@@ -1,31 +0,0 @@
-# coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
-$LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'spree_bank_transfer/version'
-
-Gem::Specification.new do |s|
-  s.platform    = Gem::Platform::RUBY
-  s.name        = 'ba_spree_bank_transfer'
-  s.version     = SpreeBankTransfer::VERSION
-  s.summary     = 'Spree extension to create bank transfer payment method. (be agile fork)'
-  s.description = 'This Spree extension provides bank transfer payment method functionality. Compatible with Ruby 3.1, Rails 7.2 and Spree 5.0.'
-  s.required_ruby_version = '>= 3.1.4'
-
-  s.author      = 'be agile Co., Ltd.'
-  s.email       = 'develop@be-agile.jp'
-  s.homepage    = 'https://github.com/be-agile/ba_spree_bank_transfer'
-  s.licenses    = ['AGPL-3.0-or-later']
-
-  s.files        = `git ls-files`.split("\n")
-  s.test_files   = `git ls-files -- spec/*`.split("\n")
-  s.require_path = 'lib'
-  s.requirements << 'none'
-
-  s.add_dependency 'spree_core', '~> 5.3.0'
-  s.add_dependency 'spree_admin', '= 5.3.6'
-  s.add_dependency 'spree_storefront', '= 5.3.6'
-  s.add_dependency 'deface'
-
-  # @gem-override マーカーの差分確認に使う開発ツール
-  s.add_development_dependency 'gem_override_marker'
-end

data/bin/rails

@@ -1,6 +0,0 @@
-
-ENGINE_ROOT = File.expand_path('../..', __FILE__)
-ENGINE_PATH = File.expand_path('../../lib/spree_bank_transfer/engine', __FILE__)
-
-require 'rails/all'
-require 'rails/engine/commands'

data/lib/payment_details.rb

@@ -1,21 +0,0 @@
-class PaymentDetails
-  attr_accessor :payment, :errors, :params
-
-  def initialize(payment, params)
-    self.payment = payment
-    self.params = params
-    self.errors = []
-  end
-
-  def save
-    if payment.details_submitted?
-      errors << "Payment Details can be submitted only once."
-    else
-      payment.validate_bank_details = true
-      unless payment.update_attributes(params)
-        errors << payment.errors.full_messages.to_sentence
-      end
-    end
-    errors.empty?
-  end
-end

data/spec/controllers/spree/admin/banks_controller_spec.rb

@@ -1,82 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Admin::BanksController, type: :controller do
-
-  let(:role) { Spree::Role.create!(name: 'user') }
-  let(:roles) { [role] }
-
-  before(:each) do
-    @user = mock_model(Spree::User, :generate_spree_api_key! => false)
-    allow(@user).to receive_message_chain(:roles, :includes).and_return([])
-    allow(@user).to receive(:has_spree_role?).with('admin').and_return(true)
-    allow(controller).to receive(:spree_user_signed_in?).and_return(true)
-    allow(controller).to receive(:spree_current_user).and_return(@user)
-    allow(@user).to receive(:roles).and_return(roles)
-    allow(roles).to receive(:includes).with(:permissions).and_return(roles)
-    allow(controller).to receive(:authorize_admin).and_return(true)
-    allow(controller).to receive(:authorize!).and_return(true)
-  end
-
-  describe "GET index" do
-    before(:each) do
-      @bank1 = mock_model(Spree::Bank)
-      allow(Spree::Bank).to receive(:page).and_return([@bank1])
-    end
-
-    def send_request
-      get :index
-
-
-
-    end
-
-    it "assigns @banks" do
-      send_request
-      expect(assigns(:banks)).to eq([@bank1])
-    end
-
-    it "paginates banks" do
-      expect(Spree::Bank).to receive(:page)
-      send_request
-    end
-
-    it "renders index template" do
-      send_request
-      expect(response).to render_template(:index)
-    end
-  end
-
-  describe "PUT toggle_activation" do
-    before(:each) do
-      @bank = mock_model(Spree::Bank, :toggle! => true)
-      allow(Spree::Bank).to receive(:find).with("1").and_return(@bank)
-    end
-
-    def send_request
-      put :toggle_activation, id: "1", format: :js
-    end
-
-    it "toggles activation status of bank" do
-      expect(@bank).to receive(:toggle!).with(:active)
-      send_request
-    end
-
-    context "when activation status toggled successfully" do
-      it "sets @success to true" do
-        send_request
-        expect(assigns(:success)).to be true
-      end
-    end
-
-    context "when activation status not toggled successfully" do
-      before(:each) do
-        allow(@bank).to receive(:toggle!).and_return(false)
-      end
-
-      it "sets @success to false" do
-        send_request
-        expect(assigns(:success)).to be false
-      end
-    end
-  end
-end

data/spec/controllers/spree/content_controller_spec.rb

@@ -1,16 +0,0 @@
-require 'spec_helper'
-
-describe Spree::ContentController, type: :controller do
-  before do
-    allow(controller).to receive(:authenticate_spree_user!).and_return(true)
-    @user = mock_model(Spree::User, :generate_spree_api_key! => false, last_incomplete_spree_order: nil)
-    allow(controller).to receive(:spree_current_user).and_return(@user)
-  end
-
-  describe "GET bank_transfer" do
-    it "renders bank_transfer" do
-      get :bank_transfer
-      expect(response).to render_template("bank_transfer")
-    end
-  end
-end

data/spec/controllers/spree/payments_controller_spec.rb

@@ -1,98 +0,0 @@
-require 'spec_helper'
-
-describe Spree::PaymentsController, type: :controller do
-  before do
-    allow(controller).to receive(:authenticate_spree_user!).and_return(true)
-    @user = mock_model(Spree::User, :generate_spree_api_key! => false, last_incomplete_spree_order: nil)
-    allow(controller).to receive(:spree_current_user).and_return(@user)
-
-    @payment = mock_model(Spree::Payment)
-    @current_user_payments = double("current_user_payments", find_by: @payment)
-    allow(@user).to receive(:payments).and_return(@current_user_payments)
-    request.env["HTTP_REFERER"] = "http://localhost"
-  end
-
-  shared_examples_for "request which finds payment" do
-
-    it "finds payment" do
-      expect(@current_user_payments).to receive(:find_by).with(number: 'payment_id')
-      send_request
-    end
-
-    it "assigns @payment" do
-      send_request
-      expect(assigns(:payment)).to eq(@payment)
-    end
-
-    context 'when payment not found' do
-      before do
-        allow(@current_user_payments).to receive(:find_by).and_return(nil)
-      end
-
-      it "sets flash message" do
-        send_request
-        expect(flash[:error]).to eq("Payment not found")
-      end
-
-      it "redirects to back" do
-        send_request
-        expect(response).to redirect_to(:back)
-      end
-    end
-  end
-
-  describe "PATCH update" do
-    before do
-      @payment_details = double('payment_details', save: true)
-      allow(PaymentDetails).to receive(:new).and_return(@payment_details)
-    end
-
-    def send_request
-      patch :update, id: 'payment_id', payment: { bank_name: 'bank_name', account_no: "account_no", transaction_reference_no: "transaction_reference_no" }
-    end
-
-    it_behaves_like "request which finds payment"
-
-
-    it "creates new payment details" do
-      expect(PaymentDetails).to receive(:new).with(@payment, { 'bank_name' => 'bank_name', 'account_no' => "account_no", 'transaction_reference_no' => "transaction_reference_no" })
-      send_request
-    end
-
-    it "saves payment_details" do
-      expect(@payment_details).to receive(:save)
-      send_request
-    end
-
-    context 'when payment details saved successfully' do
-      it 'sets flash message' do
-        send_request
-        expect(flash[:notice]).to eq("Payment successfully updated")
-      end
-    end
-
-    context 'when payment details not saved' do
-      before do
-        allow(@payment_details).to receive(:save).and_return(false)
-        allow(@payment_details).to receive(:errors).and_return(["some error occurred"])
-      end
-
-      it 'sets flash error' do
-        send_request
-        expect(flash[:error]).to eq("some error occurred")
-      end
-    end
-
-    it "redirects to back" do
-      send_request
-      expect(response).to redirect_to(:back)
-    end
-  end
-
-  describe "#payment_params" do
-    it "permits only bank_name, account_no, transaction_reference_no, deposited_on" do
-      controller.params = { payment: { bank_name: 'Bank Name', account_no: 'Account number', transaction_reference_no: "transaction reference number", order_id: 'order_id', deposited_on: 'deposited_on' } }
-      expect(controller.send(:payment_params)).to eq({ "bank_name" => "Bank Name", "account_no" => "Account number", "transaction_reference_no" => "transaction reference number", 'deposited_on' => 'deposited_on' })
-    end
-  end
-end

data/spec/lib/payment_details_spec.rb

@@ -1,84 +0,0 @@
-require 'spec_helper'
-
-describe PaymentDetails do
-  before do
-    @payment = mock_model(Spree::Payment, update_attributes: true, :validate_bank_details= => nil)
-    @params = { 'bank_name' => 'bank_name', 'account_no' => "account_no", 'transaction_reference_no' => "transaction_reference_no" }
-    @payment_details = PaymentDetails.new(@payment, @params)
-  end
-
-  describe "#initialize" do
-    it "sets payment" do
-      expect(@payment_details.payment).to eq(@payment)
-    end
-
-    it "sets params" do
-      expect(@payment_details.params).to eq(@params)
-    end
-
-    it "initializes errors" do
-      expect(@payment_details.errors).to eq([])
-    end
-  end
-
-  describe "#save" do
-    context 'when payment details submitted' do
-      before do
-        allow(@payment).to receive(:details_submitted?).and_return(true)
-      end
-
-      it "sets errors" do
-        @payment_details.save
-        expect(@payment_details.errors).to eq(["Payment Details can be submitted only once."])
-      end
-
-      it "does not update payment" do
-        expect(@payment).not_to receive(:update_attributes)
-        @payment_details.save
-      end
-
-      it "returns false" do
-        expect(@payment_details.save).to be false
-      end
-
-    end
-
-    context 'when payment details not submitted' do
-      before do
-        allow(@payment).to receive(:details_submitted?).and_return(false)
-      end
-
-      it 'sets validates_bank_details to true' do
-        expect(@payment).to receive(:validate_bank_details=).with(true)
-        @payment_details.save
-      end
-
-      it "updates payment attributes" do
-        expect(@payment).to receive(:update_attributes)
-        @payment_details.save
-      end
-
-      context 'when payment attributes not updated successfully' do
-        before do
-          allow(@payment).to receive(:update_attributes).and_return(false)
-          allow(@payment).to receive_message_chain(:errors, :full_messages).and_return(["some error occurred"])
-        end
-
-        it 'sets errors' do
-          @payment_details.save
-          expect(@payment_details.errors).to eq(["some error occurred"])
-        end
-
-        it 'returns false' do
-          expect(@payment_details.save).to be false
-        end
-      end
-
-      context 'when payment attributes updated successfully' do
-        it 'returns true' do
-          expect(@payment_details.save).to be true
-        end
-      end
-    end
-  end
-end

data/spec/models/spree/bank_spec.rb

@@ -1,19 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Bank, type: :model do
-  it { is_expected.to validate_presence_of :name }
-  it { is_expected.to validate_presence_of :account_no }
-  it { is_expected.to validate_uniqueness_of(:account_no).scoped_to(:name) }
-
-  describe ".active" do
-    before(:each) do
-      @active_bank1 = Spree::Bank.create! name: 'bank1', :account_no => '98765'
-      @active_bank2 = Spree::Bank.create! name: 'bank2', :account_no => '98765'
-      @bank3 = Spree::Bank.create! name: 'bank3', :account_no => '98765', active: false
-    end
-
-    it "returns active banks" do
-      expect(Spree::Bank.active).to match([@active_bank1, @active_bank2])
-    end
-  end
-end

data/spec/models/spree/payment_method/bank_transfer_spec.rb

@@ -1,114 +0,0 @@
-require 'spec_helper'
-
-describe Spree::PaymentMethod::BankTransfer do
-  before(:each) do
-    @order = Spree::Order.new
-    @payment_method = Spree::PaymentMethod::BankTransfer.create! name: "Bank Transfer", active: true
-    @payment = Spree::Payment.new amount: 0.0
-    @payment.order = @order
-    @payment.payment_method = @payment_method
-    @payment.save!
-  end
-
-  describe "#actions" do
-    it "returns actions" do
-      expect(@payment_method.actions).to eq(["capture", "void"])
-    end
-  end
-
-  describe "#can_capture?" do
-    context "when payment state is pending or checkout" do
-      before(:each) do
-        @payment.state = 'checkout'
-        @payment.save!
-      end
-
-      it "returns true" do
-        expect(@payment_method.can_capture?(@payment)).to be true
-      end
-    end
-
-    context "when payment state is pending" do
-      before(:each) do
-        @payment.state = 'pending'
-        @payment.save!
-      end
-
-      it "returns true" do
-        expect(@payment_method.can_capture?(@payment)).to be true
-      end
-
-    end
-
-    context "when payment state is not pending or checkout" do
-      before(:each) do
-        @payment.state = 'void'
-        @payment.save!
-      end
-
-      it "returns false" do
-        expect(@payment_method.can_capture?(@payment)).to be false
-      end
-    end
-  end
-
-  describe "#can_void?" do
-    context "when payment state is not void" do
-      before(:each) do
-        @payment.state = 'pending'
-        @payment.save!
-      end
-
-      it "returns true" do
-        expect(@payment_method.can_void?(@payment)).to be true
-      end
-    end
-
-    context "when payment state is void" do
-      before(:each) do
-        @payment.update_column(:state, 'void')
-      end
-
-      it "returns false" do
-        expect(@payment_method.can_void?(@payment)).to be false
-      end
-    end
-  end
-
-  describe "#capture" do
-    it "creates a new active merchant billing response" do
-      expect(ActiveMerchant::Billing::Response).to receive(:new).with(true, "", {}, {})
-      @payment_method.capture
-    end
-    it "returns active merchant billing response" do
-      expect(@payment_method.capture).to be_a(ActiveMerchant::Billing::Response)
-    end
-  end
-
-  describe "#void" do
-    it "creates a new active merchant billing response" do
-      expect(ActiveMerchant::Billing::Response).to receive(:new).with(true, "", {}, {})
-      @payment_method.void
-    end
-    it "returns active merchant billing response" do
-      expect(@payment_method.void).to be_a(ActiveMerchant::Billing::Response)
-    end
-  end
-
-  describe "#source_required?" do
-    it "returns false" do
-      expect(@payment_method.source_required?).to be false
-    end
-  end
-
-  describe "#credit" do
-    it "creates a new active merchant billing response" do
-      expect(ActiveMerchant::Billing::Response).to receive(:new).with(true, "", {}, {})
-      @payment_method.credit
-    end
-    it "returns active merchant billing response" do
-      expect(@payment_method.credit).to be_a(ActiveMerchant::Billing::Response)
-    end
-  end
-
-end

data/spec/models/spree/payment_spec.rb

@@ -1,81 +0,0 @@
-require 'spec_helper'
-
-describe Spree::Payment, type: :model do
-  context 'when validate_bank_details' do
-    before do
-      subject.validate_bank_details = true
-    end
-
-    it { is_expected.to validate_presence_of(:bank_name) }
-    it { is_expected.to validate_presence_of(:account_no) }
-    it { is_expected.to validate_presence_of(:transaction_reference_no) }
-    it { is_expected.to validate_presence_of(:deposited_on) }
-  end
-
-  context 'when validate_bank_details not true' do
-    it { is_expected.not_to validate_presence_of(:bank_name) }
-    it { is_expected.not_to validate_presence_of(:account_no) }
-    it { is_expected.not_to validate_presence_of(:transaction_reference_no) }
-  end
-
-  describe ".from_bank_transfer" do
-    before do
-      check_payment_method = Spree::PaymentMethod::Check.new
-      check_payment_method.name = "Check Payment Method"
-      check_payment_method.save!
-
-      bank_transfer_payment_method = Spree::PaymentMethod::BankTransfer.new
-      bank_transfer_payment_method.name = "Bank Transfer"
-      bank_transfer_payment_method.save!
-
-      @order = Spree::Order.new
-      @order.save!
-
-      @payment1 = Spree::Payment.new
-      @payment1.order_id = @order.id
-      @payment1.payment_method_id = check_payment_method.id
-      @payment1.save!
-
-      @payment2 = Spree::Payment.new
-      @payment2.order_id = @order.id
-      @payment2.payment_method_id = check_payment_method.id
-      @payment2.save!
-
-      @bank_transfer_payment1 = Spree::Payment.new
-      @bank_transfer_payment1.order_id = @order.id
-      @bank_transfer_payment1.payment_method_id = bank_transfer_payment_method.id
-      @bank_transfer_payment1.save!
-
-      @bank_transfer_payment2 = Spree::Payment.new
-      @bank_transfer_payment2.order_id = @order.id
-      @bank_transfer_payment2.payment_method_id = bank_transfer_payment_method.id
-      @bank_transfer_payment2.save!
-    end
-
-    it "returns payments with payment method as bank_transfer" do
-      expect(Spree::Payment.from_bank_transfer).to match_array([@bank_transfer_payment1, @bank_transfer_payment2])
-    end
-  end
-
-  describe "#details_submitted?" do
-    before do
-      @payment = Spree::Payment.new
-    end
-
-    context 'when transaction_reference_no present' do
-      before do
-        @payment.transaction_reference_no = 'transaction_reference_no'
-      end
-
-      it 'is true' do
-        expect(@payment.details_submitted?).to be true
-      end
-    end
-
-    context 'when transaction_reference_no not present' do
-      it 'is false' do
-        expect(@payment.details_submitted?).to be false
-      end
-    end
-  end
-end

data/spec/models/spree/user_spec.rb

@@ -1,5 +0,0 @@
-require 'spec_helper'
-
-describe Spree::User, type: :model do
-  it { is_expected.to have_many(:payments).through(:orders) }
-end

data/spec/spec_helper.rb

@@ -1,98 +0,0 @@
-# Run Coverage report
-require 'simplecov'
-SimpleCov.start do
-  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'
-
-begin
-  require File.expand_path('../dummy/config/environment', __FILE__)
-rescue LoadError
-  puts 'Could not load dummy application. Please ensure you have run `bundle exec rake test_app`'
-  exit
-end
-
-require 'rspec/rails'
-require 'shoulda/matchers'
-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 defined in spree_core
-# require 'spree/testing_support/factories'
-require 'spree/testing_support/controller_requests'
-require 'spree/testing_support/authorization_helpers'
-require 'spree/testing_support/url_helpers'
-
-# Requires factories defined in lib/spree_favorite_products/factories.rb
-# require 'spree_favorite_products/factories'
-
-Shoulda::Matchers.configure do |config|
-  config.integrate do |with|
-    with.test_framework :rspec
-    with.library :rails
-  end
-end
-
-RSpec.configure do |config|
-  # config.include FactoryGirl::Syntax::Methods
-
-  # == 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 seperate 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 |example|
-    DatabaseCleaner.strategy = 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.include Spree::TestingSupport::ControllerRequests, type: :controller
-
-  config.infer_spec_type_from_file_location!
-end