divisio 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 8aae102437166c67d6dc2dfe8277969afee58978
-  data.tar.gz: 34d89f2cf876329fa57d519c1ee76cdd17d6666f
+  metadata.gz: cd36702ea18fb40ba9952362c384f24cb0a41da1
+  data.tar.gz: 6a12c149173875cf9d9cade5705c447f114dd792
 SHA512:
-  metadata.gz: afa5878c8dae28f8848b3f095fe55240931c4b21711109bd033e3761d40934ae7c292f76d201c87ddb5b40809c2467fb973c071499eb8eacd48731e33936555c
-  data.tar.gz: 9822d1d7741be88a91cb1725be4af78eb343b34c5f2481559763fb1099ef5f6b5dcd326a57f16ff0f7bc7126133e2e42a2467c7fd494e0f200fcb2e6cc5c5771
+  metadata.gz: 70781a34c9383b36b4e68b7cf2bc12af388cfb8db71a92ba3b97c6d4686a258bfafe823f449bff2e12fcd595b6242538cc253130ac550715e41e239b637bae7a
+  data.tar.gz: f038c089575bbb2ed3b61f0024f35c25a928a7f8a543f9ec69d3de647d3e1a0bc93ffbeda957f52b57a70084fc07535408251ffd7d17f82574629ad247b21769

data/.rubocop.yml

@@ -0,0 +1,8 @@
+Metrics/LineLength:
+  Max: 116
+
+Style/Documentation:
+  Enabled: false
+
+Style/ModuleFunction:
+  EnforcedStyle: 'extend_self'

data/Gemfile

@@ -2,12 +2,3 @@ source 'https://rubygems.org'
 
 # Specify your gem's dependencies in divisio.gemspec
 gemspec
-
-group :development, :test do
-  gem 'pry-plus'
-  gem 'mongoid'
-end
-
-group :test do
-  gem 'rspec'
-end

data/README.md

@@ -43,7 +43,7 @@ Divisio.new.split(experiment_name, variants, identity) # ==>> 1
 
 ### Mongoid adaper
 
-_Requires mongoid v4.0.0 or greater_
+_Requires mongoid v5.0.0 or greater_
 
 This adapter will persist the experiment name, identifier, and variant information in a MongoDb collection called `experiments`. Note: The variant returned will be cast to a string.
 

data/divisio.gemspec

@@ -8,8 +8,8 @@ Gem::Specification.new do |spec|
   spec.version       = Divisio::VERSION
   spec.authors       = ['Dragos Miron']
   spec.email         = ['tech@simplybusiness.co.uk']
-  spec.summary       = %q{Provides a splitting framework similar to AB testing}
-  spec.description   = %q{Provides a splitting framework similar to AB testing}
+  spec.summary       = 'Provides a splitting framework similar to AB testing'
+  spec.description   = 'Provides a splitting framework similar to AB testing'
   spec.homepage      = 'http://www.simplybusiness.co.uk'
   spec.license       = 'MIT'
 
@@ -17,4 +17,8 @@ Gem::Specification.new do |spec|
   spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
+
+  spec.add_development_dependency 'mongoid', '~> 6.0'
+  spec.add_development_dependency 'rspec', '~> 3.5'
+  spec.add_development_dependency 'rubocop', '~> 0.46'
 end

data/lib/divisio.rb

@@ -4,7 +4,6 @@ require 'divisio/modulo_algorithm'
 require 'divisio/mongoid_adapter' if defined? Mongoid
 
 class Divisio
-
   @default_adapter = Divisio::NoPersistenceAdapter
   class << self
     attr_accessor :default_adapter

data/lib/divisio/base_adapter.rb

@@ -3,10 +3,10 @@ class Divisio
     extend self
 
     def split(experiment_name, variants, identity)
-      ModuloAlgorithm.new(experiment_name.to_s+identity.to_s, variants).calc
+      ModuloAlgorithm.new(experiment_name.to_s + identity.to_s, variants).calc
     end
 
-    def delete_experiment_for_identity(identity, experiment_name)
+    def delete_experiment_for_identity(_identity, _experiment_name)
       raise NotImplementedError
     end
   end

data/lib/divisio/modulo_algorithm.rb

@@ -1,6 +1,5 @@
 class Divisio
   class ModuloAlgorithm
-
     attr_reader :key, :variants
     private :key, :variants
 

data/lib/divisio/mongoid_adapter.rb

@@ -23,7 +23,7 @@ class Divisio
       experiment_object = Experiment.where(identifier: identity, name: experiment_name).first
 
       return experiment_object.destroy if experiment_object
-      return false
+      false
     end
   end
 end

data/lib/divisio/mongoid_adapter/experiment.rb

@@ -1,7 +1,6 @@
 class Divisio
   module MongoidAdapter
     class Experiment
-
       include Mongoid::Document
       include Mongoid::Timestamps
 
@@ -12,8 +11,7 @@ class Divisio
       field :identifier, type: String
       field :variant, type: String
 
-      index({ name: 1, identifier: 1}, { unique: true})
-
+      index({ name: 1, identifier: 1 }, unique: true)
     end
   end
 end

data/lib/divisio/version.rb

@@ -1,3 +1,3 @@
 class Divisio
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'.freeze
 end

data/mongoid.yml

@@ -1,8 +1,6 @@
 test:
-  sessions:
+  clients:
     default:
       database: divisio_test
       hosts:
         - localhost:27017
-      options:
-        <%= "safe: true" if Mongoid::VERSION =~ /^3/%>

data/spec/divisio/base_adapter_spec.rb

@@ -1,4 +1,3 @@
 describe Divisio::BaseAdapter do
-
   it_behaves_like 'a base adapter'
 end

data/spec/divisio/modulo_algorithm_spec.rb

@@ -1,6 +1,6 @@
 describe Divisio::ModuloAlgorithm do
   describe '#calc' do
-    let(:variants) {  }
+    let(:variants) {}
 
     context 'when there is one variant provided' do
       let(:variants) { 1 }
@@ -11,9 +11,9 @@ describe Divisio::ModuloAlgorithm do
     end
 
     context 'when there are multiple variants provided' do
-      let(:variants) { ['1', '2', '3'] }
+      let(:variants) { %w(1 2 3) }
 
-      { blah8: '1', blah4: '2', blah0: '3'}.each_pair do |key, expected_variant|
+      { blah8: '1', blah4: '2', blah0: '3' }.each_pair do |key, expected_variant|
         it "returns the variant based on the key provided: #{key}" do
           expect(Divisio::ModuloAlgorithm.new(key, variants).calc).to eq(expected_variant)
         end
@@ -23,7 +23,7 @@ describe Divisio::ModuloAlgorithm do
     context 'when there are multiple weighted variants provided' do
       let(:variants) { { a: 1, b: 2, c: 3 } }
 
-      { blah8: :a, blah4: :b, blah0: :c}.each_pair do |key, expected_variant|
+      { blah8: :a, blah4: :b, blah0: :c }.each_pair do |key, expected_variant|
         it "returns the variant based on the key provided and weight: #{key}" do
           expect(Divisio::ModuloAlgorithm.new(key, variants).calc).to eq(expected_variant)
         end

data/spec/divisio/mongoid_adapter/experiment_spec.rb

@@ -1,36 +1,34 @@
 describe Divisio::MongoidAdapter::Experiment do
   describe '#save' do
-
-    let(:required_fields) { {:name => 'experiment', :identifier => 'hash', :variant => '2'} }
+    let(:required_fields) { { name: 'experiment', identifier: 'hash', variant: '2' } }
     subject { Divisio::MongoidAdapter::Experiment.new }
 
     context 'all fields are present' do
       it 'gets saved to the database' do
         subject.attributes = required_fields
 
-        expect{ subject.save }.to change{ described_class.count }.from(0).to(1)
+        expect { subject.save }.to change { described_class.count }.from(0).to(1)
       end
     end
 
     context 'missing fields' do
       it 'does not get saved if identifier is missing' do
-        subject.attributes = required_fields.tap{ |h| h.delete(:identifier) }
+        subject.attributes = required_fields.tap { |h| h.delete(:identifier) }
         expect(subject.save).to be_falsey
       end
 
       it 'does not get saved if name is missing' do
-        subject.attributes = required_fields.tap{ |h| h.delete(:name) }
+        subject.attributes = required_fields.tap { |h| h.delete(:name) }
         expect(subject.save).to be_falsey
       end
 
       it 'does not get saved if variant is missing' do
-        subject.attributes = required_fields.tap{ |h| h.delete(:variant) }
+        subject.attributes = required_fields.tap { |h| h.delete(:variant) }
         expect(subject.save).to be_falsey
       end
     end
 
     context 'uniqueness' do
-
       it 'does not save second object with the same name and identifier' do
         subject.attributes = required_fields
         subject.save
@@ -39,16 +37,16 @@ describe Divisio::MongoidAdapter::Experiment do
         required_fields[:variant] = '3'
         new_object = described_class.new(required_fields)
 
-        expect{ new_object.save }.to_not change(described_class, :count)
+        expect { new_object.save }.to_not change(described_class, :count)
       end
 
       it 'does not save second object in case of race condition because of mongo index' do
         described_class.create_indexes
-        Mongoid.default_session[:divisio_mongoid_adapter_experiments].insert(required_fields)
+        Mongoid::Clients.default[:divisio_mongoid_adapter_experiments].insert_one(required_fields)
         expect(described_class.count).to eq(1)
 
         collection = described_class.collection
-        expect{collection.insert(required_fields)}.to raise_exception(Moped::Errors::OperationFailure)
+        expect { collection.insert_one(required_fields) }.to raise_exception(Mongo::Error::OperationFailure)
       end
     end
   end

data/spec/divisio/mongoid_adapter_spec.rb

@@ -4,12 +4,11 @@ describe Divisio::MongoidAdapter do
   let(:identity)   { 'identity' }
 
   describe '::split' do
-    subject{ described_class.split(experiment, variants, identity) }
+    subject { described_class.split(experiment, variants, identity) }
 
     it_behaves_like 'a base adapter'
 
     context 'new record' do
-
       it 'saves the experiment to the database' do
         expect_any_instance_of(Divisio::MongoidAdapter::Experiment).to receive(:save)
         subject
@@ -21,7 +20,6 @@ describe Divisio::MongoidAdapter do
     end
 
     context 'old record' do
-
       before do
         Divisio::MongoidAdapter::Experiment.create(name: experiment, identifier: identity, variant: 'random')
       end
@@ -38,7 +36,7 @@ describe Divisio::MongoidAdapter do
   end
 
   describe '::delete_experiment_for_identity' do
-    subject{ described_class.delete_experiment_for_identity(identity, experiment) }
+    subject { described_class.delete_experiment_for_identity(identity, experiment) }
 
     context 'record exists in the database' do
       before do
@@ -46,7 +44,7 @@ describe Divisio::MongoidAdapter do
       end
 
       it 'deletes the record' do
-        expect{subject}.to change { Divisio::MongoidAdapter::Experiment.count }.from(1).to(0)
+        expect { subject }.to change { Divisio::MongoidAdapter::Experiment.count }.from(1).to(0)
       end
 
       it 'returns true' do

data/spec/divisio/no_persistence_adapter_spec.rb

@@ -1,4 +1,3 @@
 describe Divisio::NoPersistenceAdapter do
-
   it_behaves_like 'a base adapter'
 end

data/spec/spec_helper.rb

@@ -1,6 +1,6 @@
 ENV['RACK_ENV'] = 'test'
 
-Dir['./spec/support/**/*.rb'].sort.each { |f| require f}
+Dir['./spec/support/**/*.rb'].sort.each { |f| require f }
 
 require 'mongoid'
 require 'divisio'
@@ -9,6 +9,6 @@ Mongoid.load!('mongoid.yml')
 
 RSpec.configure do |config|
   config.before(:each) do
-    Mongoid::Sessions.default.collections.each(&:drop)
+    Mongoid.purge!
   end
 end

data/spec/support/shared_examples_for_base_adapter.rb

@@ -1,10 +1,9 @@
 shared_examples_for 'a base adapter' do
-
   describe '::split' do
     let(:experiment) { 'experiment' }
-    let(:variants)   { ['1', '2', '3'] }
+    let(:variants)   { %w(1 2 3) }
     let(:identity)   { 'identity' }
-    subject{ described_class.split(experiment, variants, identity) }
+    subject { described_class.split(experiment, variants, identity) }
 
     it 'returns a variant for the given experiment and identity' do
       expect(subject).to eq('3')

metadata

@@ -1,15 +1,57 @@
 --- !ruby/object:Gem::Specification
 name: divisio
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Dragos Miron
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2015-02-13 00:00:00.000000000 Z
-dependencies: []
+date: 2017-01-04 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
+  name: mongoid
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '6.0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '6.0'
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '3.5'
+- !ruby/object:Gem::Dependency
+  name: rubocop
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.46'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '0.46'
 description: Provides a splitting framework similar to AB testing
 email:
 - tech@simplybusiness.co.uk
@@ -19,6 +61,7 @@ extra_rdoc_files: []
 files:
 - ".gitignore"
 - ".rspec"
+- ".rubocop.yml"
 - Gemfile
 - LICENSE.txt
 - README.md
@@ -59,7 +102,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 2.0.14
+rubygems_version: 2.4.5.1
 signing_key: 
 specification_version: 4
 summary: Provides a splitting framework similar to AB testing
@@ -72,4 +115,3 @@ test_files:
 - spec/divisio_spec.rb
 - spec/spec_helper.rb
 - spec/support/shared_examples_for_base_adapter.rb
-has_rdoc: