trailguide 0.1.30 → 0.1.31

data/app/controllers/trail_guide/admin/application_controller.rb

@@ -18,7 +18,7 @@ module TrailGuide
 
       def experiment_peekable?(experiment)
         return false unless TrailGuide::Admin.configuration.peek_parameter
-        return false unless experiment.running?
+        return false unless experiment.started? && !experiment.stopped?
         return false if experiment.target_sample_size_reached?
         return true
       end
@@ -30,7 +30,7 @@ module TrailGuide
       helper_method :experiment_peeking?
 
       def experiment_metrics_visible?(experiment)
-        return true unless experiment.running?
+        return true unless experiment.started? && !experiment.stopped?
         return true if params[TrailGuide::Admin.configuration.peek_parameter] == experiment.experiment_name.to_s
         return true if experiment.target_sample_size_reached?
         return false
@@ -38,7 +38,7 @@ module TrailGuide
       helper_method :experiment_metrics_visible?
 
       def experiment_metric(experiment, metric)
-        return metric if experiment_metrics_visible?(experiment)
+        return helpers.number_with_delimiter(metric) if experiment_metrics_visible?(experiment)
         return helpers.content_tag('span', nil, class: 'fas fa-eye-slash', data: {toggle: 'tooltip'}, title: "metrics are hidden until this experiment reaches it's target sample size")
       end
       helper_method :experiment_metric

data/app/controllers/trail_guide/admin/experiments_controller.rb

@@ -5,32 +5,42 @@ module TrailGuide
         (redirect_to :back rescue redirect_to trail_guide_admin.experiments_path) and return unless experiment.present?
       end
 
+      before_action :experiments, only: [:index]
+
       def index
       end
 
       def start
         experiment.start!(self)
-        redirect_to trail_guide_admin.experiments_path(anchor: experiment.experiment_name)
+        redirect_to trail_guide_admin.scoped_experiments_path(scope: :running, anchor: experiment.experiment_name)
+      end
+
+      def pause
+        experiment.pause!(self)
+        redirect_to trail_guide_admin.scoped_experiments_path(scope: :paused, anchor: experiment.experiment_name)
       end
 
       def stop
         experiment.stop!(self)
-        redirect_to trail_guide_admin.experiments_path(anchor: experiment.experiment_name)
+        redirect_to trail_guide_admin.scoped_experiments_path(scope: :stopped, anchor: experiment.experiment_name)
       end
 
       def reset
+        experiment.stop!(self)
         experiment.reset!(self)
         redirect_to trail_guide_admin.experiments_path(anchor: experiment.experiment_name)
       end
 
       def resume
         experiment.resume!(self)
-        redirect_to trail_guide_admin.experiments_path(anchor: experiment.experiment_name)
+        redirect_to trail_guide_admin.scoped_experiments_path(scope: :running, anchor: experiment.experiment_name)
       end
 
       def restart
-        experiment.reset!(self) && experiment.start!(self)
-        redirect_to trail_guide_admin.experiments_path(anchor: experiment.experiment_name)
+        experiment.stop!(self)
+        experiment.reset!(self)
+        experiment.start!(self)
+        redirect_to trail_guide_admin.scoped_experiments_path(scope: :running, anchor: experiment.experiment_name)
       end
 
       def join
@@ -58,6 +68,12 @@ module TrailGuide
 
       private
 
+      def experiments
+        @experiments = TrailGuide.catalog
+        @experiments = @experiments.send(params[:scope]) if params[:scope].present?
+        @experiments = @experiments.by_started
+      end
+
       def experiment
         @experiment ||= TrailGuide.catalog.find(params[:id])
       end

data/app/views/layouts/trail_guide/admin/_header.erb

@@ -4,11 +4,39 @@
     <%= TrailGuide::Admin.configuration.title %>
   <% end %>
   <div class="col-sm text-center">
-    <strong class="total"><%= TrailGuide.catalog.count %></strong> experiments
+    <%= link_to trail_guide_admin.experiments_path, class: "text-dark" do %>
+      <strong class="total">
+        <%= TrailGuide.catalog.count %>
+      </strong>
+      <span>experiments</span>
+    <% end %>
+
     <span>/</span>
-    <strong class="running"><%= TrailGuide.catalog.running.count %></strong> running
+
+    <%= link_to trail_guide_admin.scoped_experiments_path(scope: :running), class: "text-dark" do %>
+      <strong class="total">
+        <%= TrailGuide.catalog.running.count %>
+      </strong>
+      <span>running</span>
+    <% end %>
+
     <span>/</span>
-    <strong class="stopped"><%= TrailGuide.catalog.stopped.count %></strong> stopped
+
+    <%= link_to trail_guide_admin.scoped_experiments_path(scope: :paused), class: "text-dark" do %>
+      <strong class="total">
+        <%= TrailGuide.catalog.paused.count %>
+      </strong>
+      <span>paused</span>
+    <% end %>
+
+    <span>/</span>
+
+    <%= link_to trail_guide_admin.scoped_experiments_path(scope: :stopped), class: "text-dark" do %>
+      <strong class="total">
+        <%= TrailGuide.catalog.stopped.count %>
+      </strong>
+      <span>stopped</span>
+    <% end %>
   </div>
   <span class="navbar-brand"><small class="text-muted"><%= TrailGuide::Admin.configuration.subtitle %></small></span>
 </nav>

data/app/views/trail_guide/admin/experiments/_combined_experiment.html.erb

@@ -19,17 +19,37 @@
               <% end %>
             <% end %>
           <% end %>
-          <%= link_to trail_guide_admin.stop_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-warning', method: :put, data: {toggle: :tooltip}, title: 'pause this experiment - you will have the option to resume or reset' do %>
-            <span class="fas fa-pause" />
+
+          <% if combined_experiment.configuration.can_resume? %>
+            <%= link_to trail_guide_admin.pause_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-warning', method: :put, data: {toggle: :tooltip}, title: 'pause this experiment - you will have the option to resume or reset' do %>
+              <span class="fas fa-pause" />
+            <% end %>
+          <% end %>
+
+          <%= link_to trail_guide_admin.stop_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-danger', method: :put, data: {toggle: :tooltip}, title: 'stop this experiment - once stopped you will need to reset before restarting' do %>
+            <span class="fas fa-stop" />
           <% end %>
+
           <%= link_to trail_guide_admin.restart_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
             <span class="fas fa-redo" />
           <% end %>
-        <% elsif combined_experiment.started? %>
-          <%= link_to trail_guide_admin.resume_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'resume this experiment to start bucketing users and serving variants again' do %>
+        <% elsif combined_experiment.paused? %>
+          <% if combined_experiment.configuration.can_resume? %>
+            <%= link_to trail_guide_admin.resume_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'resume this experiment to start bucketing users and serving variants again' do %>
+              <span class="fas fa-redo" />
+            <% end %>
+            <%= link_to trail_guide_admin.stop_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-danger', method: :put, data: {toggle: :tooltip}, title: 'stop this experiment - once stopped you will need to reset before restarting' do %>
+              <span class="fas fa-stop" />
+            <% end %>
+          <% end %>
+          <%= link_to trail_guide_admin.restart_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
             <span class="fas fa-redo" />
           <% end %>
-        <% else %>
+        <% elsif combined_experiment.stopped? %>
+          <%= link_to trail_guide_admin.restart_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
+            <span class="fas fa-redo" />
+          <% end %>
+        <% elsif !combined_experiment.started? %>
           <%= link_to trail_guide_admin.start_experiment_path(combined_experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'start this experiment' do %>
             <span class="fas fa-play" />
           <% end %>
@@ -50,6 +70,8 @@
           <span class="text-muted">&mdash;</span>
           <% if combined_experiment.stopped? %>
             <small class="text-muted"><%= combined_experiment.stopped_at.strftime('%b %e %Y @ %l:%M %p') %></small>
+          <% elsif combined_experiment.paused? %>
+            <small class="text-muted">paused</small>
           <% else %>
             <small class="text-muted">running</small>
           <% end %>
@@ -147,9 +169,9 @@
         <tr>
           <th scope="row">&nbsp;</th>
           <th>
-            <span><%= combined_experiment.participants %></span>
+            <span><%= number_with_delimiter combined_experiment.participants %></span>
             <% if combined_experiment.configuration.target_sample_size %>
-              <span class="text-muted">/ <%= combined_experiment.configuration.target_sample_size %></span>
+              <span class="text-muted">/ <%= number_with_delimiter combined_experiment.configuration.target_sample_size %></span>
             <% end %>
           </th>
           <% if combined_experiment.goals.empty? %>

data/app/views/trail_guide/admin/experiments/_experiment.html.erb

@@ -19,17 +19,37 @@
               <% end %>
             <% end %>
           <% end %>
-          <%= link_to trail_guide_admin.stop_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-warning', method: :put, data: {toggle: :tooltip}, title: 'pause this experiment - you will have the option to resume or reset' do %>
-            <span class="fas fa-pause" />
+
+          <% if experiment.configuration.can_resume? %>
+            <%= link_to trail_guide_admin.pause_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-warning', method: :put, data: {toggle: :tooltip}, title: 'pause this experiment - you will have the option to resume or reset' do %>
+              <span class="fas fa-pause" />
+            <% end %>
+          <% end %>
+
+          <%= link_to trail_guide_admin.stop_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-danger', method: :put, data: {toggle: :tooltip}, title: 'stop this experiment - once stopped you will need to reset before restarting' do %>
+            <span class="fas fa-stop" />
           <% end %>
+
           <%= link_to trail_guide_admin.restart_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
             <span class="fas fa-redo" />
           <% end %>
-        <% elsif experiment.started? %>
-          <%= link_to trail_guide_admin.resume_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'resume this experiment to start bucketing users and serving variants again' do %>
+        <% elsif experiment.paused? %>
+          <% if experiment.configuration.can_resume? %>
+            <%= link_to trail_guide_admin.resume_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'resume this experiment to start bucketing users and serving variants again' do %>
+              <span class="fas fa-redo" />
+            <% end %>
+            <%= link_to trail_guide_admin.stop_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-danger', method: :put, data: {toggle: :tooltip}, title: 'stop this experiment - once stopped you will need to reset before restarting' do %>
+              <span class="fas fa-stop" />
+            <% end %>
+          <% end %>
+          <%= link_to trail_guide_admin.restart_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
             <span class="fas fa-redo" />
           <% end %>
-        <% else %>
+        <% elsif experiment.stopped? %>
+          <%= link_to trail_guide_admin.restart_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-danger',method: :put, data: {toggle: :tooltip}, title: 'restart this experiment - will reset all data and restart the experiment' do %>
+            <span class="fas fa-redo" />
+          <% end %>
+        <% elsif !experiment.started? %>
           <%= link_to trail_guide_admin.start_experiment_path(experiment.experiment_name), class: 'btn btn-sm btn-success',method: :put, data: {toggle: :tooltip}, title: 'start this experiment' do %>
             <span class="fas fa-play" />
           <% end %>
@@ -50,6 +70,8 @@
           <span class="text-muted">&mdash;</span>
           <% if experiment.stopped? %>
             <small class="text-muted"><%= experiment.stopped_at.strftime('%b %e %Y @ %l:%M %p') %></small>
+          <% elsif experiment.paused? %>
+            <small class="text-muted">paused</small>
           <% else %>
             <small class="text-muted">running</small>
           <% end %>
@@ -140,9 +162,9 @@
         <tr>
           <th scope="row">&nbsp;</th>
           <th>
-            <span><%= experiment.participants %></span>
+            <span><%= number_with_delimiter experiment.participants %></span>
             <% if experiment.configuration.target_sample_size %>
-              <span class="text-muted">/ <%= experiment.configuration.target_sample_size %></span>
+              <span class="text-muted">/ <%= number_with_delimiter experiment.configuration.target_sample_size %></span>
             <% end %>
           </th>
           <% if experiment.goals.empty? %>

data/app/views/trail_guide/admin/experiments/index.html.erb

@@ -9,7 +9,7 @@
 <% end %>
 
 <div class="experiments">
-  <% TrailGuide.catalog.by_started.each do |experiment| %>
+  <% @experiments.each do |experiment| %>
     <% if experiment.combined? %>
       <%= render 'combined_experiment', combined_experiment: experiment %>
     <% else %>

data/config/initializers/trailguide.rb

@@ -1,6 +1,9 @@
 # top-level trailguide rails engine configuration
 #
 TrailGuide.configure do |config|
+  # logger object
+  config.logger = Rails.logger
+
   # url string or initialized Redis object
   config.redis = ENV['REDIS_URL']
 
@@ -73,7 +76,7 @@ TrailGuide.configure do |config|
   # callback when your participant adapter fails to initialize, and trailguide
   # falls back to the anonymous adapter
   config.on_adapter_failover = -> (adapter, error) do
-    Rails.logger.error("#{error.class.name}: #{error.message}")
+    TrailGuide.logger.error error
   end
 
   # list of user agents used by the default request filter proc below when
@@ -175,6 +178,13 @@ TrailGuide::Experiment.configure do |config|
   # false   default behavior, requests will be filtered based on your config
   config.skip_request_filter = false
 
+  # whether or not this experiment can be resumed after it's stopped - allows
+  # temporarily "pausing" an experiment then resuming it without losing metrics
+  #
+  # true    this experiment can be paused and resumed
+  # false   this experiment can only be stopped and reset/restarted
+  config.can_resume = false
+
   # set a default target sample size for all experiments - this will prevent
   # metrics and stats from being displayed in the admin UI until the sample size
   # is reached or the experiment is stopped
@@ -184,7 +194,7 @@ TrailGuide::Experiment.configure do |config|
   # callback when connecting to redis fails and trailguide falls back to always
   # returning control variants
   config.on_redis_failover = -> (experiment, error) do
-    Rails.logger.error("#{error.class.name}: #{error.message}")
+    TrailGuide.logger.error error
   end
 
   # callback on experiment start, either manually via UI/console or
@@ -208,6 +218,16 @@ TrailGuide::Experiment.configure do |config|
   #
   # config.on_stop = -> (experiment, context) { ... }
 
+  # callback on experiment pause manually via UI/console, can be used for
+  # logging, tracking, etc.
+  #
+  # context may or may not be present depending on how you're triggering the
+  # action - if you're using the admin, this will be the admin controller
+  # context, if you're in a console you have the option to pass a context to
+  # `experiment.pause!` or not
+  #
+  # config.on_pause = -> (experiment, context) { ... }
+
   # callback on experiment resume manually via UI/console, can be used for
   # logging, tracking, etc.
   #
@@ -218,8 +238,18 @@ TrailGuide::Experiment.configure do |config|
   #
   # config.on_resume = -> (experiment, context) { ... }
 
+  # callback on experiment delete manually via UI/console, can be used for
+  # logging, tracking, etc. - will also be triggered by a reset
+  #
+  # context may or may not be present depending on how you're triggering the
+  # action - if you're using the admin, this will be the admin controller
+  # context, if you're in a console you have the option to pass a context to
+  # `experiment.delete!` or not
+  #
+  # config.on_delete = -> (experiment, context) { ... }
+
   # callback on experiment reset manually via UI/console, can be used for
-  # logging, tracking, etc.
+  # logging, tracking, etc. - will also trigger any on_delete callbacks
   #
   # context may or may not be present depending on how you're triggering the
   # action - if you're using the admin, this will be the admin controller

data/config/routes/admin.rb

@@ -0,0 +1,22 @@
+TrailGuide::Admin::Engine.routes.draw do
+  resources :experiments, path: '/', only: [:index] do
+    member do
+      match :start,   via: [:put, :post, :get]
+      match :pause,   via: [:put, :post, :get]
+      match :stop,    via: [:put, :post, :get]
+      match :reset,   via: [:put, :post, :get]
+      match :resume,  via: [:put, :post, :get]
+      match :restart, via: [:put, :post, :get]
+
+      match :join,    via: [:put, :post, :get], path: 'join/:variant'
+      match :leave,   via: [:put, :post, :get]
+
+      match :winner,  via: [:put, :post, :get], path: 'winner/:variant'
+      match :clear,   via: [:put, :post, :get]
+    end
+
+    collection do
+      get  '/:scope', action: :index, as: :scoped
+    end
+  end
+end

data/config/routes/engine.rb

@@ -0,0 +1,13 @@
+TrailGuide::Engine.routes.draw do
+  get   '/' => 'experiments#index',
+        defaults: { format: :json }
+  match '/:experiment_name' => 'experiments#choose',
+        defaults: { format: :json },
+        via: [:get, :post]
+  match '/:experiment_name' => 'experiments#convert',
+        defaults: { format: :json },
+        via: [:put]
+  match '/:experiment_name/:checkpoint' => 'experiments#convert',
+        defaults: { format: :json },
+        via: [:put]
+end

data/config/routes.rb

@@ -1,31 +1,5 @@
-TrailGuide::Engine.routes.draw do
-  get   '/' => 'experiments#index',
-        defaults: { format: :json }
-  match '/:experiment_name' => 'experiments#choose',
-        defaults: { format: :json },
-        via: [:get, :post]
-  match '/:experiment_name' => 'experiments#convert',
-        defaults: { format: :json },
-        via: [:put]
-  match '/:experiment_name/:checkpoint' => 'experiments#convert',
-        defaults: { format: :json },
-        via: [:put]
-end
-
-TrailGuide::Admin::Engine.routes.draw do
-  resources :experiments, path: '/', only: [:index] do
-    member do
-      match :start,   via: [:put, :post, :get]
-      match :stop,    via: [:put, :post, :get]
-      match :reset,   via: [:put, :post, :get]
-      match :resume,  via: [:put, :post, :get]
-      match :restart, via: [:put, :post, :get]
-
-      match :join,    via: [:put, :post, :get], path: 'join/:variant'
-      match :leave,   via: [:put, :post, :get]
-
-      match :winner,  via: [:put, :post, :get], path: 'winner/:variant'
-      match :clear,   via: [:put, :post, :get]
-    end
-  end
-end
+# this not only organizes the routes, but also ensures they're only included
+# once (via ruby require behavior), because having two separate engines in the
+# gem can cause routes to be parsed twice and double up
+require_relative 'routes/engine'
+require_relative 'routes/admin'

data/lib/trail_guide/adapters/participants/multi.rb

@@ -6,7 +6,8 @@ module TrailGuide
         def initialize(&block)
           configure do |config|
             config.adapter = -> (context) do
-              if context.respond_to?(:current_user, true) && context.send(:current_user).present?
+              if (context.respond_to?(:trailguide_user, true) && context.send(:trailguide_user).present?) ||
+                  (context.respond_to?(:current_user, true) && context.send(:current_user).present?)
                 TrailGuide::Adapters::Participants::Redis
               elsif context.respond_to?(:cookies, true)
                 TrailGuide::Adapters::Participants::Cookie

data/lib/trail_guide/adapters/participants/redis.rb

@@ -6,7 +6,10 @@ module TrailGuide
         def initialize(&block)
           configure do |config|
             config.namespace = :participants
-            config.lookup = -> (context) { context.current_user.id }
+            config.lookup = -> (context) {
+              context.try(:trailguide_user).try(:id) ||
+                context.try(:current_user).try(:id)
+            }
             config.expiration = nil
 
             yield(config) if block_given?

data/lib/trail_guide/catalog.rb

@@ -99,10 +99,22 @@ module TrailGuide
       self.class.new(to_a.select(&:running?))
     end
 
+    def paused
+      self.class.new(to_a.select(&:paused?))
+    end
+
     def stopped
       self.class.new(to_a.select(&:stopped?))
     end
 
+    def ended
+      self.class.new(to_a.select(&:winner?))
+    end
+
+    def not_running
+      self.class.new(to_a.select { |e| !e.running? })
+    end
+
     def by_started
       scoped = to_a.sort do |a,b|
         if a.running? && !b.running?

data/lib/trail_guide/combined_experiment.rb

@@ -19,6 +19,10 @@ module TrailGuide
         parent.start!
       end
 
+      def pause!
+        parent.pause!
+      end
+
       def stop!
         parent.stop!
       end
@@ -31,6 +35,10 @@ module TrailGuide
         parent.started_at
       end
 
+      def paused_at
+        parent.paused_at
+      end
+
       def stopped_at
         parent.stopped_at
       end
@@ -41,7 +49,7 @@ module TrailGuide
 
     # use the parent experiment as the algorithm and map to the matching variant
     def algorithm_choose!(metadata: nil)
-      variant = parent.new(participant).choose!(metadata: metadata)
+      variant = parent.new(participant.participant).choose!(metadata: metadata)
       variants.find { |var| var == variant.name }
     end
   end

data/lib/trail_guide/config.rb

@@ -1,10 +1,10 @@
 module TrailGuide
   class Config < Canfig::Config
     DEFAULT_KEYS = [
-      :redis, :disabled, :override_parameter, :allow_multiple_experiments,
-      :adapter, :on_adapter_failover, :filtered_ip_addresses,
-      :filtered_user_agents, :request_filter, :include_helpers,
-      :cleanup_participant_experiments, :unity_ttl
+      :logger, :redis, :disabled, :override_parameter,
+      :allow_multiple_experiments, :adapter, :on_adapter_failover,
+      :filtered_ip_addresses, :filtered_user_agents, :request_filter,
+      :include_helpers, :cleanup_participant_experiments, :unity_ttl
     ].freeze
 
     def initialize(*args, **opts, &block)

data/lib/trail_guide/experiments/base.rb

@@ -52,18 +52,25 @@ module TrailGuide
           started
         end
 
+        def pause!(context=nil)
+          return false unless running? && configuration.can_resume?
+          paused = TrailGuide.redis.hset(storage_key, 'paused_at', Time.now.to_i)
+          run_callbacks(:on_pause, context)
+          paused
+        end
+
         def stop!(context=nil)
-          return false unless running?
+          return false unless started? && !stopped?
           stopped = TrailGuide.redis.hset(storage_key, 'stopped_at', Time.now.to_i)
           run_callbacks(:on_stop, context)
           stopped
         end
 
         def resume!(context=nil)
-          return false unless started? && stopped?
-          restarted = TrailGuide.redis.hdel(storage_key, 'stopped_at')
+          return false unless paused? && configuration.can_resume?
+          resumed = TrailGuide.redis.hdel(storage_key, 'paused_at')
           run_callbacks(:on_resume, context)
-          restarted
+          resumed
         end
 
         def started_at
@@ -71,6 +78,11 @@ module TrailGuide
           return Time.at(started.to_i) if started
         end
 
+        def paused_at
+          paused = TrailGuide.redis.hget(storage_key, 'paused_at')
+          return Time.at(paused.to_i) if paused
+        end
+
         def stopped_at
           stopped = TrailGuide.redis.hget(storage_key, 'stopped_at')
           return Time.at(stopped.to_i) if stopped
@@ -80,12 +92,16 @@ module TrailGuide
           started_at && started_at <= Time.now
         end
 
+        def paused?
+          paused_at && paused_at <= Time.now
+        end
+
         def stopped?
           stopped_at && stopped_at <= Time.now
         end
 
         def running?
-          started? && !stopped?
+          started? && !paused? && !stopped?
         end
 
         def declare_winner!(variant, context=nil)

data/lib/trail_guide/experiments/config.rb

@@ -5,13 +5,13 @@ module TrailGuide
         :name, :summary, :preview_url, :algorithm, :metric, :variants, :goals,
         :start_manually, :reset_manually, :store_override, :track_override,
         :combined, :allow_multiple_conversions, :allow_multiple_goals,
-        :track_winner_conversions, :skip_request_filter, :target_sample_size
+        :track_winner_conversions, :skip_request_filter, :target_sample_size,
+        :can_resume
       ].freeze
 
       CALLBACK_KEYS = [
-        :on_start, :on_stop, :on_resume, :on_winner, :on_reset, :on_delete,
-        :on_choose, :on_use, :on_convert,
-        :on_redis_failover,
+        :on_start, :on_stop, :on_pause, :on_resume, :on_winner, :on_reset,
+        :on_delete, :on_choose, :on_use, :on_convert, :on_redis_failover,
         :allow_participation, :allow_conversion, :rollout_winner
       ].freeze
 
@@ -73,6 +73,10 @@ module TrailGuide
         !!skip_request_filter
       end
 
+      def can_resume?
+        !!can_resume
+      end
+
       def name
         @name ||= (self[:name] || experiment.name).try(:to_s).try(:underscore).try(:to_sym)
       end
@@ -160,6 +164,11 @@ module TrailGuide
         self[:on_stop] << (meth || block)
       end
 
+      def on_pause(meth=nil, &block)
+        self[:on_pause] ||= []
+        self[:on_pause] << (meth || block)
+      end
+
       def on_resume(meth=nil, &block)
         self[:on_resume] ||= []
         self[:on_resume] << (meth || block)