test_track_rails_client 7.0.0 → 7.1.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ef99958de5b5a33d340983ad667aa5232a0209b5e5e2aa0ecb249fe8ba92e35
-  data.tar.gz: edb76d28b667e451a5b9a284cb90cd3c3eb2a1a55f71471763842dcaf3df052b
+  metadata.gz: d4d2661b24f3074456010fedcd9db386a21203d01012a8a7d994305b85e81830
+  data.tar.gz: f5b831de28f2b7543be6e53649d212670edc2561970e05cfd6e7758e08db348a
 SHA512:
-  metadata.gz: 7fce8893812f7ce818b849e471d77cc4f8ce596189ef8a7f1b9aedfa17761b951f0cfcfedbee317e772562c640a5f303f37e52bff03bfd36220a3c67a49b6c0c
-  data.tar.gz: 668e3db85f8939b1322ec4fb821381278ac0331d41ccdc75cb460b6787f3b8e3e6ed5ae101b42f71bba003f76a13297879faf5e870895ac6876fbbb4769e244c
+  metadata.gz: 7726be38a0eb5b6d81a555b9dd489d1e2d0a33faf30481f922d0faf1a7a79bd150f5e66ce541bc5eb9dcb51c17c0fa2da003a67b316ea3ec124955cf1b98240a
+  data.tar.gz: ffceabaffbeb7966c1de27b47f3e820d1a4322fcd871371ba1c90e4fa3d6ea5d8679caae0082ee1a9580341265b96f897125d6c9c9b89f7880011926ee076e88

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/application_identity.rb

@@ -12,7 +12,7 @@ class TestTrack::ApplicationIdentity
   end
 
   def identity
-    Identity.new(app_name)
+    @identity ||= Identity.new(app_name)
   end
 
   class Identity

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/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 = "7.0.0".freeze
+  VERSION = "7.1.2".freeze
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: test_track_rails_client
 version: !ruby/object:Gem::Version
-  version: 7.0.0
+  version: 7.1.2
 platform: ruby
 authors:
 - Ryan O'Neill
@@ -13,7 +13,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2023-03-06 00:00:00.000000000 Z
+date: 2024-07-24 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activejob
@@ -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
@@ -447,14 +446,14 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: 2.7.0
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.2.33
+rubygems_version: 3.5.15
 signing_key:
 specification_version: 4
 summary: Rails client for TestTrack

data/config/initializers/set_build_timestamp.rb

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