vanity 3.1.0 → 4.0.0

data/lib/vanity/experiment/definition.rb

@@ -8,14 +8,14 @@ module Vanity
     #     metrics :signup
     #   end
     module Definition
-
       attr_reader :playground
 
       # Defines a new experiment, given the experiment's name, type and
       # definition block.
       def define(name, type, options = nil, &block)
-        fail "Experiment #{@experiment_id} already defined in playground" if playground.experiments[@experiment_id]
-        klass = Experiment.const_get(type.to_s.gsub(/\/(.?)/) { "::#{$1.upcase}" }.gsub(/(?:^|_)(.)/) { $1.upcase })
+        raise "Experiment #{@experiment_id} already defined in playground" if playground.experiments[@experiment_id]
+
+        klass = Experiment.const_get(type.to_s.gsub(/\/(.?)/) { "::#{Regexp.last_match(1).upcase}" }.gsub(/(?:^|_)(.)/) { Regexp.last_match(1).upcase })
         experiment = klass.new(playground, @experiment_id, name, options)
         experiment.instance_eval(&block)
         experiment.save
@@ -23,10 +23,10 @@ module Vanity
       end
 
       def new_binding(playground, id)
-        @playground, @experiment_id = playground, id
+        @playground = playground
+        @experiment_id = id
         binding
       end
-
     end
   end
-end
+end

data/lib/vanity/frameworks/rails.rb

@@ -69,7 +69,7 @@ module Vanity
           @vanity_identity
         elsif cookies[Vanity.configuration.cookie_name]
           @vanity_identity = cookies[Vanity.configuration.cookie_name]
-        elsif identity = vanity_identity_from_method(vanity_identity_method)
+        elsif identity = vanity_identity_from_method(vanity_identity_method) # rubocop:todo Lint/AssignmentInCondition
           @vanity_identity = identity
         elsif response # everyday use
           @vanity_identity = cookies[Vanity.configuration.cookie_name] || SecureRandom.hex(16)
@@ -96,7 +96,7 @@ module Vanity
           path: Vanity.configuration.cookie_path,
           domain: Vanity.configuration.cookie_domain,
           secure: Vanity.configuration.cookie_secure,
-          httponly: Vanity.configuration.cookie_httponly
+          httponly: Vanity.configuration.cookie_httponly,
         }
         result[:domain] ||= ::Rails.application.config.session_options[:domain]
         result
@@ -118,13 +118,13 @@ module Vanity
         if symbol && (@object = symbol)
           class << self
             define_method :vanity_identity do
-              @vanity_identity = (String === @object ? @object : @object.id)
+              @vanity_identity = (@object.is_a?(String) ? @object : @object.id)
             end
           end
         else
           class << self
             define_method :vanity_identity do
-              @vanity_identity = @vanity_identity || SecureRandom.hex(16)
+              @vanity_identity ||= SecureRandom.hex(16)
             end
           end
         end
@@ -137,7 +137,8 @@ module Vanity
     module Filters
       # Around filter that sets Vanity.context to controller.
       def vanity_context_filter
-        previous, Vanity.context = Vanity.context, self
+        previous = Vanity.context
+        Vanity.context = self
         yield
       ensure
         Vanity.context = previous
@@ -158,9 +159,9 @@ module Vanity
       # http://example.com/.
       def vanity_query_parameter_filter
         query_params = request.query_parameters
-        if request.get? && query_params[:_vanity]
+        if request.get? && query_params[:_vanity] # rubocop:todo Style/GuardClause
           hashes = Array(query_params.delete(:_vanity))
-          Vanity.playground.experiments.each do |id, experiment|
+          Vanity.playground.experiments.each do |_id, experiment|
             if experiment.respond_to?(:alternatives)
               experiment.alternatives.each do |alt|
                 if hashes.delete(experiment.fingerprint(alt))
@@ -185,15 +186,12 @@ module Vanity
       # Filter to track metrics. Pass _track param along to call track! on that
       # alternative.
       def vanity_track_filter
-        if request.get? && params[:_track]
-          Vanity.track! params[:_track]
-        end
+        Vanity.track! params[:_track] if request.get? && params[:_track]
       end
 
       protected :vanity_context_filter, :vanity_query_parameter_filter, :vanity_reload_filter
     end
 
-
     # Introduces ab_test helper (controllers and views). Similar to the generic
     # ab_test method, with the ability to capture content (applicable to views,
     # see examples).
@@ -221,13 +219,13 @@ module Vanity
       #     <%= count %> features to choose from!
       #   <% end %>
       def ab_test(name, &block)
-        current_request = respond_to?(:request) ? self.request : nil
+        current_request = respond_to?(:request) ? request : nil
         value = Vanity.ab_test(name, current_request)
 
         if block
           content = capture(value, &block)
           if defined?(block_called_from_erb?) && block_called_from_erb?(block)
-             concat(content)
+            concat(content)
           else
             content
           end
@@ -238,20 +236,21 @@ module Vanity
 
       # Generate url with the identity of the current user and the metric to track on click
       def vanity_track_url_for(identity, metric, options = {})
-        options = options.merge(:_identity => identity, :_track => metric)
+        options = options.merge(_identity: identity, _track: metric)
         url_for(options)
       end
 
       # Generate url with the fingerprint for the current Vanity experiment
       def vanity_tracking_image(identity, metric, options = {})
-        options = options.merge(:controller => :vanity, :action => :image, :_identity => identity, :_track => metric)
-        image_tag(url_for(options), :width => "1px", :height => "1px", :alt => "")
+        options = options.merge(controller: :vanity, action: :image, _identity: identity, _track: metric)
+        image_tag(url_for(options), width: "1px", height: "1px", alt: "")
       end
 
       def vanity_js
         return if Vanity.context.vanity_active_experiments.nil? || Vanity.context.vanity_active_experiments.empty?
+
         javascript_tag do
-          render :file => Vanity.template("_vanity.js.erb"), :formats => [:js]
+          render file: Vanity.template("_vanity.js.erb"), formats: [:js]
         end
       end
 
@@ -267,7 +266,7 @@ module Vanity
         end
       end
 
-      def vanity_simple_format(text, html_options={})
+      def vanity_simple_format(text, html_options = {})
         vanity_html_safe(simple_format(text, html_options))
       end
 
@@ -299,7 +298,6 @@ module Vanity
       end
     end
 
-
     # When configuring use_js to true, you must set up a route to
     # add_participant_route.
     #
@@ -323,17 +321,21 @@ module Vanity
         h = {}
         params[:v].split(',').each do |pair|
           exp_id, answer = pair.split('=')
-          exp = Vanity.playground.experiment(exp_id.to_s.to_sym) rescue nil
+          exp = begin
+            Vanity.playground.experiment(exp_id.to_s.to_sym)
+          rescue StandardError
+            nil
+          end
           answer = answer.to_i
 
           if !exp || !exp.alternatives[answer]
             head 404
-            return
+            return # rubocop:todo Lint/NonLocalExitFromIterator
           end
           h[exp] = exp.alternatives[answer].value
         end
 
-        h.each{ |e,a| e.chooses(a, request) }
+        h.each { |e, a| e.chooses(a, request) }
         head 200
       end
     end
@@ -354,16 +356,16 @@ module Vanity
 
       def index
         set_vanity_view_path
-        render :template=>"_report", :content_type=>Mime[:html], :locals=>{
-          :experiments=>Vanity.playground.experiments,
-          :experiments_persisted=>Vanity.playground.experiments_persisted?,
-          :metrics=>Vanity.playground.metrics
+        render template: "_report", content_type: Mime[:html], locals: {
+          experiments: Vanity.playground.experiments,
+          experiments_persisted: Vanity.playground.experiments_persisted?,
+          metrics: Vanity.playground.metrics,
         }
       end
 
       def participant
         set_vanity_view_path
-        render :template=>"_participant", :locals=>{:participant_id => params[:id], :participant_info => Vanity.playground.participant_info(params[:id])}, :content_type=>Mime[:html]
+        render template: "_participant", locals: { participant_id: params[:id], participant_info: Vanity.playground.participant_info(params[:id]) }, content_type: Mime[:html]
       end
 
       def complete
@@ -372,12 +374,12 @@ module Vanity
         alt = exp.alternatives[params[:a].to_i]
         confirmed = params[:confirmed]
         # make the user confirm before completing an experiment
-        if confirmed && confirmed.to_i==alt.id && exp.active?
+        if confirmed && confirmed.to_i == alt.id && exp.active?
           exp.complete!(alt.id)
-          render :template=>"_experiment", :locals=>{:experiment=>exp}
+          render template: "_experiment", locals: { experiment: exp }
         else
           @to_confirm = alt.id
-          render :template=>"_experiment", :locals=>{:experiment=>exp}
+          render template: "_experiment", locals: { experiment: exp }
         end
       end
 
@@ -385,21 +387,21 @@ module Vanity
         set_vanity_view_path
         exp = Vanity.playground.experiment(params[:e].to_sym)
         exp.enabled = false
-        render :template=>"_experiment", :locals=>{:experiment=>exp}
+        render template: "_experiment", locals: { experiment: exp }
       end
 
       def enable
         set_vanity_view_path
         exp = Vanity.playground.experiment(params[:e].to_sym)
         exp.enabled = true
-        render :template=>"_experiment", :locals=>{:experiment=>exp}
+        render template: "_experiment", locals: { experiment: exp }
       end
 
       def chooses
         set_vanity_view_path
         exp = Vanity.playground.experiment(params[:e].to_sym)
         exp.chooses(exp.alternatives[params[:a].to_i].value)
-        render :template=>"_experiment", :locals=>{:experiment=>exp}
+        render template: "_experiment", locals: { experiment: exp }
       end
 
       def reset
@@ -407,7 +409,7 @@ module Vanity
         exp = Vanity.playground.experiment(params[:e].to_sym)
         exp.reset
         flash[:notice] = I18n.t 'vanity.experiment_has_been_reset', name: exp.name
-        render :template=>"_experiment", :locals=>{:experiment=>exp}
+        render template: "_experiment", locals: { experiment: exp }
       end
 
       include AddParticipant
@@ -416,13 +418,12 @@ module Vanity
     module TrackingImage
       def image
         # send image
-        send_file(File.expand_path("../images/x.gif", File.dirname(__FILE__)), :type => 'image/gif', :stream => false, :disposition => 'inline')
+        send_file(File.expand_path("../images/x.gif", File.dirname(__FILE__)), type: 'image/gif', stream: false, disposition: 'inline')
       end
     end
   end
 end
 
-
 # Enhance ActionController with use_vanity, filters and helper methods.
 ActiveSupport.on_load(:action_controller) do
   # Include in controller, add view helper methods.
@@ -434,7 +435,6 @@ ActiveSupport.on_load(:action_controller) do
   end
 end
 
-
 # Include in mailer, add view helper methods.
 ActiveSupport.on_load(:action_mailer) do
   include Vanity::Rails::UseVanityMailer
@@ -448,8 +448,8 @@ if defined?(PhusionPassenger)
     if forked
       begin
         Vanity.playground.reconnect! if Vanity.playground.collecting?
-      rescue Exception=>ex
-        Rails.logger.error "Error reconnecting: #{ex.to_s}"
+      rescue Exception => e # rubocop:todo Lint/RescueException
+        Rails.logger.error "Error reconnecting: #{e}"
       end
     end
   end

data/lib/vanity/frameworks.rb

@@ -1,8 +1,8 @@
-# TODO turn this into a real rails engine jobbie
+# TODO: turn this into a real rails engine jobbie
 # Automatically configure Vanity.
 if defined?(Rails)
   class Plugin < Rails::Railtie # :nodoc:
-    initializer "vanity.require" do |app|
+    initializer "vanity.require" do |_app|
       require 'vanity/frameworks/rails'
 
       Vanity::Rails.load!

data/lib/vanity/helpers.rb

@@ -32,7 +32,7 @@ module Vanity
     #     render action: Vanity.ab_test(:new_page)
     #   end
     # @since 1.2.0
-    def ab_test(name, request=nil, &block)
+    def ab_test(name, request = nil, &block)
       request ||= Vanity.context.respond_to?(:request) ? Vanity.context.request : nil
 
       alternative = Vanity.playground.experiment(name).choose(request)

data/lib/vanity/metric/active_record.rb

@@ -1,6 +1,5 @@
 module Vanity
   class Metric
-
     AGGREGATES = [:average, :minimum, :maximum, :sum]
 
     # Use an ActiveRecord model to get metric data from database table. Also
@@ -35,15 +34,15 @@ module Vanity
     # @since 1.2.0
     # @see Vanity::Metric::ActiveRecord
     def model(class_or_scope, options = nil)
-      ActiveSupport.on_load(:active_record, :yield=>true) do
+      ActiveSupport.on_load(:active_record, yield: true) do
         class_or_scope = class_or_scope.constantize if class_or_scope.is_a?(String)
-        options = options || {}
+        options ||= {}
         conditions = options.delete(:conditions)
 
         @ar_scoped = conditions ? class_or_scope.where(conditions) : class_or_scope
         @ar_aggregate = AGGREGATES.find { |key| options.has_key?(key) }
         @ar_column = options.delete(@ar_aggregate)
-        fail "Cannot use multiple aggregates in a single metric" if AGGREGATES.find { |key| options.has_key?(key) }
+        raise "Cannot use multiple aggregates in a single metric" if AGGREGATES.find { |key| options.has_key?(key) }
 
         @ar_timestamp = options.delete(:timestamp) || :created_at
         @ar_timestamp, @ar_timestamp_table = @ar_timestamp.to_s.split('.').reverse
@@ -51,7 +50,7 @@ module Vanity
 
         @ar_identity_block = options.delete(:identity)
 
-        fail "Unrecognized options: #{options.keys * ", "}" unless options.empty?
+        raise "Unrecognized options: #{options.keys * ', '}" unless options.empty?
 
         @ar_scoped.after_create(self)
         extend ActiveRecord
@@ -63,31 +62,31 @@ module Vanity
     #
     # @since 1.3.0
     module ActiveRecord
-
       # This values method queries the database.
       def values(sdate, edate)
         time = Time.now.in_time_zone
         difference = time.to_date - Date.today
-        sdate = sdate + difference
-        edate = edate + difference
+        sdate += difference
+        edate += difference
 
         grouped = @ar_scoped
-            .where(@ar_timestamp_table => { @ar_timestamp => (sdate.to_time...(edate + 1).to_time) })
-            .group("date(#{@ar_scoped.quoted_table_name}.#{@ar_scoped.connection.quote_column_name(@ar_timestamp)})")
+                  .where(@ar_timestamp_table => { @ar_timestamp => (sdate.to_time...(edate + 1).to_time) })
+                  .group("date(#{@ar_scoped.quoted_table_name}.#{@ar_scoped.connection.quote_column_name(@ar_timestamp)})")
 
-        if @ar_column
-          grouped = grouped.send(@ar_aggregate, @ar_column)
-        else
-          grouped = grouped.count
-        end
+        grouped = if @ar_column
+                    grouped.send(@ar_aggregate, @ar_column)
+                  else
+                    grouped.count
+                  end
 
-        grouped = Hash[grouped.map {|k,v| [k.to_date, v] }]
+        grouped = grouped.map { |k, v| [k.to_date, v] }.to_h
         (sdate..edate).inject([]) { |ordered, date| ordered << (grouped[date] || 0) }
       end
 
       # This track! method stores nothing, but calls the hooks.
       def track!(args = nil)
         return unless @playground.collecting?
+
         call_hooks(*track_args(args))
       end
 
@@ -100,12 +99,15 @@ module Vanity
       # AR model after_create callback notifies all the hooks.
       def after_create(record)
         return unless @playground.collecting?
+
         count = @ar_column ? (record.send(@ar_column) || 0) : 1
 
-        identity = Vanity.context.vanity_identity rescue nil
-        identity ||= if @ar_identity_block
-          @ar_identity_block.call(record)
+        identity = begin
+          Vanity.context.vanity_identity
+        rescue StandardError
+          nil
         end
+        identity ||= (@ar_identity_block.call(record) if @ar_identity_block)
 
         call_hooks record.send(@ar_timestamp), identity, [count] if count > 0 && @ar_scoped.exists?(record.id)
       end

data/lib/vanity/metric/base.rb

@@ -1,5 +1,4 @@
 module Vanity
-
   # A metric is an object that implements two methods: +name+ and +values+. It
   # can also respond to addition methods (+track!+, +bounds+, etc), these are
   # optional.
@@ -10,7 +9,6 @@ module Vanity
   #
   # @since 1.1.0
   class Metric
-
     # These methods are available when defining a metric in a file loaded
     # from the +experiments/metrics+ directory.
     #
@@ -20,22 +18,22 @@ module Vanity
     #     description "Most boring metric ever"
     #   end
     module Definition
-
       attr_reader :playground
 
       # Defines a new metric, using the class Vanity::Metric.
       def metric(name, &block)
-        fail "Metric #{@metric_id} already defined in playground" if playground.metrics[@metric_id]
+        raise "Metric #{@metric_id} already defined in playground" if playground.metrics[@metric_id]
+
         metric = Metric.new(playground, name.to_s, @metric_id)
         metric.instance_eval(&block)
         playground.metrics[@metric_id] = metric
       end
 
       def new_binding(playground, id)
-        @playground, @metric_id = playground, id
+        @playground = playground
+        @metric_id = id
         binding
       end
-
     end
 
     # Startup metrics for pirates. AARRR stands for:
@@ -46,7 +44,6 @@ module Vanity
     # * Revenue
     # Read more: http://500hats.typepad.com/500blogs/2007/09/startup-metrics.html
     class << self
-
       # Helper method to return description for a metric.
       #
       # A metric object may have a +description+ method that returns a detailed
@@ -68,7 +65,7 @@ module Vanity
       # @example
       #   upper = Vanity::Metric.bounds(metric).last
       def bounds(metric)
-        metric.respond_to?(:bounds) && metric.bounds || [nil, nil]
+        (metric.respond_to?(:bounds) && metric.bounds) || [nil, nil]
       end
 
       # Returns data set for a given date range. The data set is an array of
@@ -92,37 +89,37 @@ module Vanity
 
       # Playground uses this to load metric definitions.
       def load(playground, stack, file)
-        fail "Circular dependency detected: #{stack.join('=>')}=>#{file}" if stack.include?(file)
+        raise "Circular dependency detected: #{stack.join('=>')}=>#{file}" if stack.include?(file)
+
         source = File.read(file)
         stack.push file
         id = File.basename(file, ".rb").downcase.gsub(/\W/, "_").to_sym
         context = Object.new
         context.instance_eval do
           extend Definition
-          metric = eval(source, context.new_binding(playground, id), file)
-          fail NameError.new("Expected #{file} to define metric #{id}", id) unless playground.metrics[id]
+          metric = eval(source, context.new_binding(playground, id), file) # rubocop:todo Security/Eval
+          raise NameError.new("Expected #{file} to define metric #{id}", id) unless playground.metrics[id]
+
           metric
         end
-      rescue
+      rescue StandardError
         error = NameError.exception($!.message, id)
         error.set_backtrace $!.backtrace
         raise error
       ensure
         stack.pop
       end
-
     end
 
-
     # Takes playground (need this to access Redis), friendly name and optional
     # id (can infer from name).
     def initialize(playground, name, id = nil)
-      @playground, @name = playground, name.to_s
+      @playground = playground
+      @name = name.to_s
       @id = (id || name.to_s.downcase.gsub(/\W+/, '_')).to_sym
       @hooks = []
     end
 
-
     # -- Tracking --
 
     # Called to track an action associated with this metric. Most common is not
@@ -135,9 +132,10 @@ module Vanity
     #   foo_and_bar.track! [5,11]
     def track!(args = nil)
       return unless @playground.collecting?
+
       timestamp, identity, values = track_args(args)
       connection.metric_track @id, timestamp, identity, values
-      @playground.logger.info "vanity: #{@id} with value #{values.join(", ")}"
+      @playground.logger.info "vanity: #{@id} with value #{values.join(', ')}"
       call_hooks timestamp, identity, values
     end
 
@@ -152,7 +150,11 @@ module Vanity
       when Numeric
         values = [args]
       end
-      identity ||= Vanity.context.vanity_identity rescue nil
+      identity ||= begin
+        Vanity.context.vanity_identity
+      rescue StandardError
+        nil
+      end
       [timestamp || Time.now, identity, values || [1]]
     end
     protected :track_args
@@ -183,12 +185,11 @@ module Vanity
     def bounds
     end
 
-
     #  -- Reporting --
 
     # Human readable metric name. All metrics must implement this method.
     attr_reader :name
-    alias :to_s :name
+    alias to_s name
 
     # Human readable description. Use two newlines to break paragraphs.
     attr_writer :description
@@ -218,7 +219,6 @@ module Vanity
       connection.get_metric_last_update_at(@id)
     end
 
-
     # -- Storage --
 
     def destroy!
@@ -235,9 +235,8 @@ module Vanity
 
     def call_hooks(timestamp, identity, values)
       @hooks.each do |hook|
-        hook.call @id, timestamp, values.first || 1, :identity=>identity
+        hook.call @id, timestamp, values.first || 1, identity: identity
       end
     end
-
   end
 end

data/lib/vanity/metric/google_analytics.rb

@@ -1,6 +1,5 @@
 module Vanity
   class Metric
-
     # Use Google Analytics metric. Note: you must +require "garb"+ before
     # vanity.
     #
@@ -17,13 +16,13 @@ module Vanity
     # @see Vanity::Metric::GoogleAnalytics
     def google_analytics(web_property_id, *args)
       require "garb"
-      options = Hash === args.last ? args.pop : {}
+      options = args.last.is_a?(Hash) ? args.pop : {}
       metric = args.shift || :pageviews
       @ga_resource = Vanity::Metric::GoogleAnalytics::Resource.new(web_property_id, metric)
-      @ga_mapper = options[:mapper] ||= lambda { |entry| entry.send(@ga_resource.metrics.elements.first).to_i }
+      @ga_mapper = options[:mapper] ||= ->(entry) { entry.send(@ga_resource.metrics.elements.first).to_i }
       extend GoogleAnalytics
     rescue LoadError
-      fail LoadError, "Google Analytics metrics require Garb, please gem install garb first"
+      raise LoadError, "Google Analytics metrics require Garb, please gem install garb first"
     end
 
     # Calling google_analytics method on a metric extends it with these modules,
@@ -31,19 +30,18 @@ module Vanity
     #
     # @since 1.3.0
     module GoogleAnalytics
-
       # Returns values from GA using parameters specified by prior call to
       # google_analytics.
       def values(from, to)
-        data = @ga_resource.results(from, to).inject({}) do |hash,entry|
-          hash.merge(entry.date=>@ga_mapper.call(entry))
+        data = @ga_resource.results(from, to).inject({}) do |hash, entry|
+          hash.merge(entry.date => @ga_mapper.call(entry))
         end
         (from..to).map { |day| data[day.strftime('%Y%m%d')] || 0 }
       end
 
       # Hooks not supported for GA metrics.
       def hook
-        fail "Cannot use hooks with Google Analytics methods"
+        raise "Cannot use hooks with Google Analytics methods"
       end
 
       # Garb report.
@@ -77,7 +75,6 @@ module Vanity
           Garb::ReportResponse.new(send_request_for_body).results
         end
       end
-
     end
   end
 end

data/lib/vanity/metric/remote.rb

@@ -3,7 +3,6 @@ require "cgi"
 
 module Vanity
   class Metric
-
     # Specifies the base URL to use for a remote metric. For example:
     #   metric :sandbox do
     #     remote "http://api.vanitydash.com/metrics/sandbox"
@@ -26,9 +25,9 @@ module Vanity
     # - Set values by their index using +values[0]+, +values[1]+, etc or
     # - Set values by series name using +values[foo]+, +values[bar]+, etc.
     module Remote
-
       def track!(args = nil)
         return unless @playground.collecting?
+
         timestamp, identity, values = track_args(args)
         params = ["metric=#{CGI.escape @id.to_s}", "timestamp=#{CGI.escape timestamp.httpdate}"]
         params << "identity=#{CGI.escape identity.to_s}" if identity
@@ -36,9 +35,9 @@ module Vanity
         params << @remote_url.query if @remote_url.query
         @mutex.synchronize do
           @http ||= Net::HTTP.start(@remote_url.host, @remote_url.port)
-          @http.request Net::HTTP::Post.new(@remote_url.path, "Content-Type"=>"application/x-www-form-urlencoded"), params.join("&")
+          @http.request Net::HTTP::Post.new(@remote_url.path, "Content-Type" => "application/x-www-form-urlencoded"), params.join("&")
         end
-      rescue Timeout::Error, StandardError
+      rescue Timeout::Error, StandardError # rubocop:todo Lint/ShadowedException
         @playground.logger.error "Error sending data for metric #{name}: #{$!}"
         @http = nil
       ensure
@@ -47,7 +46,6 @@ module Vanity
 
       # "Don't worry, be crappy. Revolutionary means you ship and then test."
       # -- Guy Kawazaki
-
     end
   end
 end