test_track_rails_client 6.0.0 → 7.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 435ccd01e25d78c6a1198b18fcc898a34b76ba6949173672f65f801408dbe789
-  data.tar.gz: 87de3abad19abb1a7691959c09597ac2c4133b310c2d70397bbda07526b51649
+  metadata.gz: b1c0d5e3f14e82a63f34d8ab019c2f433014cdcf590ba339e70cf54996178a1a
+  data.tar.gz: e45737b7d5dba5cffec0219292091252fbac298ac48189cbf827cde52c408d5c
 SHA512:
-  metadata.gz: 727ec20876be5bfc7447ea1bf04db3cc1ad88460bf9fac72a7656f41b2481689b3c936713e2196bd5ec807aabbd9084c313470abb94687eab94f43bbe862b6fe
-  data.tar.gz: 1fbd5e5b9afea8b03b2ae1664098bbfcc528af98d3c785cfbc4a66c80a4535f6debe35e1a3d1f7b2815f3943fc7a63f1c574c9749fedd4fac6c03e7f37720548
+  metadata.gz: 2b892995788d2baccf116d1cf233fd763159b9047a068196feb4fa89dbc70c0ffc6ab1670d1d57da9bdd5359f9d8f455619a456e0a4b70f93b84df966a1122af
+  data.tar.gz: 15e46fb3134a62060b3bf931f0b99912151bc75c3be74493c8b12956951d42a67014101086f773f002abfcce661a058485969c24f0f613413fd2ec57e6fd6160

data/Rakefile

@@ -28,12 +28,12 @@ RuboCop::RakeTask.new
 
 desc "Pull the latest versions of all dependencies into the gem for distribution"
 task :vendor_deps do
-  TEST_TRACK_JS_CLIENT_VERSION = '2.0.0'.freeze
-  TEST_TRACK_CLI_VERSION = 'v1.2.0'.freeze
+  test_track_js_client_version = '2.0.0'.freeze
+  test_track_cli_version = 'v1.2.0'.freeze
 
   # Bundle JS client
   sh 'npm init -y'
-  sh "npm install --no-save test_track_js_client@#{TEST_TRACK_JS_CLIENT_VERSION}"
+  sh "npm install --no-save test_track_js_client@#{test_track_js_client_version}"
   sh 'cp', 'node_modules/test_track_js_client/dist/testTrack.bundle.js', 'app/assets/javascripts/testTrack.bundle.min.js'
   sh 'rm package.json'
 
@@ -44,7 +44,7 @@ task :vendor_deps do
       rm_r Dir.glob('*')
 
       %w(darwin linux).each do |arch|
-        `curl -L https://github.com/Betterment/testtrack-cli/releases/download/#{TEST_TRACK_CLI_VERSION}/testtrack.#{arch} \
+        `curl -L https://github.com/Betterment/testtrack-cli/releases/download/#{test_track_cli_version}/testtrack.#{arch} \
           > testtrack.#{arch}`
         chmod 'u=wrx,go=rx', "testtrack.#{arch}"
       end

data/app/controllers/concerns/test_track/controller.rb

@@ -27,10 +27,8 @@ module TestTrack::Controller
     test_track_session.visitor_dsl
   end
 
-  def manage_test_track_session
+  def manage_test_track_session(&block)
     RequestStore[:test_track_web_session] = test_track_session
-    test_track_session.manage do
-      yield
-    end
+    test_track_session.manage(&block)
   end
 end

data/app/models/test_track/ab_configuration.rb

@@ -1,7 +1,7 @@
 class TestTrack::AbConfiguration
   include TestTrack::RequiredOptions
 
-  def initialize(opts) # rubocop:disable Metrics/AbcSize
+  def initialize(opts)
     @split_name = require_option!(opts, :split_name).to_s
     true_variant = require_option!(opts, :true_variant, allow_nil: true)
     @split_registry = require_option!(opts, :split_registry)

data/app/models/test_track/identity_session_locator.rb

@@ -5,7 +5,7 @@ class TestTrack::IdentitySessionLocator
     @identity = identity
   end
 
-  def with_visitor # rubocop:disable Metrics/AbcSize
+  def with_visitor(&block) # rubocop:disable Metrics/AbcSize
     raise ArgumentError, "must provide block to `with_visitor`" unless block_given?
 
     if web_session.present?
@@ -13,9 +13,7 @@ class TestTrack::IdentitySessionLocator
     elsif job_session.present?
       yield job_session.visitor_dsl_for(identity)
     else
-      TestTrack::OfflineSession.with_visitor_for(identity.test_track_identifier_type, identity.test_track_identifier_value) do |v|
-        yield v
-      end
+      TestTrack::OfflineSession.with_visitor_for(identity.test_track_identifier_type, identity.test_track_identifier_value, &block)
     end
   end
 

data/app/models/test_track/offline_session.rb

@@ -3,24 +3,20 @@ class TestTrack::OfflineSession
     @remote_visitor = remote_visitor
   end
 
-  def self.with_visitor_for(identifier_type, identifier_value)
+  def self.with_visitor_for(identifier_type, identifier_value, &block)
     raise ArgumentError, "must provide block to `with_visitor_for`" unless block_given?
 
     remote_visitor = TestTrack::Remote::Visitor.from_identifier(identifier_type, identifier_value)
 
-    new(remote_visitor).send :manage do |visitor_dsl|
-      yield visitor_dsl
-    end
+    new(remote_visitor).send(:manage, &block)
   end
 
-  def self.with_visitor_id(visitor_id)
+  def self.with_visitor_id(visitor_id, &block)
     raise ArgumentError, "must provide block to `with_visitor_id`" unless block_given?
 
     remote_visitor = TestTrack::Remote::Visitor.find(visitor_id)
 
-    new(remote_visitor).send :manage do |visitor_dsl|
-      yield visitor_dsl
-    end
+    new(remote_visitor).send(:manage, &block)
   end
 
   private

data/app/models/test_track/unsynced_assignments_notifier.rb

@@ -13,10 +13,10 @@ class TestTrack::UnsyncedAssignmentsNotifier
 
   def notify
     assignments.each do |assignment|
-      TestTrack::AssignmentEventJob.perform_now assignment_job_args(assignment)
+      TestTrack::AssignmentEventJob.perform_now(**assignment_job_args(assignment))
     rescue *TestTrack::SERVER_ERRORS => e
       Rails.logger.error "TestTrack failed to notify unsynced assignments, retrying. #{e}"
-      TestTrack::AssignmentEventJob.perform_later assignment_job_args(assignment)
+      TestTrack::AssignmentEventJob.perform_later(**assignment_job_args(assignment))
     end
   end
 

data/app/models/test_track/vary_dsl.rb

@@ -33,6 +33,7 @@ class TestTrack::VaryDsl
   private
 
   attr_reader :split_registry, :assignment, :context
+
   delegate :split_name, to: :assignment
 
   def split

data/app/models/test_track/web_session.rb

@@ -51,7 +51,7 @@ class TestTrack::WebSession
   attr_reader :controller
 
   def current_identity
-    raise <<~ERROR unless controller.class.test_track_identity&.is_a?(Symbol)
+    raise <<~ERROR unless controller.class.test_track_identity.is_a?(Symbol)
       Your controller (or controller base class) must set test_track_identity for
       TestTrack to work properly. e.g.:
 
@@ -106,7 +106,7 @@ class TestTrack::WebSession
   end
 
   def wildcard_domain
-    "." + (public_suffix_domain || request.host)
+    ".#{public_suffix_domain || request.host}"
   end
 
   def public_suffix_domain

data/lib/test_track.rb

@@ -17,7 +17,7 @@ module TestTrack
 
   SERVER_ERRORS = [Faraday::ConnectionFailed, Faraday::TimeoutError, Her::Errors::RemoteServerError].freeze
   BUILD_TIMESTAMP_FILE_PATH = 'testtrack/build_timestamp'.freeze
-  BUILD_TIMESTAMP_REGEX = /\A\d{4}-[01]\d-[0-3]\dT[0-2]\d:[0-5]\d:[0-5]\d(.\d+)?([+-][0-2]\d:[0-5]\d|Z)\z/.freeze
+  BUILD_TIMESTAMP_REGEX = /\A\d{4}-[01]\d-[0-3]\dT[0-2]\d:[0-5]\d:[0-5]\d(.\d+)?([+-][0-2]\d:[0-5]\d|Z)\z/
 
   mattr_accessor :enabled_override, :app_name
   mattr_accessor(:job_base_class_name) { 'ActiveJob::Base' }

data/lib/test_track_rails_client/engine.rb

@@ -9,5 +9,9 @@ module TestTrackRailsClient
     config.generators do |g|
       g.test_framework :rspec
     end
+
+    initializer after: 'active_support.initialize_time_zone' do
+      TestTrack.set_build_timestamp! unless ENV['SKIP_TESTTRACK_SET_BUILD_TIMESTAMP'] == '1'
+    end
   end
 end

data/lib/test_track_rails_client/version.rb

@@ -1,3 +1,3 @@
 module TestTrackRailsClient
-  VERSION = "6.0.0" # rubocop:disable Style/MutableConstant
+  VERSION = "7.1.1".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test_track_rails_client
 version: !ruby/object:Gem::Version
-  version: 6.0.0
+  version: 7.1.1
 platform: ruby
 authors:
 - Ryan O'Neill
@@ -10,10 +10,10 @@ authors:
 - John Mileham
 - Alan Norton
 - Sam Moore
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-01-12 00:00:00.000000000 Z
+date: 2024-03-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob
@@ -21,28 +21,28 @@ dependencies:
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: activemodel
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '5.1'
+        version: '6.0'
 - !ruby/object:Gem::Dependency
   name: faraday
   requirement: !ruby/object:Gem::Requirement
@@ -217,14 +217,14 @@ dependencies:
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.81.0
+        version: 1.48.0
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: 0.81.0
+        version: 1.48.0
 - !ruby/object:Gem::Dependency
   name: rubocop-performance
   requirement: !ruby/object:Gem::Requirement
@@ -382,7 +382,6 @@ files:
 - app/views/tt/api/v1/split_details/show.json.jbuilder
 - app/views/tt/api/v1/visitors/_show.json.jbuilder
 - app/views/tt/api/v1/visitors/show.json.jbuilder
-- config/initializers/set_build_timestamp.rb
 - config/initializers/test_track_api.rb
 - config/routes.rb
 - lib/generators/test_track/migration_generator.rb
@@ -439,7 +438,7 @@ homepage: https://github.com/Betterment
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -447,15 +446,15 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.5.0
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.3.7
-signing_key: 
+rubygems_version: 3.5.6
+signing_key:
 specification_version: 4
 summary: Rails client for TestTrack
 test_files: []

data/config/initializers/set_build_timestamp.rb

@@ -1 +0,0 @@
-TestTrack.set_build_timestamp! unless ENV['SKIP_TESTTRACK_SET_BUILD_TIMESTAMP'] == '1'