spree_purchase_order 2.3.0.beta → 2.3.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: a6868ac4dc810cfcd75c34a21fd59fb387e0db35
-  data.tar.gz: 8e6b5d31274f438f06167debf9636c84f41dfbe9
+  metadata.gz: 687aa8d8212fd254800cacddc3ca0da7bf832365
+  data.tar.gz: 91ea5e93eef1934ba81f1138778995c229cc866d
 SHA512:
-  metadata.gz: 69b319f7f42cd625c357a6d05367a8e64a75d5bc518e21a69f9963622c23de2213ee01ebb169b77d6187b45dc6e741480242b89b4b638f9f44297d652f17e67f
-  data.tar.gz: c8157f21bb8bb5eebb2a702a445ff02fd1f2fa58d2b0c551b5065632ef366ecb4eb08694414eae1985592dfd11c54e7124e4e651e202fd3d5af87fea5d91c724
+  metadata.gz: 02d7e9d600d525734270379f784a48044741f0933499b9fc509695bc46012538b1ec5cbde62365dfe8b71aa0261577847011a39e29fba5563fb052a005a8d074
+  data.tar.gz: 3e554dd76ec313c1bae0137154258f856fd5f1a2b7ef0d12ca3d2eb5d3e712faed1270da63591c369bac2edbd232658860670f0fa8c5642dcb2288717a59ced5

data/README.md

@@ -1,3 +1,5 @@
+[![Build Status](https://travis-ci.org/deepakmahakale/spree_purchase_order.svg?branch=master)](https://travis-ci.org/deepakmahakale/spree_purchase_order)
+
 # Spree Purchase Order
 
 Add Purchase Order payment method in Spree
@@ -5,15 +7,18 @@ Add Purchase Order payment method in Spree
 ## Installation
 
 1. Add this line to your Gemfile:
-
-        gem 'spree_purchase_order'
+  ```ruby
+  gem 'spree_purchase_order'
+  ```
 
 2. Install the gem using Bundler:
-
-        bundle install
+  ```bash
+  bundle install
+  ```
 
 3. Copy & run migrations
+  ```bash
+  bundle exec rails g spree_purchase_order:install
+  ```
 
-        bundle exec rails g spree_purchase_order:install
-
-3. Restart your server
+4. Restart your server

data/Rakefile

@@ -1,3 +1,4 @@
+#!/usr/bin/env rake
 begin
   require 'bundler/setup'
 rescue LoadError
@@ -10,24 +11,21 @@ RDoc::Task.new(:rdoc) do |rdoc|
   rdoc.rdoc_dir = 'rdoc'
   rdoc.title    = 'SpreePurchaseOrder'
   rdoc.options << '--line-numbers'
-  rdoc.rdoc_files.include('README.rdoc')
+  rdoc.rdoc_files.include('README.md')
   rdoc.rdoc_files.include('lib/**/*.rb')
 end
 
-APP_RAKEFILE = File.expand_path("../test/dummy/Rakefile", __FILE__)
+APP_RAKEFILE = File.expand_path("../spec/dummy/Rakefile", __FILE__)
 load 'rails/tasks/engine.rake'
 
-load 'rails/tasks/statistics.rake'
-
 Bundler::GemHelper.install_tasks
 
-require 'rake/testtask'
+Dir[File.join(File.dirname(__FILE__), 'tasks/**/*.rake')].each {|f| load f }
 
-Rake::TestTask.new(:test) do |t|
-  t.libs << 'lib'
-  t.libs << 'test'
-  t.pattern = 'test/**/*_test.rb'
-  t.verbose = false
-end
+require 'rspec/core'
+require 'rspec/core/rake_task'
+
+desc "Run all specs in spec directory (excluding plugin specs)"
+RSpec::Core::RakeTask.new(spec: 'app:db:test:prepare')
 
-task default: :test
+task default: :spec

data/app/assets/stylesheets/spree/backend/spree_purchase_order.css

@@ -1,3 +1,29 @@
 /*
  *= require spree/backend
  */
+table th.actions .fa-complete:hover, table td.actions .fa-complete:hover{
+  background-color: #9FC820;
+  color: #FFFFFF;
+}
+table tbody tr.before-highlight.action-complete td {
+  border-bottom-color: #e0f1a9;
+}
+table tbody tr.highlight.action-complete td {
+  background-color: #f8fceb;
+  border-color: #e0f1a9;
+}
+.fa-complete:before {
+  content: "\f00c";
+}
+#powerTip.complete {
+  background-color: #9FC820;
+}
+#powerTip.complete.n:before, #powerTip.complete.ne:before, #powerTip.complete.nw:before {
+  border-top-color: #9FC820;
+}
+#powerTip.complete.s:before, #powerTip.complete.se:before, #powerTip.complete.sw:before {
+  border-bottom-color: #9FC820;
+}
+#powerTip.complete.w:before {
+  border-left-color: #9FC820;
+}

data/app/models/spree/payment_decorator.rb

@@ -1,7 +1,7 @@
 Spree::Payment.class_eval do
-  has_one :purchase_order
+  scope :from_purchase_order, -> { where(source_type: 'Spree::PurchaseOrder') }
 
   def po?
-    payment_method.type == 'Spree::PaymentMethod::PurchaseOrder'
+    source_type == 'Spree::PurchaseOrder'
   end
 end

data/app/models/spree/payment_method/purchase_order.rb

@@ -4,28 +4,12 @@ module Spree
       Spree::PurchaseOrder
     end
 
-    def actions
-      %w(complete void)
-    end
-
-    def can_complete?(payment)
-      %w(checkout pending).include?(payment.state)
-    end
-
-    def can_void?(payment)
-      payment.state != 'void'
-    end
-
     def authorize(*args)
-      ActiveMerchant::Billing::Response.new(true, 'Purchase Order: Success', {}, {})
-    end
-
-    def complete(*)
-      ActiveMerchant::Billing::Response.new(true, 'Purchase Order: Success', {}, {})
+      simulated_successful_billing_response('Purchase Order: Success')
     end
 
     def void(*args)
-      ActiveMerchant::Billing::Response.new(true, '', {}, {})
+      simulated_successful_billing_response
     end
 
     def source_required?
@@ -35,5 +19,11 @@ module Spree
     def auto_capture?
       false
     end
+
+    private
+
+    def simulated_successful_billing_response(message = "")
+      ActiveMerchant::Billing::Response.new(true, message, {}, {})
+    end
   end
 end

data/app/models/spree/purchase_order.rb

@@ -1,7 +1,22 @@
 module Spree
-  class PurchaseOrder < ActiveRecord::Base
-    has_one :payment, as: :source
+  class PurchaseOrder < Spree::Base
+    belongs_to :payment_method
+    has_many :payments, as: :source
 
     validates_presence_of :po_number, :organization_name
+
+    def actions
+      %w(complete void)
+    end
+
+    # Indicates whether its possible to complete the payment
+    def can_complete?(payment)
+      payment.pending? || payment.checkout?
+    end
+
+    # Indicates whether its possible to void the payment.
+    def can_void?(payment)
+      !payment.failed? && !payment.void?
+    end
   end
 end

data/app/views/spree/checkout/payment/_purchaseorder.html.erb

@@ -1,15 +1,9 @@
-<div class='well clearfix'>
-  <% param_prefix = "payment_source[#{payment_method.id}]" %>
-  <p class='field'>
-    <%= label_tag :po_number do %>
-      PO Number<abbr class='required' title='required'>*</abbr>
-    <% end %>
-    <%= text_field_tag "#{param_prefix}[po_number]", '', id: :po_number, class: 'form-control required', size: 10, maxlength: 10, autocomplete: "off" %>
-  </p>
-  <p class='field'>
-    <%= label_tag :organization_name do %>
-      Organization Name<abbr class='required' title='required'>*</abbr>
-    <% end %>
-    <%= text_field_tag "#{param_prefix}[organization_name]", '', id: :organization_name, class: 'form-control required'%>
-  </p>
-</div>
+<% param_prefix = "payment_source[#{payment_method.id}]" %>
+<p class='field'>
+  <%= label_tag :po_number, "PO Number" %><span class="required">*</span><br />
+  <%= text_field_tag "#{param_prefix}[po_number]", '', id: :po_number, size: 10, maxlength: 10, autocomplete: "off" %>
+</p>
+<p class='field'>
+  <%= label_tag :organization_name, "Organization Name" %><span class="required">*</span><br />
+  <%= text_field_tag "#{param_prefix}[organization_name]", '', id: :organization_name, class: 'form-control required'%>
+</p>

data/config/locales/en.yml

@@ -0,0 +1,10 @@
+en:
+  spree:
+    purchase_order: Purchase order
+  activerecord:
+    models:
+      spree/purchase_order: Purchase order
+    attributes:
+      spree/purchase_order:
+        po_number: Number
+        organization_name: Organization name

data/lib/spree_purchase_order/engine.rb

@@ -7,7 +7,10 @@ module SpreePurchaseOrder
     config.autoload_paths << "#{config.root}/lib"
 
     config.generators do |g|
-      g.test_framework :rspec
+      g.test_framework :rspec, fixture: false
+      g.fixture_replacement :factory_girl, dir: 'spec/factories'
+      g.assets false
+      g.helper false
     end
 
     def self.activate

data/lib/spree_purchase_order/version.rb

@@ -1,3 +1,3 @@
 module SpreePurchaseOrder
-  VERSION = '2.3.0.beta'
+  VERSION = '2.3.0'
 end

data/spec/dummy/Rakefile

@@ -0,0 +1,3 @@
+require File.expand_path('../config/application', __FILE__)
+
+Rails.application.load_tasks

data/spec/dummy/app/assets/javascripts/application.js

@@ -0,0 +1 @@
+//= require_tree .

data/spec/dummy/app/assets/stylesheets/application.css

@@ -0,0 +1,4 @@
+/*
+ *= require_tree .
+ *= require_self
+ */

data/spec/dummy/app/controllers/application_controller.rb

@@ -0,0 +1,3 @@
+class ApplicationController < ActionController::Base
+  protect_from_forgery with: :exception
+end

data/spec/dummy/app/helpers/application_helper.rb

@@ -0,0 +1,2 @@
+module ApplicationHelper
+end

data/spec/dummy/app/views/layouts/application.html.erb

@@ -0,0 +1,14 @@
+<!DOCTYPE html>
+<html>
+<head>
+  <title>Dummy</title>
+  <%= stylesheet_link_tag    'application', media: 'all', 'data-turbolinks-track' => true %>
+  <%= javascript_include_tag 'application', 'data-turbolinks-track' => true %>
+  <%= csrf_meta_tags %>
+</head>
+<body>
+
+<%= yield %>
+
+</body>
+</html>

data/spec/dummy/bin/bundle

@@ -0,0 +1,3 @@
+#!/usr/bin/env ruby
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../Gemfile', __FILE__)
+load Gem.bin_path('bundler', 'bundle')

data/spec/dummy/bin/rails

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+APP_PATH = File.expand_path('../../config/application', __FILE__)
+require_relative '../config/boot'
+require 'rails/commands'

data/spec/dummy/bin/rake

@@ -0,0 +1,4 @@
+#!/usr/bin/env ruby
+require_relative '../config/boot'
+require 'rake'
+Rake.application.run

data/spec/dummy/bin/setup

@@ -0,0 +1,22 @@
+#!/usr/bin/env ruby
+require 'pathname'
+
+# path to your application root.
+APP_ROOT = Pathname.new File.expand_path('../../',  __FILE__)
+
+Dir.chdir APP_ROOT do
+
+  puts "== Installing dependencies =="
+  system "gem install bundler --conservative"
+  system "bundle check || bundle install"
+
+  puts "\n== Preparing database =="
+  system "bin/rake db:setup"
+
+  puts "\n== Removing old logs and tempfiles =="
+  system "rm -f log/*"
+  system "rm -rf tmp/cache"
+
+  puts "\n== Restarting application server =="
+  system "touch tmp/restart.txt"
+end

data/spec/dummy/config/application.rb

@@ -0,0 +1,26 @@
+require File.expand_path('../boot', __FILE__)
+
+require "active_record/railtie"
+require "action_controller/railtie"
+require "action_mailer/railtie"
+require "action_view/railtie"
+require "sprockets/railtie"
+
+Bundler.require(*Rails.groups)
+require "spree_purchase_order"
+
+module Dummy
+  class Application < Rails::Application
+
+    config.to_prepare do
+      Dir.glob(File.join(File.dirname(__FILE__), "../app/**/*_decorator*.rb")) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+
+      Dir.glob(File.join(File.dirname(__FILE__), "../app/overrides/*.rb")) do |c|
+        Rails.configuration.cache_classes ? require(c) : load(c)
+      end
+    end
+  end
+end
+

data/spec/dummy/config/boot.rb

@@ -0,0 +1,5 @@
+# Set up gems listed in the Gemfile.
+ENV['BUNDLE_GEMFILE'] ||= File.expand_path('../../../../Gemfile', __FILE__)
+
+require 'bundler/setup' if File.exist?(ENV['BUNDLE_GEMFILE'])
+$LOAD_PATH.unshift File.expand_path('../../../../lib', __FILE__)

data/spec/dummy/config/database.yml

@@ -0,0 +1,8 @@
+default: &default
+  adapter: sqlite3
+  pool: 5
+  timeout: 5000
+
+test:
+  <<: *default
+  database: db/test.sqlite3

data/spec/dummy/config/environment.rb

@@ -0,0 +1,3 @@
+require File.expand_path('../application', __FILE__)
+
+Rails.application.initialize!

data/spec/dummy/config/environments/test.rb

@@ -0,0 +1,13 @@
+Rails.application.configure do
+  config.cache_classes = true
+  config.eager_load = false
+  config.serve_static_files   = true
+  config.static_cache_control = 'public, max-age=3600'
+  config.consider_all_requests_local       = true
+  config.action_controller.perform_caching = false
+  config.action_dispatch.show_exceptions = false
+  config.action_controller.allow_forgery_protection = false
+  config.action_mailer.delivery_method = :test
+  config.active_support.test_order = :random
+  config.active_support.deprecation = :stderr
+end

data/spec/dummy/config/initializers/assets.rb

@@ -0,0 +1 @@
+Rails.application.config.assets.version = '1.0'

data/spec/dummy/config/initializers/cookies_serializer.rb

@@ -0,0 +1 @@
+Rails.application.config.action_dispatch.cookies_serializer = :json

data/spec/dummy/config/initializers/filter_parameter_logging.rb

@@ -0,0 +1 @@
+Rails.application.config.filter_parameters += [:password]

data/spec/dummy/config/initializers/session_store.rb

@@ -0,0 +1 @@
+Rails.application.config.session_store :cookie_store, key: '_dummy_session'

data/spec/dummy/config/initializers/spree.rb

@@ -0,0 +1,2 @@
+Spree.config do |config|
+end

data/spec/dummy/config/initializers/wrap_parameters.rb

@@ -0,0 +1,3 @@
+ActiveSupport.on_load(:action_controller) do
+  wrap_parameters format: [:json] if respond_to?(:wrap_parameters)
+end

data/spec/dummy/config/locales/en.yml

@@ -0,0 +1,2 @@
+en:
+  hello: "Hello world"

data/spec/dummy/config/routes.rb

@@ -0,0 +1,2 @@
+Rails.application.routes.draw do
+end

data/spec/dummy/config/secrets.yml

@@ -0,0 +1,8 @@
+development:
+  secret_key_base: a21ce77d12504975623f2bc74d70b7cb4973f7495a5d4b1df2759dab2aa684a99a9391c4d2c0e2cf7e82409ffe37bb29ca978622a9c234b2aa25309fd1ee0404
+
+test:
+  secret_key_base: 26cf60e63c4cf25d010087eff97d56793bddce5a85e63a1966c8922d2b2fa790a25d2ef5581d6e9ebfcfc69cd67d9a5d15c28c091231cd5029bfe00d357beaa1
+
+production:
+  secret_key_base: <%= ENV["SECRET_KEY_BASE"] %>

data/spec/dummy/config.ru

@@ -0,0 +1,2 @@
+require ::File.expand_path('../config/environment', __FILE__)
+run Rails.application

data/spec/factories/purchase_orders.rb

@@ -0,0 +1,75 @@
+FactoryGirl.define do
+  sequence :user_authentication_token do |n|
+    "xxxx#{Time.current.to_i}#{rand(1000)}#{n}xxxxxxxxxxxxx"
+  end
+
+  factory :purchase_order, class: Spree::PurchaseOrder do
+    po_number 'PO-123'
+    organization_name 'Spree Commerce'
+    association(:payment_method, factory: :purchase_order_payment_method)
+  end
+
+  factory :purchase_order_payment_method, class: Spree::PaymentMethod::PurchaseOrder do
+    name 'Purchase Order'
+  end
+
+  factory :payment, class: Spree::Payment do
+    amount 45.75
+    association(:source, factory: :purchase_order)
+    association(:payment_method, factory: :purchase_order_payment_method)
+    order
+    state 'checkout'
+    response_code '12345'
+  end
+
+  factory :order, class: Spree::Order do
+    bill_address
+    completed_at nil
+    email { 'email@example.com' }
+
+    transient do
+      line_items_price BigDecimal.new(10)
+    end
+  end
+
+  factory :state, class: Spree::State do
+    sequence(:name) { |n| "STATE_NAME_#{n}" }
+    sequence(:abbr) { |n| "STATE_ABBR_#{n}" }
+    country do |country|
+      if usa = Spree::Country.find_by_numcode(840)
+        country = usa
+      else
+        country.association(:country)
+      end
+    end
+  end
+
+  factory :country, class: Spree::Country do
+    sequence(:iso_name) { |n| "ISO_NAME_#{n}" }
+    sequence(:name) { |n| "NAME_#{n}" }
+    iso 'US'
+    iso3 'USA'
+    numcode 840
+  end
+
+  factory :address, aliases: [:bill_address, :ship_address], class: Spree::Address do
+    firstname 'John'
+    lastname 'Doe'
+    company 'Company'
+    address1 '10 Lovely Street'
+    address2 'Northwest'
+    city 'Herndon'
+    zipcode '35005'
+    phone '555-555-0199'
+    alternative_phone '555-555-0199'
+
+    state { |address| address.association(:state) || Spree::State.last }
+    country do |address|
+      if address.state
+        address.state.country
+      else
+        address.association(:country)
+      end
+    end
+  end
+end

data/spec/models/spree/purchase_order_spec.rb

@@ -0,0 +1,61 @@
+require 'spec_helper'
+
+describe Spree::PurchaseOrder, type: :model do
+  let(:valid_purchase_order_attributes) do
+    {
+      po_number: 'PO-123',
+      organization_name: 'Spree Commerce'
+    }
+  end
+
+  def self.payment_states
+    Spree::Payment.state_machine.states.keys
+  end
+
+  let(:purchase_order) { Spree::PurchaseOrder.new }
+
+  context "#can_complete?" do
+    it "should be true if payment is pending" do
+      payment = create(:payment, state: 'pending')
+      expect(purchase_order.can_complete?(payment)).to be true
+    end
+
+    it "should be true if payment is checkout" do
+      payment = create(:payment, state: 'checkout')
+      expect(purchase_order.can_complete?(payment)).to be true
+    end
+  end
+
+  context "#can_void?" do
+    it "should be true if payment is not void" do
+      payment = create(:payment, state: 'checkout')
+      expect(purchase_order.can_void?(payment)).to be true
+    end
+  end
+
+  context "#valid?" do
+    it "should validate presence of po number" do
+      purchase_order.attributes = valid_purchase_order_attributes.except(:po_number)
+      expect(purchase_order).not_to be_valid
+      expect(purchase_order.errors[:po_number]).to eq(["can't be blank"])
+    end
+
+    it "should validate presence of organization name" do
+      purchase_order.attributes = valid_purchase_order_attributes.except(:organization_name)
+      expect(purchase_order).not_to be_valid
+      expect(purchase_order.errors[:organization_name]).to eq(["can't be blank"])
+    end
+
+    it "should only validate on create" do
+      purchase_order.attributes = valid_purchase_order_attributes
+      purchase_order.save
+      expect(purchase_order).to be_valid
+    end
+  end
+
+  context "#associations" do
+    it "should be able to access its payments" do
+      expect { purchase_order.payments.to_a }.not_to raise_error
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,20 @@
+ENV['RAILS_ENV'] ||= 'test'
+
+require File.expand_path("../dummy/config/environment.rb", __FILE__)
+require 'rspec/rails'
+require 'rspec/autorun'
+require 'factory_girl_rails'
+
+Rails.backtrace_cleaner.remove_silencers!
+
+# Load support files
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
+
+RSpec.configure do |config|
+ config.mock_with :rspec
+ config.use_transactional_fixtures = true
+ config.infer_base_class_for_anonymous_controllers = false
+ config.order = "random"
+
+ config.include FactoryGirl::Syntax::Methods
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: spree_purchase_order
 version: !ruby/object:Gem::Version
-  version: 2.3.0.beta
+  version: 2.3.0
 platform: ruby
 authors:
 - Deepak Mahakale
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-12-15 00:00:00.000000000 Z
+date: 2016-12-18 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: spree_core
@@ -24,6 +24,48 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: 2.3.0
+- !ruby/object:Gem::Dependency
+  name: sqlite3
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: rspec-rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+- !ruby/object:Gem::Dependency
+  name: factory_girl_rails
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: Adds Purchase Order as a Payment Method to Spree Commerce
 email: deepakmahakale@gmail.com
 executables: []
@@ -37,20 +79,47 @@ files:
 - app/assets/javascripts/spree/frontend/spree_purchase_order.js
 - app/assets/stylesheets/spree/backend/spree_purchase_order.css
 - app/assets/stylesheets/spree/frontend/spree_purchase_order.css
-- app/models/spree/order_decorator.rb
 - app/models/spree/payment_decorator.rb
 - app/models/spree/payment_method/purchase_order.rb
 - app/models/spree/purchase_order.rb
 - app/views/spree/checkout/payment/_purchaseorder.html.erb
+- config/locales/en.yml
 - config/routes.rb
 - db/migrate/20160111123141_create_spree_purchase_orders.rb
-- db/migrate/20160111163926_add_po_number_to_spree_payments.rb
 - db/migrate/20161214190805_add_payment_method_id_to_spree_purchase_orders.rb
 - db/migrate/20161214191512_add_user_id_to_spree_purchase_orders.rb
 - lib/generators/spree_purchase_order/install/install_generator.rb
 - lib/spree_purchase_order.rb
 - lib/spree_purchase_order/engine.rb
 - lib/spree_purchase_order/version.rb
+- spec/dummy/Rakefile
+- spec/dummy/app/assets/javascripts/application.js
+- spec/dummy/app/assets/stylesheets/application.css
+- spec/dummy/app/controllers/application_controller.rb
+- spec/dummy/app/helpers/application_helper.rb
+- spec/dummy/app/views/layouts/application.html.erb
+- spec/dummy/bin/bundle
+- spec/dummy/bin/rails
+- spec/dummy/bin/rake
+- spec/dummy/bin/setup
+- spec/dummy/config.ru
+- spec/dummy/config/application.rb
+- spec/dummy/config/boot.rb
+- spec/dummy/config/database.yml
+- spec/dummy/config/environment.rb
+- spec/dummy/config/environments/test.rb
+- spec/dummy/config/initializers/assets.rb
+- spec/dummy/config/initializers/cookies_serializer.rb
+- spec/dummy/config/initializers/filter_parameter_logging.rb
+- spec/dummy/config/initializers/session_store.rb
+- spec/dummy/config/initializers/spree.rb
+- spec/dummy/config/initializers/wrap_parameters.rb
+- spec/dummy/config/locales/en.yml
+- spec/dummy/config/routes.rb
+- spec/dummy/config/secrets.yml
+- spec/factories/purchase_orders.rb
+- spec/models/spree/purchase_order_spec.rb
+- spec/spec_helper.rb
 homepage: https://github.com/deepakmahakale/spree_purchase_order
 licenses:
 - MIT
@@ -66,9 +135,9 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: 1.9.3
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
-  - - ">"
+  - - ">="
     - !ruby/object:Gem::Version
-      version: 1.3.1
+      version: '0'
 requirements:
 - none
 rubyforge_project: 
@@ -76,4 +145,32 @@ rubygems_version: 2.5.1
 signing_key: 
 specification_version: 4
 summary: Adds Purchase Order as a Payment Method to Spree Commerce
-test_files: []
+test_files:
+- spec/dummy/Rakefile
+- spec/dummy/config/initializers/filter_parameter_logging.rb
+- spec/dummy/config/initializers/assets.rb
+- spec/dummy/config/initializers/spree.rb
+- spec/dummy/config/initializers/cookies_serializer.rb
+- spec/dummy/config/initializers/wrap_parameters.rb
+- spec/dummy/config/initializers/session_store.rb
+- spec/dummy/config/application.rb
+- spec/dummy/config/database.yml
+- spec/dummy/config/boot.rb
+- spec/dummy/config/environments/test.rb
+- spec/dummy/config/secrets.yml
+- spec/dummy/config/environment.rb
+- spec/dummy/config/routes.rb
+- spec/dummy/config/locales/en.yml
+- spec/dummy/config.ru
+- spec/dummy/bin/rake
+- spec/dummy/bin/bundle
+- spec/dummy/bin/rails
+- spec/dummy/bin/setup
+- spec/dummy/app/assets/javascripts/application.js
+- spec/dummy/app/assets/stylesheets/application.css
+- spec/dummy/app/controllers/application_controller.rb
+- spec/dummy/app/views/layouts/application.html.erb
+- spec/dummy/app/helpers/application_helper.rb
+- spec/spec_helper.rb
+- spec/models/spree/purchase_order_spec.rb
+- spec/factories/purchase_orders.rb

data/app/models/spree/order_decorator.rb

@@ -1,15 +0,0 @@
-Spree::Order.class_eval do
-  def authorize_digital?
-    return true if paid?
-    if pay_with_po?
-      payments.each do |payment|
-        return true if payment.po? && payment.source.attachment_file_name
-      end
-      return false
-    end
-  end
-
-  def pay_with_po?
-    payments.collect(&:po?).include?(true)
-  end
-end

data/db/migrate/20160111163926_add_po_number_to_spree_payments.rb

@@ -1,5 +0,0 @@
-class AddPoNumberToSpreePayments < ActiveRecord::Migration
-  def change
-    add_column :spree_payments, :po_number, :string
-  end
-end