test_track_rails_client 7.0.0 → 7.1.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 1ef99958de5b5a33d340983ad667aa5232a0209b5e5e2aa0ecb249fe8ba92e35
-  data.tar.gz: edb76d28b667e451a5b9a284cb90cd3c3eb2a1a55f71471763842dcaf3df052b
+  metadata.gz: b1c0d5e3f14e82a63f34d8ab019c2f433014cdcf590ba339e70cf54996178a1a
+  data.tar.gz: e45737b7d5dba5cffec0219292091252fbac298ac48189cbf827cde52c408d5c
 SHA512:
-  metadata.gz: 7fce8893812f7ce818b849e471d77cc4f8ce596189ef8a7f1b9aedfa17761b951f0cfcfedbee317e772562c640a5f303f37e52bff03bfd36220a3c67a49b6c0c
-  data.tar.gz: 668e3db85f8939b1322ec4fb821381278ac0331d41ccdc75cb460b6787f3b8e3e6ed5ae101b42f71bba003f76a13297879faf5e870895ac6876fbbb4769e244c
+  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/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.1".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.1
 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-03-13 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.6
 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'