spree_backend 3.1.5 → 3.1.6

data/spec/features/admin/products/edit/products_spec.rb

@@ -1,64 +0,0 @@
-# encoding: UTF-8
-require 'spec_helper'
-
-describe 'Product Details', type: :feature, js: true do
-  stub_authorization!
-
-  context 'editing a product' do
-    it 'should list the product details' do
-      create(:product, :name => 'Bún thịt nướng', :sku => 'A100',
-              :description => 'lorem ipsum', :available_on => '2013-08-14 01:02:03')
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      click_link 'Details'
-
-      expect(find('.content-header h1').text.strip).to eq('Products / Bún thịt nướng')
-      expect(find('input#product_name').value).to eq('Bún thịt nướng')
-      expect(find('input#product_slug').value).to eq('bun-th-t-n-ng')
-      expect(find('textarea#product_description').text.strip).to eq('lorem ipsum')
-      expect(find('input#product_price').value).to eq('19.99')
-      expect(find('input#product_cost_price').value).to eq('17.00')
-      expect(find('input#product_available_on').value).to eq("2013/08/14")
-      expect(find('input#product_sku').value).to eq('A100')
-    end
-
-    it "should handle slug changes" do
-      create(:product, name: 'Bún thịt nướng', sku: 'A100',
-               description: 'lorem ipsum', available_on: '2011-01-01 01:01:01')
-
-      visit spree.admin_products_path
-      within('table.table tbody tr:nth-child(1)') do
-        click_icon(:edit)
-      end
-
-      fill_in "product_slug", with: 'random-slug-value'
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      fill_in "product_slug", with: 'a'
-      click_button "Update"
-      within('#product_slug_field') { expect(page).to have_content("is too short") }
-
-      fill_in "product_slug", with: 'another-random-slug-value'
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-    end
-  end
-
-  # Regression test for #3385
-  context "deleting a product" do
-    it "is still able to find the master variant" do
-      create(:product)
-
-      visit spree.admin_products_path
-      within_row(1) do
-        accept_alert do
-          click_icon :delete
-        end
-      end
-      wait_for_ajax
-    end
-  end
-end

data/spec/features/admin/products/edit/taxons_spec.rb

@@ -1,41 +0,0 @@
-require 'spec_helper'
-
-describe "Product Taxons", type: :feature, js: true do
-  stub_authorization!
-
-  after do
-    Capybara.ignore_hidden_elements = true
-  end
-
-  before do
-    Capybara.ignore_hidden_elements = false
-  end
-
-  context "managing taxons" do
-    def selected_taxons
-      find("#product_taxon_ids").value.split(',').map(&:to_i).uniq
-    end
-
-    it "should allow an admin to manage taxons" do
-      taxon_1 = create(:taxon)
-      taxon_2 = create(:taxon, name: 'Clothing')
-      product = create(:product)
-      product.taxons << taxon_1
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      expect(find(".select2-search-choice").text).to eq(taxon_1.name)
-      expect(selected_taxons).to match_array([taxon_1.id])
-
-      select2_search "Clothing", from: "Taxons"
-      click_button "Update"
-      expect(selected_taxons).to match_array([taxon_1.id, taxon_2.id])
-
-      # Regression test for #2139
-      sleep(1)
-      expect(first(".select2-search-choice", text: taxon_1.name)).to be_present
-      expect(first(".select2-search-choice", text: taxon_2.name)).to be_present
-    end
-  end
-end

data/spec/features/admin/products/edit/variants_spec.rb

@@ -1,56 +0,0 @@
-require 'spec_helper'
-
-describe "Product Variants", type: :feature, js: true do
-  stub_authorization!
-
-  let!(:product) { create(:product) }
-
-  before(:each) do
-    visit spree.admin_products_path
-  end
-
-  context "editing variant option types" do
-    it "should allow an admin to create option types for a variant" do
-      within_row(1) { click_icon :edit }
-
-      within('#sidebar') { click_link "Variants" }
-      expect(page).to have_content("To add variants, you must first define")
-    end
-
-    it "allows admin to create a variant if there are option types" do
-      click_link "Products"
-      click_link "Option Types"
-      click_link "new_option_type_link"
-      fill_in "option_type_name", with: "shirt colors"
-      fill_in "option_type_presentation", with: "colors"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      page.find('#option_type_option_values_attributes_0_name').set('color')
-      page.find('#option_type_option_values_attributes_0_presentation').set('black')
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      visit spree.admin_products_path
-      within_row(1) { click_icon :edit }
-
-      select2_search "shirt", from: "Option Types"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-
-      within('#sidebar') { click_link "Variants" }
-      click_link "New Variant"
-
-      targetted_select2 "black", from: "#s2id_variant_option_value_ids"
-      fill_in "variant_sku", with: "A100"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      within(".table") do
-        expect(page).to have_content("19.99")
-        expect(page).to have_content("black")
-        expect(page).to have_content("A100")
-      end
-    end
-  end
-end

data/spec/features/admin/products/option_types_spec.rb

@@ -1,114 +0,0 @@
-require 'spec_helper'
-
-describe "Option Types", type: :feature, js: true do
-  stub_authorization!
-
-  before(:each) do
-    visit spree.admin_path
-    click_link "Products"
-  end
-
-  context "listing option types" do
-    it "should list existing option types" do
-      create(:option_type, name: "tshirt-color", presentation: "Color")
-      create(:option_type, name: "tshirt-size", presentation: "Size")
-      click_link "Option Types"
-      within("table#listing_option_types") do
-        expect(page).to have_content("Color")
-        expect(page).to have_content("tshirt-color")
-        expect(page).to have_content("Size")
-        expect(page).to have_content("tshirt-size")
-      end
-    end
-  end
-
-  context "creating a new option type" do
-    it "should allow an admin to create a new option type" do
-      click_link "Option Types"
-      click_link "new_option_type_link"
-      expect(page).to have_content("New Option Type")
-      fill_in "option_type_name", with: "shirt colors"
-      fill_in "option_type_presentation", with: "colors"
-      click_button "Create"
-      expect(page).to have_content("successfully created!")
-
-      page.find('#option_type_option_values_attributes_0_name').set('color')
-      page.find('#option_type_option_values_attributes_0_presentation').set('black')
-
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-    end
-  end
-
-  context "editing an existing option type" do
-    it "should allow an admin to update an existing option type" do
-      create(:option_type, name: "tshirt-color", presentation: "Color")
-      create(:option_type, name: "tshirt-size", presentation: "Size")
-      click_link "Option Types"
-      within('table#listing_option_types') { click_icon :edit }
-      fill_in "option_type_name", with: "foo-size 99"
-      click_button "Update"
-      expect(page).to have_content("successfully updated!")
-      expect(page).to have_content("foo-size 99")
-    end
-  end
-
-  # Regression test for #2277
-  it "can remove an option value from an option type" do
-    option_type = create(:option_type)
-    create(:option_value, option_type: option_type)
-    click_link "Option Types"
-    within('table#listing_option_types') { click_icon :edit }
-    expect(page).to have_content(option_type.name)
-    expect(all("tbody#option_values tr").count).to eq(1)
-    within("tbody#option_values") do
-      find('.spree_remove_fields').click
-    end
-    # Assert that the field is hidden automatically
-    expect(all("tbody#option_values tr").select(&:visible?).count).to eq(0)
-
-    # Then assert that on a page refresh that it's still not visible
-    visit page.current_url
-    # What *is* visible is a new option value field, with blank values
-    # Sometimes the page doesn't load before the all check is done
-    # lazily finding the element gives the page 10 seconds
-    expect(page).to have_css("tbody#option_values")
-    all("tbody#option_values tr input").all? { |input| input.value.blank? }
-  end
-
-  # Regression test for #3204
-  it "can remove a non-persisted option value from an option type" do
-    create(:option_type)
-    click_link "Option Types"
-    within('table#listing_option_types') { click_icon :edit }
-
-    wait_for_ajax
-    page.find("tbody#option_values", :visible => true)
-
-    expect(all("tbody#option_values tr").select(&:visible?).count).to eq(1)
-
-    # Add a new option type
-    click_link "Add Option Value"
-    expect(all("tbody#option_values tr").select(&:visible?).count).to eq(2)
-
-    # Remove default option type
-    within("tbody#option_values") do
-      click_icon :delete
-    end
-    # Check that there was no HTTP request
-    expect(all("div#progress[style]").count).to eq(0)
-    # Assert that the field is hidden automatically
-    expect(all("tbody#option_values tr").select(&:visible?).count).to eq(1)
-
-    # Remove added option type
-    within("tbody#option_values") do
-      click_icon :delete
-    end
-    # Check that there was no HTTP request
-    expect(all("div#progress[style]").count).to eq(0)
-    # Assert that the field is hidden automatically
-    expect(all("tbody#option_values tr").select(&:visible?).count).to eq(0)
-
-  end
-
-end

data/spec/features/admin/products/products_spec.rb

@@ -1,445 +0,0 @@
-# encoding: utf-8
-require 'spec_helper'
-
-describe "Products", type: :feature do
-  context "as admin user" do
-    stub_authorization!
-
-    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
-
-    context "listing products" do
-      context "sorting" do
-        before do
-          create(:product, name: 'apache baseball cap', price: 10)
-          create(:product, name: 'zomg shirt', price: 5)
-        end
-
-        it "should list existing products with correct sorting by name" do
-          visit spree.admin_products_path
-          # Name ASC
-          within_row(1) { expect(page).to have_content('apache baseball cap') }
-          within_row(2) { expect(page).to have_content("zomg shirt") }
-
-          # Name DESC
-          click_link "admin_products_listing_name_title"
-          within_row(1) { expect(page).to have_content("zomg shirt")  }
-          within_row(2) { expect(page).to have_content('apache baseball cap') }
-        end
-
-        it "should list existing products with correct sorting by price" do
-          visit spree.admin_products_path
-          # Name ASC (default)
-          within_row(1) { expect(page).to have_content('apache baseball cap') }
-          within_row(2) { expect(page).to have_content("zomg shirt") }
-
-          # Price DESC
-          click_link "admin_products_listing_price_title"
-          within_row(1) { expect(page).to have_content("zomg shirt") }
-          within_row(2) { expect(page).to have_content('apache baseball cap') }
-        end
-      end
-
-      context "currency displaying" do
-        context "using Russian Rubles" do
-          before do
-            Spree::Config[:currency] = "RUB"
-          end
-
-          let!(:product) do
-            create(:product, name: "Just a product", price: 19.99)
-          end
-
-          # Regression test for #2737
-          context "uses руб as the currency symbol" do
-            it "on the products listing page" do
-              visit spree.admin_products_path
-              within_row(1) { expect(page).to have_content("19.99 ₽") }
-            end
-          end
-        end
-      end
-    end
-
-    context "searching products" do
-      it "should be able to search deleted products" do
-        create(:product, name: 'apache baseball cap', deleted_at: "2011-01-06 18:21:13")
-        create(:product, name: 'zomg shirt')
-
-        visit spree.admin_products_path
-        expect(page).to have_content("zomg shirt")
-        expect(page).not_to have_content("apache baseball cap")
-
-        check "Show Deleted"
-        click_on 'Search'
-
-        expect(page).to have_content("zomg shirt")
-        expect(page).to have_content("apache baseball cap")
-
-        uncheck "Show Deleted"
-        click_on 'Search'
-
-        expect(page).to have_content("zomg shirt")
-        expect(page).not_to have_content("apache baseball cap")
-      end
-
-      it "should be able to search products by their properties" do
-        create(:product, name: 'apache baseball cap', sku: "A100")
-        create(:product, name: 'apache baseball cap2', sku: "B100")
-        create(:product, name: 'zomg shirt')
-
-        visit spree.admin_products_path
-        fill_in "q_name_cont", with: "ap"
-        click_on 'Search'
-
-        expect(page).to have_content("apache baseball cap")
-        expect(page).to have_content("apache baseball cap2")
-        expect(page).not_to have_content("zomg shirt")
-
-        fill_in "q_variants_including_master_sku_cont", with: "A1"
-        click_on 'Search'
-
-        expect(page).to have_content("apache baseball cap")
-        expect(page).not_to have_content("apache baseball cap2")
-        expect(page).not_to have_content("zomg shirt")
-      end
-    end
-
-    context "creating a new product from a prototype", js: true 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(:simple_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
-
-      before(:each) do
-        @option_type_prototype = prototype
-        @property_prototype = create(:prototype, name: "Random")
-        @shipping_category = create(:shipping_category)
-        visit spree.admin_products_path
-        click_link "admin_new_product"
-        within('#new_product') do
-          expect(page).to have_content("SKU")
-        end
-      end
-
-      it "should allow an admin to create a new product and variants from a prototype" do
-        fill_in "product_name", with: "Baseball Cap"
-        fill_in "product_sku", with: "B100"
-        fill_in "product_price", with: "100"
-        fill_in "product_available_on", with: "2012/01/24"
-        # Just so the datepicker gets out of poltergeists way.
-        page.execute_script("$('#ui-datepicker-div').hide();")
-        select "Size", from: "Prototype"
-        wait_for_ajax
-        check "Large"
-        select @shipping_category.name, from: "product_shipping_category_id"
-        click_button "Create"
-        expect(page).to have_content("successfully created!")
-        expect(Spree::Product.last.variants.length).to eq(1)
-      end
-
-      it "should not display variants when prototype does not contain option types" do
-        select "Random", from: "Prototype"
-
-        fill_in "product_name", with: "Baseball Cap"
-
-        expect(page).not_to have_content("Variants")
-      end
-
-      it "should keep option values selected if validation fails" do
-        fill_in "product_name", with: "Baseball Cap"
-        fill_in "product_sku", with: "B100"
-        fill_in "product_price", with: "100"
-        select "Size", from: "Prototype"
-        wait_for_ajax
-        check "Large"
-        click_button "Create"
-        expect(page).to have_content("Shipping Category can't be blank")
-        expect(field_labeled("Size")).to be_checked
-        expect(field_labeled("Large")).to be_checked
-        expect(field_labeled("Small")).not_to be_checked
-      end
-    end
-
-    context "creating a new product" do
-      before(:each) do
-        @shipping_category = create(:shipping_category)
-        visit spree.admin_products_path
-        click_link "admin_new_product"
-        within('#new_product') do
-          expect(page).to have_content("SKU")
-        end
-      end
-
-      it "should allow an admin to create a new product" do
-        fill_in "product_name", with: "Baseball Cap"
-        fill_in "product_sku", with: "B100"
-        fill_in "product_price", with: "100"
-        fill_in "product_available_on", with: "2012/01/24"
-        select @shipping_category.name, from: "product_shipping_category_id"
-        click_button "Create"
-        expect(page).to have_content("successfully created!")
-        click_button "Update"
-        expect(page).to have_content("successfully updated!")
-      end
-
-      it "should show validation errors" do
-        fill_in "product_name", with: "Baseball Cap"
-        fill_in "product_sku", with: "B100"
-        fill_in "product_price", with: "100"
-        click_button "Create"
-        expect(page).to have_content("Shipping Category can't be blank")
-      end
-
-      context "using a locale with a different decimal format " do
-        before do
-          # change English locale’s separator and delimiter to match 19,99 format
-          I18n.backend.store_translations(:en,
-            :number => {
-              :currency => {
-                :format => {
-                  :separator => ",",
-                  :delimiter => "."
-                }
-              }
-            })
-        end
-
-        after do
-          # revert changes to English locale
-          I18n.backend.store_translations(:en,
-            :number => {
-              :currency => {
-                :format => {
-                  :separator => ".",
-                  :delimiter => ","
-                }
-              }
-            })
-        end
-
-        it "should show localized price value on validation errors", :js => true do
-          fill_in "product_price", :with => "19,99"
-          click_button "Create"
-          expect(find('input#product_price').value).to eq('19,99')
-        end
-      end
-
-      # Regression test for #2097
-      it "can set the count on hand to a null value" do
-        fill_in "product_name", with: "Baseball Cap"
-        fill_in "product_price", with: "100"
-        select @shipping_category.name, from: "product_shipping_category_id"
-        click_button "Create"
-        expect(page).to have_content("successfully created!")
-        click_button "Update"
-        expect(page).to have_content("successfully updated!")
-      end
-    end
-
-
-    context "cloning a product", js: true do
-      it "should allow an admin to clone a product" do
-        create(:product)
-
-        visit spree.admin_products_path
-        within_row(1) do
-          click_icon :clone
-        end
-
-        expect(page).to have_content("Product has been cloned")
-      end
-
-      context "cloning a deleted product" do
-        it "should allow an admin to clone a deleted product" do
-          create(:product, name: "apache baseball cap")
-
-          visit spree.admin_products_path
-          click_on 'Filter'
-          check "Show Deleted"
-          click_on 'Search'
-
-          expect(page).to have_content("apache baseball cap")
-
-          within_row(1) do
-            click_icon :clone
-          end
-
-          expect(page).to have_content("Product has been cloned")
-        end
-      end
-    end
-
-    context 'updating a product' do
-      let(:product) { create(:product) }
-
-      let(:prototype) do
-        size = build_option_type_with_values("size", %w(Small Medium Large))
-        FactoryGirl.create(:prototype, name: "Size", option_types: [ size ])
-      end
-
-      before(:each) do
-        @option_type_prototype = prototype
-        @property_prototype = create(:prototype, name: "Random")
-      end
-
-      it 'should parse correctly available_on' do
-        visit spree.admin_product_path(product)
-        fill_in "product_available_on", with: "2012/12/25"
-        click_button "Update"
-        expect(page).to have_content("successfully updated!")
-        expect(Spree::Product.last.available_on.to_s).to eq("2012-12-25 00:00:00 UTC")
-      end
-
-      it 'should add option_types when selecting a prototype', js: true do
-        visit spree.admin_product_path(product)
-        within('#sidebar') do
-          click_link 'Properties'
-        end
-        click_link "Select From Prototype"
-
-        within("#prototypes tr#row_#{prototype.id}") do
-          click_link 'Select'
-          wait_for_ajax
-        end
-
-        within(:css, "tr.product_property:first-child") do
-          expect(first('input[type=text]').value).to eq('baseball_cap_color')
-        end
-      end
-
-      context "using a locale with a different decimal format" do
-        before do
-          # change English locale’s separator and delimiter to match 19,99 format
-          I18n.backend.store_translations(
-            :en,
-            number: {
-              currency: {
-                format: {
-                  separator: ",",
-                  delimiter: "."
-                }
-              },
-              format: {
-                separator: ",",
-                delimiter: "."
-              }
-            }
-          )
-        end
-
-        after do
-          # revert changes to English locale
-          I18n.backend.store_translations(
-            :en,
-            number: {
-              currency: {
-                format: {
-                  separator: ".",
-                  delimiter: ","
-                }
-              },
-              format: {
-                separator: ".",
-                delimiter: ","
-              }
-            }
-          )
-        end
-
-        it 'should parse correctly decimal values like weight' do
-          visit spree.admin_product_path(product)
-          fill_in 'product_weight', with: '1'
-          click_button 'Update'
-          weight_prev = find('#product_weight').value
-          click_button 'Update'
-          expect(find('#product_weight').value).to eq(weight_prev)
-        end
-      end
-    end
-
-    context 'deleting a product', :js => true do
-      let!(:product) { create(:product) }
-
-      it "is still viewable" do
-        visit spree.admin_products_path
-        accept_alert do
-          click_icon :delete
-          wait_for_ajax
-        end
-        click_on 'Filter'
-        # This will show our deleted product
-        check "Show Deleted"
-        click_on 'Search'
-        click_link product.name
-        expect(find("#product_price").value.to_f).to eq(product.price.to_f)
-      end
-    end
-  end
-
-  context 'with only product permissions' do
-
-    before do
-      allow_any_instance_of(Spree::Admin::BaseController).to receive(:spree_current_user).and_return(nil)
-    end
-
-    custom_authorization! do |user|
-      can [:admin, :update, :index, :read], Spree::Product
-    end
-    let!(:product) { create(:product) }
-
-    it "should only display accessible links on index" do
-      visit spree.admin_products_path
-
-      expect(page).to have_link('Products')
-      expect(page).not_to have_link('Option Types')
-      expect(page).not_to have_link('Properties')
-      expect(page).not_to have_link('Prototypes')
-      expect(page).not_to have_link('New Product')
-      expect(page).not_to have_css('.icon-clone')
-      expect(page).to have_css('.icon-edit')
-      expect(page).not_to have_css('.delete-resource')
-    end
-
-    it "should only display accessible links on edit" do
-      visit spree.admin_product_path(product)
-
-      # product tabs should be hidden
-      expect(page).to have_link('Details')
-      expect(page).not_to have_link('Images')
-      expect(page).not_to have_link('Variants')
-      expect(page).not_to have_link('Properties')
-      expect(page).not_to have_link('Stock Management')
-
-      # no create permission
-      expect(page).not_to have_link('New Product')
-    end
-  end
-end