test_track_rails_client 0.9.19 → 0.9.20

checksums.yaml

@@ -1,15 +1,15 @@
 ---
 !binary "U0hBMQ==":
   metadata.gz: !binary |-
-    ZjBiNDc0ODg2MTNiMjFjYjdjMzg5Y2EyOGFlNjk3MTNjN2I4ZWY2OA==
+    YjMxNjA4MWVlMjgwYjM3Yjg2MjlmMGRhNDU5MTRjZmYzZDEyMGZhYw==
   data.tar.gz: !binary |-
-    ZmVkMzc4NmNjNDgwZDM2OTE1YTZjOGJhYzM5ZmM4MTVmOTI1OWMwYg==
+    YzM5NTQ3YTEwODQ4MDJiMjczYWI1ZDI5MmYwM2ZkNzVjMWYzOWFhNg==
 SHA512:
   metadata.gz: !binary |-
-    YzNhNjg4ZmJjNTE0NWY1MjgxMjY2YzM0ZmE1ZGI4NDYzNmIyNzc5MTYxMzhk
-    OGZkYmI5YTVjMDc1N2VkNjllNDZiNzE0MjNlMDgyNGU1ZDExZDQ3MGQyMjE4
-    NzM0ZmM5MDk2MGQ3OWJlYzRhNjRhOTVmZDY3NGE3NmEyMWMwMGY=
+    ZTM5NzZhZjA2YzZjYjI1MzdiYWMxYTEzM2RkZTBlNzg0NjYxNTcyYjU1M2Ey
+    N2JmNzNlYTQxYmRlMTA2NzAzMTJkNWI0NmZlMTQ1YTlhNWExZDdlYzJjN2Uz
+    OTEzMjljMDQ0NGI2NDg3NTNkYTc3NWU1MDVmYTBlMDcwM2ViYjc=
   data.tar.gz: !binary |-
-    N2E1MGUxYzc4YzRjNjZiMjU1OWUyZWVlYWNiNzEzOTZkYzBmNzJhOWNiZTc4
-    OTMxY2U2YTU2MmJmZmJjZTAxNjg4YWI2ZjEwYTUzZDE1NzRlYjc2N2Q1NzBh
-    ZjQ4NmY5OWU1ZDBmYjM4NjZmNjFjNzc1OTI3ZjhhMmUyYTg3ODM=
+    ZWU4ZDZjNjg2ODUyNjA5ODc1OGIzODA5NzA3NzAzYzJjNjBhMjQxMTA2ZDcw
+    Mjg0ODI2ODE2MDBlOGQ0ZjA4ZjEwZDI2ZjJlNjU5Yzc2N2QwM2VhZjFkOWFi
+    OGU3N2UwZGE3ZDMwYTU1YzJmNjQzMmFiMDY4YmE2Mjg4MjYzZmU=

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

@@ -6,11 +6,7 @@ class TestTrack::Fake::SplitRegistry
   end
 
   def to_h
-    if test_track_schema_yml.present?
-      test_track_schema_yml[:splits]
-    else
-      {}
-    end
+    @to_h ||= splits_with_deterministic_weights
   end
 
   def splits
@@ -21,6 +17,14 @@ class TestTrack::Fake::SplitRegistry
 
   private
 
+  def split_hash
+    if test_track_schema_yml.present?
+      test_track_schema_yml[:splits]
+    else
+      {}
+    end
+  end
+
   def test_track_schema_yml
     unless instance_variable_defined?(:@test_track_schema_yml)
       @test_track_schema_yml = _test_track_schema_yml
@@ -29,8 +33,25 @@ class TestTrack::Fake::SplitRegistry
   end
 
   def _test_track_schema_yml
-    YAML.load_file("#{Rails.root}/db/test_track_schema.yml").with_indifferent_access
+    YAML.load_file(test_track_schema_yml_path).with_indifferent_access
   rescue
     nil
   end
+
+  def test_track_schema_yml_path
+    ENV["TEST_TRACK_SCHEMA_FILE_PATH"] || "#{Rails.root}/db/test_track_schema.yml"
+  end
+
+  def splits_with_deterministic_weights
+    split_hash.each_with_object({}) do |(split_name, weighting_registry), split_registry|
+      default_variant = weighting_registry.keys.sort.first
+
+      adjusted_weights = { default_variant => 100 }
+      weighting_registry.except(default_variant).keys.each do |variant|
+        adjusted_weights[variant] = 0
+      end
+
+      split_registry[split_name] = adjusted_weights
+    end
+  end
 end

data/app/models/test_track/fake/visitor.rb

@@ -1,5 +1,3 @@
-require 'digest'
-
 class TestTrack::Fake::Visitor
   attr_reader :id
 
@@ -21,21 +19,16 @@ class TestTrack::Fake::Visitor
     @assignments ||= _assignments
   end
 
+  def split_registry
+    TestTrack::Fake::SplitRegistry.instance.to_h
+  end
+
   private
 
   def _assignments
-    TestTrack::Fake::SplitRegistry.instance.splits.map do |split|
-      index = hash_fixnum(split.name) % split.registry.keys.size
-      variant = split.registry.keys[index]
-      Assignment.new(split.name, variant, false, "the_context")
+    split_registry.keys.map do |split_name|
+      variant = TestTrack::VariantCalculator.new(visitor: self, split_name: split_name).variant
+      Assignment.new(split_name, variant, false, "the_context")
     end
   end
-
-  def hash_fixnum(split_name)
-    split_visitor_hash(split_name).slice(0, 8).to_i(16)
-  end
-
-  def split_visitor_hash(split_name)
-    Digest::MD5.new.update(split_name.to_s + id.to_s).hexdigest
-  end
 end

data/lib/test_track_rails_client/assignment_helper.rb

@@ -1,13 +1,12 @@
 module TestTrackRailsClient::AssignmentHelper
-  def stub_test_track_assignments(assignment_registry) # rubocop:disable Metrics/AbcSize, Metrics/MethodLength
+  def stub_test_track_assignments(assignment_registry) # rubocop:disable Metrics/AbcSize
     raise "Cannot stub test track assignments when TestTrack is enabled" if TestTrack.enabled?
 
-    split_registry = {}
+    split_registry = TestTrack::Fake::SplitRegistry.instance.to_h.dup
     assignments = []
 
     assignment_registry.each do |split_name, variant|
-      assignment_registry[split_name] = variant.to_s
-      split_registry[split_name] = { variant => 100 }
+      split_registry[split_name] = { variant => 100 } unless split_registry[split_name]
       assignments << { split_name: split_name.to_s, variant: variant.to_s, unsynced: false }
     end
 

data/lib/test_track_rails_client/version.rb

@@ -1,3 +1,3 @@
 module TestTrackRailsClient
-  VERSION = "0.9.19" # rubocop:disable Style/MutableConstant
+  VERSION = "0.9.20" # 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.19
+  version: 0.9.20
 platform: ruby
 authors:
 - Ryan O'Neill
@@ -13,7 +13,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2017-02-22 00:00:00.000000000 Z
+date: 2017-03-04 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails