solidus_backend_devise_token_auth 2.8.0.alpha.2

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

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'Product Details', type: :feature 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_path
+      click_nav "Products"
+      within_row(1) { click_icon :edit }
+
+      click_link 'Product Details'
+
+      expect(page).to have_content('ProductsBún thịt nướng')
+      expect(page).to have_field('product_name', with: 'Bún thịt nướng')
+      expect(page).to have_field('product_slug', with: 'bun-th-t-n-ng')
+      expect(page).to have_field('product_description', with: 'lorem ipsum')
+      expect(page).to have_field('product_price', with: '19.99')
+      expect(page).to have_field('product_cost_price', with: '17.00')
+      expect(page).to have_field('product_available_on', with: "2013/08/14")
+      expect(page).to have_field('product_sku', with: '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_path
+      click_nav "Products"
+      within('table.index 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: ''
+      click_button "Update"
+      within('#product_slug_field') { expect(page).to have_content("can't be blank") }
+
+      fill_in "product_slug", with: 'x'
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+    end
+  end
+
+  # Regression test for https://github.com/spree/spree/issues/3385
+  context "deleting a product", js: true 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 :trash
+        end
+      end
+      expect(page).to have_content('Product has been deleted')
+    end
+  end
+end

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

@@ -0,0 +1,47 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Product Display Order", type: :feature do
+  stub_authorization!
+
+  context "managing display order", js: true do
+    def assert_selected_taxons(taxons)
+      # Regression test for https://github.com/spree/spree/issues/2139
+      taxons.each do |taxon|
+        expect(page).to have_css(".select2-search-choice", text: taxon.name)
+      end
+
+      expected_value = taxons.map(&:id).join(",")
+      expect(page).to have_xpath("//*[@id = 'product_taxon_ids' and @value = '#{expected_value}']", visible: :all)
+    end
+
+    let(:product) { create(:product) }
+
+    it "should allow an admin to manage display order (taxons)" do
+      taxon_1 = create(:taxon)
+      taxon_2 = create(:taxon, name: 'Clothing')
+      product.taxons << taxon_1
+
+      visit spree.edit_admin_product_path(product)
+
+      assert_selected_taxons([taxon_1])
+
+      select2_search "Clothing", from: "Taxon"
+      click_button "Update"
+      assert_selected_taxons([taxon_1, taxon_2])
+    end
+
+    xcontext "with an XSS attempt" do
+      let(:taxon_name) { %(<script>throw("XSS")</script>) }
+      let!(:taxon) { create(:taxon, name: taxon_name) }
+      it "displays the escaped HTML without executing it" do
+        visit spree.edit_admin_product_path(product)
+
+        select2_search "<script>", from: "Taxon"
+
+        expect(page).to have_content(taxon_name)
+      end
+    end
+  end
+end

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

@@ -0,0 +1,63 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Product Variants", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+  end
+
+  context "editing variant option types", js: true do
+    let!(:product) { create(:product) }
+
+    it "should allow an admin to create option types for a variant" do
+      click_nav "Products"
+
+      within_row(1) { click_icon :edit }
+
+      within('nav > ul.tabs') { 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_nav "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_path
+      click_nav "Products"
+      within('table.index tbody tr:nth-child(1)') do
+        click_icon :edit
+      end
+
+      select2_search "shirt", from: "Option Types"
+      click_button "Update"
+      expect(page).to have_content("successfully updated!")
+
+      within('nav > ul.tabs') { click_link "Variants" }
+      click_link "New Variant"
+
+      select "black", from: "Colors"
+      fill_in "variant_sku", with: "A100"
+      click_button "Create"
+      expect(page).to have_content("successfully created!")
+
+      within(".index") 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

@@ -0,0 +1,116 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe "Option Types", type: :feature do
+  stub_authorization!
+
+  before(:each) do
+    visit spree.admin_path
+    click_nav "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", js: true 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') do
+        find('tr', text: 'Size').click_link "Edit"
+      end
+      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 https://github.com/spree/spree/issues/2277
+  it "can remove an option value from an option type", js: true do
+    option_value = create(:option_value)
+    click_link "Option Types"
+    within('table#listing_option_types') { click_icon :edit }
+    expect(page).to have_title("#{option_value.option_type.name} - Option Types - Products")
+    expect(page).to have_css("tbody#option_values tr", count: 1)
+    within("tbody#option_values") do
+      find('.spree_remove_fields').click
+    end
+    # Assert that the field is hidden automatically
+    expect(page).to have_no_css("tbody#option_values tr")
+
+    # Ensure the DELETE request finishes
+    expect(page).to have_no_css("#progress")
+
+    # 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", count: 2).each do |input|
+      expect(input.value).to be_blank
+    end
+  end
+
+  # Regression test for https://github.com/spree/spree/issues/3204
+  it "can remove a non-persisted option value from an option type", js: true do
+    create(:option_type)
+    click_link "Option Types"
+    within('table#listing_option_types') { click_icon :edit }
+
+    expect(page).to have_css("tbody#option_values tr", count: 1)
+
+    # Add a new option type
+    click_link "Add Option Value"
+    expect(page).to have_css("tbody#option_values tr", count: 2)
+
+    # Remove default option type
+    within("tbody#option_values") do
+      within_row(1) do
+        find('.fa-trash').click
+      end
+    end
+    # Assert that the field is hidden automatically
+    expect(page).to have_css("tbody#option_values tr", count: 1)
+
+    # Remove added option type
+    within("tbody#option_values") do
+      find('.fa-trash').click
+    end
+    # Assert that the field is hidden automatically
+    expect(page).to have_css("tbody#option_values tr", count: 0)
+  end
+end

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

@@ -0,0 +1,109 @@
+# frozen_string_literal: true
+
+require 'spec_helper'
+
+describe 'Pricing' do
+  stub_authorization!
+
+  let(:product) { create(:product) }
+
+  before do
+    visit spree.edit_admin_product_path(product)
+  end
+
+  it 'has a Prices tab' do
+    within(".tabs") do
+      expect(page).to have_link("Prices")
+    end
+  end
+
+  context "in the prices tab" do
+    let!(:country) { create :country, iso: "DE" }
+    let(:master_price) { product.master.default_price }
+    let!(:other_price) { product.master.prices.create(amount: 34.56, currency: "RUB", country_iso: "DE") }
+
+    before do
+      visit spree.admin_product_prices_path(product)
+    end
+
+    it 'displays a table with the prices' do
+      expect(page).to have_content(product.name)
+      within(".tabs .active") do
+        expect(page).to have_content("Prices")
+      end
+
+      within('table.master_prices') do
+        expect(page).to have_content("$19.99")
+        expect(page).to have_content("USD")
+        expect(page).to have_content("34.56 ₽")
+        expect(page).to have_content("RUB")
+        expect(page).to have_content("Any Country")
+        expect(page).to have_content("Germany")
+      end
+    end
+
+    context "searching" do
+      let(:variant) { create(:variant, price: 20) }
+      let(:product) { variant.product }
+
+      before do
+        product.master.update(price: 49.99)
+      end
+
+      it 'has a working table filter' do
+        expect(page).to have_selector("#table-filter")
+        within "#table-filter" do
+          within "fieldset legend" do
+            expect(page).to have_content("Search")
+          end
+        end
+        select variant.options_text, from: "q_variant_id_eq", exact: false
+        click_button "Filter Results"
+        expect(page).to have_content("20")
+        expect(page).to_not have_content("49.99")
+      end
+    end
+
+    context "editing" do
+      let(:variant) { create(:variant, price: 20) }
+      let(:product) { variant.product }
+
+      before do
+        product.master.update(price: 49.99)
+      end
+
+      it 'has a working edit page' do
+        within "#spree_price_#{product.master.prices.first.id}" do
+          click_icon :edit
+        end
+        expect(page).to have_content("Edit Price")
+      end
+    end
+
+    context "deleting", js: true do
+      let(:product) { create(:product, price: 65.43) }
+      let!(:variant) { product.master }
+      let!(:other_price) { product.master.prices.create(amount: 34.56, currency: "EUR") }
+
+      it "will delete the non-default price" do
+        within "#spree_price_#{other_price.id}" do
+          accept_alert do
+            click_icon :trash
+          end
+        end
+        expect(page).to have_content("Price has been successfully removed")
+      end
+
+      it "does not break when default price is deleted" do
+        within "#spree_price_#{variant.default_price.id}" do
+          accept_alert do
+            click_icon :trash
+          end
+        end
+        expect(page).to have_content("Price has been successfully removed")
+        visit spree.admin_products_path
+        expect(page).to have_selector("#spree_product_#{product.id}")
+      end
+    end
+  end
+end