makara 0.5.0 → 0.5.1

data/spec/active_record/connection_adapters/makara_abstract_adapter_error_handling_spec.rb

@@ -2,7 +2,6 @@ require 'spec_helper'
 require 'active_record/connection_adapters/makara_abstract_adapter'
 
 describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter::ErrorHandler do
-
   let(:handler){ described_class.new }
   let(:proxy) { FakeAdapter.new(config(1,1)) }
   let(:connection){ proxy.master_pool.connections.first }
@@ -60,9 +59,8 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter::ErrorHandler d
   end
 
   describe 'custom errors' do
-
     let(:config_path) { File.join(File.expand_path('../../../', __FILE__), 'support', 'mysql2_database_with_custom_errors.yml') }
-    let(:config) { YAML.load_file(config_path)['test'] }
+    let(:config) { YAML.load(ERB.new(File.read(config_path)).result)['test'] }
     let(:handler){ described_class.new }
     let(:proxy) { FakeAdapter.new(config) }
     let(:connection){ proxy.master_pool.connections.first }
@@ -85,8 +83,5 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter::ErrorHandler d
         end
       }.to raise_error(Makara::Errors::BlacklistConnection)
     end
-
   end
-
-
 end

data/spec/active_record/connection_adapters/makara_abstract_adapter_spec.rb

@@ -2,7 +2,6 @@ require 'spec_helper'
 require 'active_record/connection_adapters/makara_abstract_adapter'
 
 describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter do
-
   let(:klass){ FakeAdapter }
 
   {
@@ -38,15 +37,12 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter do
     'select pg_advisory_lock(12345)' => true,
     'select pg_advisory_unlock(12345)' => true
   }.each do |sql, should_go_to_master|
-
     it "determines that \"#{sql}\" #{should_go_to_master ? 'requires' : 'does not require'} master" do
       proxy = klass.new(config(1,1))
       expect(proxy.master_for?(sql)).to eq(should_go_to_master)
     end
-
   end
 
-
   {
     "SET @@things" => true,
     "INSERT INTO wisdom ('The truth will set you free.')" => false,
@@ -61,7 +57,6 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter do
         max_treats IS NULL
     } => false
   }.each do |sql, should_send_to_all_connections|
-
     it "determines that \"#{sql}\" #{should_send_to_all_connections ? 'should' : 'should not'} be sent to all underlying connections" do
       proxy = klass.new(config(1,1))
       proxy.master_pool.connections.each{|con| expect(con).to receive(:execute).with(sql).once}
@@ -75,7 +70,6 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter do
 
       proxy.execute(sql)
     end
-
   end
 
   {
@@ -108,12 +102,9 @@ describe ActiveRecord::ConnectionAdapters::MakaraAbstractAdapter do
         max_treats IS NULL
     } => true
   }.each do |sql,should_stick|
-
     it "should #{should_stick ? 'stick' : 'not stick'} to master if handling sql like \"#{sql}\"" do
       proxy = klass.new(config(0,0))
       expect(proxy.would_stick?(sql)).to eq(should_stick)
     end
-
   end
-
 end

data/spec/active_record/connection_adapters/makara_mysql2_adapter_spec.rb

@@ -2,12 +2,10 @@ require 'spec_helper'
 require 'active_record/connection_adapters/mysql2_adapter'
 
 describe 'MakaraMysql2Adapter' do
-
-  let(:db_username){ ENV['TRAVIS'] ? 'travis' : 'root' }
-
   let(:config){
-    base = YAML.load_file(File.expand_path('spec/support/mysql2_database.yml'))['test']
-    base
+    file = File.expand_path('spec/support/mysql2_database.yml')
+    hash = YAML.load(ERB.new(File.read(file)).result)
+    hash['test']
   }
 
   let(:connection) { ActiveRecord::Base.connection }
@@ -18,7 +16,6 @@ describe 'MakaraMysql2Adapter' do
   end
 
   context "unconnected" do
-
     it 'should allow a connection to be established' do
       establish_connection(config)
       expect(ActiveRecord::Base.connection).to be_instance_of(ActiveRecord::ConnectionAdapters::MakaraMysql2Adapter)
@@ -56,7 +53,6 @@ describe 'MakaraMysql2Adapter' do
       expect{
         connection.execute('SET @t1 = 1')
       }.to raise_error(Makara::Errors::NoConnectionsAvailable)
-
     end
 
     context "unconnect afterwards" do
@@ -84,8 +80,7 @@ describe 'MakaraMysql2Adapter' do
         load(File.dirname(__FILE__) + '/../../support/schema.rb')
         change_context
 
-        allow(ActiveRecord::Base).to receive(:mysql2_connection) do |config|
-          config[:username] = db_username
+        allow(ActiveRecord::Base).to receive(:mysql2_connection) do
           original_method.call(config)
         end
 
@@ -102,18 +97,15 @@ describe 'MakaraMysql2Adapter' do
         ActiveRecord::Base.remove_connection
       end
     end
-
   end
 
   context 'with the connection established and schema loaded' do
-
     before do
       establish_connection(config)
       load(File.dirname(__FILE__) + '/../../support/schema.rb')
       change_context
     end
 
-
     it 'should have one master and two slaves' do
       expect(connection.master_pool.connection_count).to eq(1)
       expect(connection.slave_pool.connection_count).to eq(2)
@@ -159,17 +151,14 @@ describe 'MakaraMysql2Adapter' do
     end
 
     it 'should send exists? to slave' do
-      next if ActiveRecord::VERSION::MAJOR == 3 && ActiveRecord::VERSION::MINOR == 0 # query doesn't work?
-
       allow_any_instance_of(Makara::Strategies::RoundRobin).to receive(:single_one?){ true }
       Test::User.exists? # flush other (schema) things that need to happen
-      
+
       con = connection.slave_pool.connections.first
-      if (ActiveRecord::VERSION::MAJOR == 5 && ActiveRecord::VERSION::MINOR <= 0)
-        expect(con).to receive(:execute).with(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/, any_args).once.and_call_original
-      else
-        expect(con).to receive(:exec_query).with(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/, any_args).once.and_call_original
-      end
+      expect(con).to receive(:exec_query) do |query|
+        expect(query).to match(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/)
+      end.once.and_call_original
+
       Test::User.exists?
     end
 
@@ -198,7 +187,6 @@ describe 'MakaraMysql2Adapter' do
         }.to raise_error(Makara::Errors::AllConnectionsBlacklisted)
       end
     end
-
   end
 
   describe 'transaction support' do
@@ -256,5 +244,4 @@ describe 'MakaraMysql2Adapter' do
       it_behaves_like 'a transaction supporter'
     end
   end
-
 end

data/spec/active_record/connection_adapters/makara_postgis_adapter_spec.rb

@@ -1,16 +1,12 @@
 # RGeo doesn't play well with JRuby and to avoid complicated test setup
 # we're only testing ActiveRecord version ~> 4.2
-# also rgeo only works on 2.1+
 
-rmajor, rminor, rpatch = RUBY_VERSION.split(/[^\d]/)[0..2].map(&:to_i)
 require 'active_record'
 
 # TODO: test this in AR 5+ ?
-
 if RUBY_ENGINE == 'ruby' &&
     ActiveRecord::VERSION::MAJOR == 4 &&
-    ActiveRecord::VERSION::MINOR >= 2 &&
-    (rmajor > 2 || (rmajor == 2 && rminor >= 1))
+    ActiveRecord::VERSION::MINOR >= 2
 
   require 'spec_helper'
   require 'rgeo'
@@ -18,12 +14,8 @@ if RUBY_ENGINE == 'ruby' &&
   require 'active_record/connection_adapters/postgis_adapter'
 
   describe 'MakaraPostgisAdapter' do
-    let(:db_username){ ENV['TRAVIS'] ? 'postgres' : `whoami`.chomp }
-
     let(:config) do
-      base = YAML.load_file(File.expand_path('spec/support/postgis_database.yml'))['test']
-      base['username'] = db_username
-      base
+      YAML.load_file(File.expand_path('spec/support/postgis_database.yml'))['test']
     end
 
     let(:connection) { ActiveRecord::Base.connection }

data/spec/active_record/connection_adapters/makara_postgresql_adapter_spec.rb

@@ -3,13 +3,8 @@ require 'active_record/connection_adapters/postgresql_adapter'
 require 'active_record/errors'
 
 describe 'MakaraPostgreSQLAdapter' do
-
-  let(:db_username){ ENV['TRAVIS'] ? 'postgres' : `whoami`.chomp }
-
   let(:config) do
-    base = YAML.load_file(File.expand_path('spec/support/postgresql_database.yml'))['test']
-    base['username'] = db_username
-    base
+    YAML.load_file(File.expand_path('spec/support/postgresql_database.yml'))['test']
   end
 
   let(:connection) { ActiveRecord::Base.connection }
@@ -19,21 +14,18 @@ describe 'MakaraPostgreSQLAdapter' do
     change_context
   end
 
-
   it 'should allow a connection to be established' do
     establish_connection(config)
     expect(ActiveRecord::Base.connection).to be_instance_of(ActiveRecord::ConnectionAdapters::MakaraPostgreSQLAdapter)
   end
 
   context 'with the connection established and schema loaded' do
-
     before do
       establish_connection(config)
       load(File.dirname(__FILE__) + '/../../support/schema.rb')
       change_context
     end
 
-
     it 'should have one master and two slaves' do
       expect(connection.master_pool.connection_count).to eq(1)
       expect(connection.slave_pool.connection_count).to eq(2)
@@ -74,18 +66,15 @@ describe 'MakaraPostgreSQLAdapter' do
     end
 
     it 'should send exists? to slave' do
-      next if ActiveRecord::VERSION::MAJOR == 3 && ActiveRecord::VERSION::MINOR == 0 # query doesn't work?
-
       allow_any_instance_of(Makara::Strategies::RoundRobin).to receive(:single_one?){ true }
       Test::User.exists? # flush other (schema) things that need to happen
-      
+
       con = connection.slave_pool.connections.first
-      if (ActiveRecord::VERSION::MAJOR == 4 && ActiveRecord::VERSION::MINOR >= 2) ||
-         (ActiveRecord::VERSION::MAJOR == 5 && ActiveRecord::VERSION::MINOR <= 0)
-        expect(con).to receive(:exec_no_cache).with(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/, any_args).once.and_call_original
-      else
-        expect(con).to receive(:exec_query).with(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/, any_args).once.and_call_original
-      end
+
+      expect(con).to receive(:exec_query) do |query|
+        expect(query).to match(/SELECT\s+1\s*(AS one)?\s+FROM .?users.?\s+LIMIT\s+.?1/)
+      end.once.and_call_original
+
       Test::User.exists?
     end
 
@@ -194,7 +183,6 @@ describe 'MakaraPostgreSQLAdapter' do
   end
 
   context 'with two activerecord connection pools' do
-
     before :each do
       class Model1 < ActiveRecord::Base
       end
@@ -204,7 +192,6 @@ describe 'MakaraPostgreSQLAdapter' do
 
       Model1.establish_connection(config)
       Model2.establish_connection(config)
-
     end
 
     it 'should not leak raw connection into activerecord pool' do

data/spec/cache_spec.rb

@@ -1,7 +1,6 @@
 require 'spec_helper'
 
 describe Makara::Cache do
-
   it 'shows a warning' do
     expect(Makara::Logging::Logger).to receive(:log).with(/Setting the Makara::Cache\.store won't have any effects/, :warn)
     described_class.store = :noop

data/spec/config_parser_spec.rb

@@ -1,21 +1,20 @@
 require 'spec_helper'
 
 describe Makara::ConfigParser do
-
   let(:config){
     {
-      :top_level => 'value',
-      :makara => {
-        :connections => [
+      top_level: 'value',
+      makara: {
+        connections: [
           {
-            :role => 'master',
-            :name => 'themaster'
+            role: 'master',
+            name: 'themaster'
           },
           {
-            :name => 'slave1'
+            name: 'slave1'
           },
           {
-            :name => 'slave2'
+            name: 'slave2'
           }
         ]
       }
@@ -25,27 +24,27 @@ describe Makara::ConfigParser do
   context '::merge_and_resolve_default_url_config' do
     let(:config_without_url) do
       {
-        :master_ttl => 5,
-        :blacklist_duration => 30,
-        :sticky => true,
-        :adapter => 'mysql2_makara',
-        :encoding => 'utf8',
-        :host => 'localhost',
-        :database => 'db_name',
-        :username => 'db_username',
-        :password => 'db_password',
-        :port =>     3306
+        master_ttl: 5,
+        blacklist_duration: 30,
+        sticky: true,
+        adapter: 'mysql2_makara',
+        encoding: 'utf8',
+        host: 'localhost',
+        database: 'db_name',
+        username: 'db_username',
+        password: 'db_password',
+        port: 3306
       }
     end
 
     let(:config_with_url) do
       {
-        :master_ttl => 5,
-        :blacklist_duration => 30,
-        :sticky => true,
-        :adapter => 'mysql2_makara',
-        :encoding => 'utf8',
-        :url => 'mysql2://db_username:db_password@localhost:3306/db_name'
+        master_ttl: 5,
+        blacklist_duration: 30,
+        sticky: true,
+        adapter: 'mysql2_makara',
+        encoding: 'utf8',
+        url: 'mysql2://db_username:db_password@localhost:3306/db_name'
       }
     end
 
@@ -76,12 +75,11 @@ describe Makara::ConfigParser do
         ENV['DATABASE_URL'] = database_url
       end
     end
-
   end
 
   it 'should provide a default proxy id based on the recursively sorted config' do
     parsera = described_class.new(config)
-    parserb = described_class.new(config.merge(:other => 'value'))
+    parserb = described_class.new(config.merge(other: 'value'))
     parserc = described_class.new(config)
 
     expect(parsera.id).not_to eq(parserb.id)
@@ -113,27 +111,27 @@ describe Makara::ConfigParser do
       parser = described_class.new(config)
       expect(parser.master_configs).to eq([
         {
-          :name => 'themaster',
-          :top_level => 'value',
-          :sticky => true,
-          :blacklist_duration => 30,
-          :master_ttl => 5
+          name: 'themaster',
+          top_level: 'value',
+          sticky: true,
+          blacklist_duration: 30,
+          master_ttl: 5
         }
       ])
       expect(parser.slave_configs).to eq([
         {
-          :name => 'slave1',
-          :top_level => 'value',
-          :sticky => true,
-          :blacklist_duration => 30,
-          :master_ttl => 5
+          name: 'slave1',
+          top_level: 'value',
+          sticky: true,
+          blacklist_duration: 30,
+          master_ttl: 5
         },
         {
-          :name => 'slave2',
-          :top_level => 'value',
-          :sticky => true,
-          :blacklist_duration => 30,
-          :master_ttl => 5
+          name: 'slave2',
+          top_level: 'value',
+          sticky: true,
+          blacklist_duration: 30,
+          master_ttl: 5
         }
       ])
     end
@@ -146,27 +144,27 @@ describe Makara::ConfigParser do
       parser = described_class.new(config)
       expect(parser.master_configs).to eq([
         {
-          :name => 'themaster',
-          :top_level => 'value',
-          :sticky => true,
-          :blacklist_duration => 456,
-          :master_ttl => 5
+          name: 'themaster',
+          top_level: 'value',
+          sticky: true,
+          blacklist_duration: 456,
+          master_ttl: 5
         }
       ])
       expect(parser.slave_configs).to eq([
         {
-          :name => 'slave1',
-          :top_level => 'slave value',
-          :sticky => true,
-          :blacklist_duration => 123,
-          :master_ttl => 5
+          name: 'slave1',
+          top_level: 'slave value',
+          sticky: true,
+          blacklist_duration: 123,
+          master_ttl: 5
         },
         {
-          :name => 'slave2',
-          :top_level => 'value',
-          :sticky => true,
-          :blacklist_duration => 123,
-          :master_ttl => 5
+          name: 'slave2',
+          top_level: 'value',
+          sticky: true,
+          blacklist_duration: 123,
+          master_ttl: 5
         }
       ])
     end