referer_tracking 4.4.0 → 4.5.0

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/CS/CS8nEse4iNQsIQp6JJMkrfiNOJ0C4AsDb2TR7hOGIH0.cache

@@ -0,0 +1 @@
+I"}app/assets/stylesheets/application.css?type=text/css&id=df21cedf5c076ad230f056a5c4a18223ecb59d20cce688c3c2cdb5505ac5242c:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/Dm/DmmfrCpXtt74Hr6NO54lxyOCDv6klnDyBqeDFR7oDU8.cache

@@ -0,0 +1,3 @@
+[o:Set:
+@hash}
+I"environment-version:ETTI"environment-paths;TTI"rails-env;TTI">processors:type=text/css&file_type=text/css&pipeline=self;TTI"6file-digest://app/assets/stylesheets/scaffold.css;TTF

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/Ka/KagYVzDXOrwIV8OqoCjrLu5kwqlqtzrwhLbuOdO_Lrs.cache

@@ -0,0 +1 @@
+"%��A�I8���߅�u7�!��H���j���\��|x

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/Z4/Z41nxEwpWQWFoc3tdahPn8d-nQygg6rN5wPU0iWRYrA.cache

@@ -0,0 +1 @@
+I"
�app/assets/stylesheets/users.css?type=text/css&pipeline=self&id=12a0a025bee7b6c0775f47da5959c9b7d81b52ca144340e110b957f600bc60f2:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/bP/bPBEsM47rrracC6h0DUpa76b38d_hdPadAGWeHXJIj8.cache

@@ -0,0 +1,3 @@
+[o:Set:
+@hash}
+I"environment-version:ETTI"environment-paths;TTI"rails-env;TTI"Zprocessors:type=application/javascript&file_type=application/javascript&pipeline=self;TTI"0file-digest://app/assets/config/manifest.js;TTF

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/cG/cGfnqX6g3P0WuiBZ7nc-f8N13CSgrV163hKk2ry0-n8.cache

@@ -0,0 +1 @@
+I"
}app/assets/config/manifest.js?type=application/javascript&id=96c2df0017491eafb55fb7e2f788a1973899ae13c0635d0d5bad071778f2da39:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/dz/dzSD3sm4znNDRdp04nrjxADibaOAuBl1B_WUyfsaNac.cache

@@ -0,0 +1 @@
+I"
�app/assets/stylesheets/application.css?type=text/css&pipeline=self&id=a65253230ad4b2a0794169089fa585ecf231dc15af74cf6c80bc0a2e2886c0af:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/i1/i1DwWMZeHC_QdByjF6bFQ3-UXJyXEvY7zJfsvk6rSl4.cache

@@ -0,0 +1 @@
+I"
�app/assets/config/manifest.js?type=application/javascript&pipeline=self&id=7ca5d7b067545f9ea3db0d2d0521c464159c5096c8546c0ed15b4129d9418b97:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/iD/iDTnVSbZnIij5Xm4jkwCyIXAPDn8RwaRe5Ja8YPKibw.cache

@@ -0,0 +1,3 @@
+[o:Set:
+@hash}
+I"environment-version:ETTI"environment-paths;TTI"rails-env;TTI">processors:type=text/css&file_type=text/css&pipeline=self;TTI"3file-digest://app/assets/stylesheets/users.css;TTF

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/l8/l8s8Rg4fjDwXddzNqmy9aLM9gcOiiuYpZZbbreeV7Xk.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash}I"environment-version:ETTI"environment-paths;TTI"rails-env;TTI"Lprocessors:type=application/javascript&file_type=application/javascript;TTI"0file-digest://app/assets/config/manifest.js;TTI"Zprocessors:type=application/javascript&file_type=application/javascript&pipeline=self;TTF

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/qp/qpYHEqUJcqJpx4Zm9PbO4LVVjAqI1wQUcy83KPrUK7M.cache

@@ -0,0 +1 @@
+"%֧�NTZq�בH�L��#p3��Jg!��)

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/r6/r6KjMmQ9n_jOrJlySdyY3ghq7-QyiM3ai5wHX6eTp-g.cache

@@ -0,0 +1 @@
+"%9�#�U�-���Jnݶ�W��Gx5>T�u�\x��:�

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/rb/rb_u_gHDnEZUoDZl9KxS9eteCoe-SaXymHkbl47PpVs.cache

@@ -0,0 +1 @@
+"%u�0b��^�#�ߜj�E�b��/��e�=

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/wX/wXZYB3qZL8CcqH7bLqFclD7G1KHZdxh-PxR2zVSSQe0.cache

@@ -0,0 +1 @@
+I"
�app/assets/stylesheets/scaffold.css?type=text/css&pipeline=self&id=bd3274ddee93013d3d6d96e79334a08b2a6a305c44a0acb4be9780665d121b24:ET

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/wn/wnvJ74QIdj8tSD10cXmY9KyXCpaQIv0FfZJWw-EudW8.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash}

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/xP/xPMfTXGxxCCgIip6XsG-paelKjnDzbpAYugLmUfykPs.cache

@@ -0,0 +1,2 @@
+[o:Set:
+@hash}I"environment-version:ETTI"environment-paths;TTI"rails-env;TTI"0processors:type=text/css&file_type=text/css;TTI"9file-digest://app/assets/stylesheets/application.css;TTI">processors:type=text/css&file_type=text/css&pipeline=self;TTI")file-digest://app/assets/stylesheets;TTI"6file-digest://app/assets/stylesheets/scaffold.css;TTI"3file-digest://app/assets/stylesheets/users.css;TTF

data/test/dummy/tmp/cache/assets/sprockets/v4.0.0/yw/ywTOTWuvOMHV0wh02S8szPOIWIaS_244Z0lUeU8Dstw.cache

@@ -0,0 +1 @@
+"%Ȥ���o�A�$w�NN�Dj�mJ�8��ӌ�~�w

data/test/integration/referer_tracking/referer_session_test.rb

@@ -133,6 +133,22 @@ class RefererSessionTest < ActionDispatch::IntegrationTest
     assert_equal 1, RefererTracking::Tracking.count, "should create one item"
   end
 
+  test "custom referer_tracking save work with too long user agent" do
+    RefererTracking.save_cookies = true
+
+    too_long_user_agent = "Mozilla/5.0 (iPhone; CPU iPhone OS 11_2_6 like Mac OS X) AppleWebKit/604.5.6 (KHTML, like Gecko) Mobile/15D100 [FBAN/MessengerForiOS; ... #{(0...400).map { (65+rand(26)).chr }.join}"
+    post '/users/create_with_custom_saving', params: {:user => {:name => 'test name'}},
+         headers: {"HTTP_USER_AGENT" => (@user_agent = too_long_user_agent)}
+    assert_equal 1, RefererTracking::Tracking.count, "should create one item"
+
+    assert too_long_user_agent.size > 400
+    rt = RefererTracking::Tracking.last
+    assert_equal too_long_user_agent.first(100), rt.user_agent.first(100)
+    assert rt.user_agent.size < 255, "should handle db limits" # testing manually because sqlite does not enforce limits same way as mysql
+  end
+
+
+
   test "custom referer_tracking save should not save if item itself is not saved" do
     RefererTracking.save_cookies = true
 

data/test/test_helper.rb

@@ -4,7 +4,7 @@ require File.expand_path("../../test/dummy/config/environment.rb", __FILE__)
 ActiveRecord::Migrator.migrations_paths = [File.expand_path("../../test/dummy/db/migrate", __FILE__)]
 ActiveRecord::Migrator.migrations_paths << File.expand_path('../../db/migrate', __FILE__)
 require "rails/test_help"
-require "mocha/setup"
+require 'mocha/minitest'
 
 # Filter out Minitest backtrace while allowing backtrace from other libraries
 # to be shown.

data/test/tracking_test.rb

@@ -22,8 +22,26 @@ class TrackingTest < ActiveSupport::TestCase
 
     assert (log = RefererTracking::Tracking.last.log)
     assert_equal 2, log.to_s.lines.count
-    assert_match /#{Date.today.to_s(:db)}.*: first_line$/, log.lines.first
-    assert_match /#{Date.today.to_s(:db)}.*: second line/, log.lines.last
+    assert_match(/#{Date.today.to_s(:db)}.*: first_line$/, log.lines.first)
+    assert_match(/#{Date.today.to_s(:db)}.*: second line/, log.lines.last)
+  end
+
+  test "add_log_line should not save if new record" do
+    rt = RefererTracking::Tracking.new
+
+    assert rt.new_record?
+    rt.add_log_line 'first_line'
+    assert rt.new_record?
+    assert rt.changed?
+
+    rt.save!
+    rt.add_log_line "second\nline"
+    assert !rt.changed?
+
+    assert (log = RefererTracking::Tracking.last.log)
+    assert_equal 2, log.to_s.lines.count
+    assert_match(/#{Date.today.to_s(:db)}.*: first_line$/, log.lines.first)
+    assert_match(/#{Date.today.to_s(:db)}.*: second line/, log.lines.last)
   end
 
   test "status_without_save" do
@@ -41,7 +59,7 @@ class TrackingTest < ActiveSupport::TestCase
     rt = RefererTracking::Tracking.last
     assert_equal 'active', rt.status
     assert_equal 1, rt.log.lines.count
-    assert_match /#{Date.today.to_s(:db)}.*: status active$/, rt.log
+    assert_match(/#{Date.today.to_s(:db)}.*: status active$/, rt.log)
 
     rt.update_status 'active'
     assert_equal 1, rt.log.lines.count, "should not alter anything if status stays the same"