excursion 0.0.5 → 0.0.6

data/spec/excursion/datastores/active_record_spec.rb

@@ -0,0 +1,106 @@
+require 'spec_helper'
+require 'excursion/datastores/active_record'
+
+describe 'Excursion::Datastores::ActiveRecord' do
+
+  def fill_pool
+    Excursion::RoutePool.all.each { |m| m.destroy }
+    Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
+      Excursion::RoutePool.create key: key, value: val
+    end
+  end
+  
+  subject do
+    fill_pool
+    Excursion::Datastores::ActiveRecord.new
+  end
+
+  describe '#read' do
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.read }.to raise_exception
+        expect { subject.read('test_key') }.to_not raise_exception
+      end
+
+      it 'should accept a symbol or string' do
+        expect { subject.read('test_key') }.to_not raise_exception
+        expect { subject.read(:test_key) }.to_not raise_exception
+      end
+
+      it 'should convert symbols to strings' do
+        expect(subject.read(:key1)).to eql(subject.read('key1'))
+      end
+    end
+
+    context 'when the requested key does not exist' do
+      it 'should return nil' do
+        subject.read('non_existent_key').should be_nil
+      end
+    end
+
+    context 'when the requested key exists' do
+      it 'should return the value of the requested key' do
+        Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
+          expect(subject.read(key)).to eql(val)
+        end
+      end
+    end
+  end
+
+  describe '#write' do
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.write }.to raise_exception
+      end
+
+      it 'should accept a symbol or string' do
+        expect { subject.write('str_key', 'strval') }.to_not raise_exception
+        expect { subject.write(:sym_key, 'symval') }.to_not raise_exception
+      end
+
+      it 'should convert symbols to strings' do
+        subject.write(:sym_key, 'symval')
+        subject.read('sym_key').should == 'symval'
+      end
+    end
+
+    describe 'value' do
+      it 'should be required' do
+        expect { subject.write('test_key') }.to raise_exception(ArgumentError)
+      end
+    end
+
+    it 'should add the key to the datastore and set the value' do
+        subject.write('test_key', 'testval')
+        subject.read('test_key').should == 'testval'
+    end
+
+    it 'should return the value of the added key' do
+      subject.write('test_key', 'testval').should == 'testval'
+    end
+  end
+
+  context '#delete' do
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.delete }.to raise_exception(ArgumentError)
+      end
+    end
+    
+    it 'should remove the key from the datastore' do
+      subject.read('key1').should_not eql(nil)
+      subject.delete('key1')
+      subject.read('key1').should be(nil)
+    end
+
+    it 'should return the value of the deleted key if it exists' do
+      keyval = subject.read('key1')
+      subject.delete('key1').should eql(keyval)
+    end
+
+    it 'should return nil if the deleted key does not exist' do
+      subject.delete('non_existent_key').should eql(nil)
+    end
+  end
+
+end

data/spec/excursion/datastores/active_record_with_memcache_spec.rb

@@ -0,0 +1,158 @@
+require 'spec_helper'
+require 'excursion/datastores/active_record_with_memcache'
+
+describe 'Excursion::Datastores::ActiveRecordWithMemcache' do
+  
+  def dalli_client
+    @dalli_client ||= Dalli::Client.new dummy_pool, {namespace: 'excursion'}
+  end
+
+  def dummy_pool
+    Excursion::Specs::DUMMY_MEMCACHE_SERVER
+  end
+
+  def fill_pool
+    Excursion::RoutePool.all.each { |m| m.destroy }
+    dalli_client.flush_all
+    Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
+      dalli_client.set(key, val)
+      Excursion::RoutePool.create key: key, value: val
+    end
+  end
+  
+  subject do
+    fill_pool
+    Excursion::Datastores::ActiveRecordWithMemcache.new dummy_pool
+  end
+
+  describe '#read' do
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.read }.to raise_exception
+        expect { subject.read('test_key') }.to_not raise_exception
+      end
+
+      it 'should accept a symbol or string' do
+        expect { subject.read('test_key') }.to_not raise_exception
+        expect { subject.read(:test_key) }.to_not raise_exception
+      end
+
+      it 'should convert symbols to strings' do
+        expect(subject.read(:key1)).to eql(subject.read('key1'))
+      end
+    end
+
+    context 'when the requested key does not exist' do
+      it 'should return nil' do
+        subject.read('non_existent_key').should be_nil
+      end
+    end
+
+    context 'when the requested key exists' do
+      context 'in the cache' do
+        it 'should return the value of the requested key' do
+          Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
+            expect(dalli_client.get(key)).to eql(val)
+            expect(subject.read(key)).to eql(val)
+          end
+        end
+      end
+
+      context 'in the database' do
+        it 'should return the value of the requested key' do
+          Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
+            dalli_client.delete(key)
+            expect(dalli_client.get(key)).to be_nil
+            expect(subject.read(key)).to eql(val)
+          end
+        end
+      end
+    end
+  end
+
+  describe '#write' do
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.write }.to raise_exception
+      end
+
+      it 'should accept a symbol or string' do
+        expect { subject.write('str_key', 'strval') }.to_not raise_exception
+        expect { subject.write(:sym_key, 'symval') }.to_not raise_exception
+      end
+
+      it 'should convert symbols to strings' do
+        subject.write(:sym_key, 'symval')
+        subject.read('sym_key').should == 'symval'
+      end
+    end
+
+    describe 'value' do
+      it 'should be required' do
+        expect { subject.write('test_key') }.to raise_exception(ArgumentError)
+      end
+    end
+
+    it 'should add the key to the datastore and set the value' do
+      subject.write('test_key', 'testval')
+      subject.read('test_key').should == 'testval'
+    end
+    
+    it 'should add the key to the active record table and set the value' do
+      subject.write('test_key', 'testval')
+      Excursion::RoutePool.find_by(key: 'test_key').value.should eql('testval') if Excursion.rails4?
+      Excursion::RoutePool.find_by_key('test_key').value.should eql('testval') if Excursion.rails3?
+    end
+
+    it 'should add the key to the cache and set the value' do
+      subject.write('test_key', 'testval')
+      dalli_client.get('test_key').should eql('testval')
+    end
+
+    it 'should return the value of the added key' do
+      subject.write('test_key', 'testval').should == 'testval'
+    end
+  end
+
+  context '#delete' do
+    before(:each) do
+      fill_pool
+    end
+
+    describe 'key' do
+      it 'should be required' do
+        expect { subject.delete }.to raise_exception(ArgumentError)
+      end
+    end
+    
+    it 'should remove the key from the datastore' do
+      subject.read('key1').should_not eql(nil)
+      subject.delete('key1')
+      subject.read('key1').should be(nil)
+    end
+    
+    it 'should remove the key from the active record table' do
+      Excursion::RoutePool.find_by(key: 'key1').should_not be_nil if Excursion.rails4?
+      Excursion::RoutePool.find_by_key('key1').should_not be_nil if Excursion.rails3?
+      subject.delete('key1')
+      Excursion::RoutePool.find_by(key: 'key1').should be_nil if Excursion.rails4?
+      Excursion::RoutePool.find_by_key('key1').should be_nil if Excursion.rails3?
+    end
+
+    it 'should remove the key from the cache if it exists' do
+      dalli_client.get('key1').should_not be_nil
+      subject.delete('key1')
+      dalli_client.get('key1').should be_nil
+    end
+
+    it 'should return the value of the deleted key if it exists' do
+      keyval = subject.read('key1')
+      subject.delete('key1').should eql(keyval)
+    end
+
+    it 'should return nil if the deleted key does not exist' do
+      subject.delete('non_existent_key').should eql(nil)
+    end
+  end
+
+end

data/spec/excursion/datastores/file_spec.rb

@@ -8,6 +8,7 @@ describe 'Excursion::Datastores::File' do
   end
 
   def fill_pool
+    FileUtils.mkpath(::File.dirname(Excursion::Specs::DUMMY_POOL_FILE))
     File.open(dummy_pool, 'w') do |f|
       f.write(Excursion::Specs::Mocks::SIMPLE_VALUES.to_yaml)
     end

data/spec/excursion/datastores/memcache_spec.rb

@@ -9,6 +9,7 @@ describe 'Excursion::Datastores::Memcache' do
 
   def fill_pool
     dc = Dalli::Client.new dummy_pool, {namespace: 'excursion'}
+    dc.flush_all
     Excursion::Specs::Mocks::SIMPLE_VALUES.each do |key,val|
       dc.set(key,val)
     end

data/spec/excursion/route_pool_spec.rb

@@ -0,0 +1,4 @@
+require 'spec_helper'
+
+describe 'Excursion::RoutePool' do
+end

data/spec/spec_helper.rb

@@ -1,7 +1,9 @@
 require File.expand_path("../dummy/config/environment", __FILE__)
 require 'rspec/core'
-#require 'rspec/rails'
-#require 'rspec/autorun'
+
+ActiveRecord::Base.establish_connection :adapter => "sqlite3",
+                                        :database => ":memory:"
+ActiveRecord::Migrator.migrate(File.join(File.dirname(__FILE__), "dummy/db/migrate"))
 
 Dir[File.join(File.dirname(__FILE__), "support/**/*.rb")].each { |f| require f }
 

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: excursion
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
 platform: ruby
 authors:
 - Mark Rebec
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2013-06-17 00:00:00.000000000 Z
+date: 2013-06-19 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rails
@@ -16,14 +16,14 @@ dependencies:
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 4.0.0.rc2
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - '>='
       - !ruby/object:Gem::Version
-        version: 4.0.0.rc2
+        version: '0'
 - !ruby/object:Gem::Dependency
   name: rspec-rails
   requirement: !ruby/object:Gem::Requirement
@@ -78,6 +78,7 @@ files:
 - lib/excursion/pool.rb
 - lib/excursion/exceptions/datastores.rb
 - lib/excursion/exceptions/pool.rb
+- lib/excursion/exceptions/active_record.rb
 - lib/excursion/exceptions/memcache.rb
 - lib/excursion/pool/application.rb
 - lib/excursion/railtie.rb
@@ -86,10 +87,15 @@ files:
 - lib/excursion/helpers/application_helper.rb
 - lib/excursion/exceptions.rb
 - lib/excursion/datastores/datastore.rb
+- lib/excursion/datastores/active_record_with_memcache.rb
 - lib/excursion/datastores/file.rb
+- lib/excursion/datastores/active_record.rb
 - lib/excursion/datastores/memcache.rb
 - lib/excursion/helpers.rb
+- lib/excursion/route_pool.rb
 - lib/excursion/version.rb
+- lib/generators/excursion/active_record_generator.rb
+- lib/generators/excursion/templates/migration.rb
 - lib/excursion.rb
 - spec/excursion_spec.rb
 - spec/dummy/public/500.html
@@ -115,7 +121,10 @@ files:
 - spec/dummy/config/environment.rb
 - spec/dummy/config/boot.rb
 - spec/dummy/db/seeds.rb
+- spec/dummy/db/migrate/20130618222840_create_excursion_route_pool.rb
 - spec/dummy/db/development.sqlite3
+- spec/dummy/db/test.sqlite3
+- spec/dummy/db/schema.rb
 - spec/dummy/config.ru
 - spec/dummy/README.rdoc
 - spec/dummy/log/development.log
@@ -130,10 +139,13 @@ files:
 - spec/dummy/Rakefile
 - spec/support/mocks.rb
 - spec/excursion/helpers_spec.rb
+- spec/excursion/route_pool_spec.rb
 - spec/excursion/pool/application_spec.rb
 - spec/excursion/pool_spec.rb
 - spec/excursion/helpers/url_helper_spec.rb
 - spec/excursion/helpers/application_helper_spec.rb
+- spec/excursion/datastores/active_record_with_memcache_spec.rb
+- spec/excursion/datastores/active_record_spec.rb
 - spec/excursion/datastores/memcache_spec.rb
 - spec/excursion/datastores/file_spec.rb
 - spec/spec_helper.rb
@@ -185,7 +197,10 @@ test_files:
 - spec/dummy/config/environment.rb
 - spec/dummy/config/boot.rb
 - spec/dummy/db/seeds.rb
+- spec/dummy/db/migrate/20130618222840_create_excursion_route_pool.rb
 - spec/dummy/db/development.sqlite3
+- spec/dummy/db/test.sqlite3
+- spec/dummy/db/schema.rb
 - spec/dummy/config.ru
 - spec/dummy/README.rdoc
 - spec/dummy/log/development.log
@@ -200,10 +215,13 @@ test_files:
 - spec/dummy/Rakefile
 - spec/support/mocks.rb
 - spec/excursion/helpers_spec.rb
+- spec/excursion/route_pool_spec.rb
 - spec/excursion/pool/application_spec.rb
 - spec/excursion/pool_spec.rb
 - spec/excursion/helpers/url_helper_spec.rb
 - spec/excursion/helpers/application_helper_spec.rb
+- spec/excursion/datastores/active_record_with_memcache_spec.rb
+- spec/excursion/datastores/active_record_spec.rb
 - spec/excursion/datastores/memcache_spec.rb
 - spec/excursion/datastores/file_spec.rb
 - spec/spec_helper.rb