workarea-core 3.5.0.beta.1 → 3.5.0

data/test/lib/workarea/latest_version_test.rb

@@ -0,0 +1,11 @@
+require 'test_helper'
+
+module Workarea
+  class LatestVersionTest < TestCase
+    def test_get
+      VCR.use_cassette 'get_latest_version' do
+        assert_equal('3.4.21', Workarea::LatestVersion.get)
+      end
+    end
+  end
+end

data/test/models/workarea/checkout/collect_payment_test.rb

@@ -39,19 +39,19 @@ module Workarea
 
       def test_action
         Workarea.config.checkout_payment_action = {
-          not_shipped: 'foo!',
-          shipped: 'bar!',
-          mixed: 'baz!'
+          shipping: 'bar!',
+          partial_shipping: 'baz!',
+          no_shipping: 'foo!'
         }
 
-        order.items.build(requires_shipping: false)
+        order.items.build(fulfillment: 'download')
         assert_equal('foo!', collect_payment.action)
 
-        order.items.build(requires_shipping: true)
+        order.items.build(fulfillment: 'shipping')
         assert_equal('baz!', collect_payment.action)
 
         order.items.clear
-        order.items.build(requires_shipping: true)
+        order.items.build(fulfillment: 'shipping')
         assert_equal('bar!', collect_payment.action)
       end
     end

data/test/models/workarea/data_file/csv_test.rb

@@ -401,6 +401,21 @@ module Workarea
           assert_equal("Bowl Light\u0099", product.name)
         end
       end
+
+      def test_bom_characters_in_unicode
+        csv = %(\xEF\xBB\xBF_id,name,slug,details_keywords\n653911,Bowl Light,Bowl_Light,"testing")
+        import = create_import(
+          model_type: Catalog::Product.name,
+          file: create_tempfile(csv, extension: 'csv'),
+          file_type: 'csv'
+        )
+
+        Csv.new(import).import!
+
+        product = Catalog::Product.find_by(slug: 'bowl_light')
+
+        assert_equal('653911', product.id)
+      end
     end
   end
 end

data/test/models/workarea/fulfillment/sku_test.rb

@@ -4,7 +4,7 @@ module Workarea
   class Fulfillment
     class SkuTest < TestCase
       def test_validation
-        sku = Sku.new(id: 'SKU1', policy: 'ignore')
+        sku = Sku.new(id: 'SKU1', policy: 'shipping')
         assert(sku.valid?)
 
         sku.policy = 'download'
@@ -16,10 +16,10 @@ module Workarea
       end
 
       def test_find_or_initialize_all
-        create_fulfillment_sku(id: 'sku1', policy: 'ignore')
+        create_fulfillment_sku(id: 'sku1')
         skus = Sku.find_or_initialize_all(%w(sku1 sku2))
 
-        assert(2, skus.size)
+        assert_equal(2, skus.size)
         assert(skus.first.persisted?)
         refute(skus.second.persisted?)
       end
@@ -28,7 +28,7 @@ module Workarea
         order = Order.new
         item = order.items.build(sku: 'SKU1')
 
-        sku = Sku.new(id: 'SKU1', policy: 'ignore')
+        sku = Sku.new(id: 'SKU1', policy: 'shipping')
         assert_nil(sku.process!(order_item: item))
 
         sku = Sku.new(id: 'SKU1', policy: 'foobar')
@@ -40,7 +40,7 @@ module Workarea
       end
 
       def test_downloadable?
-        sku = Sku.new(id: 'SKU1', policy: 'ignore')
+        sku = Sku.new(id: 'SKU1', policy: 'shipping')
         refute(sku.downloadable?)
 
         sku.policy = 'download'

data/test/models/workarea/insights/cold_searches_test.rb

@@ -5,32 +5,32 @@ module Workarea
     class ColdSearchesTest < TestCase
       def test_results
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: -1,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: -4,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: -2,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           revenue_change: -5,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'quz',
+          query_string: 'quz',
           revenue_change: -15,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qux',
+          query_string: 'qux',
           revenue_change: 0,
           reporting_on: Time.current.last_week
         )
@@ -41,32 +41,33 @@ module Workarea
         cold_searches = ColdSearches.first
         assert_equal(1, cold_searches.results.size)
         assert_equal('quz', cold_searches.results.first['query_id'])
+        assert_equal('quz', cold_searches.results.first['query_string'])
         assert_equal(-15, cold_searches.results.first['revenue_change'])
       end
 
       def test_falling_back_to_fewer_deviations
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: -1,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: -4,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: -2,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           revenue_change: -5,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qux',
+          query_string: 'qux',
           revenue_change: 0,
           reporting_on: Time.current.last_week
         )
@@ -77,6 +78,7 @@ module Workarea
         cold_searches = ColdSearches.first
         assert_equal(1, cold_searches.results.size)
         assert_equal('qoo', cold_searches.results.first['query_id'])
+        assert_equal('qoo', cold_searches.results.first['query_string'])
         assert_equal(-5, cold_searches.results.first['revenue_change'])
       end
 

data/test/models/workarea/insights/hot_searches_test.rb

@@ -5,32 +5,32 @@ module Workarea
     class HotSearchesTest < TestCase
       def test_results
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: 1,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: 4,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: 2,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           revenue_change: 5,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'quz',
+          query_string: 'quz',
           revenue_change: 15,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qux',
+          query_string: 'qux',
           revenue_change: 0,
           reporting_on: Time.current.last_week
         )
@@ -41,32 +41,33 @@ module Workarea
         hot_searches = HotSearches.first
         assert_equal(1, hot_searches.results.size)
         assert_equal('quz', hot_searches.results.first['query_id'])
+        assert_equal('quz', hot_searches.results.first['query_string'])
         assert_equal(15, hot_searches.results.first['revenue_change'])
       end
 
       def test_falling_back_to_fewer_deviations
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: 1,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: 4,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: 2,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           revenue_change: 5,
           reporting_on: Time.current.last_week
         )
         create_search_by_week(
-          query_id: 'qux',
+          query_string: 'qux',
           revenue_change: 0,
           reporting_on: Time.current.last_week
         )
@@ -77,6 +78,7 @@ module Workarea
         hot_searches = HotSearches.first
         assert_equal(1, hot_searches.results.size)
         assert_equal('qoo', hot_searches.results.first['query_id'])
+        assert_equal('qoo', hot_searches.results.first['query_string'])
         assert_equal(5, hot_searches.results.first['revenue_change'])
       end
 

data/test/models/workarea/insights/searches_to_improve_test.rb

@@ -5,25 +5,25 @@ module Workarea
     class SearchesToImproveTest < TestCase
       def test_results
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.1
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           searches_percentile: 90,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.2
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.01
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.02
@@ -35,20 +35,22 @@ module Workarea
         searches_to_improve = SearchesToImprove.first
         assert_equal(2, searches_to_improve.results.size)
         assert_equal('baz', searches_to_improve.results.first['query_id'])
+        assert_equal('baz', searches_to_improve.results.first['query_string'])
         assert_equal(0.01, searches_to_improve.results.first['conversion_rate'])
         assert_equal('qoo', searches_to_improve.results.second['query_id'])
+        assert_equal('qoo', searches_to_improve.results.second['query_string'])
         assert_equal(0.02, searches_to_improve.results.second['conversion_rate'])
       end
 
       def test_falling_back_to_second_pass
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           searches_percentile: 90,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.1
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           searches_percentile: 80,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.2
@@ -62,6 +64,7 @@ module Workarea
         searches_to_improve = SearchesToImprove.first
         assert_equal(1, searches_to_improve.results.size)
         assert_equal('foo', searches_to_improve.results.first['query_id'])
+        assert_equal('foo', searches_to_improve.results.first['query_string'])
         assert_equal(0.1, searches_to_improve.results.first['conversion_rate'])
       end
 

data/test/models/workarea/insights/star_searches_test.rb

@@ -5,25 +5,25 @@ module Workarea
     class StarSearchesTest < TestCase
       def test_generate_weekly!
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.1
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           searches_percentile: 90,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.2
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.01
         )
         create_search_by_week(
-          query_id: 'qoo',
+          query_string: 'qoo',
           searches_percentile: 100,
           reporting_on: Time.current.last_week,
           conversion_rate: 0.02
@@ -35,6 +35,7 @@ module Workarea
         star_searches = StarSearches.first
         assert_equal(1, star_searches.results.size)
         assert_equal('foo', star_searches.results.first['query_id'])
+        assert_equal('foo', star_searches.results.first['query_string'])
         assert_equal(0.1, star_searches.results.first['conversion_rate'])
       end
 

data/test/models/workarea/insights/trending_searches_test.rb

@@ -5,55 +5,55 @@ module Workarea
     class TrendingSearchesTest < TestCase
       def test_generate_monthly!
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: 10,
           orders: 5,
           reporting_on: Time.zone.local(2018, 12, 3)
         )
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: -10,
           orders: 0,
           reporting_on: Time.zone.local(2018, 12, 10)
         )
         create_search_by_week(
-          query_id: 'foo',
+          query_string: 'foo',
           revenue_change: 20,
           orders: 10,
           reporting_on: Time.zone.local(2018, 12, 17)
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: 10,
           orders: 5,
           reporting_on: Time.zone.local(2018, 12, 3)
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: -10,
           orders: 0,
           reporting_on: Time.zone.local(2018, 12, 10)
         )
         create_search_by_week(
-          query_id: 'bar',
+          query_string: 'bar',
           revenue_change: 0,
           orders: 0,
           reporting_on: Time.zone.local(2018, 12, 17)
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: 10,
           orders: 5,
           reporting_on: Time.zone.local(2018, 12, 3)
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: -10,
           orders: 1,
           reporting_on: Time.zone.local(2018, 12, 10)
         )
         create_search_by_week(
-          query_id: 'baz',
+          query_string: 'baz',
           revenue_change: 0,
           orders: 0,
           reporting_on: Time.zone.local(2018, 12, 17)
@@ -67,14 +67,17 @@ module Workarea
         assert_equal(3, trending_searches.results.size)
 
         assert_equal('foo', trending_searches.results.first['query_id'])
+        assert_equal('foo', trending_searches.results.first['query_string'])
         assert_equal(2, trending_searches.results.first['improving_weeks'])
         assert_equal(15, trending_searches.results.first['orders'])
 
         assert_equal('baz', trending_searches.results.second['query_id'])
+        assert_equal('baz', trending_searches.results.second['query_string'])
         assert_equal(1, trending_searches.results.second['improving_weeks'])
         assert_equal(6, trending_searches.results.second['orders'])
 
         assert_equal('bar', trending_searches.results.third['query_id'])
+        assert_equal('bar', trending_searches.results.third['query_string'])
         assert_equal(1, trending_searches.results.third['improving_weeks'])
         assert_equal(5, trending_searches.results.third['orders'])
       end