flipper-mongo 0.3.0 → 0.5.0

data/Gemfile

@@ -5,8 +5,6 @@ gem 'rake'
 gem 'rspec'
 gem 'timecop'
 gem 'bson_ext'
-gem 'mongo'
-gem 'rack-test'
 
 group(:guard) do
   gem 'guard'

data/README.md

@@ -1,6 +1,6 @@
 # Flipper Mongo
 
-A mongo adapter for [Flipper](https://github.com/jnunemaker/flipper), the feature flipping gems.
+A [MongoDB](https://github.com/mongodb/mongo-ruby-driver) adapter for [Flipper](https://github.com/jnunemaker/flipper).
 
 ## Installation
 
@@ -20,12 +20,62 @@ Or install it yourself with:
 
 ```ruby
 require 'flipper/adapters/mongo'
-collection = Mongo::Connection.new.db('testing')['flipper']
-adapter = Flipper::Adapters::Mongo.new(collection, 'features')
+collection = Mongo::MongoClient.new.db('testing')['flipper']
+adapter = Flipper::Adapters::Mongo.new(collection)
 flipper = Flipper.new(adapter)
 # profit...
 ```
 
+## Internals
+
+Each feature is stored in a document, which means getting a feature is single query.
+
+```ruby
+require 'flipper/adapters/mongo'
+collection = Mongo::MongoClient.new.db('testing')['flipper']
+adapter = Flipper::Adapters::Mongo.new(collection)
+flipper = Flipper.new(adapter)
+
+# Register a few groups.
+Flipper.register(:admins) { |thing| thing.admin? }
+Flipper.register(:early_access) { |thing| thing.early_access? }
+
+# Create a user class that has flipper_id instance method.
+User = Struct.new(:flipper_id)
+
+flipper[:stats].enable
+flipper[:stats].enable flipper.group(:admins)
+flipper[:stats].enable flipper.group(:early_access)
+flipper[:stats].enable User.new('25')
+flipper[:stats].enable User.new('90')
+flipper[:stats].enable User.new('180')
+flipper[:stats].enable flipper.random(15)
+flipper[:stats].enable flipper.actors(45)
+
+flipper[:search].enable
+
+puts 'all docs in collection'
+pp collection.find.to_a
+# all docs in collection
+# [{"_id"=>"stats",
+#   "actors"=>["25", "90", "180"],
+#   "boolean"=>"true",
+#   "groups"=>["admins", "early_access"],
+#   "percentage_of_actors"=>"45",
+#   "percentage_of_random"=>"15"},
+#  {"_id"=>"flipper_features", "features"=>["stats", "search"]},
+#  {"_id"=>"search", "boolean"=>"true"}]
+
+puts 'flipper get of feature'
+pp adapter.get(flipper[:stats])
+# flipper get of feature
+# {:boolean=>"true",
+#  :groups=>#<Set: {"admins", "early_access"}>,
+#  :actors=>#<Set: {"25", "90", "180"}>,
+#  :percentage_of_actors=>"45",
+#  :percentage_of_random=>"15"}
+```
+
 ## Contributing
 
 1. Fork it

data/examples/basic.rb

@@ -0,0 +1,27 @@
+require 'pathname'
+require 'logger'
+
+root_path = Pathname(__FILE__).dirname.join('..').expand_path
+lib_path  = root_path.join('lib')
+$:.unshift(lib_path)
+
+require 'flipper/adapters/mongo'
+collection = Mongo::MongoClient.new.db('testing')['flipper']
+adapter = Flipper::Adapters::Mongo.new(collection)
+flipper = Flipper.new(adapter)
+
+flipper[:stats].enable
+
+if flipper[:stats].enabled?
+  puts "Enabled!"
+else
+  puts "Disabled!"
+end
+
+flipper[:stats].disable
+
+if flipper[:stats].enabled?
+  puts "Enabled!"
+else
+  puts "Disabled!"
+end

data/examples/internals.rb

@@ -0,0 +1,52 @@
+require 'pp'
+require 'pathname'
+require 'logger'
+
+root_path = Pathname(__FILE__).dirname.join('..').expand_path
+lib_path  = root_path.join('lib')
+$:.unshift(lib_path)
+
+require 'flipper/adapters/mongo'
+collection = Mongo::MongoClient.new.db('testing')['flipper']
+adapter = Flipper::Adapters::Mongo.new(collection)
+flipper = Flipper.new(adapter)
+
+# Register a few groups.
+Flipper.register(:admins) { |thing| thing.admin? }
+Flipper.register(:early_access) { |thing| thing.early_access? }
+
+# Create a user class that has flipper_id instance method.
+User = Struct.new(:flipper_id)
+
+flipper[:stats].enable
+flipper[:stats].enable flipper.group(:admins)
+flipper[:stats].enable flipper.group(:early_access)
+flipper[:stats].enable User.new('25')
+flipper[:stats].enable User.new('90')
+flipper[:stats].enable User.new('180')
+flipper[:stats].enable flipper.random(15)
+flipper[:stats].enable flipper.actors(45)
+
+flipper[:search].enable
+
+puts 'all docs in collection'
+pp collection.find.to_a
+# all docs in collection
+# [{"_id"=>"stats",
+#   "actors"=>["25", "90", "180"],
+#   "boolean"=>"true",
+#   "groups"=>["admins", "early_access"],
+#   "percentage_of_actors"=>"45",
+#   "percentage_of_random"=>"15"},
+#  {"_id"=>"flipper_features", "features"=>["stats", "search"]},
+#  {"_id"=>"search", "boolean"=>"true"}]
+puts
+
+puts 'flipper get of feature'
+pp adapter.get(flipper[:stats])
+# flipper get of feature
+# {:boolean=>"true",
+#  :groups=>#<Set: {"admins", "early_access"}>,
+#  :actors=>#<Set: {"25", "90", "180"}>,
+#  :percentage_of_actors=>"45",
+#  :percentage_of_random=>"15"}

data/flipper-mongo.gemspec

@@ -2,17 +2,19 @@
 require File.expand_path('../lib/flipper/adapters/mongo/version', __FILE__)
 
 Gem::Specification.new do |gem|
+  gem.name          = "flipper-mongo"
+  gem.version       = Flipper::Adapters::Mongo::VERSION
   gem.authors       = ["John Nunemaker"]
   gem.email         = ["nunemaker@gmail.com"]
   gem.description   = %q{Mongo adapter for Flipper}
   gem.summary       = %q{Mongo adapter for Flipper}
   gem.homepage      = "http://jnunemaker.github.com/flipper-mongo"
-
-  gem.executables   = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
-  gem.files         = `git ls-files`.split("\n")
-  gem.test_files    = `git ls-files -- {test,spec,features}/*`.split("\n")
-  gem.name          = "flipper-mongo"
   gem.require_paths = ["lib"]
-  gem.version       = Flipper::Adapters::Mongo::VERSION
-  gem.add_dependency 'flipper', '~> 0.4'
+
+  gem.files         = `git ls-files`.split($/)
+  gem.executables   = gem.files.grep(%r{^bin/}).map{ |f| File.basename(f) }
+  gem.test_files    = gem.files.grep(%r{^(test|spec|features)/})
+
+  gem.add_dependency 'flipper', '~> 0.5.0'
+  gem.add_dependency 'mongo', '~> 1.8'
 end

data/lib/flipper-mongo.rb

@@ -1,2 +1 @@
-require 'flipper'
 require 'flipper/adapters/mongo'

data/lib/flipper/adapters/mongo.rb

@@ -1,59 +1,123 @@
 require 'set'
-require 'forwardable'
+require 'flipper'
 require 'mongo'
 
 module Flipper
   module Adapters
     class Mongo
-      extend Forwardable
+      include Flipper::Adapter
+
+      # Private: The key that stores the set of known features.
+      FeaturesKey = :flipper_features
+
+      # Public: The name of the adapter.
+      attr_reader :name
 
       def initialize(collection)
         @collection = collection
-        @update_options = {:safe => true, :upsert => true}
+        @name = :mongo
       end
 
-      def read(key)
-        find_one key
-      end
+      # Public: Gets the values for all gates for a given feature.
+      #
+      # Returns a Hash of Flipper::Gate#key => value.
+      def get(feature)
+        result = {}
+        doc = find(feature.key)
 
-      def write(key, value)
-        update key, {'$set' => {'v' => value.to_s}}
-      end
+        feature.gates.each do |gate|
+          result[gate.key] = case gate.data_type
+          when :boolean, :integer
+            doc[gate.key.to_s]
+          when :set
+            doc.fetch(gate.key.to_s) { Set.new }.to_set
+          else
+            unsupported_data_type gate.data_type
+          end
+        end
 
-      def delete(key)
-        remove key
+        result
       end
 
-      def set_members(key)
-        (find_one(key) || Set.new).to_set
+      # Public: Enables a gate for a given thing.
+      #
+      # feature - The Flipper::Feature for the gate.
+      # gate - The Flipper::Gate to disable.
+      # thing - The Flipper::Type being disabled for the gate.
+      #
+      # Returns true.
+      def enable(feature, gate, thing)
+        case gate.data_type
+        when :boolean, :integer
+          update feature.key, '$set' => {
+            gate.key.to_s => thing.value.to_s,
+          }
+        when :set
+          update feature.key, '$addToSet' => {
+            gate.key.to_s => thing.value.to_s,
+          }
+        else
+          unsupported_data_type gate.data_type
+        end
+
+        true
       end
 
-      def set_add(key, value)
-        update key, {'$addToSet' => {'v' => value.to_s}}
+      # Public: Disables a gate for a given thing.
+      #
+      # feature - The Flipper::Feature for the gate.
+      # gate - The Flipper::Gate to disable.
+      # thing - The Flipper::Type being disabled for the gate.
+      #
+      # Returns true.
+      def disable(feature, gate, thing)
+        case gate.data_type
+        when :boolean
+          remove feature.key
+        when :integer
+          update feature.key, '$set' => {gate.key.to_s => thing.value.to_s}
+        when :set
+          update feature.key, '$pull' => {gate.key.to_s => thing.value.to_s}
+        else
+          unsupported_data_type gate.data_type
+        end
+
+        true
       end
 
-      def set_delete(key, value)
-        update key, {'$pull' => {'v' => value.to_s}}
+      # Public: Adds a feature to the set of known features.
+      def add(feature)
+        update FeaturesKey, '$addToSet' => {'features' => feature.name.to_s}
+        true
       end
 
-      private
+      # Public: The set of known features.
+      def features
+        find(FeaturesKey).fetch('features') { Set.new }.to_set
+      end
 
-      def find_one(key)
-        doc = @collection.find_one(criteria(key))
+      # Private
+      def unsupported_data_type(data_type)
+        raise "#{data_type} is not supported by this adapter"
+      end
 
-        unless doc.nil?
-          doc['v']
-        end
+      # Private
+      def find(key)
+        @collection.find_one(criteria(key)) || {}
       end
 
+      # Private
       def update(key, updates)
-        @collection.update criteria(key), updates, @update_options
+        options = {:upsert => true}
+        @collection.update criteria(key), updates, options
       end
 
+      # Private
       def remove(key)
         @collection.remove criteria(key)
       end
 
+      # Private
       def criteria(key)
         {:_id => key.to_s}
       end

data/lib/flipper/adapters/mongo/version.rb

@@ -1,7 +1,7 @@
 module Flipper
   module Adapters
     class Mongo
-      VERSION = "0.3.0"
+      VERSION = "0.5.0"
     end
   end
 end

data/spec/helper.rb

@@ -1,29 +1,17 @@
 $:.unshift(File.expand_path('../../lib', __FILE__))
 
-require 'pathname'
-require 'logger'
-
-root_path = Pathname(__FILE__).dirname.join('..').expand_path
-lib_path  = root_path.join('lib')
-log_path  = root_path.join('log')
-log_path.mkpath
-
-require 'rubygems'
 require 'bundler'
-
-Bundler.require(:default, :test)
-
+Bundler.setup :default
 require 'flipper-mongo'
 
-Logger.new(log_path.join('test.log'))
-
-require 'support/accessor_helpers'
-
 RSpec.configure do |config|
   config.filter_run :focused => true
   config.alias_example_to :fit, :focused => true
   config.alias_example_to :xit, :pending => true
   config.run_all_when_everything_filtered = true
+  config.fail_fast = true
 
-  config.include AccessorHelpers
+  config.backtrace_clean_patterns = [
+    /rspec-(core|expectations)/,
+  ]
 end

data/spec/mongo_spec.rb

@@ -0,0 +1,15 @@
+require 'helper'
+require 'flipper/adapters/mongo'
+require 'flipper/spec/shared_adapter_specs'
+
+describe Flipper::Adapters::Mongo do
+  let(:collection) { Mongo::MongoClient.new.db('testing')['testing'] }
+
+  subject { described_class.new(collection) }
+
+  before do
+    collection.remove
+  end
+
+  it_should_behave_like 'a flipper adapter'
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: flipper-mongo
 version: !ruby/object:Gem::Version
-  version: 0.3.0
+  version: 0.5.0
   prerelease: 
 platform: ruby
 authors:
@@ -9,7 +9,7 @@ authors:
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-02-06 00:00:00.000000000 Z
+date: 2013-02-16 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: flipper
@@ -18,7 +18,7 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: 0.5.0
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
@@ -26,7 +26,23 @@ dependencies:
     requirements:
     - - ~>
       - !ruby/object:Gem::Version
-        version: '0.4'
+        version: 0.5.0
+- !ruby/object:Gem::Dependency
+  name: mongo
+  requirement: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.8'
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - ~>
+      - !ruby/object:Gem::Version
+        version: '1.8'
 description: Mongo adapter for Flipper
 email:
 - nunemaker@gmail.com
@@ -41,19 +57,14 @@ files:
 - LICENSE
 - README.md
 - Rakefile
+- examples/basic.rb
+- examples/internals.rb
 - flipper-mongo.gemspec
 - lib/flipper-mongo.rb
 - lib/flipper/adapters/mongo.rb
-- lib/flipper/adapters/mongo/document.rb
 - lib/flipper/adapters/mongo/version.rb
-- lib/flipper/adapters/mongo_single_document.rb
-- lib/flipper/middleware/mongo_single_document_query_cache.rb
-- spec/flipper/adapters/mongo/document_spec.rb
-- spec/flipper/adapters/mongo_single_document_spec.rb
-- spec/flipper/adapters/mongo_spec.rb
-- spec/flipper/middleware/mongo_single_document_query_cache_spec.rb
 - spec/helper.rb
-- spec/support/accessor_helpers.rb
+- spec/mongo_spec.rb
 homepage: http://jnunemaker.github.com/flipper-mongo
 licenses: []
 post_install_message: 
@@ -68,7 +79,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1196015138740871413
+      hash: 1536259830562571499
 required_rubygems_version: !ruby/object:Gem::Requirement
   none: false
   requirements:
@@ -77,7 +88,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
       segments:
       - 0
-      hash: 1196015138740871413
+      hash: 1536259830562571499
 requirements: []
 rubyforge_project: 
 rubygems_version: 1.8.23
@@ -85,9 +96,5 @@ signing_key:
 specification_version: 3
 summary: Mongo adapter for Flipper
 test_files:
-- spec/flipper/adapters/mongo/document_spec.rb
-- spec/flipper/adapters/mongo_single_document_spec.rb
-- spec/flipper/adapters/mongo_spec.rb
-- spec/flipper/middleware/mongo_single_document_query_cache_spec.rb
 - spec/helper.rb
-- spec/support/accessor_helpers.rb
+- spec/mongo_spec.rb

data/lib/flipper/adapters/mongo/document.rb

@@ -1,80 +0,0 @@
-require 'set'
-require 'mongo'
-
-module Flipper
-  module Adapters
-    class MongoSingleDocument
-      class Document
-        DefaultId = 'flipper'
-
-        def initialize(collection, options = {})
-          @collection = collection
-          @options = options
-          @id = @options[:id] || DefaultId
-          @source = @options.fetch(:source) { {} }
-          @criteria = {:_id => @id}
-          @mongo_options = {:safe => true, :upsert => true}
-        end
-
-        def read(key)
-          source[key.to_s]
-        end
-
-        def write(key, value)
-          value = value.to_s
-          @collection.update @criteria, {'$set' => {key.to_s => value}}, @mongo_options
-          @source[key.to_s] = value
-        end
-
-        def delete(key)
-          @collection.update @criteria, {'$unset' => {key.to_s => 1}}, @mongo_options
-          @source.delete key.to_s
-        end
-
-        def set_members(key)
-          members = source.fetch(key.to_s) { @source[key.to_s] = Set.new }
-
-          if members.is_a?(Array)
-            @source[key.to_s] = members.to_set
-          else
-            members
-          end
-        end
-
-        def set_add(key, value)
-          value = value.to_s
-          @collection.update @criteria, {'$addToSet' => {key.to_s => value}}, @mongo_options
-          set_members(key.to_s).add(value)
-        end
-
-        def set_delete(key, value)
-          value = value.to_s
-          @collection.update @criteria, {'$pull' => {key.to_s => value}}, @mongo_options
-          set_members(key.to_s).delete(value)
-        end
-
-        def clear
-          @loaded = nil
-          @source.clear
-        end
-
-        def loaded?
-          @loaded == true
-        end
-
-        private
-
-        def source
-          load unless loaded?
-          @source
-        end
-
-        def load
-          @loaded = true
-          @source.clear
-          @source.update @collection.find_one(@criteria) || {}
-        end
-      end
-    end
-  end
-end

data/lib/flipper/adapters/mongo_single_document.rb

@@ -1,55 +0,0 @@
-require 'set'
-require 'forwardable'
-require 'mongo'
-require 'flipper/adapters/mongo/document'
-
-module Flipper
-  module Adapters
-    class MongoSingleDocument
-      extend Forwardable
-
-      def initialize(collection, options = {})
-        @collection = collection
-        @options = options
-        @document_cache = false
-      end
-
-      def_delegators :document, :read, :write, :delete, :set_members, :set_add, :set_delete
-
-      def using_document_cache?
-        @document_cache == true
-      end
-
-      def document_cache=(value)
-        reset_document_cache
-        @document_cache = value
-      end
-
-      def use_document_cache(&block)
-        original = @document_cache
-        @document_cache = true
-        yield
-      ensure
-        @document_cache = original
-      end
-
-      def reset_document_cache
-        @document = nil
-      end
-
-      private
-
-      def document
-        if @document_cache == true
-          @document ||= fresh_document
-        else
-          fresh_document
-        end
-      end
-
-      def fresh_document
-        Document.new(@collection, :id => @options[:id])
-      end
-    end
-  end
-end

data/lib/flipper/middleware/mongo_single_document_query_cache.rb

@@ -1,36 +0,0 @@
-module Flipper
-  module Middleware
-    class MongoSingleDocumentQueryCache
-      class Body
-        def initialize(target, adapter, original)
-          @target   = target
-          @adapter  = adapter
-          @original = original
-        end
-
-        def each(&block)
-          @target.each(&block)
-        end
-
-        def close
-          @target.close if @target.respond_to?(:close)
-        ensure
-          @adapter.document_cache = @original
-        end
-      end
-
-      def initialize(app, adapter)
-        @app = app
-        @adapter = adapter
-      end
-
-      def call(env)
-        original = @adapter.using_document_cache?
-        @adapter.document_cache = true
-
-        status, headers, body = @app.call(env)
-        [status, headers, Body.new(body, @adapter, original)]
-      end
-    end
-  end
-end

data/spec/flipper/adapters/mongo/document_spec.rb

@@ -1,279 +0,0 @@
-require 'helper'
-require 'flipper/adapters/mongo/document'
-
-describe Flipper::Adapters::MongoSingleDocument::Document do
-  subject          { described_class.new(collection, :id => id, :source => source) }
-  let(:collection) { Mongo::Connection.new.db('testing')['testing'] }
-  let(:id)         { described_class::DefaultId }
-  let(:source)     { {} }
-  let(:criteria)   { {:_id => id} }
-  let(:options)    { {:safe => true, :upsert => true} }
-
-  def document
-    collection.find_one(criteria)
-  end
-
-  before do
-    collection.remove(criteria)
-  end
-
-  it "defaults id to flipper" do
-    described_class.new(collection).instance_variable_get("@id").should eq('flipper')
-  end
-
-  it "defaults id to flipper even if nil passed in for id" do
-    described_class.new(collection, :id => nil).instance_variable_get("@id").should eq('flipper')
-  end
-
-  describe "loading document" do
-    before do
-      collection.update(criteria, {'$set' => {'foo' => 'bar', 'people' => ['1', '2', '3']}}, options)
-    end
-
-    it "only happens once" do
-      collection.should_receive(:find_one).with(criteria).once.and_return({})
-      subject.read('foo')
-      subject.set_members('people')
-    end
-
-    it "happens again if document is cleared" do
-      collection.should_receive(:find_one).with(criteria)
-      subject.read('foo')
-      subject.set_members('people')
-
-      subject.clear
-
-      collection.should_receive(:find_one).with(criteria)
-      subject.read('foo')
-      subject.set_members('people')
-    end
-  end
-
-  describe "#read" do
-    context "existing key" do
-      before do
-        source['baz'] = 'wick'
-        collection.update(criteria, {'$set' => {'foo' => 'bar'}}, options)
-        @result = subject.read('foo')
-      end
-
-      it "returns value" do
-        @result.should eq('bar')
-      end
-
-      it "clears and loads source hash" do
-        source.should eq({
-          '_id' => id,
-          'foo' => 'bar',
-        })
-      end
-    end
-
-    context "missing key" do
-      before do
-        collection.update(criteria, {'$set' => {'foo' => 'bar'}}, options)
-        @result = subject.read('apple')
-      end
-
-      it "returns nil" do
-        @result.should be_nil
-      end
-    end
-
-    context "missing document" do
-      before do
-        @result = subject.read('foo')
-      end
-
-      it "returns nil" do
-        @result.should be_nil
-      end
-    end
-  end
-
-  describe "#write" do
-    context "existing key" do
-      before do
-        collection.update(criteria, {'$set' => {'foo' => 'bar'}}, options)
-        subject.write('foo', 'new value')
-      end
-
-      it "sets key" do
-        document.fetch('foo').should eq('new value')
-        source.fetch('foo').should eq('new value')
-      end
-    end
-
-    context "missing key" do
-      before do
-        collection.update(criteria, {'$set' => {'foo' => 'bar'}}, options)
-        subject.write('apple', 'orange')
-      end
-
-      it "sets key" do
-        document.fetch('apple').should eq('orange')
-        source.fetch('apple').should eq('orange')
-      end
-    end
-
-    context "missing document" do
-      before do
-        subject.write('foo', 'bar')
-      end
-
-      it "creates document" do
-        document.should_not be_nil
-      end
-
-      it "sets key" do
-        document.fetch('foo').should eq('bar')
-        source.fetch('foo').should eq('bar')
-      end
-    end
-  end
-
-  describe "#delete" do
-    before do
-      collection.update(criteria, {'$set' => {'foo' => 'bar', 'apple' => 'orange'}}, options)
-      @result = subject.delete('foo')
-    end
-
-    it "removes the key" do
-      document.key?('foo').should be_false
-      source.key?('foo').should be_false
-    end
-
-    it "does not remove other keys" do
-      document.fetch('apple').should eq('orange')
-    end
-  end
-
-  describe "#set_members" do
-    context "existing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3'], 'foo' => 'bar'}}, options)
-        @result = subject.set_members('people')
-      end
-
-      it "returns set" do
-        @result.should eq(Set['1', '2', '3'])
-      end
-
-      it "loads source hash" do
-        source.should eq({
-          '_id'    => id,
-          'people' => Set['1', '2', '3'],
-          'foo'    => 'bar',
-        })
-      end
-    end
-
-    context "missing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3']}}, options)
-        @result = subject.set_members('users')
-      end
-
-      it "returns empty set" do
-        @result.should eq(Set.new)
-      end
-    end
-
-    context "missing document" do
-      it "returns empty set" do
-        subject.set_members('people').should eq(Set.new)
-      end
-    end
-  end
-
-  describe "#set_add" do
-    context "existing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3']}}, options)
-        subject.set_add('people', 4)
-      end
-
-      it "adds value to set" do
-        document.fetch('people').should eq(['1', '2', '3', '4'])
-        source.fetch('people').should eq(Set['1', '2', '3', '4'])
-      end
-    end
-
-    context "missing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3']}}, options)
-        subject.set_add('users', '1')
-      end
-
-      it "adds value to set" do
-        document.fetch('users').should eq(['1'])
-        source.fetch('users').should eq(Set['1'])
-      end
-    end
-
-    context "missing document" do
-      before do
-        subject.set_add('people', '1')
-      end
-
-      it "creates document" do
-        document.should_not be_nil
-      end
-
-      it "adds value to set" do
-        document.fetch('people').should eq(['1'])
-        source.fetch('people').should eq(Set['1'])
-      end
-    end
-  end
-
-  describe "#set_delete" do
-    context "existing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3']}}, options)
-        subject.set_delete 'people', '3'
-      end
-
-      it "removes value to key" do
-        document.fetch('people').should eq(['1', '2'])
-        source.fetch('people').should eq(Set['1', '2'])
-      end
-    end
-
-    context "missing key" do
-      before do
-        collection.update(criteria, {'$set' => {'people' => ['1', '2', '3']}}, options)
-      end
-
-      it "does not error" do
-        expect { subject.set_delete 'foo', '1' }.to_not raise_error
-      end
-    end
-
-    context "missing document" do
-      it "does not error" do
-        expect { subject.set_delete 'foo', 1 }.to_not raise_error
-      end
-    end
-  end
-
-  describe "#clear" do
-    before do
-      collection.update(criteria, {'$set' => {'foo' => 'bar'}}, options)
-      subject.read('foo') # load the source hash
-      subject.clear
-    end
-
-    it "clears the source hash" do
-      source.should be_empty
-    end
-
-    it "does not remove the document" do
-      document.should_not be_empty
-    end
-
-    it "marks the document as not loaded" do
-      subject.loaded?.should be_false
-    end
-  end
-end

data/spec/flipper/adapters/mongo_single_document_spec.rb

@@ -1,95 +0,0 @@
-require 'helper'
-require 'flipper/adapters/mongo_single_document'
-require 'flipper/spec/shared_adapter_specs'
-
-describe Flipper::Adapters::MongoSingleDocument do
-  let(:collection) { Mongo::Connection.new.db('testing')['testing'] }
-  let(:criteria)   { {:_id => id} }
-  let(:id)         { 'flipper' }
-
-  subject { described_class.new(collection, :id => id) }
-
-  before do
-    collection.remove(criteria)
-  end
-
-  def read_key(key)
-    if (doc = collection.find_one(criteria))
-      value = doc[key.to_s]
-
-      if value.is_a?(::Array)
-        value = value.to_set
-      end
-
-      value
-    end
-  end
-
-  def write_key(key, value)
-    value = if value.is_a?(::Set)
-      value.to_a.map(&:to_s)
-    else
-      value.to_s
-    end
-
-    options = {:upsert => true}
-    updates = {'$set' => {key.to_s => value}}
-    collection.update criteria, updates, options
-  end
-
-  context "with cache" do
-    before do
-      subject.document_cache = true
-    end
-
-    it_should_behave_like 'a flipper adapter'
-
-    it "should only query mongo once until reloaded" do
-      collection.should_receive(:find_one).with(criteria).once.and_return({})
-      subject.read('foo')
-      subject.read('foo')
-      subject.read('foo')
-      subject.set_members('users')
-
-      subject.reset_document_cache
-
-      collection.should_receive(:find_one).with(criteria).once.and_return({})
-      subject.read('foo')
-      subject.read('foo')
-      subject.set_members('users')
-    end
-  end
-
-  context "without cache" do
-    before do
-      subject.document_cache = false
-    end
-
-    it_should_behave_like 'a flipper adapter'
-  end
-
-  describe "#use_document_cache" do
-    it "turns cache on for block and restores to original after block" do
-      subject.using_document_cache?.should be_false
-      subject.use_document_cache do
-        subject.using_document_cache?.should be_true
-      end
-      subject.using_document_cache?.should be_false
-    end
-  end
-
-  describe "#document_cache=" do
-    it "sets document cache" do
-      subject.document_cache = true
-      subject.using_document_cache?.should be_true
-
-      subject.document_cache = false
-      subject.using_document_cache?.should be_false
-    end
-
-    it "resets cached document" do
-      subject.should_receive(:reset_document_cache)
-      subject.document_cache = true
-    end
-  end
-end

data/spec/flipper/adapters/mongo_spec.rb

@@ -1,41 +0,0 @@
-require 'helper'
-require 'flipper/adapters/mongo'
-require 'flipper/spec/shared_adapter_specs'
-
-describe Flipper::Adapters::Mongo do
-  let(:collection) { Mongo::Connection.new.db('testing')['testing'] }
-  let(:id)         { 'flipper' }
-
-  subject { described_class.new(collection) }
-
-  before do
-    collection.remove
-  end
-
-  def read_key(key)
-    if (doc = collection.find_one(:_id => key.to_s))
-      value = doc['v']
-
-      if value.is_a?(::Array)
-        value = value.to_set
-      end
-
-      value
-    end
-  end
-
-  def write_key(key, value)
-    value = if value.is_a?(::Set)
-      value.to_a.map(&:to_s)
-    else
-      value.to_s
-    end
-
-    criteria = {:_id => key.to_s}
-    updates  = {'$set' => {'v' => value}}
-    options  = {:upsert => true}
-    collection.update criteria, updates, options
-  end
-
-  it_should_behave_like 'a flipper adapter'
-end

data/spec/flipper/middleware/mongo_single_document_query_cache_spec.rb

@@ -1,121 +0,0 @@
-require 'helper'
-require 'rack/test'
-require 'flipper/middleware/mongo_single_document_query_cache'
-
-describe Flipper::Middleware::MongoSingleDocumentQueryCache do
-  include Rack::Test::Methods
-
-  let(:collection) { Mongo::Connection.new.db('testing')['testing'] }
-  let(:adapter)    { Flipper::Adapters::MongoSingleDocument.new(collection) }
-  let(:flipper)    { Flipper.new(adapter) }
-
-  class Enum < Struct.new(:iter)
-    def each(&b)
-      iter.call(&b)
-    end
-  end
-
-  let(:app) {
-    # ensure scoped for builder block, annoying...
-    instance = adapter
-    middleware = described_class
-
-    Rack::Builder.new do
-      use middleware, instance
-
-      map "/" do
-        run lambda {|env| [200, {}, []] }
-      end
-
-      map "/fail" do
-        run lambda {|env| raise "FAIL!" }
-      end
-    end.to_app
-  }
-
-  it "delegates" do
-    called = false
-    app = lambda { |env|
-      called = true
-      [200, {}, nil]
-    }
-    middleware = described_class.new app, adapter
-    middleware.call({})
-    called.should be_true
-  end
-
-  it "enables document cache during delegation" do
-    app = lambda { |env|
-      adapter.using_document_cache?.should be_true
-      [200, {}, nil]
-    }
-    middleware = described_class.new app, adapter
-    middleware.call({})
-  end
-
-  it "enables document cache for body each" do
-    app = lambda { |env|
-      [200, {}, Enum.new(lambda { |&b|
-        adapter.using_document_cache?.should be_true
-        b.call "hello"
-      })]
-    }
-    middleware = described_class.new app, adapter
-    body = middleware.call({}).last
-    body.each { |x| x.should eql('hello') }
-  end
-
-  it "disables document cache after body close" do
-    app = lambda { |env| [200, {}, []] }
-    middleware = described_class.new app, adapter
-    body = middleware.call({}).last
-
-    adapter.using_document_cache?.should be_true
-    body.close
-    adapter.using_document_cache?.should be_false
-  end
-
-  it "clears document cache after body close" do
-    app = lambda { |env| [200, {}, []] }
-    middleware = described_class.new app, adapter
-    body = middleware.call({}).last
-    adapter.write('hello', 'world')
-
-    adapter.instance_variable_get("@document").should_not be_nil
-    body.close
-    adapter.instance_variable_get("@document").should be_nil
-  end
-
-  it "really does cache" do
-    flipper[:stats].enable
-
-    collection.should_receive(:find_one).once.and_return({})
-
-    app = lambda { |env|
-      flipper[:stats].enabled?
-      flipper[:stats].enabled?
-      flipper[:stats].enabled?
-      flipper[:stats].enabled?
-      flipper[:stats].enabled?
-      flipper[:stats].enabled?
-
-      [200, {}, []]
-    }
-    middleware = described_class.new app, adapter
-    middleware.call({})
-  end
-
-  context "with a successful request" do
-    it "clears the document cache" do
-      adapter.should_receive(:reset_document_cache).twice
-      get '/'
-    end
-  end
-
-  context "when the request raises an error" do
-    it "clears the document cache" do
-      adapter.should_receive(:reset_document_cache).once
-      get '/fail' rescue nil
-    end
-  end
-end

data/spec/support/accessor_helpers.rb

@@ -1,23 +0,0 @@
-module AccessorHelpers
-  def read_key(key)
-    if (doc = collection.find_one(criteria))
-      value = doc[key]
-
-      if value.is_a?(::Array)
-        value = value.to_set
-      end
-
-      value
-    end
-  end
-
-  def write_key(key, value)
-    if value.is_a?(::Set)
-      value = value.to_a
-    end
-
-    options = {:upsert => true}
-    updates = {'$set' => {key => value}}
-    collection.update criteria, updates, options
-  end
-end