test_track_rails_client 0.9.11 → 0.9.12

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    OTg3ZWViNmFlNjgyYmY5MzcxNTkwZmIxZTAyM2U1ZWNhMTZmNzE2MQ==
+    OTI3NTIwMjQ2OTcwMDg5ZTA0NDkxNDUzYjBiYzMyMTNkODg1YzZjNg==
   data.tar.gz: !binary |-
-    YTczNDM5NzAzMmM3OWI5NjZlNmE2ZTU4NDFmMDc3NDgwOWFmNzgwNw==
+    NzQ3Yzc0YmE2MTRkYTg5NTU1MTM0MGQxNjk4ZjdjOTUwYzYxODE0Nw==
 SHA512:
   metadata.gz: !binary |-
-    MGU1ZTNjNDE3OGRhM2NjYTRjNzRhMWNhZmYwMGIzMzUxNjRlZmRkN2EyMzAz
-    ZDZhMmNjZWM3YmZkN2VlNDFhMGZlZmQxYmNjNjlkNThjOGNhMzg1ZTMyYWQy
-    ZjY4NTQwMzZiMjUzZWMwNzU5M2E4YWM5MGJmOWExMGY0ZDBhNTQ=
+    MDdkNmNmZWM5NWE2MjQ5NGNhNGNjZjc4Y2JhZDUzMTYzYjZmMTIxYWM5OTU2
+    Y2EwODI2YThjMGIwYWY4ZmM5MDA3YzE3OWJjMjczNDI5NDc4MTgzOWMxMWIz
+    NDg1NTRjNTA5NGJiNjJiN2E2NTYyNDc0YzFmMDMyNGY4Y2ZmMjc=
   data.tar.gz: !binary |-
-    YzRkNzBkM2FiZjQzMzNkYjA3Mjk3MWM2ZTc1MDg0MGYwODRmNzUwYmExZGRm
-    ODIxNmZiNjMyY2IzYjFhOGY0ZmU3MWRkY2M3NWIyYmVmMGZmYTQzNWIyYTg2
-    YzJmMmRjODc3NzcxZWZkMDhhZWZhMjdjMTQ0ZWNhMTYzYzEwODA=
+    M2I3NWUwMWYxNTY1ZDgyMzE0MzVlZGQ2ZTU0ZjI3OWVlOWNmYjg5ODdhNzFj
+    NjhmNWQ5NjgyYzRjNjVkNmVmYmJiMTA0ZWNmNzhjMDJjYTAwOWYxNWFlMDQ3
+    MzUyYTM5MDU0NzA0MTBiMWEwODA5NDQ2MmRkZTc2Y2Y2NDU3NDk=

data/app/models/test_track/remote/split_registry.rb

@@ -30,7 +30,8 @@ class TestTrack::Remote::SplitRegistry
         instance.attributes
       end.freeze
     end
-  rescue *TestTrack::SERVER_ERRORS
+  rescue *TestTrack::SERVER_ERRORS => e
+    Rails.logger.error "TestTrack failed to load split registry. #{e}"
     nil # if we can't get a split registry
   end
 end

data/app/models/test_track/session.rb

@@ -129,7 +129,7 @@ class TestTrack::Session
   end
 
   def sync_assignments?
-    !visitor.offline? && visitor.unsynced_assignments.present?
+    visitor.loaded? && visitor.unsynced_assignments.present?
   end
 
   def mixpanel_distinct_id

data/app/models/test_track/unsynced_assignments_notifier.rb

@@ -17,7 +17,8 @@ class TestTrack::UnsyncedAssignmentsNotifier
       build_notify_assignment_job(assignment).tap do |job|
         begin
           job.perform
-        rescue *TestTrack::SERVER_ERRORS
+        rescue *TestTrack::SERVER_ERRORS => e
+          Rails.logger.error "TestTrack failed to notify unsynced assignments, retrying. #{e}"
           Delayed::Job.enqueue(build_notify_assignment_job(assignment))
         end
       end

data/app/models/test_track/visitor.rb

@@ -70,7 +70,9 @@ class TestTrack::Visitor
     begin
       identifier = TestTrack::Remote::Identifier.create!(identifier_opts)
       merge!(identifier.visitor)
-    rescue *TestTrack::SERVER_ERRORS
+    rescue *TestTrack::SERVER_ERRORS => e
+      Rails.logger.error "TestTrack failed to link identifier, retrying. #{e}"
+
       # If at first you don't succeed, async it - we may not display 100% consistent UX this time,
       # but subsequent requests will be better off
       TestTrack::Remote::Identifier.delay.create!(identifier_opts)
@@ -92,6 +94,10 @@ class TestTrack::Visitor
     @tt_offline
   end
 
+  def loaded?
+    !offline? && @remote_visitor.present?
+  end
+
   private
 
   def assignments
@@ -100,7 +106,8 @@ class TestTrack::Visitor
 
   def remote_visitor
     @remote_visitor ||= TestTrack::Remote::Visitor.find(id) unless tt_offline?
-  rescue *TestTrack::SERVER_ERRORS
+  rescue *TestTrack::SERVER_ERRORS => e
+    Rails.logger.error "TestTrack failed to load remote visitor. #{e}"
     @tt_offline = true
     nil
   end

data/lib/test_track.rb

@@ -14,7 +14,7 @@ require 'request_store'
 module TestTrack
   module_function
 
-  SERVER_ERRORS = [Faraday::TimeoutError, Her::Errors::RemoteServerError].freeze
+  SERVER_ERRORS = [Faraday::ConnectionFailed, Faraday::TimeoutError, Her::Errors::RemoteServerError].freeze
 
   mattr_accessor :enabled_override
 

data/lib/test_track_rails_client/version.rb

@@ -1,3 +1,3 @@
 module TestTrackRailsClient
-  VERSION = "0.9.11" # rubocop:disable Style/MutableConstant
+  VERSION = "0.9.12" # rubocop:disable Style/MutableConstant
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test_track_rails_client
 version: !ruby/object:Gem::Version
-  version: 0.9.11
+  version: 0.9.12
 platform: ruby
 authors:
 - Ryan O'Neill
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2016-11-08 00:00:00.000000000 Z
+date: 2016-11-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails