spree_frontend 2.4.10 → 3.0.0.rc1

data/spec/features/taxons_spec.rb

@@ -0,0 +1,147 @@
+require 'spec_helper'
+
+describe "viewing products", type: :feature, inaccessible: true do
+  let!(:taxonomy) { create(:taxonomy, name: "Category") }
+  let!(:super_clothing) { taxonomy.root.children.create(name: "Super Clothing") }
+  let!(:t_shirts) { super_clothing.children.create(name: "T-Shirts") }
+  let!(:xxl) { t_shirts.children.create(name: "XXL") }
+  let!(:product) do
+    product = create(:product, name: "Superman T-Shirt")
+    product.taxons << t_shirts
+  end
+  let(:metas) { { meta_description: 'Brand new Ruby on Rails TShirts', meta_title: "Ruby On Rails TShirt", meta_keywords: 'ror, tshirt, ruby' } }
+  let(:store_name) do
+    ((first_store = Spree::Store.first) && first_store.name).to_s
+  end
+
+  # Regression test for #1796
+  it "can see a taxon's products, even if that taxon has child taxons" do
+    visit '/t/category/super-clothing/t-shirts'
+    expect(page).to have_content("Superman T-Shirt")
+  end
+
+  it "shouldn't show nested taxons with a search" do
+    visit '/t/category/super-clothing?keywords=shirt'
+    expect(page).to have_content("Superman T-Shirt")
+    expect(page).not_to have_selector("div[data-hook='taxon_children']")
+  end
+
+  describe 'breadcrumbs' do
+    before do
+      visit '/t/category/super-clothing/t-shirts'
+    end
+    it "should render breadcrumbs" do
+      expect(page.find("#breadcrumbs")).to have_link("T-Shirts")
+    end
+    it "should mark last breadcrumb as active" do
+      expect(page.find('#breadcrumbs .active')).to have_link("T-Shirts")
+    end
+  end
+
+  describe 'meta tags and title' do
+    it 'displays metas' do
+      t_shirts.update_attributes metas
+      visit '/t/category/super-clothing/t-shirts'
+      expect(page).to have_meta(:description, 'Brand new Ruby on Rails TShirts')
+      expect(page).to have_meta(:keywords, 'ror, tshirt, ruby')
+    end
+
+    it 'display title if set' do
+      t_shirts.update_attributes metas
+      visit '/t/category/super-clothing/t-shirts'
+      expect(page).to have_title("Ruby On Rails TShirt")
+    end
+
+    it 'displays title from taxon root and taxon name' do
+      visit '/t/category/super-clothing/t-shirts'
+      expect(page).to have_title('Category - T-Shirts - ' + store_name)
+    end
+
+    # Regression test for #2814
+    it "doesn't use meta_title as heading on page" do
+      t_shirts.update_attributes metas
+      visit '/t/category/super-clothing/t-shirts'
+      within("h1.taxon-title") do
+        expect(page).to have_content(t_shirts.name)
+      end
+    end
+
+    it 'uses taxon name in title when meta_title set to empty string' do
+      t_shirts.update_attributes meta_title: ''
+      visit '/t/category/super-clothing/t-shirts'
+      expect(page).to have_title('Category - T-Shirts - ' + store_name)
+    end
+  end
+
+  context "taxon pages" do
+    include_context "custom products"
+    before do
+      visit spree.root_path
+    end
+
+    it "should be able to visit brand Ruby on Rails" do
+      within(:css, '#taxonomies') { click_link "Ruby on Rails" }
+
+      expect(page.all('#products .product-list-item').size).to eq(7)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      array = ["Ruby on Rails Bag",
+       "Ruby on Rails Baseball Jersey",
+       "Ruby on Rails Jr. Spaghetti",
+       "Ruby on Rails Mug",
+       "Ruby on Rails Ringer T-Shirt",
+       "Ruby on Rails Stein",
+       "Ruby on Rails Tote"]
+      expect(tmp.sort!).to eq(array)
+    end
+
+    it "should be able to visit brand Ruby" do
+      within(:css, '#taxonomies') { click_link "Ruby" }
+
+      expect(page.all('#products .product-list-item').size).to eq(1)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      expect(tmp.sort!).to eq(["Ruby Baseball Jersey"])
+    end
+
+    it "should be able to visit brand Apache" do
+      within(:css, '#taxonomies') { click_link "Apache" }
+
+      expect(page.all('#products .product-list-item').size).to eq(1)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      expect(tmp.sort!).to eq(["Apache Baseball Jersey"])
+    end
+
+    it "should be able to visit category Clothing" do
+      click_link "Clothing"
+
+      expect(page.all('#products .product-list-item').size).to eq(5)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      expect(tmp.sort!).to eq(["Apache Baseball Jersey",
+     "Ruby Baseball Jersey",
+     "Ruby on Rails Baseball Jersey",
+     "Ruby on Rails Jr. Spaghetti",
+     "Ruby on Rails Ringer T-Shirt"])
+    end
+
+    it "should be able to visit category Mugs" do
+      click_link "Mugs"
+
+      expect(page.all('#products .product-list-item').size).to eq(2)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      expect(tmp.sort!).to eq(["Ruby on Rails Mug", "Ruby on Rails Stein"])
+    end
+
+    it "should be able to visit category Bags" do
+      click_link "Bags"
+
+      expect(page.all('#products .product-list-item').size).to eq(2)
+      tmp = page.all('#products .product-list-item a').map(&:text).flatten.compact
+      tmp.delete("")
+      expect(tmp.sort!).to eq(["Ruby on Rails Bag", "Ruby on Rails Tote"])
+    end
+  end
+end

data/spec/features/template_rendering_spec.rb

@@ -0,0 +1,19 @@
+require 'spec_helper'
+
+describe "Template rendering", type: :feature do
+
+  after do
+    Capybara.ignore_hidden_elements = true
+  end
+
+  before do
+    Capybara.ignore_hidden_elements = false
+  end
+
+  it 'layout should have canonical tag referencing site url' do
+    Spree::Store.create!(code: 'spree', name: 'My Spree Store', url: 'spreestore.example.com', mail_from_address: 'test@example.com')
+
+    visit spree.root_path
+    expect(find('link[rel=canonical]')[:href]).to eql('http://spreestore.example.com/')
+  end
+end

data/spec/helpers/frontend_helper_spec.rb

@@ -0,0 +1,41 @@
+require 'spec_helper'
+
+module Spree
+  describe FrontendHelper, type: :helper do
+    # Regression test for #2034
+    context "flash_message" do
+      let(:flash) { { "notice" => "ok", "foo" => "foo", "bar" => "bar" } }
+
+      it "should output all flash content" do
+        flash_messages
+        html = Nokogiri::HTML(helper.output_buffer)
+        expect(html.css(".alert-notice").text).to eq("ok")
+        expect(html.css(".alert-foo").text).to eq("foo")
+        expect(html.css(".alert-bar").text).to eq("bar")
+      end
+
+      it "should output flash content except one key" do
+        flash_messages(ignore_types: :bar)
+        html = Nokogiri::HTML(helper.output_buffer)
+        expect(html.css(".alert-notice").text).to eq("ok")
+        expect(html.css(".alert-foo").text).to eq("foo")
+        expect(html.css(".alert-bar").text).to be_empty
+      end
+
+      it "should output flash content except some keys" do
+        flash_messages(ignore_types: [:foo, :bar])
+        html = Nokogiri::HTML(helper.output_buffer)
+        expect(html.css(".alert-notice").text).to eq("ok")
+        expect(html.css(".alert-foo").text).to be_empty
+        expect(html.css(".alert-bar").text).to be_empty
+        expect(helper.output_buffer).to eq("<div class=\"alert alert-notice\">ok</div>")
+      end
+    end
+
+    # Regression test for #2759
+    it "nested_taxons_path works with a Taxon object" do
+      taxon = create(:taxon, name: "iphone")
+      expect(spree.nested_taxons_path(taxon)).to eq("/t/iphone")
+    end
+  end
+end

data/spec/helpers/order_helper_spec.rb

@@ -0,0 +1,12 @@
+require 'spec_helper'
+
+module Spree
+  describe Spree::OrdersHelper, :type => :helper do
+    # Regression test for #2518 + #2323
+    it "truncates HTML correctly in product description" do
+      product = double(:description => "<strong>" + ("a" * 95) + "</strong> This content is invisible.")
+      expected = "<strong>" + ("a" * 95) + "</strong>..."
+      expect(truncated_product_description(product)).to eq(expected)
+    end
+  end
+end

data/spec/helpers/taxons_helper_spec.rb

@@ -0,0 +1,17 @@
+require 'spec_helper'
+
+describe Spree::TaxonsHelper, :type => :helper do
+  # Regression test for #4382
+  it "#taxon_preview" do
+    taxon = create(:taxon)
+    child_taxon = create(:taxon, parent: taxon)
+    product_1 = create(:product)
+    product_2 = create(:product)
+    product_3 = create(:product)
+    taxon.products << product_1
+    taxon.products << product_2
+    child_taxon.products << product_3
+
+    expect(taxon_preview(taxon.reload)).to eql([product_1, product_2, product_3])
+  end
+end

data/spec/spec_helper.rb

@@ -0,0 +1,115 @@
+if ENV["COVERAGE"]
+  # 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
+end
+
+# This file is copied to ~/spec when you run 'ruby script/generate rspec'
+# from the project root directory.
+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 'ffaker'
+
+# Requires supporting files with custom matchers and macros, etc,
+# in ./support/ and its subdirectories.
+Dir["#{File.dirname(__FILE__)}/support/**/*.rb"].each {|f| require f}
+
+require 'database_cleaner'
+
+if ENV["CHECK_TRANSLATIONS"]
+  require "spree/testing_support/i18n"
+end
+
+require 'spree/testing_support/authorization_helpers'
+require 'spree/testing_support/capybara_ext'
+require 'spree/testing_support/factories'
+require 'spree/testing_support/preferences'
+require 'spree/testing_support/controller_requests'
+require 'spree/testing_support/flash'
+require 'spree/testing_support/url_helpers'
+require 'spree/testing_support/order_walkthrough'
+require 'spree/testing_support/caching'
+
+require 'paperclip/matchers'
+
+if ENV['WEBDRIVER'] == 'accessible'
+  require 'capybara/accessible'
+  Capybara.javascript_driver = :accessible
+else
+  require 'capybara/poltergeist'
+  Capybara.javascript_driver = :poltergeist
+end
+
+RSpec.configure do |config|
+  config.color = true
+  config.fail_fast = ENV['FAIL_FAST'] || false
+  config.fixture_path = File.join(File.expand_path(File.dirname(__FILE__)), "fixtures")
+  config.infer_spec_type_from_file_location!
+  config.mock_with :rspec
+  config.raise_errors_for_deprecations!
+
+  # If you're not using ActiveRecord, or you'd prefer not to run each of your
+  # examples within a transaction, comment the following line or assign false
+  # instead of true.
+  config.use_transactional_fixtures = false
+
+  if ENV['WEBDRIVER'] == 'accessible'
+    config.around(:each, :inaccessible => true) do |example|
+      Capybara::Accessible.skip_audit { example.run }
+    end
+  end
+
+  config.before(:each) do
+    WebMock.disable!
+    if RSpec.current_example.metadata[:js]
+      DatabaseCleaner.strategy = :truncation
+    else
+      DatabaseCleaner.strategy = :transaction
+    end
+    # TODO: Find out why open_transactions ever gets below 0
+    # See issue #3428
+    if ActiveRecord::Base.connection.open_transactions < 0
+      ActiveRecord::Base.connection.increment_open_transactions
+    end
+    DatabaseCleaner.start
+    reset_spree_preferences
+  end
+
+  config.after(:each) do
+    DatabaseCleaner.clean
+  end
+
+  config.after(:each, :type => :feature) do |example|
+    missing_translations = page.body.scan(/translation missing: #{I18n.locale}\.(.*?)[\s<\"&]/)
+    if missing_translations.any?
+      #binding.pry
+      puts "Found missing translations: #{missing_translations.inspect}"
+      puts "In spec: #{example.location}"
+    end
+  end
+
+
+  config.include FactoryGirl::Syntax::Methods
+
+  config.include Spree::TestingSupport::Preferences
+  config.include Spree::TestingSupport::UrlHelpers
+  config.include Spree::TestingSupport::ControllerRequests
+  config.include Spree::TestingSupport::Flash
+
+  config.include Paperclip::Shoulda::Matchers
+end

data/spec/support/shared_contexts/checkout_setup.rb

@@ -0,0 +1,9 @@
+shared_context 'checkout setup' do
+  let!(:country) { create(:country, states_required: true) }
+  let!(:state) { create(:state, country: country) }
+  let!(:shipping_method) { create(:shipping_method) }
+  let!(:stock_location) { create(:stock_location) }
+  let!(:mug) { create(:product, name: "RoR Mug") }
+  let!(:payment_method) { create(:check_payment_method) }
+  let!(:zone) { create(:zone) }
+end

data/spec/support/shared_contexts/custom_products.rb

@@ -0,0 +1,25 @@
+shared_context "custom products" do
+  before(:each) do
+    taxonomy = FactoryGirl.create(:taxonomy, :name => 'Categories')
+    root = taxonomy.root
+    clothing_taxon = FactoryGirl.create(:taxon, :name => 'Clothing', :parent_id => root.id)
+    bags_taxon = FactoryGirl.create(:taxon, :name => 'Bags', :parent_id => root.id)
+    mugs_taxon = FactoryGirl.create(:taxon, :name => 'Mugs', :parent_id => root.id)
+
+    taxonomy = FactoryGirl.create(:taxonomy, :name => 'Brands')
+    root = taxonomy.root
+    apache_taxon = FactoryGirl.create(:taxon, :name => 'Apache', :parent_id => root.id)
+    rails_taxon = FactoryGirl.create(:taxon, :name => 'Ruby on Rails', :parent_id => root.id)
+    ruby_taxon = FactoryGirl.create(:taxon, :name => 'Ruby', :parent_id => root.id)
+
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Ringer T-Shirt', :price => '19.99', :taxons => [rails_taxon, clothing_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Mug', :price => '15.99', :taxons => [rails_taxon, mugs_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Tote', :price => '15.99', :taxons => [rails_taxon, bags_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Bag', :price => '22.99', :taxons => [rails_taxon, bags_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Baseball Jersey', :price => '19.99', :taxons => [rails_taxon, clothing_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Stein', :price => '16.99', :taxons => [rails_taxon, mugs_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby on Rails Jr. Spaghetti', :price => '19.99', :taxons => [rails_taxon, clothing_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Ruby Baseball Jersey', :price => '19.99', :taxons => [ruby_taxon, clothing_taxon])
+    FactoryGirl.create(:custom_product, :name => 'Apache Baseball Jersey', :price => '19.99', :taxons => [apache_taxon, clothing_taxon])
+  end
+end

data/spec/support/shared_contexts/product_prototypes.rb

@@ -0,0 +1,30 @@
+shared_context "product prototype" do
+
+  def build_option_type_with_values(name, values)
+    ot = FactoryGirl.create(:option_type, :name => name)
+    values.each do |val|
+      ot.option_values.create(:name => val.downcase, :presentation => val)
+    end
+    ot
+  end
+
+  let(:product_attributes) do
+    # FactoryGirl.attributes_for is un-deprecated!
+    #   https://github.com/thoughtbot/factory_girl/issues/274#issuecomment-3592054
+    FactoryGirl.attributes_for(:base_product)
+  end
+
+  let(:prototype) do
+    size = build_option_type_with_values("size", %w(Small Medium Large))
+    FactoryGirl.create(:prototype, :name => "Size", :option_types => [ size ])
+  end
+
+  let(:option_values_hash) do
+    hash = {}
+    prototype.option_types.each do |i|
+      hash[i.id.to_s] = i.option_value_ids
+    end
+    hash
+  end
+
+end

data/spec/views/spree/checkout/_summary_spec.rb

@@ -0,0 +1,11 @@
+require 'spec_helper'
+
+describe "spree/checkout/_summary.html.erb", :type => :view do
+  # Regression spec for #4223
+  it "does not use the @order instance variable" do
+    order = stub_model(Spree::Order)
+    expect do
+      render :partial => "spree/checkout/summary", :locals => {:order => order}
+    end.not_to raise_error
+  end
+end

data/spree_frontend.gemspec

@@ -0,0 +1,30 @@
+# encoding: UTF-8
+version = File.read(File.expand_path("../../SPREE_VERSION", __FILE__)).strip
+
+Gem::Specification.new do |s|
+  s.platform    = Gem::Platform::RUBY
+  s.name        = 'spree_frontend'
+  s.version     = version
+  s.summary     = 'Frontend e-commerce functionality for the Spree project.'
+  s.description = s.summary
+
+  s.author      = 'Sean Schofield'
+  s.email       = 'sean@spreecommerce.com'
+  s.homepage    = 'https://spreecommerce.com'
+  s.license     = %q{BSD-3}
+
+  s.files        = `git ls-files`.split("\n")
+  s.test_files   = `git ls-files -- {test,spec,features}/*`.split("\n")
+  s.require_path = 'lib'
+  s.requirements << 'none'
+
+  s.add_dependency 'spree_api', version
+  s.add_dependency 'spree_core', version
+
+  s.add_dependency 'bootstrap-sass',  '~> 3.3.1'
+  s.add_dependency 'canonical-rails', '~> 0.0.4'
+  s.add_dependency 'jquery-rails',    '~> 4.0.3'
+  s.add_dependency 'sprockets-rails', '~> 2.2'
+
+  s.add_development_dependency 'capybara-accessible'
+end