spree 0.4.1 → 0.5.0

data/spec/models/zone_spec.rb

@@ -24,19 +24,19 @@ describe Zone do
     end
   end
 
-  describe "#in_zone?" do
+  describe "#include?" do
     describe "with countries based zone" do 
       it "should return true when the address country is included in the zones list of countries" do
         country = mock_model(Country)
         address = mock_model(Address, :country => country, :null_object => true)
         @zone.countries << country
-        @zone.in_zone?(address).should be_true
+        @zone.include?(address).should be_true
       end
       it "should return false when the address country is not included in the zones list of countries " do
         country = mock_model(Country)
         address = mock_model(Address, :country => country, :null_object => true)
         @zone.countries << mock_model(Country)
-        @zone.in_zone?(address).should be_false
+        @zone.include?(address).should be_false
       end
     end
     describe "with states based zone" do 
@@ -44,29 +44,29 @@ describe Zone do
         state = mock_model(State)
         address = mock_model(Address, :state => state, :null_object => true)
         @zone.states << state
-        @zone.in_zone?(address).should be_true
+        @zone.include?(address).should be_true
       end
       it "should return false when the address state is not included in the zones list of states" do
         state = mock_model(State)
         address = mock_model(Address, :state => state, :null_object => true)
         @zone.states << mock_model(State)
-        @zone.in_zone?(address).should be_false
+        @zone.include?(address).should be_false
       end
     end
     describe "with zones based zone" do
       it "should return true when the address satisfies at least one of the zones in the list of zones" do
         address = mock_model(Address, :null_object => true)
         zone = mock_model(Zone)
-        zone.should_receive(:in_zone?).with(address).and_return(true)
+        zone.should_receive(:include?).with(address).and_return(true)
         @zone.zones << zone
-        @zone.in_zone?(address).should be_true
+        @zone.include?(address).should be_true
       end
       it "should return false when the address satisfies none of the zones in the list of zones" do
         address = mock_model(Address, :null_object => true)
         zone = mock_model(Zone)
-        zone.should_receive(:in_zone?).with(address).and_return(false)
+        zone.should_receive(:include?).with(address).and_return(false)
         @zone.zones << zone
-        @zone.in_zone?(address).should be_false
+        @zone.include?(address).should be_false
       end
     end
   end
@@ -84,30 +84,50 @@ describe Zone do
     
     it "should return only one zone if the address matches only one zone" do
       zone1 = mock_model(Zone)
-      zone1.should_receive(:in_zone?).with(@address).and_return(true)
+      zone1.should_receive(:include?).with(@address).and_return(true)
       zone2 = mock_model(Zone)
-      zone2.should_receive(:in_zone?).with(@address).and_return(false)
+      zone2.should_receive(:include?).with(@address).and_return(false)
       Zone.should_receive(:all).and_return([zone1, zone2])
       Zone.match(@address).should == [zone1]
     end
 
     it "should return both zones if the address matches both zones" do
       zone1 = mock_model(Zone)
-      zone1.should_receive(:in_zone?).with(@address).and_return(true)
+      zone1.should_receive(:include?).with(@address).and_return(true)
       zone2 = mock_model(Zone)
-      zone2.should_receive(:in_zone?).with(@address).and_return(true)
+      zone2.should_receive(:include?).with(@address).and_return(true)
       Zone.should_receive(:all).and_return([zone1, zone2])
       Zone.match(@address).should == [zone1, zone2]
     end
 
     it "should return no zones if address matches neither of the zones" do
       zone1 = mock_model(Zone)
-      zone1.should_receive(:in_zone?).with(@address).and_return(false)
+      zone1.should_receive(:include?).with(@address).and_return(false)
       zone2 = mock_model(Zone)
-      zone2.should_receive(:in_zone?).with(@address).and_return(false)
+      zone2.should_receive(:include?).with(@address).and_return(false)
       Zone.should_receive(:all).and_return([zone1, zone2])
       Zone.match(@address).should == []
     end
 
   end
+  
+  describe "country_list" do
+    it "should return an empty array if the zone type is state" do
+      @zone.stub!(:type).and_return("state")
+      @zone.country_list.should == []
+    end
+    it "should return the corresponding countries if zone type is country" do
+      country = mock_model(Country)
+      @zone.should_receive(:countries).and_return([country])
+      @zone.country_list.should == [country]
+    end
+    it "should return the countries of the zone children if the type is zone" do
+      country1 = mock_model(Country)
+      country2 = mock_model(Country)
+      zone1 = mock_model(Zone, :country_list => [country1])
+      zone2 = mock_model(Zone, :country_list => [country2])
+      @zone.stub!(:members).and_return([zone1, zone2])
+      @zone.country_list.should == [country1, country2]
+    end
+  end
 end

data/spec/spec_helper.rb

@@ -63,3 +63,18 @@ unless defined? SPEC_ROOT
     # for all of your examples, even those that don't use them.
   end
 end
+
+
+class Hash
+  def except(*keys)
+    self.reject { |k,v| keys.include?(k || k.to_sym) }
+  end
+
+  def with(overrides = {})
+    self.merge overrides
+  end
+
+  def only(*keys)
+    self.reject { |k,v| !keys.include?(k || k.to_sym) }
+  end
+end

data/spec/views/admin/configurations/index.html.erb_spec.rb

@@ -1,8 +1,37 @@
 require File.expand_path(File.dirname(__FILE__) + '/../../../spec_helper')
 
+include PreferenceFactory
+
 describe "/admin/configurations" do
+
+
   before(:each) do
+    assigns[:extension_links] = []
     render 'admin/configurations/index'
   end
   
+  it "should display a Mail Server Settings link" do
+    response.should have_tag('a[href=?]', admin_mail_settings_path)
+  end
+
+  it "should display a Tax Categories link" do
+    response.should have_tag('a[href=?]', admin_tax_categories_path)
+  end
+
+  it "should display a Zones link" do
+    response.should have_tag('a[href=?]', admin_zones_path)
+  end
+
+  it "should display a States link" do
+    response.should have_tag('a[href=?]', admin_country_states_path(214))
+  end
+
+  it "should display a Gateway link" do
+    response.should have_tag('a[href=?]', admin_gateways_path)
+  end
+
+  it "should display an Inventory Settings link" do
+    response.should have_tag('a[href=?]', admin_inventory_settings_path)
+  end
+
 end

data/spec/views/admin/mail_settings/show.html.erb_spec.rb

@@ -6,7 +6,7 @@ describe "/admin/configurations/:id/mail_settings" do
   end
   
   #Delete this example and add some real ones or delete this file
-  #   it "should have a div for mails settings" do
-  #     response.should have_tag('div#mail_settings')
-  #   end
+  it "should have a div for mails settings" do
+    response.should have_tag('div#mail_settings')
+  end
 end

data/spec/views/products/index.html.erb_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+describe '/products/index' do
+  before(:each) do
+    @product = mock_model(Product)
+    template.stub!(:small_image)
+    template.stub!(:breadcrumbs).and_return("")
+    @product.stub!(:master_price)
+    @product.stub!(:name).and_return('Delicious Cows')
+    @product.stub!(:has_stock?).and_return(true)
+    assigns[:products] = [@product]
+
+    # TODO: put these pagination stubs into a helper?
+    assigns[:products].stub!(:page_count).and_return(1)
+    assigns[:products].stub!(:first_page).and_return(true)
+    assigns[:products].stub!(:previous_page?).and_return(false)
+    assigns[:products].stub!(:next_page?).and_return(false)
+    template.stub!(:windowed_pagination_links).and_return(false)
+    template.stub_render(:partial => 'shared/taxonomies')
+  end
+
+  it "should display items in stock" do
+    render '/products/index'
+    response.should have_tag('a[href=?]', product_path(@product), /#{@product.name}/)
+  end
+
+  describe 'with out-of-stock items' do
+    before(:each) do
+      @product.stub!(:has_stock?).and_return(false)
+    end
+
+    it "should not display if show_zero_stock_products is not set" do
+      Spree::Config.stub!(:[]).with(:show_zero_stock_products).and_return(false)
+      render '/products/index'
+      response.should_not have_tag('a[href=?]', product_path(@product), /#{@product.name}/)
+    end
+
+    it "should display if show_zero_stock_products is set" do
+      Spree::Config.stub!(:[]).with(:show_zero_stock_products).and_return(true)
+      render '/products/index'
+      response.should have_tag('a[href=?]', product_path(@product), /#{@product.name}/)
+    end
+
+  end
+end
+

data/spec/views/products/show.html.erb_spec.rb

@@ -0,0 +1,46 @@
+require File.expand_path(File.dirname(__FILE__) + '/../../spec_helper')
+
+
+describe '/product/show' do
+  before(:each) do
+    template.stub!(:product_image)
+    @product = Product.new(:name => 'Delicious Cows', :permalink => 'delicious-cows',
+                           :description => 'Cows are delicious!', :master_price => 45.00,
+                           :variants => [Variant.new]
+    )
+    @product.stub!(:images).and_return(mock(Image, :size => 1))
+    assigns[:product] = @product
+  end
+
+  it "should display the item" do
+    render '/products/show'
+    response.should have_tag('td[class=?]', 'product-name', /#{@product.name}/)
+  end
+
+  describe 'with out-of-stock items' do
+    before(:each) do
+      @product.stub!(:has_stock?).and_return(false)
+    end
+
+    it "should not display product if allow_backorders is not set" do
+      Spree::Config.stub!(:[]).with(:allow_backorders).and_return(false)
+      render '/products/show'
+      response.should have_tag('strong', /Out of Stock/)
+      response.should_not have_tag('input[type=?]', 'submit')
+    end
+
+    it "should display product if allow_backorders is set" do
+      Spree::Config.stub!(:[]).with(:allow_backorders).and_return(true)
+      render '/products/show'
+      response.should_not have_tag('strong', /Out of Stock/)
+      response.should have_tag('input[type=?]', 'submit')
+    end
+  end
+
+  # NOTE: the behavior of variants is largely described thru the variant_options
+  #       helper method in app/helpers/spree/base_helpers.rb.  For that reason,
+  #       the variant testing lives there
+
+end
+
+

data/vendor/extensions/flat_rate_shipping/README.markdown

@@ -0,0 +1,3 @@
+= Flat Rate Shipping
+
+Description goes here

data/vendor/extensions/flat_rate_shipping/Rakefile

@@ -0,0 +1,120 @@
+# I think this is the one that should be moved to the extension Rakefile template
+
+# In rails 1.2, plugins aren't available in the path until they're loaded.
+# Check to see if the rspec plugin is installed first and require
+# it if it is.  If not, use the gem version.
+
+# Determine where the RSpec plugin is by loading the boot
+unless defined? SPREE_ROOT
+  ENV["RAILS_ENV"] = "test"
+  case
+  when ENV["SPREE_ENV_FILE"]
+    require File.dirname(ENV["SPREE_ENV_FILE"]) + "/boot"
+  when File.dirname(__FILE__) =~ %r{vendor/SPREE/vendor/extensions}
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../../../")}/config/boot"
+  else
+    require "#{File.expand_path(File.dirname(__FILE__) + "/../../../")}/config/boot"
+  end
+end
+
+require 'rake'
+require 'rake/rdoctask'
+require 'rake/testtask'
+
+rspec_base = File.expand_path(SPREE_ROOT + '/vendor/plugins/rspec/lib')
+$LOAD_PATH.unshift(rspec_base) if File.exist?(rspec_base)
+require 'spec/rake/spectask'
+# require 'spec/translator'
+
+# Cleanup the SPREE_ROOT constant so specs will load the environment
+Object.send(:remove_const, :SPREE_ROOT)
+
+extension_root = File.expand_path(File.dirname(__FILE__))
+
+task :default => :spec
+task :stats => "spec:statsetup"
+
+desc "Run all specs in spec directory"
+Spec::Rake::SpecTask.new(:spec) do |t|
+  t.spec_opts = ['--options', "\"#{extension_root}/spec/spec.opts\""]
+  t.spec_files = FileList['spec/**/*_spec.rb']
+end
+
+namespace :spec do
+  desc "Run all specs in spec directory with RCov"
+  Spec::Rake::SpecTask.new(:rcov) do |t|
+    t.spec_opts = ['--options', "\"#{extension_root}/spec/spec.opts\""]
+    t.spec_files = FileList['spec/**/*_spec.rb']
+    t.rcov = true
+    t.rcov_opts = ['--exclude', 'spec', '--rails']
+  end
+  
+  desc "Print Specdoc for all specs"
+  Spec::Rake::SpecTask.new(:doc) do |t|
+    t.spec_opts = ["--format", "specdoc", "--dry-run"]
+    t.spec_files = FileList['spec/**/*_spec.rb']
+  end
+
+  [:models, :controllers, :views, :helpers].each do |sub|
+    desc "Run the specs under spec/#{sub}"
+    Spec::Rake::SpecTask.new(sub) do |t|
+      t.spec_opts = ['--options', "\"#{extension_root}/spec/spec.opts\""]
+      t.spec_files = FileList["spec/#{sub}/**/*_spec.rb"]
+    end
+  end
+  
+  # Hopefully no one has written their extensions in pre-0.9 style
+  # desc "Translate specs from pre-0.9 to 0.9 style"
+  # task :translate do
+  #   translator = ::Spec::Translator.new
+  #   dir = RAILS_ROOT + '/spec'
+  #   translator.translate(dir, dir)
+  # end
+
+  # Setup specs for stats
+  task :statsetup do
+    require 'code_statistics'
+    ::STATS_DIRECTORIES << %w(Model\ specs spec/models)
+    ::STATS_DIRECTORIES << %w(View\ specs spec/views)
+    ::STATS_DIRECTORIES << %w(Controller\ specs spec/controllers)
+    ::STATS_DIRECTORIES << %w(Helper\ specs spec/views)
+    ::CodeStatistics::TEST_TYPES << "Model specs"
+    ::CodeStatistics::TEST_TYPES << "View specs"
+    ::CodeStatistics::TEST_TYPES << "Controller specs"
+    ::CodeStatistics::TEST_TYPES << "Helper specs"
+    ::STATS_DIRECTORIES.delete_if {|a| a[0] =~ /test/}
+  end
+
+  namespace :db do
+    namespace :fixtures do
+      desc "Load fixtures (from spec/fixtures) into the current environment's database.  Load specific fixtures using FIXTURES=x,y"
+      task :load => :environment do
+        require 'active_record/fixtures'
+        ActiveRecord::Base.establish_connection(RAILS_ENV.to_sym)
+        (ENV['FIXTURES'] ? ENV['FIXTURES'].split(/,/) : Dir.glob(File.join(RAILS_ROOT, 'spec', 'fixtures', '*.{yml,csv}'))).each do |fixture_file|
+          Fixtures.create_fixtures('spec/fixtures', File.basename(fixture_file, '.*'))
+        end
+      end
+    end
+  end
+end
+
+desc 'Generate documentation for the flat_rate_shipping extension.'
+Rake::RDocTask.new(:rdoc) do |rdoc|
+  rdoc.rdoc_dir = 'rdoc'
+  rdoc.title    = 'FlatRateShippingExtension'
+  rdoc.options << '--line-numbers' << '--inline-source'
+  rdoc.rdoc_files.include('README')
+  rdoc.rdoc_files.include('lib/**/*.rb')
+end
+
+# For extensions that are in transition
+desc 'Test the flat_rate_shipping extension.'
+Rake::TestTask.new(:test) do |t|
+  t.libs << 'lib'
+  t.pattern = 'test/**/*_test.rb'
+  t.verbose = true
+end
+
+# Load any custom rakefiles for extension
+Dir[File.dirname(__FILE__) + '/tasks/*.rake'].sort.each { |f| require f }

data/vendor/extensions/flat_rate_shipping/flat_rate_shipping_extension.rb

@@ -0,0 +1,17 @@
+# Uncomment this if you reference any of your controllers in activate
+# require_dependency 'application'
+
+class FlatRateShippingExtension < Spree::Extension
+  version "1.0"
+  description "Provides FlatRate shiping_calculator" 
+
+  # define_routes do |map|
+  #   map.namespace :admin do |admin|
+  #     admin.resources :whatever
+  #   end  
+  # end
+  
+  def activate
+    # admin.tabs.add "Flat Rate Shipping", "/admin/flat_rate_shipping", :after => "Layouts", :visibility => [:all]
+  end
+end

data/vendor/extensions/flat_rate_shipping/lib/flat_rate_shipping_configuration.rb

@@ -0,0 +1,7 @@
+class FlatRateShippingConfiguration < Configuration
+
+  preference :flat_rate_amount, :decimal, :default => 12.99
+  
+  validates_presence_of :name
+  validates_uniqueness_of :name
+end

data/vendor/extensions/flat_rate_shipping/lib/spree/flat_rate_shipping/calculator.rb

@@ -0,0 +1,9 @@
+module Spree
+  module FlatRateShipping
+    class Calculator
+      def calculate_shipping(order)
+        return Spree::FlatRateShipping::Config[:flat_rate_amount]
+      end
+    end
+  end
+end

data/vendor/extensions/flat_rate_shipping/lib/spree/flat_rate_shipping/config.rb

@@ -0,0 +1,22 @@
+module Spree
+  module FlatRateShipping
+    # Singleton class to access the tax configuration object (TaxConfiguration.first by default) and it's preferences.
+    #
+    # Usage:
+    #   Spree::FlatRateShipping::Config[:foo]                  # Returns the foo preference
+    #   Spree::FlatRateShipping::Config[]                      # Returns a Hash with all the tax preferences
+    #   Spree::FlatRateShipping::Config.instance               # Returns the configuration object (TaxConfiguration.first)
+    #   Spree::FlatRateShipping::Config.set(preferences_hash)  # Set the tax preferences as especified in +preference_hash+
+    class Config
+      include Singleton
+      include PreferenceAccess
+    
+      class << self
+        def instance
+          return nil unless ActiveRecord::Base.connection.tables.include?('configurations')
+          FlatRateShippingConfiguration.find_or_create_by_name("Default tax configuration")
+        end
+      end
+    end
+  end
+end