ab-split 1.0.0 → 1.0.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 72275edbc6ab7af7f773bc30f4c585a6166e94ccdd721f2c15c497e9c594f8d8
-  data.tar.gz: 241fb4233f417cb9a74e8b1f9e49bd96d33178b594c3de98dd33f162b175833a
+  metadata.gz: 47759e39db6fc259f34e839328343c53b7ca8e1eb824d58e6f2ddc900a170ea6
+  data.tar.gz: ab97c9dab5dbe1161def9884f9b89028c6f14f8e4c5e8eb4b0d9fc6f92621b16
 SHA512:
-  metadata.gz: defdeceeae8355a5041848c5336908cc29951feb4faaae3067c135d148b5000b9e7f8de4576606f8aaf7492dc8117d65b1409fdc0309725d8d341394e5463437
-  data.tar.gz: 8a319c4a7b15a60dd9ae74e46d237344c8d384269fdddc455d44fc107ca50ffbf565ad7adccb422b67483c912c75525ef1fee43ae742ffc7d19c0322f018e92a
+  metadata.gz: b3e45a591fa2c72bcbebd4ea5d2900ede17adba222f683477baaf3a2b5da0c3826e0cc3e24459eb5610e02e086ae4ac625f90914501c2442fa73b69057127e2e
+  data.tar.gz: 878332197608d922d8bc794a5506cbe26190963f145009cf7346bae968ec372ff52dffdcb78ff00b5670908a21ab1ea9667f09a3777b593c9ee48e3214e7194d

data/ab-split.gemspec

@@ -8,18 +8,18 @@ Gem::Specification.new do |s|
   s.version     = Split::VERSION
   s.platform    = Gem::Platform::RUBY
   s.licenses    = ['MIT']
-  s.authors     = ["aibotyu"]
-  s.email       = ["284894567@qq.com"]
-  s.homepage    = "https://github.com/WeiGangqiang/split"
+  s.authors       = ["aibotyu"]
+  s.email         = ["284894567@qq.com"]
+  s.homepage    = "https://github.com/splitrb/split"
   s.summary     = "Rack based split testing framework"
 
   s.metadata    = {
-   "homepage_uri" => "https://github.com/WeiGangqiang/split",
-   "changelog_uri" => "https://github.com/WeiGangqiang/split/blob/master/CHANGELOG.md",
-   "source_code_uri" => "https://github.com/WeiGangqiang/split",
-   "bug_tracker_uri" => "https://github.com/WeiGangqiang/split/issues",
-   "wiki_uri" => "https://github.com/WeiGangqiang/split/wiki",
-   "mailing_list_uri" => "https://github.com/WeiGangqiang/split/split-ruby"
+   "homepage_uri" => "https://github.com/splitrb/split",
+   "changelog_uri" => "https://github.com/splitrb/split/blob/master/CHANGELOG.md",
+   "source_code_uri" => "https://github.com/splitrb/split",
+   "bug_tracker_uri" => "https://github.com/splitrb/split/issues",
+   "wiki_uri" => "https://github.com/splitrb/split/wiki",
+   "mailing_list_uri" => "https://groups.google.com/d/forum/split-ruby"
  }
 
   s.required_ruby_version = '>= 1.9.3'

data/lib/split/helper.rb

@@ -9,7 +9,6 @@ module Split
       begin
         experiment = ExperimentCatalog.find_or_initialize(metric_descriptor, control, *alternatives)
         alternative = if Split.configuration.enabled && !exclude_visitor?
-          experiment.save
           raise(Split::InvalidExperimentsFormatError) unless (Split.configuration.experiments || {}).fetch(experiment.name.to_sym, {})[:combined_experiments].nil?
           trial = Trial.new(:user => ab_user, :experiment => experiment,
               :override => override_alternative(experiment.name), :exclude => exclude_visitor?,
@@ -39,6 +38,11 @@ module Split
       end
     end
 
+    def ab_test_config(metric_descriptor, control = nil, *alternatives)
+      experiment = ExperimentCatalog.find_or_initialize(metric_descriptor, control, *alternatives)
+      experiment.save
+    end
+
     def reset!(experiment)
       ab_user.delete(experiment.key)
     end

data/lib/split/trial.rb

@@ -49,7 +49,7 @@ module Split
     # method is guaranteed to only run once, and will skip the alternative choosing process if run
     # a second time.
     def choose!(context = nil)
-      @user.cleanup_old_experiments!
+      @user.cleanup_old_experiments! self.experiment
       # Only run the process once
       return alternative if @alternative_choosen
 

data/lib/split/user.rb

@@ -12,10 +12,10 @@ module Split
       @cleaned_up = false
     end
 
-    def cleanup_old_experiments!
+    def cleanup_old_experiments! experiment
       return if @cleaned_up
       keys_without_finished(user.keys).each do |key|
-        experiment = ExperimentCatalog.find key_without_version(key)
+        return unless experiment.name == key_without_version(key)
         if experiment.nil? || experiment.has_winner? || experiment.start_time.nil?
           user.delete key
           user.delete Experiment.finished_key(key)

data/lib/split/version.rb

@@ -2,6 +2,6 @@
 module Split
   MAJOR   = 1
   MINOR   = 0
-  PATCH   = 0
+  PATCH   = 1
   VERSION = [MAJOR, MINOR, PATCH].join('.')
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: ab-split
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.0.1
 platform: ruby
 authors:
 - aibotyu
@@ -263,16 +263,16 @@ files:
 - spec/support/cookies_mock.rb
 - spec/trial_spec.rb
 - spec/user_spec.rb
-homepage: https://github.com/WeiGangqiang/split
+homepage: https://github.com/splitrb/split
 licenses:
 - MIT
 metadata:
-  homepage_uri: https://github.com/WeiGangqiang/split
-  changelog_uri: https://github.com/WeiGangqiang/split/blob/master/CHANGELOG.md
-  source_code_uri: https://github.com/WeiGangqiang/split
-  bug_tracker_uri: https://github.com/WeiGangqiang/split/issues
-  wiki_uri: https://github.com/WeiGangqiang/split/wiki
-  mailing_list_uri: https://github.com/WeiGangqiang/split/split-ruby
+  homepage_uri: https://github.com/splitrb/split
+  changelog_uri: https://github.com/splitrb/split/blob/master/CHANGELOG.md
+  source_code_uri: https://github.com/splitrb/split
+  bug_tracker_uri: https://github.com/splitrb/split/issues
+  wiki_uri: https://github.com/splitrb/split/wiki
+  mailing_list_uri: https://groups.google.com/d/forum/split-ruby
 post_install_message: 
 rdoc_options: []
 require_paths: