spree_core 1.3.5 → 2.0.0.rc1

data/lib/spree/testing_support/factories/stock_movement_factory.rb

@@ -0,0 +1,11 @@
+FactoryGirl.define do
+  factory :stock_movement, class: Spree::StockMovement do
+    quantity 1
+    action 'sold'
+    stock_item
+  end
+
+  trait :received do
+    action 'received'
+  end
+end

data/lib/spree/testing_support/factories/tax_category_factory.rb

@@ -0,0 +1,6 @@
+FactoryGirl.define do
+  factory :tax_category, class: Spree::TaxCategory do
+    name { "TaxCategory - #{rand(999999)}" }
+    description { generate(:random_string) }
+  end
+end

data/lib/spree/testing_support/factories/tax_rate_factory.rb

@@ -0,0 +1,8 @@
+FactoryGirl.define do
+  factory :tax_rate, class: Spree::TaxRate do
+    zone
+    amount 100.00
+    tax_category
+    # association(:calculator, factory: :default_tax_calculator, strategy: :build)
+  end
+end

data/lib/spree/testing_support/factories/taxon_factory.rb

@@ -0,0 +1,7 @@
+FactoryGirl.define do
+  factory :taxon, class: Spree::Taxon do
+    name 'Ruby on Rails'
+    taxonomy
+    parent_id nil
+  end
+end

data/lib/spree/testing_support/factories/taxonomy_factory.rb

@@ -0,0 +1,5 @@
+FactoryGirl.define do
+  factory :taxonomy, class: Spree::Taxonomy do
+    name 'Brand'
+  end
+end

data/lib/spree/{core/testing_support → testing_support}/factories/tracker_factory.rb

@@ -1,5 +1,5 @@
 FactoryGirl.define do
-  factory :tracker, :class => Spree::Tracker do
+  factory :tracker, class: Spree::Tracker do
     environment { Rails.env }
     analytics_id 'A100'
     active true

data/lib/spree/testing_support/factories/user_factory.rb

@@ -0,0 +1,17 @@
+FactoryGirl.define do
+  sequence :user_authentication_token do |n|
+    "xxxx#{Time.now.to_i}#{rand(1000)}#{n}xxxxxxxxxxxxx"
+  end
+
+  factory :user, class: Spree.user_class do
+    email { generate(:random_email) }
+    login { email }
+    password 'secret'
+    password_confirmation { password }
+    authentication_token { generate(:user_authentication_token) } if Spree.user_class.attribute_method? :authentication_token
+
+    factory :admin_user do
+      spree_roles { [Spree::Role.find_by_name('admin') || create(:role, name: 'admin')] }
+    end
+  end
+end

data/lib/spree/testing_support/factories/variant_factory.rb

@@ -0,0 +1,24 @@
+FactoryGirl.define do
+  sequence(:random_float) { BigDecimal.new("#{rand(200)}.#{rand(99)}") }
+
+  factory :base_variant, class: Spree::Variant do
+    price 19.99
+    cost_price 17.00
+    sku    { generate(:random_string) }
+    weight { generate(:random_float) }
+    height { generate(:random_float) }
+    width  { generate(:random_float) }
+    depth  { generate(:random_float) }
+
+    product { |p| p.association(:base_product) }
+    option_values { [create(:option_value)] }
+
+    # ensure stock item will be created for this variant
+    before(:create) { create(:stock_location) if Spree::StockLocation.count == 0 }
+
+    factory :variant do
+      # on_hand 5
+      product { |p| p.association(:product) }
+    end
+  end
+end

data/lib/spree/testing_support/factories/zone_factory.rb

@@ -0,0 +1,17 @@
+FactoryGirl.define do
+  factory :global_zone, class: Spree::Zone do
+    name 'GlobalZone'
+    description { generate(:random_string) }
+    zone_members do |proxy|
+      zone = proxy.instance_eval { @instance }
+      Spree::Country.all.map do |c|
+        zone_member = Spree::ZoneMember.create(zoneable: c, zone: zone)
+      end
+    end
+  end
+
+  factory :zone, class: Spree::Zone do
+    name { generate(:random_string) }
+    description { generate(:random_string) }
+  end
+end

data/lib/spree/testing_support/factories.rb

@@ -0,0 +1,17 @@
+require 'factory_girl'
+
+Spree::Zone.class_eval do
+  def self.global
+    find_by_name('GlobalZone') || FactoryGirl.create(:global_zone)
+  end
+end
+
+Dir["#{File.dirname(__FILE__)}/factories/**"].each do |f|
+  require File.expand_path(f)
+end
+
+FactoryGirl.define do
+  sequence(:random_string)      { Faker::Lorem.sentence }
+  sequence(:random_description) { Faker::Lorem.paragraphs(1 + Kernel.rand(5)).join("\n") }
+  sequence(:random_email)       { Faker::Internet.email }
+end

data/lib/spree/testing_support/flash.rb

@@ -0,0 +1,27 @@
+module Spree
+  module TestingSupport
+    module Flash
+      def assert_flash_success(flash)
+        flash = convert_flash(flash)
+
+        within("[class='flash success']") do
+          page.should have_content(flash)
+        end
+      end
+
+      def assert_successful_update_message(resource)
+        flash = Spree.t(:successfully_updated, :resource => Spree.t(resource))
+        assert_flash_success(flash)
+      end
+
+      private
+
+      def convert_flash(flash)
+        if flash.is_a?(Symbol)
+          flash = Spree.t(flash)
+        end
+        flash
+      end
+    end
+  end
+end

data/lib/spree/testing_support/i18n.rb

@@ -0,0 +1,97 @@
+# This file exists solely to test whether or not there are missing translations
+# within the code that Spree's test suite covers.
+#
+# If there is a translation referenced which has no corresponding key within the
+# .yml file, then there will be a message output at the end of the suite showing
+# that.
+#
+# If there is a translation within the locale file which *isn't* used in the
+# test, this will also be shown at the end of the suite run.
+module Spree
+  class << self
+    attr_accessor :used_translations, :missing_translation_messages,
+                  :unused_translations, :unused_translation_messages
+    alias_method :normal_t, :t
+  end
+
+  def self.t(*args)
+    original_args = args.dup
+    options = args.extract_options!
+    self.used_translations ||= []
+    [*args.first].each do |translation_key|
+      key = ([*options[:scope]] << translation_key).join('.')
+      self.used_translations << key
+    end
+    normal_t(*original_args)
+  end
+
+  def self.check_missing_translations
+    self.missing_translation_messages = []
+    self.used_translations ||= []
+    used_translations.map { |a| a.split('.') }.each do |translation_keys|
+      root = translations
+      processed_keys = []
+      translation_keys.each do |key|
+        begin
+          root = root.fetch(key.to_sym)
+          processed_keys << key.to_sym
+        rescue KeyError
+          error = "#{(processed_keys << key).join('.')} (#{I18n.locale})"
+          unless Spree.missing_translation_messages.include?(error)
+            Spree.missing_translation_messages << error
+          end
+        end
+      end
+    end
+  end
+
+  def self.check_unused_translations
+    self.used_translations ||= []
+    self.unused_translation_messages = []
+    self.unused_translations = []
+    self.load_translations(translations)
+    translation_diff = unused_translations - used_translations
+    translation_diff.each do |translation|
+      Spree.unused_translation_messages << "#{translation} (#{I18n.locale})"
+    end
+  end
+
+  private
+
+  def self.load_translations(hash, root=[])
+    hash.each do |k,v|
+      if v.is_a?(Hash)
+        load_translations(v, root.dup << k)
+      else
+        key = (root + [k]).join('.')
+        self.unused_translations << key
+      end
+    end
+  end
+
+  def self.translations
+    @translations ||= I18n.backend.send(:translations)[I18n.locale][:spree]
+  end
+end
+
+RSpec.configure do |config|
+  # Need to check here again because this is used in i18n_spec too.
+  if ENV['CHECK_TRANSLATIONS']
+    config.after :suite do
+      Spree.check_missing_translations
+      if Spree.missing_translation_messages.any?
+        puts "\nThere are missing translations within Spree:"
+        puts Spree.missing_translation_messages.sort
+        exit(1)
+      end
+
+      Spree.check_unused_translations
+      if false && Spree.unused_translation_messages.any?
+        puts "\nThere are unused translations within Spree:"
+        puts Spree.unused_translation_messages.sort
+        exit(1)
+      end
+    end
+  end
+end
+

data/lib/spree/testing_support/order_walkthrough.rb

@@ -0,0 +1,55 @@
+class OrderWalkthrough
+  def self.up_to(state)
+    # A payment method must exist for an order to proceed through the Address state
+    unless Spree::PaymentMethod.exists?
+      FactoryGirl.create(:payment_method)
+    end
+
+    # A payment method must exist for an order to proceed through the Address state
+
+    order = Spree::Order.create!(:email => "spree@example.com")
+    add_line_item!(order)
+    order.next!
+
+    end_state_position = states.index(state.to_sym)
+    states[0..end_state_position].each do |state|
+      send(state, order)
+    end
+
+    order
+  end
+
+  private
+
+  def self.add_line_item!(order)
+    FactoryGirl.create(:line_item, order: order)
+    order.reload
+  end
+
+  def self.address(order)
+    order.bill_address = FactoryGirl.create(:address, :country_id => Spree::Zone.global.members.first.zoneable.id)
+    order.ship_address = FactoryGirl.create(:address, :country_id => Spree::Zone.global.members.first.zoneable.id)
+    order.next!
+  end
+
+  def self.delivery(order)
+    order.next!
+  end
+
+  def self.payment(order)
+    order.payments.create!({:payment_method => Spree::PaymentMethod.first, :amount => order.total}, :without_protection => true)
+    # TODO: maybe look at some way of making this payment_state change automatic
+    order.payment_state = 'paid'
+    order.next!
+  end
+
+  def self.complete(order)
+    #noop?
+  end
+
+  def self.states
+    [:address, :delivery, :payment, :complete]
+  end
+
+end
+

data/lib/spree/testing_support/preferences.rb

@@ -0,0 +1,31 @@
+module Spree
+  module TestingSupport
+    module Preferences
+      # Resets all preferences to default values, you can
+      # pass a block to override the defaults with a block
+      #
+      # reset_spree_preferences do |config|
+      #   config.site_name = "my fancy pants store"
+      # end
+      #
+      def reset_spree_preferences(&config_block)
+        Spree::Preferences::Store.instance.persistence = false
+        Spree::Preferences::Store.instance.clear_cache
+
+        config = Rails.application.config.spree.preferences
+        configure_spree_preferences &config_block if block_given?
+      end
+
+      def configure_spree_preferences
+        config = Rails.application.config.spree.preferences
+        yield(config) if block_given?
+      end
+
+      def assert_preference_unset(preference)
+        find("#preferences_#{preference}")['checked'].should be_false
+        Spree::Config[preference].should be_false
+      end
+    end
+  end
+end
+

data/lib/spree/{core → testing_support}/url_helpers.rb

@@ -1,5 +1,5 @@
 module Spree
-  module Core
+  module TestingSupport
     module UrlHelpers
       def spree
         Spree::Core::Engine.routes.url_helpers

data/lib/tasks/core.rake

@@ -1,41 +1,25 @@
 require 'active_record'
-require 'spree/core/custom_fixtures'
 
 namespace :db do
   desc %q{Loads a specified fixture file:
-For .yml      use rake db:load_file[spree/filename.yml,/absolute/path/to/parent/]
-For .rb       use rake db:load_file[/absolute/path/to/sample/filename.rb]}
+use rake db:load_file[/absolute/path/to/sample/filename.rb]}
 
   task :load_file , [:file, :dir] => :environment do |t, args|
     file = Pathname.new(args.file)
 
-    if %w{.yml}.include? file.extname
-      puts "loading fixture #{Pathname.new(args.dir).join(file)}"
-      Spree::Core::Fixtures.create_fixtures(args.dir, file.to_s.sub(file.extname, ""))
-    elsif file.exist?
-      puts "loading ruby #{file}"
-      require file
-    end
+    puts "loading ruby #{file}"
+    require file
   end
 
   desc "Loads fixtures from the the dir you specify using rake db:load_dir[loadfrom]"
-  task :load_dir , [:dir] => :environment do |t , args|
+  task :load_dir , [:dir] => :environment do |t, args|
     dir = args.dir
     dir = File.join(Rails.root, "db", dir) if Pathname.new(dir).relative?
 
-    fixtures = ActiveSupport::OrderedHash.new
-    ruby_files = ActiveSupport::OrderedHash.new
-    Dir.glob(File.join(dir , '**/*.{yml,rb}')).each do |fixture_file|
+    ruby_files = {}
+    Dir.glob(File.join(dir , '**/*.{rb}')).each do |fixture_file|
       ext = File.extname fixture_file
-      if ext == ".rb"
-        ruby_files[File.basename(fixture_file, '.*')] = fixture_file
-      else
-        fixtures[fixture_file.sub(dir, "")[1..-1]] = fixture_file
-      end
-    end
-    fixtures.sort.each do |relative_path , fixture_file|
-      # an invoke will only execute the task once
-      Rake::Task["db:load_file"].execute( Rake::TaskArguments.new([:file, :dir], [relative_path, dir]) )
+      ruby_files[File.basename(fixture_file, '.*')] = fixture_file
     end
     ruby_files.sort.each do |fixture , ruby_file|
       # an invoke will only execute the task once