gitlab-experiment 0.7.0 → 0.8.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: b6d54c567ebebd09762fb8bbd93940efb449f99d546a62776889caebedda6387
-  data.tar.gz: 87a37b577f0545885ce4c2d6db92f750781017c2235b4d89f0f5db977bec045c
+  metadata.gz: c92b7e2d77e534a23b8233f11539fa9c2767b21adab9c247e916933724dd7f5d
+  data.tar.gz: 7a1144676a16835f64eaefb9ea7bac2b7055c5cf8e329d55dba6c115be17eb50
 SHA512:
-  metadata.gz: ed428cb380868eb7e94e9a5b4753c7c09dd73656fa66da4bb90fc728b2e2a460cb8d7f24170e783a63c6c28f86d61b00cb99ae9065a3c02999e7518ecc79e2e2
-  data.tar.gz: 001edc2f533b5a5c210ada04c73a335b51e0e377be05449f0ae86ba9da078655d54e860294dd8a66ea8564270345cbbd145dc789aae1eb7f76f074db1fd79be3
+  metadata.gz: b7a645b7d2c3cf13d7031fc61dfdaef2626d0092661093a19ff5e21d346adda8cbe031b5c360de21fda3ecee828eaa7bda0a6afe839b6c5ce7ba57f9e10232eb
+  data.tar.gz: 1a8e4672a903746f5f31f2cf5dcf815bb6e2c3cb8c8fa73430c93462977310fcd6ed2f6f8676d5e2ccf856289c6befde6469e9f29715341c4bde7fd0b14fa51b

data/lib/generators/gitlab/experiment/install/templates/initializer.rb.tt

@@ -48,12 +48,10 @@ Gitlab::Experiment.configure do |config|
   # end
   #
   # Included rollout strategies:
-  #   Gitlab::Experiment::Rollout::Percent, (recommended)
-  #   Gitlab::Experiment::Rollout::RoundRobin, or
-  #   Gitlab::Experiment::Rollout::Random
-  config.default_rollout = Gitlab::Experiment::Rollout::Percent.new(
+  #   :percent (recommended), :round_robin, or :random
+  config.default_rollout = :percent, {
     include_control: true # include control in possible assignments
-  )
+  }
 
   # Secret seed used in generating context keys.
   #
@@ -132,7 +130,7 @@ Gitlab::Experiment.configure do |config|
   #   level1 initiated by file_name.rb:2
   #   level2 initiated by file_name.rb:3
   config.nested_behavior = lambda do |nested_experiment|
-    raise NestingError.new(experiment: self, nested_experiment: nested_experiment)
+    raise Gitlab::Experiment::NestingError.new(experiment: self, nested_experiment: nested_experiment)
   end
 
   # Called at the end of every experiment run, with the result.

data/lib/gitlab/experiment/base_interface.rb

@@ -78,79 +78,33 @@ module Gitlab
 
       alias_method :to_param, :id
 
-      def variant_names
-        @_variant_names ||= behaviors.keys.map(&:to_sym) - [:control]
-      end
+      def process_redirect_url(url)
+        return unless Configuration.redirect_url_validator&.call(url)
 
-      def behaviors
-        @_behaviors ||= public_behaviors_with_deprecations(registered_behavior_callbacks)
+        track('visited', url: url)
+        url # return the url, which allows for mutation
       end
 
-      # @deprecated
-      def public_behaviors_with_deprecations(behaviors)
-        named_variants = %w[control candidate]
-        public_methods.each_with_object(behaviors) do |name, behaviors|
-          name = name.to_s # fixes compatibility for ruby 2.6.x
-          next unless name.end_with?('_behavior')
-
-          behavior_name = name.sub(/_behavior$/, '')
-          registration = named_variants.include?(behavior_name) ? behavior_name : "variant :#{behavior_name}"
-
-          Configuration.deprecated(<<~MESSAGE, version: '0.7.0', stack: 2)
-            using a public `#{name}` method is deprecated and will be removed from {{release}}, instead register variants using:
-
-            class #{self.class.name} < #{Configuration.base_class}
-              #{registration}
+      def key_for(source, seed = name)
+        return source if source.is_a?(String)
 
-              private
+        source = source.keys + source.values if source.is_a?(Hash)
 
-              def #{name}
-                #...
-              end
-            end
-          MESSAGE
-
-          behaviors[behavior_name] ||= -> { send(name) } # rubocop:disable GitlabSecurity/PublicSend
-        end
-      end
+        ingredients = Array(source).map { |v| identify(v) }
+        ingredients.unshift(seed).unshift(Configuration.context_key_secret)
 
-      # @deprecated
-      def session_id
-        Configuration.deprecated(:session_id, 'instead use `id` or use a custom rollout strategy', version: '0.7.0')
-        id
+        Digest::SHA2.new(Configuration.context_key_bit_length).hexdigest(ingredients.join('|')) # rubocop:disable Fips/OpenSSL
       end
 
       # @deprecated
-      def flipper_id
-        Configuration.deprecated(:flipper_id, 'instead use `id` or use a custom rollout strategy', version: '0.7.0')
-        "Experiment;#{id}"
-      end
-
-      # @deprecated
-      def use(&block)
-        Configuration.deprecated(:use, 'instead use `control`', version: '0.7.0')
-
-        control(&block)
-      end
-
-      # @deprecated
-      def try(name = nil, &block)
-        if name.present?
-          Configuration.deprecated(:try, "instead use `variant(:#{name})`", version: '0.7.0')
-          variant(name, &block)
-        else
-          Configuration.deprecated(:try, 'instead use `candidate`', version: '0.7.0')
-          candidate(&block)
-        end
-      end
-
-      protected
-
-      def cached_variant_resolver(provided_variant)
-        return :control if excluded?
+      def variant_names
+        Configuration.deprecated(
+          :variant_names,
+          'instead use `behavior.names`, which includes :control',
+          version: '0.8.0'
+        )
 
-        result = cache_variant(provided_variant) { resolve_variant_name }
-        result.to_sym if result.present?
+        behaviors.keys - [:control]
       end
     end
   end

data/lib/gitlab/experiment/cache.rb

@@ -49,7 +49,7 @@ module Gitlab
         result = migrated_cache_fetch(cache.store, &block)
         return result unless specified.present?
 
-        cache.write(specified) if result != specified
+        cache.write(specified) if result.to_s != specified.to_s
         specified
       end
 
@@ -66,10 +66,8 @@ module Gitlab
 
           store.write(cache_key, value)
           store.delete(old_key)
-          return value
-        end
-
-        store.fetch(cache_key, &block)
+          break value
+        end || store.fetch(cache_key, &block)
       end
     end
   end

data/lib/gitlab/experiment/callbacks.rb

@@ -47,14 +47,14 @@ module Gitlab
         private
 
         def build_behavior_callback(filters, variant, **options, &block)
-          if registered_behavior_callbacks[variant.to_s]
+          if registered_behavior_callbacks[variant]
             raise ExistingBehaviorError, "a behavior for the `#{variant}` variant has already been registered"
           end
 
           callback_behavior = "#{variant}_behavior".to_sym
 
           # Register a the behavior so we can define the block later.
-          registered_behavior_callbacks[variant.to_s] = callback_behavior
+          registered_behavior_callbacks[variant] = callback_behavior
 
           # Add our block or default behavior method.
           filters.push(block) if block.present?
@@ -82,7 +82,7 @@ module Gitlab
         end
 
         def build_run_callback(filters, **options)
-          set_callback(:run, *filters, **options)
+          set_callback(:run, *filters.compact, **options)
         end
 
         def build_callback(chain, *filters, **options)

data/lib/gitlab/experiment/configuration.rb

@@ -56,12 +56,8 @@ module Gitlab
       # end
       #
       # Included rollout strategies:
-      #   Gitlab::Experiment::Rollout::Percent, (recommended)
-      #   Gitlab::Experiment::Rollout::RoundRobin, or
-      #   Gitlab::Experiment::Rollout::Random
-      @default_rollout = Gitlab::Experiment::Rollout::Percent.new(
-        include_control: true # include control in possible assignments
-      )
+      #   :percent, (recommended), :round_robin, or :random
+      @default_rollout = Rollout.resolve(:percent)
 
       # Secret seed used in generating context keys.
       #
@@ -169,61 +165,6 @@ module Gitlab
       end
 
       class << self
-        # @deprecated
-        def context_hash_strategy=(block)
-          deprecated(
-            :context_hash_strategy,
-            'instead use `context_key_secret` and `context_key_bit_length`',
-            version: '0.7.0'
-          )
-
-          @__context_hash_strategy = block
-        end
-
-        # @deprecated
-        def variant_resolver
-          deprecated(
-            :variant_resolver,
-            'instead use `inclusion_resolver` with a block that returns a boolean',
-            version: '0.6.5'
-          )
-
-          @__inclusion_resolver
-        end
-
-        # @deprecated
-        def variant_resolver=(block)
-          deprecated(
-            :variant_resolver,
-            'instead use `inclusion_resolver` with a block that returns a boolean',
-            version: '0.6.5'
-          )
-
-          @__inclusion_resolver = block
-        end
-
-        # @deprecated
-        def inclusion_resolver=(block)
-          deprecated(
-            :inclusion_resolver,
-            'instead put this logic into custom rollout strategies',
-            version: '0.7.0'
-          )
-
-          @__inclusion_resolver = block
-        end
-
-        # @deprecated
-        def inclusion_resolver
-          deprecated(
-            :inclusion_resolver,
-            'instead put this logic into custom rollout strategies',
-            version: '0.7.0'
-          )
-
-          @__inclusion_resolver
-        end
-
         attr_accessor(
           :name_prefix,
           :logger,
@@ -241,7 +182,14 @@ module Gitlab
           :publishing_behavior
         )
 
-        # Internal helpers warnings.
+        # Attribute method overrides.
+
+        def default_rollout=(args) # rubocop:disable Lint/DuplicateMethods
+          @default_rollout = Rollout.resolve(*args)
+        end
+
+        # Internal warning helpers.
+
         def deprecated(*args, version:, stack: 0)
           deprecator = deprecator(version)
           args << args.pop.to_s.gsub('{{release}}', "#{deprecator.gem_name} #{deprecator.deprecation_horizon}")

data/lib/gitlab/experiment/engine.rb

@@ -21,6 +21,7 @@ module Gitlab
       private
 
       def include_dsl
+        Dsl.include_in(ActionController::API, with_helper: false) if defined?(ActionController)
         Dsl.include_in(ActionController::Base, with_helper: true) if defined?(ActionController)
         Dsl.include_in(ActionMailer::Base, with_helper: true) if defined?(ActionMailer)
       end

data/lib/gitlab/experiment/nestable.rb

@@ -25,15 +25,25 @@ module Gitlab
       class Stack
         include Singleton
 
-        @stack = []
+        delegate :pop, :length, :size, :[], to: :stack
 
         class << self
-          delegate :pop, :length, :size, :[], to: :@stack
+          delegate :pop, :push, :length, :size, :[], to: :instance
+        end
+
+        def initialize
+          @thread_key = "#{self.class};#{object_id}".to_sym
+        end
+
+        def push(instance)
+          stack.last&.nest_experiment(instance)
+          stack.push(instance)
+        end
+
+        private
 
-          def push(instance)
-            @stack.last&.nest_experiment(instance)
-            @stack.push(instance)
-          end
+        def stack
+          Thread.current[@thread_key] ||= []
         end
       end
     end

data/lib/gitlab/experiment/rollout/percent.rb

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require "zlib"
+
 # The percent rollout strategy is the most comprehensive included with Gitlab::Experiment. It allows specifying the
 # percentages per variant using an array, a hash, or will default to even distribution when no rules are provided.
 #
@@ -34,14 +36,14 @@ module Gitlab
           when nil then nil
           when Array, Hash
             if distribution_rules.length != behavior_names.length
-              raise InvalidRolloutRules, "the distribution rules don't match the number of variants defined"
+              raise InvalidRolloutRules, "the distribution rules don't match the number of behaviors defined"
             end
           else
             raise InvalidRolloutRules, 'unknown distribution options type'
           end
         end
 
-        def execute_assigment
+        def execute_assignment
           crc = normalized_id
           total = 0
 

data/lib/gitlab/experiment/rollout/random.rb

@@ -27,7 +27,7 @@ module Gitlab
       class Random < Base
         protected
 
-        def execute_assigment
+        def execute_assignment
           behavior_names.sample # pick a random variant
         end
       end

data/lib/gitlab/experiment/rollout/round_robin.rb

@@ -31,7 +31,7 @@ module Gitlab
 
         protected
 
-        def execute_assigment
+        def execute_assignment
           behavior_names[(cache.attr_inc(KEY_NAME) - 1) % behavior_names.size]
         end
       end

data/lib/gitlab/experiment/rollout.rb

@@ -7,70 +7,61 @@ module Gitlab
       autoload :Random, 'gitlab/experiment/rollout/random.rb'
       autoload :RoundRobin, 'gitlab/experiment/rollout/round_robin.rb'
 
-      def self.resolve(klass)
-        return "#{name}::#{klass.to_s.classify}".constantize if klass.is_a?(Symbol) || klass.is_a?(String)
-
-        klass
+      def self.resolve(klass, options = {})
+        options ||= {}
+        case klass
+        when String
+          Strategy.new(klass.classify.constantize, options)
+        when Symbol
+          Strategy.new("#{name}::#{klass.to_s.classify}".constantize, options)
+        when Class
+          Strategy.new(klass, options)
+        else
+          raise ArgumentError, "unable to resolve rollout from #{klass.inspect}"
+        end
       end
 
       class Base
-        DEFAULT_OPTIONS = {
-          include_control: false
-        }.freeze
-
         attr_reader :experiment, :options
 
-        delegate :variant_names, :cache, :id, to: :experiment
-
-        def initialize(options = {})
-          @options = DEFAULT_OPTIONS.merge(options)
-        end
+        delegate :cache, :id, to: :experiment
 
-        def for(experiment)
+        def initialize(experiment, options = {})
           raise ArgumentError, 'you must provide an experiment instance' unless experiment.class <= Gitlab::Experiment
 
           @experiment = experiment
-
-          self
+          @options = options
         end
 
         def enabled?
-          require_experiment(__method__)
-
           true
         end
 
         def resolve
-          require_experiment(__method__)
-
-          return nil if @experiment.respond_to?(:experiment_group?) && !@experiment.experiment_group?
-
           validate! # allow the rollout strategy to validate itself
 
-          assignment = execute_assigment
-          assignment == :control ? nil : assignment # avoid caching control
+          assignment = execute_assignment
+          assignment == :control ? nil : assignment # avoid caching control by returning nil
         end
 
-        protected
+        private
 
         def validate!
           # base is always valid
         end
 
-        def execute_assigment
+        def execute_assignment
           behavior_names.first
         end
 
-        private
-
-        def require_experiment(method_name)
-          return if @experiment.present?
-
-          raise ArgumentError, "you need to call `for` with an experiment instance before chaining `#{method_name}`"
+        def behavior_names
+          experiment.behaviors.keys
         end
+      end
 
-        def behavior_names
-          options[:include_control] ? [:control] + variant_names : variant_names
+      Strategy = Struct.new(:klass, :options) do
+        def for(experiment)
+          klass.new(experiment, options)
         end
       end
     end

data/lib/gitlab/experiment/rspec.rb

@@ -14,13 +14,13 @@ module Gitlab
       def self.track_gitlab_experiment_receiver(method, receiver)
         # Leverage the `>=` method on Gitlab::Experiment to determine if the receiver is an experiment, not the other
         # way round -- `receiver.<=` could be mocked and we want to be extra careful.
-        (@__gitlab_experiment_receivers[method.to_s] ||= []) << receiver if Gitlab::Experiment >= receiver
+        (@__gitlab_experiment_receivers[method] ||= []) << receiver if Gitlab::Experiment >= receiver
       rescue StandardError # again, let's just be extra careful
         false
       end
 
       def self.bind_gitlab_experiment_receiver(method)
-        method.unbind.bind(@__gitlab_experiment_receivers[method.to_s].pop)
+        method.unbind.bind(@__gitlab_experiment_receivers[method].pop)
       end
 
       module MethodDouble
@@ -28,6 +28,7 @@ module Gitlab
           RSpecMocks.track_gitlab_experiment_receiver(original_method, receiver)
           super
         end
+        ruby2_keywords :proxy_method_invoked if respond_to?(:ruby2_keywords, true)
       end
     end
 
@@ -37,9 +38,6 @@ module Gitlab
           wrapped_experiment(experiment, remock: true) do |instance, wrapped|
             # Stub internal methods that will make it behave as we've instructed.
             allow(instance).to receive(:enabled?) { wrapped.variant_name != false }
-            if instance.respond_to?(:experiment_group?, true)
-              allow(instance).to receive(:experiment_group?) { !(wrapped.variant_name == false) }
-            end
 
             # Stub the variant resolution logic to handle true/false, and named variants.
             allow(instance).to receive(:resolve_variant_name).and_wrap_original { |method|
@@ -72,8 +70,8 @@ module Gitlab
       def wrapped_experiment_chain_for(klass)
         @__wrapped_experiment_chains ||= {}
         @__wrapped_experiment_chains[klass.name || klass.object_id] ||= begin
-          allow(klass).to receive(:new).and_wrap_original do |method, *args, &original_block|
-            RSpecMocks.bind_gitlab_experiment_receiver(method).call(*args).tap do |instance|
+          allow(klass).to receive(:new).and_wrap_original do |method, *args, **kwargs, &original_block|
+            RSpecMocks.bind_gitlab_experiment_receiver(method).call(*args, **kwargs).tap do |instance|
               wrapped = @__wrapped_experiments[instance.instance_variable_get(:@_name)]
               wrapped&.blocks&.each { |b| b.call(instance, wrapped) }
 
@@ -128,7 +126,7 @@ module Gitlab
         match do |experiment|
           @experiment = require_experiment(experiment, 'register_behavior')
 
-          block = @experiment.behaviors[behavior_name.to_s]
+          block = @experiment.behaviors[behavior_name]
           @return_expected = false unless block
 
           if @return_expected
@@ -193,13 +191,13 @@ module Gitlab
           @experiment.run_callbacks(:segmentation)
 
           @actual_variant = @experiment.instance_variable_get(:@_assigned_variant_name)
-          @expected_variant ? @actual_variant.to_s == @expected_variant.to_s : @actual_variant.present?
+          @expected_variant ? @actual_variant == @expected_variant : @actual_variant.present?
         end
 
         chain :into do |expected|
           raise ArgumentError, 'variant name must be provided' if expected.blank?
 
-          @expected_variant = expected.to_s
+          @expected_variant = expected
         end
 
         failure_message do
@@ -239,7 +237,7 @@ module Gitlab
         chain(:for) do |expected|
           raise ArgumentError, 'variant name must be provided' if expected.blank?
 
-          @expected_variant = expected.to_s
+          @expected_variant = expected
         end
 
         chain(:with_context) do |expected|
@@ -303,17 +301,21 @@ RSpec.configure do |config|
   config.include Gitlab::Experiment::RSpecHelpers
   config.include Gitlab::Experiment::Dsl
 
-  config.before(:each, :experiment) do
-    RequestStore.clear!
+  config.before(:each) do |example|
+    if example.metadata[:experiment] == true || example.metadata[:type] == :experiment
+      RequestStore.clear!
 
-    if defined?(Gitlab::Experiment::TestBehaviors::TrackedStructure)
-      Gitlab::Experiment::TestBehaviors::TrackedStructure.reset!
+      if defined?(Gitlab::Experiment::TestBehaviors::TrackedStructure)
+        Gitlab::Experiment::TestBehaviors::TrackedStructure.reset!
+      end
     end
   end
 
   config.include Gitlab::Experiment::RSpecMatchers, :experiment
-  config.define_derived_metadata(file_path: Regexp.new('/spec/experiments/')) do |metadata|
-    metadata[:type] = :experiment
+  config.include Gitlab::Experiment::RSpecMatchers, type: :experiment
+
+  config.define_derived_metadata(file_path: Regexp.new('spec/experiments/')) do |metadata|
+    metadata[:type] ||= :experiment
   end
 
   # We need to monkeypatch rspec-mocks because there's an issue around stubbing class methods that impacts us here.

data/lib/gitlab/experiment/version.rb

@@ -2,6 +2,6 @@
 
 module Gitlab
   class Experiment
-    VERSION = '0.7.0'
+    VERSION = '0.8.0'
   end
 end

data/lib/gitlab/experiment.rb

@@ -3,6 +3,8 @@
 require 'request_store'
 require 'active_support'
 require 'active_support/core_ext/module/delegation'
+require 'active_support/core_ext/object/blank'
+require 'active_support/core_ext/string/inflections'
 
 require 'gitlab/experiment/errors'
 require 'gitlab/experiment/base_interface'
@@ -68,7 +70,7 @@ module Gitlab
       def default_rollout(rollout = nil, options = {})
         return @_rollout ||= Configuration.default_rollout if rollout.blank?
 
-        @_rollout = Rollout.resolve(rollout).new(options)
+        @_rollout = Rollout.resolve(rollout, options)
       end
 
       # Class level accessor methods.
@@ -86,34 +88,15 @@ module Gitlab
       variant(:control, &block)
     end
 
-    def candidate(name = nil, &block)
-      if name.present?
-        Configuration.deprecated(<<~MESSAGE, version: '0.7.0')
-          passing name to `candidate` is deprecated and will be removed from {{release}} (instead use `variant(#{name.inspect})`)
-        MESSAGE
-      end
-
-      variant(name || :candidate, &block)
+    def candidate(&block)
+      variant(:candidate, &block)
     end
 
-    def variant(name = nil, &block)
-      if block.present? # we know we're defining a variant block
-        raise ArgumentError, 'missing variant name' if name.blank?
-
-        return behaviors[name.to_s] = block
-      end
-
-      if name.present?
-        Configuration.deprecated(<<~MESSAGE, version: '0.7.0')
-          setting the variant using `variant` is deprecated and will be removed from {{release}} (instead use `assigned(#{name.inspect})`)
-        MESSAGE
-      else
-        Configuration.deprecated(<<~MESSAGE, version: '0.7.0')
-          getting the assigned variant using `variant` is deprecated and will be removed from {{release}} (instead use `assigned`)
-        MESSAGE
-      end
+    def variant(name, &block)
+      raise ArgumentError, 'name required' if name.blank?
+      raise ArgumentError, 'block required' unless block.present?
 
-      assigned(name)
+      behaviors[name] = block
     end
 
     def context(value = nil)
@@ -125,9 +108,7 @@ module Gitlab
 
     def assigned(value = nil)
       @_assigned_variant_name = cache_variant(value) if value.present?
-      if @_assigned_variant_name || @_resolving_variant
-        return Variant.new(name: (@_assigned_variant_name || :unresolved).to_s)
-      end
+      return Variant.new(name: @_assigned_variant_name || :unresolved) if @_assigned_variant_name || @_resolving_variant
 
       if enabled?
         @_resolving_variant = true
@@ -136,7 +117,7 @@ module Gitlab
 
       run_callbacks(segmentation_callback_chain) do
         @_assigned_variant_name ||= :control
-        Variant.new(name: @_assigned_variant_name.to_s)
+        Variant.new(name: @_assigned_variant_name)
       end
     ensure
       @_resolving_variant = false
@@ -145,7 +126,7 @@ module Gitlab
     def rollout(rollout = nil, options = {})
       return @_rollout ||= self.class.default_rollout(nil, options).for(self) if rollout.blank?
 
-      @_rollout = Rollout.resolve(rollout).new(options).for(self)
+      @_rollout = Rollout.resolve(rollout, options).for(self)
     end
 
     def exclude!
@@ -170,13 +151,6 @@ module Gitlab
       instance_exec(action, tracking_context(event_args).try(:compact) || {}, &Configuration.tracking_behavior)
     end
 
-    def process_redirect_url(url)
-      return unless Configuration.redirect_url_validator&.call(url)
-
-      track('visited', url: url)
-      url # return the url, which allows for mutation
-    end
-
     def enabled?
       rollout.enabled?
     end
@@ -192,23 +166,11 @@ module Gitlab
     end
 
     def signature
-      { variant: assigned.name, experiment: name }.merge(context.signature)
+      { variant: assigned.name.to_s, experiment: name }.merge(context.signature)
     end
 
-    def key_for(source, seed = name)
-      # TODO: Remove - deprecated in release 0.7.0
-      if (block = Configuration.instance_variable_get(:@__context_hash_strategy))
-        return instance_exec(source, seed, &block)
-      end
-
-      return source if source.is_a?(String)
-
-      source = source.keys + source.values if source.is_a?(Hash)
-
-      ingredients = Array(source).map { |v| identify(v) }
-      ingredients.unshift(seed).unshift(Configuration.context_key_secret)
-
-      Digest::SHA2.new(Configuration.context_key_bit_length).hexdigest(ingredients.join('|'))
+    def behaviors
+      @_behaviors ||= registered_behavior_callbacks
     end
 
     protected
@@ -217,23 +179,15 @@ module Gitlab
       (object.respond_to?(:to_global_id) ? object.to_global_id : object).to_s
     end
 
+    def cached_variant_resolver(provided_variant)
+      return :control if excluded?
+
+      result = cache_variant(provided_variant) { resolve_variant_name }
+      result.to_sym if result.present?
+    end
+
     def resolve_variant_name
-      if respond_to?(:experiment_group?, true)
-        # TODO: Remove - deprecated in release 0.7.0
-        Configuration.deprecated(:experiment_group?, <<~MESSAGE, version: '0.7.0')
-          instead put this logic into custom rollout strategies
-        MESSAGE
-
-        rollout.resolve if experiment_group?
-      elsif (block = Configuration.instance_variable_get(:@__inclusion_resolver))
-        # TODO: Remove - deprecated in release 0.7.0
-        rollout.resolve if instance_exec(@_assigned_variant_name, &block)
-      elsif (block = Configuration.instance_variable_get(:@__variant_resolver))
-        # TODO: Remove - deprecated in release 0.6.5
-        instance_exec(@_assigned_variant_name, &block)
-      else
-        rollout.resolve # this is the end result of all deprecations
-      end
+      rollout.resolve
     end
 
     def tracking_context(event_args)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab-experiment
 version: !ruby/object:Gem::Version
-  version: 0.7.0
+  version: 0.8.0
 platform: ruby
 authors:
 - GitLab
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 
+date: 2023-03-07 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: activesupport
@@ -38,67 +38,52 @@ dependencies:
     - - ">="
       - !ruby/object:Gem::Version
         version: '1.0'
-description: 
+description:
 email:
 - gitlab_rubygems@gitlab.com
 executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- lib/generators/test_unit
-- lib/generators/test_unit/experiment
-- lib/generators/test_unit/experiment/templates
-- lib/generators/test_unit/experiment/templates/experiment_test.rb.tt
-- lib/generators/test_unit/experiment/experiment_generator.rb
-- lib/generators/rspec
-- lib/generators/rspec/experiment
-- lib/generators/rspec/experiment/templates
-- lib/generators/rspec/experiment/templates/experiment_spec.rb.tt
-- lib/generators/rspec/experiment/experiment_generator.rb
-- lib/generators/gitlab
-- lib/generators/gitlab/experiment
+- LICENSE.txt
+- README.md
 - lib/generators/gitlab/experiment/USAGE
-- lib/generators/gitlab/experiment/templates
-- lib/generators/gitlab/experiment/templates/experiment.rb.tt
-- lib/generators/gitlab/experiment/install
-- lib/generators/gitlab/experiment/install/templates
+- lib/generators/gitlab/experiment/experiment_generator.rb
+- lib/generators/gitlab/experiment/install/install_generator.rb
+- lib/generators/gitlab/experiment/install/templates/POST_INSTALL
 - lib/generators/gitlab/experiment/install/templates/application_experiment.rb.tt
 - lib/generators/gitlab/experiment/install/templates/initializer.rb.tt
-- lib/generators/gitlab/experiment/install/templates/POST_INSTALL
-- lib/generators/gitlab/experiment/install/install_generator.rb
-- lib/generators/gitlab/experiment/experiment_generator.rb
-- lib/gitlab/experiment
-- lib/gitlab/experiment/dsl.rb
-- lib/gitlab/experiment/rspec.rb
-- lib/gitlab/experiment/context.rb
-- lib/gitlab/experiment/nestable.rb
-- lib/gitlab/experiment/configuration.rb
-- lib/gitlab/experiment/rollout.rb
-- lib/gitlab/experiment/cache
-- lib/gitlab/experiment/cache/redis_hash_store.rb
-- lib/gitlab/experiment/engine.rb
+- lib/generators/gitlab/experiment/templates/experiment.rb.tt
+- lib/generators/rspec/experiment/experiment_generator.rb
+- lib/generators/rspec/experiment/templates/experiment_spec.rb.tt
+- lib/generators/test_unit/experiment/experiment_generator.rb
+- lib/generators/test_unit/experiment/templates/experiment_test.rb.tt
+- lib/gitlab/experiment.rb
 - lib/gitlab/experiment/base_interface.rb
-- lib/gitlab/experiment/middleware.rb
-- lib/gitlab/experiment/version.rb
+- lib/gitlab/experiment/cache.rb
+- lib/gitlab/experiment/cache/redis_hash_store.rb
+- lib/gitlab/experiment/callbacks.rb
+- lib/gitlab/experiment/configuration.rb
+- lib/gitlab/experiment/context.rb
 - lib/gitlab/experiment/cookies.rb
+- lib/gitlab/experiment/dsl.rb
+- lib/gitlab/experiment/engine.rb
 - lib/gitlab/experiment/errors.rb
-- lib/gitlab/experiment/cache.rb
-- lib/gitlab/experiment/variant.rb
-- lib/gitlab/experiment/rollout
-- lib/gitlab/experiment/rollout/random.rb
+- lib/gitlab/experiment/middleware.rb
+- lib/gitlab/experiment/nestable.rb
+- lib/gitlab/experiment/rollout.rb
 - lib/gitlab/experiment/rollout/percent.rb
+- lib/gitlab/experiment/rollout/random.rb
 - lib/gitlab/experiment/rollout/round_robin.rb
-- lib/gitlab/experiment/callbacks.rb
-- lib/gitlab/experiment/test_behaviors
+- lib/gitlab/experiment/rspec.rb
 - lib/gitlab/experiment/test_behaviors/trackable.rb
-- lib/gitlab/experiment.rb
-- LICENSE.txt
-- README.md
+- lib/gitlab/experiment/variant.rb
+- lib/gitlab/experiment/version.rb
 homepage: https://gitlab.com/gitlab-org/ruby/gems/gitlab-experiment
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -113,8 +98,8 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubygems_version: 3.1.6
-signing_key: 
+rubygems_version: 3.4.7
+signing_key:
 specification_version: 4
 summary: GitLab experimentation library.
 test_files: []