workarea-core 3.4.44 → 3.5.0.beta.1

data/test/models/workarea/metrics/affinity_test.rb

@@ -0,0 +1,85 @@
+require 'test_helper'
+
+module Workarea
+  module Metrics
+    class AffinityTest < TestCase
+      def test_recents_max
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_a',
+          category_ids: 'category_a',
+          search_ids: 'search_a'
+        )
+
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_b',
+          category_ids: 'category_b',
+          search_ids: 'search_b'
+        )
+
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_c',
+          category_ids: 'category_c',
+          search_ids: 'search_c'
+        )
+
+        metrics = User.find('bcrouse@workarea.com')
+        assert_equal(%w(product_c), metrics.viewed.recent_product_ids(max: 1))
+        assert_equal(%w(product_c product_b), metrics.viewed.recent_product_ids(max: 2))
+        assert_equal(%w(product_c product_b product_a), metrics.viewed.recent_product_ids(max: 3))
+
+        assert_equal(%w(category_c), metrics.viewed.recent_category_ids(max: 1))
+        assert_equal(%w(category_c category_b), metrics.viewed.recent_category_ids(max: 2))
+        assert_equal(%w(category_c category_b category_a), metrics.viewed.recent_category_ids(max: 3))
+
+        assert_equal(%w(search_c), metrics.viewed.recent_search_ids(max: 1))
+        assert_equal(%w(search_c search_b), metrics.viewed.recent_search_ids(max: 2))
+        assert_equal(%w(search_c search_b search_a), metrics.viewed.recent_search_ids(max: 3))
+      end
+
+      def test_recents_uniqueness
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_a',
+          category_ids: 'category_a',
+          search_ids: 'search_a'
+        )
+
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_a',
+          category_ids: 'category_b',
+          search_ids: 'search_b'
+        )
+
+        User.save_affinity(
+          id: 'bcrouse@workarea.com',
+          action: 'viewed',
+          product_ids: 'product_b',
+          category_ids: 'category_a',
+          search_ids: 'search_b'
+        )
+
+        metrics = User.find('bcrouse@workarea.com')
+        assert_equal(%w(product_a product_a product_b), metrics.viewed.product_ids)
+        assert_equal(%w(product_b product_a), metrics.viewed.recent_product_ids(unique: true))
+        assert_equal(%w(product_b product_a product_a), metrics.viewed.recent_product_ids(unique: false))
+
+        assert_equal(%w(category_a category_b category_a), metrics.viewed.category_ids)
+        assert_equal(%w(category_a category_b), metrics.viewed.recent_category_ids(unique: true))
+        assert_equal(%w(category_a category_b category_a), metrics.viewed.recent_category_ids(unique: false))
+
+        assert_equal(%w(search_a search_b search_b), metrics.viewed.search_ids)
+        assert_equal(%w(search_b search_a), metrics.viewed.recent_search_ids(unique: true))
+        assert_equal(%w(search_b search_b search_a), metrics.viewed.recent_search_ids(unique: false))
+      end
+    end
+  end
+end

data/test/models/workarea/metrics/product_by_week_test.rb

@@ -48,31 +48,29 @@ module Workarea
       end
 
       def test_append_last_week!
-        Workarea.with_config do |config|
-          config.insights_aggregation_per_page = 1
-
-          ProductForLastWeek.create!(product_id: 'foo', orders: 1)
-          ProductByWeek.append_last_week!
-          assert_equal(1, ProductByWeek.count)
-
-          product = ProductByWeek.find_by(product_id: 'foo')
-          assert_equal('foo', product.product_id)
-          assert_equal(1, product.orders)
-
-          ProductForLastWeek.delete_all
-          ProductForLastWeek.create!(product_id: 'foo', orders: 1)
-          ProductForLastWeek.create!(product_id: 'bar', orders: 2)
-          ProductByWeek.append_last_week!
-          assert_equal(3, ProductByWeek.count)
-
-          foo = ProductByWeek.find_by(product_id: 'foo')
-          assert_equal('foo', foo.product_id)
-          assert_equal(1, foo.orders)
-
-          bar = ProductByWeek.find_by(product_id: 'bar')
-          assert_equal('bar', bar.product_id)
-          assert_equal(2, bar.orders)
-        end
+        Workarea.config.insights_aggregation_per_page = 1
+
+        ProductForLastWeek.create!(product_id: 'foo', orders: 1)
+        ProductByWeek.append_last_week!
+        assert_equal(1, ProductByWeek.count)
+
+        product = ProductByWeek.find_by(product_id: 'foo')
+        assert_equal('foo', product.product_id)
+        assert_equal(1, product.orders)
+
+        ProductForLastWeek.delete_all
+        ProductForLastWeek.create!(product_id: 'foo', orders: 1)
+        ProductForLastWeek.create!(product_id: 'bar', orders: 2)
+        ProductByWeek.append_last_week!
+        assert_equal(3, ProductByWeek.count)
+
+        foo = ProductByWeek.find_by(product_id: 'foo')
+        assert_equal('foo', foo.product_id)
+        assert_equal(1, foo.orders)
+
+        bar = ProductByWeek.find_by(product_id: 'bar')
+        assert_equal('bar', bar.product_id)
+        assert_equal(2, bar.orders)
       end
 
       def test_revenue_change_median
@@ -109,13 +107,11 @@ module Workarea
 
         travel_to Time.zone.local(2018, 12, 5)
 
-        Workarea.with_config do |config|
-          config.score_decay = 0.5
+        Workarea.config.score_decay = 0.5
 
-          assert_equal(3, this_week.score(:orders))
-          assert_equal(1, last_week.score(:orders))
-          assert_equal(0.25, two_weeks_ago.score(:orders))
-        end
+        assert_equal(3, this_week.score(:orders))
+        assert_equal(1, last_week.score(:orders))
+        assert_equal(0.25, two_weeks_ago.score(:orders))
       end
 
       def test_weeks_ago

data/test/models/workarea/metrics/search_by_day_test.rb

@@ -12,6 +12,9 @@ module Workarea
         SearchByDay.save_search('a', 3)
         assert_equal(0, SearchByDay.count)
 
+        SearchByDay.save_search('a', '')
+        assert_equal(0, SearchByDay.count)
+
         2.times do
           SearchByDay.save_search('Foo  Bar', '3')
           assert_equal(1, SearchByDay.count)

data/test/models/workarea/metrics/search_by_week_test.rb

@@ -48,31 +48,29 @@ module Workarea
       end
 
       def test_append_last_week!
-        Workarea.with_config do |config|
-          config.insights_aggregation_per_page = 1
-
-          SearchForLastWeek.create!(query_id: 'foo', orders: 1)
-          SearchByWeek.append_last_week!
-          assert_equal(1, SearchByWeek.count)
-
-          search = SearchByWeek.find_by(query_id: 'foo')
-          assert_equal('foo', search.query_id)
-          assert_equal(1, search.orders)
-
-          SearchForLastWeek.delete_all
-          SearchForLastWeek.create!(query_id: 'foo', orders: 1)
-          SearchForLastWeek.create!(query_id: 'bar', orders: 2)
-          SearchByWeek.append_last_week!
-          assert_equal(3, SearchByWeek.count)
-
-          foo = SearchByWeek.find_by(query_id: 'foo')
-          assert_equal('foo', foo.query_id)
-          assert_equal(1, foo.orders)
-
-          bar = SearchByWeek.find_by(query_id: 'bar')
-          assert_equal('bar', bar.query_id)
-          assert_equal(2, bar.orders)
-        end
+        Workarea.config.insights_aggregation_per_page = 1
+
+        SearchForLastWeek.create!(query_id: 'foo', orders: 1)
+        SearchByWeek.append_last_week!
+        assert_equal(1, SearchByWeek.count)
+
+        search = SearchByWeek.find_by(query_id: 'foo')
+        assert_equal('foo', search.query_id)
+        assert_equal(1, search.orders)
+
+        SearchForLastWeek.delete_all
+        SearchForLastWeek.create!(query_id: 'foo', orders: 1)
+        SearchForLastWeek.create!(query_id: 'bar', orders: 2)
+        SearchByWeek.append_last_week!
+        assert_equal(3, SearchByWeek.count)
+
+        foo = SearchByWeek.find_by(query_id: 'foo')
+        assert_equal('foo', foo.query_id)
+        assert_equal(1, foo.orders)
+
+        bar = SearchByWeek.find_by(query_id: 'bar')
+        assert_equal('bar', bar.query_id)
+        assert_equal(2, bar.orders)
       end
 
       def test_revenue_change_median
@@ -109,13 +107,11 @@ module Workarea
 
         travel_to Time.zone.local(2018, 12, 5)
 
-        Workarea.with_config do |config|
-          config.score_decay = 0.5
+        Workarea.config.score_decay = 0.5
 
-          assert_equal(3, this_week.score(:orders))
-          assert_equal(1, last_week.score(:orders))
-          assert_equal(0.25, two_weeks_ago.score(:orders))
-        end
+        assert_equal(3, this_week.score(:orders))
+        assert_equal(1, last_week.score(:orders))
+        assert_equal(0.25, two_weeks_ago.score(:orders))
       end
 
       def test_weeks_ago

data/test/models/workarea/metrics/update_user_aggregations_test.rb

@@ -0,0 +1,62 @@
+require 'test_helper'
+
+module Workarea
+  module Metrics
+    class UpdateUserAggregationsTest < TestCase
+      def test_update_calculated_fields!
+        freeze_time
+        Metrics::User.create!(id: 'bcrouse0@workarea.com')
+        User.save_order(email: 'bcrouse1@workarea.com', revenue: 1.to_m, at: 1.day.ago)
+        User.save_order(email: 'bcrouse2@workarea.com', revenue: 10.to_m, at: 2.days.ago)
+        User.save_order(email: 'bcrouse2@workarea.com', revenue: 20.to_m, at: 3.days.ago)
+        UpdateUserAggregations.update_calculated_fields!
+
+        one = User.find('bcrouse1@workarea.com')
+        assert_equal(1, one.average_order_value)
+        assert_in_delta(1.46e-08, one.frequency)
+
+        two = User.find('bcrouse2@workarea.com')
+        assert_equal(15, two.average_order_value)
+        assert_in_delta(1.29e-08, two.frequency)
+      end
+
+      def test_update_percentiles!
+        Metrics::User.create!(id: 'bcrouse0@workarea.com')
+        User.save_order(email: 'bcrouse1@workarea.com', revenue: 100.to_m, at: 30.days.ago)
+        User.save_order(email: 'bcrouse2@workarea.com', revenue: 1.to_m, at: 2.days.ago)
+        User.save_order(email: 'bcrouse2@workarea.com', revenue: 2.to_m, at: 3.days.ago)
+        UpdateUserAggregations.update_calculated_fields!
+        UpdateUserAggregations.update_percentiles!
+
+        one = User.find('bcrouse1@workarea.com')
+        assert_equal(50, one.orders_percentile)
+        assert_equal(50, one.frequency_percentile)
+        assert_equal(100, one.revenue_percentile)
+        assert_equal(100, one.average_order_value_percentile)
+
+        two = User.find('bcrouse2@workarea.com')
+        assert_equal(100, two.orders_percentile)
+        assert_equal(100, two.frequency_percentile)
+        assert_equal(50, two.revenue_percentile)
+        assert_equal(50, two.average_order_value_percentile)
+      end
+
+      def test_zero_orders
+        metrics = Metrics::User.create!(id: 'test@workarea.com')
+
+        assert_nothing_raised do
+          UpdateUserAggregations.update_calculated_fields!
+          UpdateUserAggregations.update_percentiles!
+        end
+
+        metrics.reload
+        assert_nil(metrics.average_order_value)
+        assert_nil(metrics.frequency)
+        assert_nil(metrics.orders_percentile)
+        assert_nil(metrics.frequency_percentile)
+        assert_nil(metrics.revenue_percentile)
+        assert_nil(metrics.average_order_value_percentile)
+      end
+    end
+  end
+end

data/test/models/workarea/metrics/user_test.rb

@@ -39,91 +39,174 @@ module Workarea
         assert_equal(-2, user.discounts)
       end
 
-      def test_update_calculated_fields!
-        freeze_time
-        User.save_order(email: 'bcrouse1@workarea.com', revenue: 1.to_m, at: 1.day.ago)
-        User.save_order(email: 'bcrouse2@workarea.com', revenue: 10.to_m, at: 2.days.ago)
-        User.save_order(email: 'bcrouse2@workarea.com', revenue: 20.to_m, at: 3.days.ago)
-        User.update_calculated_fields!
-
-        one = User.find('bcrouse1@workarea.com')
-        assert_equal(1, one.average_order_value)
-        assert_in_delta(1.46e-08, one.frequency)
-
-        two = User.find('bcrouse2@workarea.com')
-        assert_equal(15, two.average_order_value)
-        assert_in_delta(1.29e-08, two.frequency)
+      def test_save_cancellation
+        first = Time.zone.local(2018, 11, 14)
+
+        User.save_order(
+          email: 'bcrouse@workarea.com',
+          revenue: 50.to_m,
+          at: first
+        )
+
+        User.save_cancellation(
+          email: 'bcrouse@workarea.com',
+          refund: -5.to_m,
+          at: first
+        )
+
+        assert_equal(1, User.count)
+
+        user = User.first
+        assert_equal(1, user.cancellations)
+        assert_equal(-5, user.refund)
+        assert_equal(45, user.revenue)
+
+        last = Time.zone.local(2018, 11, 15)
+
+        User.save_cancellation(
+          email: 'bcrouse@workarea.com',
+          refund: -3.to_m,
+          at: last
+        )
+
+        assert_equal(1, User.count)
+
+        user.reload
+        assert_equal(2, user.cancellations)
+        assert_equal(-8, user.refund)
+        assert_equal(42, user.revenue)
+      end
+
+      def test_save_affinity
+        2.times do
+          User.save_affinity(
+            id: 'bcrouse@workarea.com',
+            action: 'viewed',
+            product_ids: %w(foo bar),
+            category_ids: 'baz',
+            search_ids: %(qoo)
+          )
+        end
+
+        assert_equal(1, User.count)
+        user = User.first
+        assert_equal('bcrouse@workarea.com', user.id)
+        assert_equal(%w(foo bar foo bar), user.viewed.product_ids)
+        assert_equal(%w(baz baz), user.viewed.category_ids)
+        assert_equal(%w(qoo qoo), user.viewed.search_ids)
+
+        assert_changes -> { user.reload.updated_at } do
+          User.save_affinity(
+            id: 'bcrouse@workarea.com',
+            action: 'purchased',
+            product_ids: %w(foo bar)
+          )
+        end
+
+        assert_equal(1, User.count)
+        user.reload
+        assert_equal('bcrouse@workarea.com', user.id)
+        assert_equal(%w(foo bar foo bar), user.viewed.product_ids)
+        assert_equal(%w(baz baz), user.viewed.category_ids)
+        assert_equal(%w(qoo qoo), user.viewed.search_ids)
+        assert_equal(%w(foo bar), user.purchased.product_ids)
       end
 
-      def test_update_percentiles!
-        User.save_order(email: 'bcrouse1@workarea.com', revenue: 100.to_m, at: 30.days.ago)
-        User.save_order(email: 'bcrouse2@workarea.com', revenue: 1.to_m, at: 2.days.ago)
-        User.save_order(email: 'bcrouse2@workarea.com', revenue: 2.to_m, at: 3.days.ago)
-        User.update_calculated_fields!
-        User.update_percentiles!
-
-        one = User.find('bcrouse1@workarea.com')
-        assert_equal(50, one.orders_percentile)
-        assert_equal(50, one.frequency_percentile)
-        assert_equal(100, one.revenue_percentile)
-        assert_equal(100, one.average_order_value_percentile)
-
-        two = User.find('bcrouse2@workarea.com')
-        assert_equal(100, two.orders_percentile)
-        assert_equal(100, two.frequency_percentile)
-        assert_equal(50, two.revenue_percentile)
-        assert_equal(50, two.average_order_value_percentile)
+      def test_save_affinity_limits_storage
+        Workarea.config.max_affinity_items = 3
+
+        4.times do
+          User.save_affinity(
+            id: 'bcrouse@workarea.com',
+            action: 'viewed',
+            product_ids: %w(foo bar),
+            category_ids: 'baz',
+            search_ids: %(qoo)
+          )
+        end
+
+        user = User.first
+        assert_equal(%w(foo bar foo), user.viewed.product_ids)
+        assert_equal(%w(baz baz baz), user.viewed.category_ids)
+        assert_equal(%w(qoo qoo qoo), user.viewed.search_ids)
       end
 
       def test_merging_metrics
         freeze_time
 
-        first = User.create!(
+        metrics = User.create!(
           first_order_at: 2.weeks.ago,
           last_order_at: 1.day.ago,
           orders: 2,
           revenue: 100,
           discounts: -10,
           average_order_value: 50,
+          cancellations: 1,
+          refund: -20,
+          viewed: { product_ids: ['foo'], category_ids: ['bar'], search_ids: ['baz'] },
+          purchased: { product_ids: ['qoo'], category_ids: ['quo'], search_ids: ['qux'] }
         )
 
-        first.merge!(User.new)
-        first.reload
-        assert_equal(1, Metrics::User.count)
-        assert_equal(2.weeks.ago, first.first_order_at)
-        assert_equal(1.day.ago, first.last_order_at)
-        assert_equal(2, first.orders)
-        assert_equal(100, first.revenue)
-        assert_equal(-10, first.discounts)
-        assert_equal(50, first.average_order_value)
-
-        second = User.create!(id: 'foo').tap { |u| u.merge!(first) }
-        second.reload
-        assert_equal(1, Metrics::User.count)
-        assert_equal(2.weeks.ago, second.first_order_at)
-        assert_equal(1.day.ago, second.last_order_at)
-        assert_equal(2, second.orders)
-        assert_equal(100, second.revenue)
-        assert_equal(-10, second.discounts)
-        assert_equal(50, second.average_order_value)
-
-        third = User.create!(
+        metrics.merge!(User.new)
+        metrics.reload
+        assert_equal(2.weeks.ago, metrics.first_order_at)
+        assert_equal(1.day.ago, metrics.last_order_at)
+        assert_equal(2, metrics.orders)
+        assert_equal(100, metrics.revenue)
+        assert_equal(-10, metrics.discounts)
+        assert_equal(50, metrics.average_order_value)
+        assert_equal(1, metrics.cancellations)
+        assert_equal(-20, metrics.refund)
+        assert_equal(['foo'], metrics.viewed.product_ids)
+        assert_equal(['bar'], metrics.viewed.category_ids)
+        assert_equal(['baz'], metrics.viewed.search_ids)
+        assert_equal(['qoo'], metrics.purchased.product_ids)
+        assert_equal(['quo'], metrics.purchased.category_ids)
+        assert_equal(['qux'], metrics.purchased.search_ids)
+
+        blank = User.create!(id: 'foo').tap { |u| u.merge!(metrics) }
+        blank.reload
+        assert_equal(2.weeks.ago, blank.first_order_at)
+        assert_equal(1.day.ago, blank.last_order_at)
+        assert_equal(2, blank.orders)
+        assert_equal(100, blank.revenue)
+        assert_equal(-10, blank.discounts)
+        assert_equal(50, blank.average_order_value)
+        assert_equal(1, blank.cancellations)
+        assert_equal(-20, blank.refund)
+        assert_equal(['foo'], blank.viewed.product_ids)
+        assert_equal(['bar'], blank.viewed.category_ids)
+        assert_equal(['baz'], blank.viewed.search_ids)
+        assert_equal(['qoo'], blank.purchased.product_ids)
+        assert_equal(['quo'], blank.purchased.category_ids)
+        assert_equal(['qux'], blank.purchased.search_ids)
+
+        existing = User.create!(
           first_order_at: 3.weeks.ago,
           last_order_at: 3.weeks.ago,
           orders: 2,
           revenue: 120,
           average_order_value: 60,
+          viewed: { product_ids: ['one'], category_ids: ['two'], search_ids: ['three'] },
+          purchased: { product_ids: ['four'], category_ids: ['five'], search_ids: ['six'] }
         )
 
-        third.merge!(second)
-        third.reload
-        assert_equal(1, Metrics::User.count)
-        assert_equal(3.weeks.ago, third.first_order_at)
-        assert_equal(1.day.ago, third.last_order_at)
-        assert_equal(4, third.orders)
-        assert_equal(220, third.revenue)
-        assert_equal(-10, third.discounts)
-        assert_equal(55, third.average_order_value)
+        existing.merge!(metrics)
+        existing.reload
+        assert_equal(3.weeks.ago, existing.first_order_at)
+        assert_equal(1.day.ago, existing.last_order_at)
+        assert_equal(4, existing.orders)
+        assert_equal(220, existing.revenue)
+        assert_equal(-10, existing.discounts)
+        assert_equal(55, existing.average_order_value)
+        assert_equal(1, existing.cancellations)
+        assert_equal(-20, existing.refund)
+        assert_equal(%w(one foo), existing.viewed.product_ids)
+        assert_equal(%w(two bar), existing.viewed.category_ids)
+        assert_equal(%w(three baz), existing.viewed.search_ids)
+        assert_equal(%w(four qoo), existing.purchased.product_ids)
+        assert_equal(%w(five quo), existing.purchased.category_ids)
+        assert_equal(%w(six qux), existing.purchased.search_ids)
       end
     end
   end