workarea-admin 3.4.45 → 3.5.0.beta.1

data/test/javascripts/new_navigation_taxons_spec.js

@@ -6,6 +6,7 @@
             it('initializes select2 on navigable id select', function () {
                 var markup = 'new_navigation_taxon.html',
                     $fixture = $(fixture.load(markup, true)),
+
                     $select = $('[name=navigable_id]', $fixture);
 
                 expect($select.is('.select2-hidden-accessible')).to.not.be.ok;
@@ -13,18 +14,6 @@
                 WORKAREA.newNavigationTaxons.init($fixture);
 
                 expect($select.is('.select2-hidden-accessible')).to.be.ok;
-                expect($select.val()).to.equal('');
-            });
-
-            it('saves with the existing data', function () {
-                var markup = 'existing_navigation_taxon.html',
-                    $fixture = $(fixture.load(markup, true)),
-                    $select = $('[name=navigable_id]', $fixture);
-
-                WORKAREA.newNavigationTaxons.init($fixture);
-
-                expect($select.is('.select2-hidden-accessible')).to.be.ok;
-                expect($select.val()).to.equal('bar');
             });
         });
     });

data/test/mailers/workarea/admin/comment_mailer_test.rb

@@ -0,0 +1,35 @@
+require 'test_helper'
+
+module Workarea
+  module Admin
+    class CommentMailerTest < MailerTest
+      include TestCase::SearchIndexing
+      include TestCase::Mail
+
+      delegate :t, to: :I18n
+
+      def test_notify
+        Workarea.config.email_from = 'noreply@example.com'
+        user = create_user
+        product = create_product
+        comment = product.comments.create!(body: 'test order comment')
+
+        CommentMailer.notify(user.id, comment.id).deliver_now
+
+        email = ActionMailer::Base.deliveries.last
+        html = email.parts.second.body
+
+        assert_includes(email.to, user.email)
+        assert_includes(email.from, Workarea.config.email_from)
+        assert_includes(html, comment.body)
+        assert_includes(
+          html,
+          t(
+            'workarea.admin.comment_mailer.notify.unsubscribe_from_notifications',
+            commentable_name: product.name
+          )
+        )
+      end
+    end
+  end
+end

data/test/mailers/workarea/admin/status_report_mailer_test.rb

@@ -2,36 +2,41 @@ require 'test_helper'
 
 module Workarea
   module Admin
-    class StatusReportMailerTest < TestCase
+    class StatusReportMailerTest < MailerTest
       include TestCase::SearchIndexing
       include TestCase::Mail
 
       delegate :t, to: :I18n
 
-      def test_showing_dashboard_data
-        Workarea.with_config do |config|
-          config.email_from = 'noreply@example.com'
-
-          addresses = %w(
-            bcrouse@workarea.com
-            tscott@workarea.com
-          )
-          from = [Workarea.config.email_from]
-          StatusReportMailer
-            .report(addresses, Time.zone.parse('2013/11/13'))
-            .deliver_now
-
-          email = ActionMailer::Base.deliveries.last
-          html = email.parts.second.body
-
-          assert_includes(html, Workarea.config.site_name)
-          assert_includes(html, t('workarea.admin.status_report_mailer.orders.total_orders'))
-          assert_includes(html, t('workarea.admin.status_report_mailer.alerts.title'))
-          assert_includes(html, Money.default_currency.symbol)
-          assert_equal(addresses, email.bcc)
-          assert_nil(email.to)
-          assert_equal(from, email.from)
-        end
+      def test_report
+        # create a Products Missing Images alert
+        create_product(images: [], description: nil)
+
+        Workarea.config.email_from = 'noreply@example.com'
+        user = create_user(email: 'bcrouse@workarea.com')
+
+        StatusReportMailer.report(user.email).deliver_now
+
+        email = ActionMailer::Base.deliveries.last
+        html = email.parts.second.body
+
+        assert_includes(html, Workarea.config.site_name)
+        assert_includes(html, t('workarea.admin.status_report_mailer.orders.total_orders'))
+        assert_includes(html, t('workarea.admin.status_report_mailer.alerts.title'))
+        assert_includes(html, t('workarea.admin.status_report_mailer.unsubscribe'))
+        assert_includes(html, Money.default_currency.symbol)
+        assert_includes(email.to, user.email)
+        assert_includes(email.from, Workarea.config.email_from)
+      end
+
+      def test_report_to_many
+        users = [
+          create_user(email: 'bcrouse@workarea.com'),
+          create_user(email: 'tscott@workarea.com')
+        ]
+
+        StatusReportMailer.report_to_many(users).map(&:deliver_now)
+        assert_equal(2, ActionMailer::Base.deliveries.count)
       end
     end
   end

data/test/system/workarea/admin/alerts_system_test.rb

@@ -17,6 +17,7 @@ module Workarea
         )
         create_category(product_ids: [], product_rules: [])
         create_release(name: 'Foo', publish_at: 1.hour.from_now)
+        create_inventory(id: 'NODETAILS', policy: :standard, available: 1)
 
         visit admin.root_path
         assert(page.has_content?('9 Alerts'))
@@ -28,7 +29,7 @@ module Workarea
           assert(page.has_content?('3 products missing descriptions'))
           assert(page.has_content?('1 product missing variants'))
           assert(page.has_content?('3 products missing categories'))
-          assert(page.has_content?('4 products with low inventory'))
+          assert(page.has_content?('1 product with low inventory'))
           assert(page.has_content?('2 products with variants missing details'))
           assert(page.has_content?('1 product with inconsistent variant details'))
           assert_match(/Foo publishes on/, page.text)

data/test/system/workarea/admin/bulk_actions_system_test.rb

@@ -156,7 +156,7 @@ module Workarea
         assert(page.has_content?(t('workarea.admin.bulk_action_product_edits.review.title')))
         assert(page.has_content?(t('workarea.admin.bulk_action_product_edits.worflow.selected', label: '2 Products')))
 
-        assert(page.has_content?("#{t('workarea.admin.fields.active')}: false"))
+        assert(page.has_content?("#{t('workarea.admin.fields.active')} false"))
         assert(page.has_content?('add_tag_2, add_tag_3'))
         assert(page.has_content?('remove_tag_1'))
         assert(page.has_content?('add_filter_1'))

data/test/system/workarea/admin/comments_system_test.rb

@@ -16,6 +16,49 @@ module Workarea
 
         assert(page.has_content?('Success'))
       end
+
+      def test_notifying_admins
+        me = create_user(email: 'foo@workarea.com')
+
+        create_user(
+          first_name: 'Bar',
+          last_name: 'Baby',
+          email: 'bar@workarea.com'
+        )
+
+        create_user(
+          first_name: 'Baz',
+          last_name: 'Bat',
+          email: 'baz@workarea.com',
+          admin: true
+        )
+
+        visit admin.user_path(me)
+
+        click_link 'Comments'
+
+        find_field('comment[body]').send_keys(['@'])
+
+        refute(page.has_content?('Bar Baby'))
+        assert(page.has_content?('Baz Bat'))
+
+        find('.tribute-container li', text: 'Baz Bat (baz@workarea.com)').click
+        click_button 'create_comment'
+
+        assert(page.has_content?('Success'))
+      end
+
+      def test_subscribing_to_comments
+        user = create_user(email: 'bcrouse-test@workarea.com')
+        visit admin.user_path(user)
+
+        click_link 'Comments'
+        click_button t('workarea.admin.comments.index.subscribe')
+        assert(page.has_content?('Success'))
+
+        click_button t('workarea.admin.comments.index.unsubscribe')
+        assert(page.has_content?('Success'))
+      end
     end
   end
 end

data/test/system/workarea/admin/content_system_test.rb

@@ -94,6 +94,12 @@ module Workarea
         click_link 'Advanced', match: :first
 
         assert_current_path(/#{admin.advanced_content_path(home)}/)
+
+        assert(page.has_selector?('#content_browser_title', visible: false))
+
+        find('.toggle-button__label--positive').click
+        assert(page.has_selector?('#content_browser_title', visible: true))
+
         fill_in 'content[browser_title]', with: 'Browser Title'
         click_button 'save'
 

data/test/system/workarea/admin/dashboard_system_test.rb

@@ -54,18 +54,17 @@ module Workarea
       end
 
       def test_viewing_the_settings_dashboard
-        Workarea.with_config do |config|
-          config.fooconfig = 'barvalue'
+        Workarea.config.fooconfig = 'barvalue'
 
-          visit admin.settings_dashboards_path
-          assert(page.has_content?('Fooconfig'))
-          assert(page.has_content?('barvalue'))
+        visit admin.settings_dashboards_path
+        assert(page.has_content?('Fooconfig'))
+        assert(page.has_content?('barvalue'))
+        assert(page.has_selector?('.jump-to-menu'))
 
-          config.hide_from_settings = %i(fooconfig)
+        Workarea.config.hide_from_settings = %i(fooconfig)
 
-          visit admin.settings_dashboards_path
-          refute(page.has_content?('Fooconfig'))
-        end
+        visit admin.settings_dashboards_path
+        refute(page.has_content?('Fooconfig'))
       end
     end
   end

data/test/system/workarea/admin/data_files_system_test.rb

@@ -30,7 +30,7 @@ module Workarea
           )
 
           import_two = create_import(
-            model_type: Workarea::Content::Page,
+            model_type: 'Workarea::Content::Page',
             created_by_id: user.id,
             started_at: 2.hour.ago,
             file: tempfile,

data/test/system/workarea/admin/discounts_system_test.rb

@@ -101,6 +101,24 @@ module Workarea
         assert(page.has_content?('444'))
         assert(page.has_content?('555'))
       end
+
+      def test_discount_redemptions
+        discount = create_category_discount
+        7.times { |i| discount.log_redemption("foo-#{i}@workarea.com") }
+        order = create_placed_order(email: 'foo-1@workarea.com')
+
+        visit admin.pricing_discount_path(discount)
+        within '.card--redemptions' do
+          assert(page.has_content?(7))
+        end
+
+        click_link t('workarea.admin.pricing_discounts.cards.redemptions.header')
+        assert(page.has_content?(7))
+        7.times { |i| assert(page.has_content?("foo-#{i}@workarea.com")) }
+
+        click_link 'foo-1@workarea.com'
+        assert(page.has_content?(order.id))
+      end
     end
   end
 end

data/test/system/workarea/admin/featured_products_system_test.rb

@@ -8,12 +8,15 @@ module Workarea
       def test_managing_featured_products
         create_product(name: 'Foo')
         create_product(name: 'Bar')
+        create_product(name: 'Baz', active: false)
         category = create_category
         visit admin.catalog_category_path(category)
 
         click_link t('workarea.admin.catalog_categories.cards.featured_products.title')
         assert(page.has_content?('Foo'))
         assert(page.has_content?('Bar'))
+        assert(page.has_content?('Baz'))
+        assert(page.has_content?(t('workarea.admin.featured_products.statuses.inactive')))
 
         click_link 'Foo'
         assert(page.has_content?('Success'))

data/test/system/workarea/admin/fulfillment_skus_system_test.rb

@@ -0,0 +1,48 @@
+require 'test_helper'
+
+module Workarea
+  module Admin
+    class FulfillmentSkusSystemTest < SystemTest
+      include Admin::IntegrationTest
+
+      def test_management
+        visit admin.fulfillment_skus_path
+        click_link 'add_fulfillment_sku'
+
+        fill_in 'sku[id]', with: 'SKU1'
+        click_button 'create_sku'
+
+        assert_current_path(admin.fulfillment_sku_path('SKU1'))
+        assert(page.has_content?('Success'))
+        assert(page.has_content?('SKU1'))
+
+        click_link 'Attributes'
+        select 'Download', from: 'sku[policy]'
+        attach_file 'sku[file]', product_image_file_path
+        click_button 'save_sku'
+
+        assert(page.has_content?('Success'))
+        assert_current_path(admin.fulfillment_sku_path('SKU1'))
+        assert(page.has_content?('Download'))
+
+        visit admin.fulfillment_sku_path('SKU1')
+        click_link 'Delete'
+
+        assert_current_path(admin.fulfillment_skus_path)
+        assert(page.has_content?('Success'))
+        assert(page.has_no_content?('SKU1'))
+      end
+
+      def test_accessing_from_variant
+        product = create_product(variants: [{ sku: 'SKU1', regular: 5.to_m }])
+        sku = create_fulfillment_sku(id: 'SKU1', policy: 'ignore')
+
+        visit admin.catalog_product_variants_path(product)
+        assert(page.has_content?('Ignore'))
+
+        click_link 'Ignore'
+        assert_current_path(admin.fulfillment_sku_path('SKU1'))
+      end
+    end
+  end
+end

data/test/system/workarea/admin/impersonation_system_test.rb

@@ -34,9 +34,12 @@ module Workarea
           click_button 'Stop Impersonation'
         end
 
-        assert_equal(storefront.root_path, current_path)
+        assert_equal(admin.user_path(user), current_path)
         assert(page.has_content?('Success'))
 
+        find('.view').hover # Ensure tooltipster menu isn't open
+        assert(page.has_content?('bcrouse@workarea.com'))
+
         visit storefront.users_account_path
         assert(page.has_no_content?('impersonated@workarea.com'))
 
@@ -49,7 +52,7 @@ module Workarea
           click_button 'Stop Impersonation'
         end
 
-        assert_equal(admin.root_path, current_path)
+        assert_equal(admin.user_path(user), current_path)
         assert(page.has_content?('Success'))
       end
     end

data/test/system/workarea/admin/orders_system_test.rb

@@ -56,24 +56,35 @@ module Workarea
       end
 
       def test_payment
-        Workarea.with_config do |config|
-          config.auto_capture = false
-
-          order = create_placed_order
-          visit admin.order_path(order)
-          click_link t('workarea.admin.orders.attributes.payment.title')
-
-          assert(page.has_content?('Ben Crouse'))
-          assert(page.has_content?('22 S. 3rd St.'))
-          assert(page.has_content?('Philadelphia'))
-          assert(page.has_content?('PA'))
-          assert(page.has_content?('19106'))
-          assert(page.has_content?('Credit Card'))
-          assert(page.has_content?('-1')) # Card Number
-          assert(page.has_content?('Authorize'))
-          assert(page.has_content?('Success'))
-          assert(page.has_content?("#{Money.default_currency.symbol}11.00")) # Auth Amount
-        end
+        Workarea.config.checkout_payment_action = {
+          not_shipped: 'authorize!',
+          shipped: 'authorize!',
+          mixed: 'authorize!'
+        }
+
+        order = create_placed_order
+        visit admin.order_path(order)
+        click_link t('workarea.admin.orders.attributes.payment.title')
+
+        assert(page.has_content?('Ben Crouse'))
+        assert(page.has_content?('22 S. 3rd St.'))
+        assert(page.has_content?('Philadelphia'))
+        assert(page.has_content?('PA'))
+        assert(page.has_content?('19106'))
+        assert(page.has_content?('Credit Card'))
+        assert(page.has_content?('-1')) # Card Number
+        assert(page.has_content?('Authorize'))
+        assert(page.has_content?('Success'))
+        assert(page.has_content?("#{Money.default_currency.symbol}11.00")) # Auth Amount
+      end
+
+      def test_fraud
+        order = create_fraudulent_order
+        visit admin.order_path(order)
+        click_link t('workarea.admin.orders.attributes.fraud.title')
+        assert(page.has_content?('Declined')) # decision
+        assert(page.has_content?(order.fraud_decision.message))
+        assert(page.has_content?(order.fraud_decision.response))
       end
 
       def test_timeline
@@ -89,9 +100,9 @@ module Workarea
         )
 
         visit admin.order_path(order)
-        assert(page.has_content?(t('workarea.admin.orders.timeline.placed')))
-        assert(page.has_content?(shipped_content))
-        assert(page.has_content?(auth_content))
+        assert_match(t('workarea.admin.orders.timeline.placed'), page.body)
+        assert_match(shipped_content, page.body)
+        assert_match(auth_content, page.body)
 
         click_link t('workarea.admin.timeline.card.title')
 

data/test/system/workarea/admin/payment_transactions_system_test.rb

@@ -6,38 +6,40 @@ module Workarea
       include Admin::IntegrationTest
 
       def test_viewing_transactions
-        Workarea.with_config do |config|
-          config.auto_capture = false
-
-          order = create_placed_order(id: 'FOO')
-          payment = Payment.find(order.id)
-          auth = payment.credit_card.transactions.first
-          capture = payment.credit_card.build_transaction(
-            action: 'capture',
-            amount: auth.amount,
-            reference: auth
-          )
-          capture.complete!
-
-          visit admin.payment_transactions_path
-
-          click_button 'Transaction'
-          assert(page.has_content?('Authorize (1)'))
-          assert(page.has_content?('Capture (1)'))
-          click_button 'Transaction' # closes filter dropdown
-
-          click_button 'Auth Status'
-          assert(page.has_content?('Captured (1)'))
-
-          visit admin.payment_transactions_path
-          click_link 'Authorize', match: :first
-
-          assert(page.has_content?('Transaction'))
-          assert(page.has_content?('Credit Card'))
-          assert(page.has_content?("#{Money.default_currency.symbol}11.00"))
-          assert(page.has_content?('Bogus Gateway: Forced success'))
-          assert(page.has_content?('Capture'))
-        end
+        Workarea.config.checkout_payment_action = {
+          not_shipped: 'authorize!',
+          shipped: 'authorize!',
+          mixed: 'authorize!'
+        }
+
+        order = create_placed_order(id: 'FOO')
+        payment = Payment.find(order.id)
+        auth = payment.credit_card.transactions.first
+        capture = payment.credit_card.build_transaction(
+          action: 'capture',
+          amount: auth.amount,
+          reference: auth
+        )
+        capture.complete!
+
+        visit admin.payment_transactions_path
+
+        click_button 'Transaction'
+        assert(page.has_content?('Authorize (1)'))
+        assert(page.has_content?('Capture (1)'))
+        click_button 'Transaction' # closes filter dropdown
+
+        click_button 'Auth Status'
+        assert(page.has_content?('Captured (1)'))
+
+        visit admin.payment_transactions_path
+        click_link 'Authorize', match: :first
+
+        assert(page.has_content?('Transaction'))
+        assert(page.has_content?('Credit Card'))
+        assert(page.has_content?("#{Money.default_currency.symbol}11.00"))
+        assert(page.has_content?('Bogus Gateway: Forced success'))
+        assert(page.has_content?('Capture'))
       end
     end
   end