office_clerk 0.8 → 0.9

data/spec/features/orders/inventory_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe "Orders" do
+  before(:each) do
+    sign_in
+  end
+  it "deduct inventory when shipping" do
+    order = create(:order_paid)
+    visit_path order_path(order)
+    start = order.basket.items.first.product.inventory
+    find(".ship_now").click
+    expect(order.basket.reload.items.first.product.inventory).to equal start - order.basket.reload.items.first.quantity
+  end
+  it "not possible to ship twice" do
+    order = create(:order_shipped)
+    visit_path order_path(order)
+    expect {find(".ship_now").click}.to raise_error Capybara::ElementNotFound
+  end
+end

data/spec/features/orders/ordering_spec.rb

@@ -0,0 +1,84 @@
+require 'spec_helper'
+
+describe "Orders" do
+  before(:each) do
+    sign_in
+  end
+  it "should render" do
+    order = create :order
+    visit_path order_path order
+  end
+  it "creates order from basket" do
+    basket = create(:basket_with_item)
+    visit_path edit_basket_path(basket)
+    find(".make_order").click
+    ensure_path order_path(basket.reload.kori)
+  end
+  it "creates order from order" do
+    order = create(:order_ordered)
+    order.name = "Tester Man"
+    order.save
+    email = order.email
+    visit_path order_path(order)
+    find(".copy_order").click
+    copy = Order.find order.id + 1
+    ensure_path edit_basket_path(copy.basket)
+    expect(copy.email).to eq email
+    expect(order.name).to eq copy.name
+  end
+  it "ships an order" do
+    order = create(:order_ordered)
+    visit_path order_path(order)
+    find(".shipment_type").click
+    ensure_path shipment_order_path(order)
+  end
+  it "pays an order" do
+    order = create(:order_ordered)
+    visit_path order_path(order)
+    find(".pay_now").click
+  end
+  it "cant pay twice" do
+    order = create(:order_paid)
+    visit_path order_path(order)
+    expect {find(".pay_now").click}.to raise_error Capybara::ElementNotFound
+  end
+  it "can edit basket of ordered order" do
+    order = create(:order_ordered)
+    visit_path order_path(order)
+    find(".edit_basket").click
+    ensure_path edit_basket_path(order.basket)
+  end
+  it "can edit basket of paid order" do
+    order = create(:order_ordered)
+    visit_path order_path(order)
+    find(".edit_basket").click
+    ensure_path edit_basket_path(order.basket)
+  end
+  it "can cancel shipped order and edit" do
+    order = create(:order_shipped)
+    visit_path order_path(order)
+    find(".cancel_order").click
+    find(".edit_basket").click
+    ensure_path edit_basket_path(order.basket)
+  end
+  it "can not edit basket of shipped order" do
+    order = create(:order_paid)
+    visit_path order_path(order)
+    find(".ship_now").click
+    expect {find(".edit_basket").click}.to raise_error Capybara::ElementNotFound
+  end
+  it  "allows shipment info to be changed" do
+    address = { :name => "Markus Janhunen" , :street => "123 my street", :city => "123 my town" , :phone => "0400404"}
+    order = create(:order_paid)
+    visit_path shipment_order_path(order)
+    address.each do |atrr, value|
+      fill_in("order_#{atrr}" , :with => value)
+    end
+    find("#make_order").click
+    ensure_path order_path(order)
+    order.reload
+    address.each do |atrr, value|
+      expect(order.send atrr).to eq value
+    end
+  end
+end

data/spec/features/paginate_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe "Pagination"  do
+  before(:each) do
+    sign_in
+  end
+  it "lists 30 clerks" do
+    30.times{ create(:clerk) }
+    visit_path clerks_path
+  end
+end

data/spec/features/sessions_spec.rb

@@ -1,12 +1,13 @@
 require 'spec_helper'
 
 describe "Sessions" do
+  
   it "redirect to sign in when accessing admin" do
     visit suppliers_path
     ensure_path sign_in_path
   end
   it "signs in with account" do
-    user = create :clerk 
+    user = create :clerk
     visit_path sign_in_path
     fill_in(:email , :with => user.email)
     fill_in(:password , :with => "password")

data/spec/{models/locale_spec.rb → i18n_spec.rb}

@@ -1,6 +1,8 @@
+require 'spec_helper'
+require 'i18n/tasks'
 require "i18n-spec"
 
-RSpec.describe 'locale:' do
+describe 'locale:' do
 
   Dir.glob('config/locales/*.yml') do |locale_file|
     next if locale_file == "config/locales/config.yml"
@@ -19,3 +21,19 @@ RSpec.describe 'locale:' do
     it { is_expected.to be_a_complete_translation_of 'config/locales/fi.yml' }
   end
 end
+
+describe 'I18n' do
+  let(:i18n) { I18n::Tasks::BaseTask.new }
+  let(:missing_keys) { i18n.missing_keys }
+  let(:unused_keys) { i18n.unused_keys }
+
+  it 'does not have missing keys' do
+    expect(missing_keys).to be_empty,
+                            "Missing #{missing_keys.leaves.count} i18n keys, run `i18n-tasks missing' to show them"
+  end
+
+  it 'does not have unused keys' do
+    expect(i18n.unused_keys).to be_empty,
+                                "#{unused_keys.leaves.count} unused i18n keys, run `i18n-tasks unused' to show them"
+  end
+end

data/spec/mailers/order_mailer_spec.rb

@@ -54,7 +54,8 @@ RSpec.describe OrderMailer, :type => :mailer do
 
   describe 'shipped mail' do
     let(:mail) do
-      order.pay_now!
+      order.pay_now
+      order.save
       OrderMailer.shipped(order)
     end
     it_should_behave_like "an order mail" do

data/spec/models/baskets/inventory_spec.rb

@@ -23,12 +23,19 @@ describe "Basket inventory" do
     basket = create :basket_2_items
     expect(basket.receive!).to be 3
     expect(basket.locked).not_to be nil
-    expect { basket.receive! }.to raise_error 
+    expect { basket.receive! }.to raise_error
   end
   it "locks after deduct" do
     basket = create :basket_2_items
     basket.deduct!
     expect(basket.locked).not_to be nil
-    expect { basket.deduct! }.to raise_error 
+    expect { basket.deduct! }.to raise_error
+  end
+  it "cancel order" do
+    basket = create :basket_2_items
+    basket.deduct!
+    basket.cancel_order!
+    expect(basket.locked).to be nil
+    expect(basket.deduct!).to be 3 
   end
 end

data/spec/models/baskets/items_spec.rb

@@ -2,9 +2,10 @@ require 'spec_helper'
 
 describe "Basket totals" do
   let(:basket) { create :basket_2_items }
-  
+
   it "creates proper quantities" do
     basket = create :basket_3_items
+    expect(basket.items.length).to eq 3
   end
   it "updates total on add" do
     items = basket.items
@@ -25,4 +26,8 @@ describe "Basket totals" do
     basket.save!
     expect(basket.items.length).to be  1
   end
+  it "zeros basket" do
+    basket.zero_prices!
+    expect(basket.total_price).to eq 0.0
+  end
 end

data/spec/models/order_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 describe Order do
   let(:order) { create :order }
-  let(:shipped_order) { create :order_shippped }
+  let(:shipped_order) { create :order_shipped }
 
   it "factory is ok" do
     o = build :order
@@ -10,6 +10,14 @@ describe Order do
     expect(ok).to be true
   end
 
+  it "doesnt save a blank order" do
+    expect(Order.new.save).to eq false
+  end
+
+  it "doesnt save order with just mail" do
+    expect(Order.new(:email => "torsten@villataika.fi").save).to eq false
+  end
+
   it "verification works" do
     o = Order.new attributes_for :order
     ok = o.save
@@ -18,6 +26,12 @@ describe Order do
 
   it "creates ok" do
     order
+    expect(order.basket.locked?).to eq false
+  end
+
+  it "creates shipped order" do
+    o = shipped_order
+    expect(o.basket.locked?).to eq true
   end
 
   it "calculates total tax" do
@@ -39,4 +53,9 @@ describe Order do
   it "calculates total tax with shipping" do
     expect(shipped_order.total_tax).to eq 2.5767
   end
+
+  it "cancels after shipping" do
+    expect(shipped_order.cancel!).to be true
+  end
+
 end

data/spec/routing/baskets_routing_spec.rb

@@ -2,7 +2,7 @@ require "spec_helper"
 
 describe BasketsController do
  routes { OfficeClerk::Engine.routes }
-  
+
   describe "routing" do
 
     it "routes to #index" do
@@ -21,6 +21,10 @@ describe BasketsController do
       expect(get("/baskets/1/edit")).to route_to("baskets#edit", :id => "1")
     end
 
+    it "routes to #zero" do
+      expect(get("/baskets/1/zero")).to route_to("baskets#zero", :id => "1")
+    end
+
     it "routes to #create" do
       expect(post("/baskets")).to route_to("baskets#create")
     end

data/spec/routing/orders_routing_spec.rb

@@ -16,18 +16,6 @@ describe OrdersController do
       expect(get("/orders/1")).to route_to("orders#show", :id => "1")
     end
 
-    it "routes to #edit" do
-      expect(get("/orders/1/edit")).to route_to("orders#edit", :id => "1")
-    end
-
-    it "routes to #create" do
-      expect(post("/orders")).to route_to("orders#create")
-    end
-
-    it "routes to #update" do
-      expect(put("/orders/1")).to route_to("orders#update", :id => "1")
-    end
-
     it "routes to #destroy" do
       expect(delete("/orders/1")).to route_to("orders#destroy", :id => "1")
     end

data/spec/spec_helper.rb

@@ -13,13 +13,26 @@ require 'rspec/rails'
 Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each { |f| require f }
 
 
+module OfficeClerk
+  module RSpec
+    module ControllerRoutes
+      extend ActiveSupport::Concern
+      included do
+        routes { ::OfficeClerk::Engine.routes }
+      end
+    end
+  end
+end
+
 RSpec.configure do |config|
-  
+
   config.include OfficeClerk::Engine.routes.url_helpers
 
+  config.include OfficeClerk::RSpec::ControllerRoutes, type: :controller
+
   config.infer_base_class_for_anonymous_controllers = false
   config.infer_spec_type_from_file_location!
-  
+
   # Run specs in random order to surface order dependencies. If you find an
   # order dependency and want to debug it, you can fix the order by providing
   # the seed, which is printed after each run.
@@ -30,7 +43,7 @@ RSpec.configure do |config|
     config.default_formatter = 'doc'
   end
 
-  # Print the 10 slowest examples and example groups at the end of the spec run, 
+  # Print the 10 slowest examples and example groups at the end of the spec run,
   #to help surface which specs are running  particularly slow.
 #  config.profile_examples = 10
 end

data/test_app/bin/setup

@@ -0,0 +1,29 @@
+#!/usr/bin/env ruby
+require 'pathname'
+
+# path to your application root.
+APP_ROOT = Pathname.new File.expand_path('../../',  __FILE__)
+
+Dir.chdir APP_ROOT do
+  # This script is a starting point to setup your application.
+  # Add necessary setup steps to this file:
+
+  puts "== Installing dependencies =="
+  system "gem install bundler --conservative"
+  system "bundle check || bundle install"
+
+  # puts "\n== Copying sample files =="
+  # unless File.exist?("config/database.yml")
+  #   system "cp config/database.yml.sample config/database.yml"
+  # end
+
+  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/test_app/config/application.rb

@@ -24,5 +24,8 @@ module TestApp
     # The default locale is :en and all translations from config/locales/*.rb,yml are auto loaded.
     # config.i18n.load_path += Dir[Rails.root.join('my', 'locales', '*.{rb,yml}').to_s]
     # config.i18n.default_locale = :de
+
+    # Do not swallow errors in after_commit/after_rollback callbacks.
+    config.active_record.raise_in_transactional_callbacks = true
   end
 end

data/test_app/config/boot.rb

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

data/test_app/config/environment.rb

@@ -2,4 +2,4 @@
 require File.expand_path('../application', __FILE__)
 
 # Initialize the Rails application.
-TestApp::Application.initialize!
+Rails.application.initialize!

data/test_app/config/environments/development.rb

@@ -1,4 +1,4 @@
-TestApp::Application.configure do
+Rails.application.configure do
   # Settings specified here will take precedence over those in config/application.rb.
 
   # In the development environment your application's code is reloaded on

data/test_app/config/environments/production.rb

@@ -1,11 +1,11 @@
-TestApp::Application.configure do
+Rails.application.configure do
   # Settings specified here will take precedence over those in config/application.rb.
 
   # Code is not reloaded between requests.
   config.cache_classes = true
 
   # Eager load code on boot. This eager loads most of Rails and
-  # your application in memory, allowing both thread web servers
+  # your application in memory, allowing both threaded web servers
   # and those relying on copy on write to perform better.
   # Rake tasks automatically ignore this option for performance.
   config.eager_load = true
@@ -16,11 +16,13 @@ TestApp::Application.configure do
 
   # Enable Rack::Cache to put a simple HTTP cache in front of your application
   # Add `rack-cache` to your Gemfile before enabling this.
-  # For large-scale production use, consider using a caching reverse proxy like nginx, varnish or squid.
+  # For large-scale production use, consider using a caching reverse proxy like
+  # NGINX, varnish or squid.
   # config.action_dispatch.rack_cache = true
 
-  # Disable Rails's static asset server (Apache or nginx will already do this).
-  config.serve_static_assets = false
+  # Disable serving static files from the `/public` folder by default since
+  # Apache or NGINX already handles this.
+  config.serve_static_files = ENV['RAILS_SERVE_STATIC_FILES'].present?
 
   # Compress JavaScripts and CSS.
   config.assets.js_compressor = :uglifier
@@ -29,21 +31,22 @@ TestApp::Application.configure do
   # Do not fallback to assets pipeline if a precompiled asset is missed.
   config.assets.compile = false
 
-  # Generate digests for assets URLs.
+  # Asset digests allow you to set far-future HTTP expiration dates on all assets,
+  # yet still be able to expire them through the digest params.
   config.assets.digest = true
 
-  # Version of your assets, change this if you want to expire all your assets.
-  config.assets.version = '1.0'
+  # `config.assets.precompile` and `config.assets.version` have moved to config/initializers/assets.rb
 
   # Specifies the header that your server uses for sending files.
-  # config.action_dispatch.x_sendfile_header = "X-Sendfile" # for apache
-  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for nginx
+  # config.action_dispatch.x_sendfile_header = 'X-Sendfile' # for Apache
+  # config.action_dispatch.x_sendfile_header = 'X-Accel-Redirect' # for NGINX
 
   # Force all access to the app over SSL, use Strict-Transport-Security, and use secure cookies.
   # config.force_ssl = true
 
-  # Set to :debug to see everything in the log.
-  config.log_level = :info
+  # Use the lowest log level to ensure availability of diagnostic information
+  # when problems arise.
+  config.log_level = :debug
 
   # Prepend all log lines with the following tags.
   # config.log_tags = [ :subdomain, :uuid ]
@@ -55,26 +58,22 @@ TestApp::Application.configure do
   # config.cache_store = :mem_cache_store
 
   # Enable serving of images, stylesheets, and JavaScripts from an asset server.
-  # config.action_controller.asset_host = "http://assets.example.com"
-
-  # Precompile additional assets.
-  # application.js, application.css, and all non-JS/CSS in app/assets folder are already added.
-  # config.assets.precompile += %w( search.js )
+  # config.action_controller.asset_host = 'http://assets.example.com'
 
   # Ignore bad email addresses and do not raise email delivery errors.
   # Set this to true and configure the email server for immediate delivery to raise delivery errors.
   # config.action_mailer.raise_delivery_errors = false
 
   # Enable locale fallbacks for I18n (makes lookups for any locale fall back to
-  # the I18n.default_locale when a translation can not be found).
+  # the I18n.default_locale when a translation cannot be found).
   config.i18n.fallbacks = true
 
   # Send deprecation notices to registered listeners.
   config.active_support.deprecation = :notify
 
-  # Disable automatic flushing of the log to improve performance.
-  # config.autoflush_log = false
-
   # Use default logging formatter so that PID and timestamp are not suppressed.
   config.log_formatter = ::Logger::Formatter.new
+
+  # Do not dump schema after migrations.
+  config.active_record.dump_schema_after_migration = false
 end