fresh_connection 0.1.8 → 0.2.0

data/spec/prepare.rb

@@ -0,0 +1,33 @@
+require 'yaml'
+
+system("mysql -uroot < spec/db_schema.sql")
+
+module ActiveRecord
+  class Base
+    self.configurations = YAML.load_file(File.join(File.dirname(__FILE__), "database.yml"))
+    establish_connection
+    establish_fresh_connection :slave1
+  end
+end
+
+class Parent < ActiveRecord::Base
+  self.abstract_class = true
+end
+
+class Slave2 < ActiveRecord::Base
+  self.abstract_class = true
+  establish_fresh_connection :slave2
+end
+
+class User < ActiveRecord::Base
+  has_one :address
+  has_many :tels
+end
+
+class Address < ActiveRecord::Base
+  belongs_to :user
+end
+
+class Tel < Slave2
+  belongs_to :user
+end

data/spec/spec_helper.rb

@@ -0,0 +1,8 @@
+ENV["RAILS_ENV"]="test"
+$LOAD_PATH.unshift File.expand_path('../../lib', __FILE__)
+require 'fresh_connection'
+
+FreshConnection::Initializer.extend_active_record
+require File.join(File.dirname(__FILE__), "prepare.rb")
+
+Dir[File.join(File.dirname(__FILE__), "/support/**/*.rb")].each {|f| require f}

data/spec/support/active_record_logger.rb

@@ -0,0 +1,3 @@
+require 'logger'
+
+ActiveRecord::Base.logger = Logger.new(File.join(File.dirname(__FILE__), '../../log/sql.log'))

data/spec/unit/access_control_spec.rb

@@ -0,0 +1,63 @@
+require 'spec_helper'
+
+describe FreshConnection::AccessControl do
+  before(:each) do
+    @ac = FreshConnection::AccessControl
+  end
+
+  context ".access" do
+    it "persisted first state(slave)" do
+      ret = []
+      @ac.access(true) do
+        @ac.access(true) do
+          ret << @ac.slave_access?
+          @ac.access(false) do
+            ret << @ac.slave_access?
+          end
+        end
+      end
+
+      expect(ret).to be_all{|item| item}
+    end
+
+    it "persisted first state(master)" do
+      ret = []
+      @ac.access(false) do
+        @ac.access(true) do
+          ret << @ac.slave_access?
+          @ac.access(false) do
+            ret << @ac.slave_access?
+          end
+        end
+      end
+
+      expect(ret).to_not be_all{|item| item}
+    end
+
+    it "outside is always master" do
+      ret = []
+      ret << @ac.slave_access?
+      @ac.access(true){}
+      ret << @ac.slave_access?
+
+      expect(ret).to_not be_all{|item| item}
+    end
+  end
+
+  context ".force_master_access" do
+    it "forced master state" do
+      @ac.access(true) do
+        @ac.force_master_access do
+          expect(@ac.slave_access?).to be_false
+        end
+      end
+    end
+
+    it "not effect outside" do
+      @ac.access(true) do
+        @ac.force_master_access {}
+        expect(@ac.slave_access?).to be_true
+      end
+    end
+  end
+end

data/spec/unit/ar_base_spec.rb

@@ -0,0 +1,42 @@
+require 'spec_helper'
+
+describe ActiveRecord::Base do
+  context ".slave_connection" do
+    it "return Mysql2Adapter object" do
+      expect(User.slave_connection).to be_a(ActiveRecord::ConnectionAdapters::Mysql2Adapter)
+    end
+  end
+
+  context ".slave_group" do
+    class Tel2 < Slave2
+      self.table_name = :tel
+    end
+
+    it "equal group_name of establish_fresh_connection" do
+      expect(User.slave_group).to eq("slave1")
+      expect(Tel.slave_group).to eq("slave2")
+    end
+
+    it "equal 'slave' when not specific group_name" do
+      Tel2.establish_fresh_connection
+      expect(Tel2.slave_group).to eq("slave")
+    end
+  end
+
+  context ".master_db_only?" do
+    class User2 < Parent
+      self.table_name = :users
+    end
+
+    it "childrend of master_db_only class is master_db_only" do
+      expect(User2.master_db_only?).to be_false
+      Parent.master_db_only!
+      expect(User2.master_db_only?).to be_true
+    end
+
+    it "not effect other class" do
+      Parent.master_db_only!
+      expect(Address.master_db_only?).to be_false
+    end
+  end
+end

data/spec/unit/fresh_connection_spec.rb

@@ -0,0 +1,106 @@
+require 'spec_helper'
+
+describe FreshConnection do
+  before(:each) do
+    @user = User.first
+  end
+
+  context "access to slave" do
+    it "select from User is to access to slave1" do
+      data = [
+        @user.name,
+        Address.first.user.name,
+        Tel.first.user.name,
+      ]
+      
+      expect(data).to be_all{|n| n.include?("slave1")}
+    end
+
+    it "select from Address is to access to slave1" do
+      data = [
+        Address.first.prefecture,
+        @user.address.prefecture
+      ]
+      
+      expect(data).to be_all{|n| n.include?("slave1")}
+    end
+
+    it "select from Address is to access to slave2" do
+      data = [
+        Tel.first.number,
+        @user.tels.first.number
+      ]
+      
+      expect(data).to be_all{|n| n.include?("slave2")}
+    end
+
+    it "select with join is to access to slave1" do
+      name = User.joins(:address).where("addresses.user_id = 1").first.name
+      expect(name).to be_include("slave1")
+    end
+  end
+
+  context "access to master" do
+    it "in transaction" do
+      User.transaction do
+        expect(@user.name).to be_include("slave1")
+        data = [
+          Address.first.user.name,
+          Address.first.prefecture,
+          Tel.first.number,
+          Tel.first.user.name,
+          @user.address.prefecture,
+          @user.tels.first.number,
+          User.joins(:address).where("addresses.user_id = 1").first.name
+        ]
+        
+        expect(data).to be_all{|n| n.include?("master")}
+
+      end
+    end
+
+    it "specify readonly(false)" do
+      data = [
+        Address.readonly(false).first.prefecture,
+        Address.includes(:user).readonly(false).first.user.name,
+        Tel.readonly(false).first.number,
+        Tel.includes(:user).readonly(false).first.user.name,
+        @user.tels.readonly(false).first.number,
+        User.includes(:tels).readonly(false).first.tels.first.number,
+        User.includes(:address).readonly(false).first.address.prefecture,
+        User.joins(:address).where("addresses.user_id = 1").readonly(false).first.name
+      ]
+      
+      expect(data).to be_all{|n| n.include?("master")}
+    end
+  end
+
+  context "master_db_only model always access to master" do
+    class Address3 < ActiveRecord::Base
+      self.table_name = "addresses"
+      master_db_only!
+    end
+
+    class Master < ActiveRecord::Base
+      self.abstract_class = true
+      master_db_only!
+    end
+
+    class Tel3 < Master
+      self.table_name = "tels"
+    end
+
+    it "self is master_db_only model" do
+      expect(Address3.first.prefecture).to be_include("master")
+    end
+
+    it "parent is master_db_only model" do
+      expect(Tel3.first.number).to be_include("master")
+    end
+
+    it "not effect other models" do
+      expect(Address.first.prefecture).to be_include("slave1")
+      expect(Tel.first.number).to be_include("slave2")
+    end
+  end
+end

data/spec/unit/recovery_spec.rb

@@ -0,0 +1,39 @@
+require 'spec_helper'
+
+describe "recovery when Mysql down" do
+  class User3 < ActiveRecord::Base
+    self.table_name = "users"
+
+    class << self
+      attr_writer :limit_time
+
+      def slave_connection
+        @access_time ||= 0
+        @access_time += 1
+        if @access_time > limit_time
+          super
+        else
+          raise ActiveRecord::StatementInvalid, "MySQL server has gone away"
+        end
+      end
+
+      def limit_time
+        @limit_time || 1
+      end
+    end
+  end
+
+  it "enable recovery" do
+    User3.limit_time = 1
+    expect {
+      User3.first
+    }.not_to raise_error
+  end
+
+  it "raise exception when retry over" do
+    User3.limit_time = 100
+    expect {
+      User3.first
+    }.to raise_error(ActiveRecord::StatementInvalid)
+  end
+end

metadata

@@ -1,126 +1,199 @@
---- !ruby/object:Gem::Specification 
+--- !ruby/object:Gem::Specification
 name: fresh_connection
-version: !ruby/object:Gem::Version 
-  hash: 11
-  prerelease: 
-  segments: 
-  - 0
-  - 1
-  - 8
-  version: 0.1.8
+version: !ruby/object:Gem::Version
+  version: 0.2.0
 platform: ruby
-authors: 
+authors:
 - Tsukasa OISHI
 autorequire: 
 bindir: bin
 cert_chain: []
-
-date: 2014-01-28 00:00:00 +09:00
-default_executable: 
-dependencies: 
-- !ruby/object:Gem::Dependency 
+date: 2014-04-11 00:00:00.000000000 Z
+dependencies:
+- !ruby/object:Gem::Dependency
   name: activerecord
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+  type: :runtime
   prerelease: false
-  requirement: &id001 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+- !ruby/object:Gem::Dependency
+  name: activesupport
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 15
-        segments: 
-        - 3
-        - 2
-        - 0
+      - !ruby/object:Gem::Version
         version: 3.2.0
   type: :runtime
-  version_requirements: *id001
-- !ruby/object:Gem::Dependency 
-  name: bundler
   prerelease: false
-  requirement: &id002 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
-    - - ~>
-      - !ruby/object:Gem::Version 
-        hash: 9
-        segments: 
-        - 1
-        - 3
-        version: "1.3"
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+- !ruby/object:Gem::Dependency
+  name: railties
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+  type: :runtime
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 3.2.0
+- !ruby/object:Gem::Dependency
+  name: bundler
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
   type: :development
-  version_requirements: *id002
-- !ruby/object:Gem::Dependency 
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - "~>"
+      - !ruby/object:Gem::Version
+        version: '1.3'
+- !ruby/object:Gem::Dependency
   name: rake
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 10.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 10.0.0
+- !ruby/object:Gem::Dependency
+  name: rspec
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 2.14.1
+  type: :development
   prerelease: false
-  requirement: &id003 !ruby/object:Gem::Requirement 
-    none: false
-    requirements: 
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
     - - ">="
-      - !ruby/object:Gem::Version 
-        hash: 3
-        segments: 
-        - 0
-        version: "0"
+      - !ruby/object:Gem::Version
+        version: 2.14.1
+- !ruby/object:Gem::Dependency
+  name: mysql2
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.15
   type: :development
-  version_requirements: *id003
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.3.15
+- !ruby/object:Gem::Dependency
+  name: appraisal
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 1.0.0
 description: https://github.com/tsukasaoishi/fresh_connection
-email: 
+email:
 - tsukasa.oishi@gmail.com
 executables: []
-
 extensions: []
-
 extra_rdoc_files: []
-
-files: 
+files:
+- ".gitignore"
+- Appraisals
 - Gemfile
 - LICENSE.txt
 - README.md
 - Rakefile
 - fresh_connection.gemspec
+- gemfiles/rails3.gemfile
+- gemfiles/rails40.gemfile
+- gemfiles/rails41.gemfile
 - lib/fresh_connection.rb
 - lib/fresh_connection/abstract_connection_manager.rb
-- lib/fresh_connection/active_record/abstract_adapter.rb
-- lib/fresh_connection/active_record/mysql2_adapter.rb
-- lib/fresh_connection/active_record/relation.rb
+- lib/fresh_connection/access_control.rb
 - lib/fresh_connection/connection_manager.rb
+- lib/fresh_connection/extend/ar_base.rb
+- lib/fresh_connection/extend/ar_relation.rb
+- lib/fresh_connection/extend/connection_handler.rb
+- lib/fresh_connection/extend/mysql2_adapter.rb
+- lib/fresh_connection/initializer.rb
 - lib/fresh_connection/rack/connection_management.rb
 - lib/fresh_connection/railtie.rb
 - lib/fresh_connection/slave_connection.rb
+- lib/fresh_connection/slave_connection_handler.rb
 - lib/fresh_connection/version.rb
-has_rdoc: true
+- log/.gitkeep
+- spec/database.yml
+- spec/db_schema.sql
+- spec/prepare.rb
+- spec/spec_helper.rb
+- spec/support/active_record_logger.rb
+- spec/unit/access_control_spec.rb
+- spec/unit/ar_base_spec.rb
+- spec/unit/fresh_connection_spec.rb
+- spec/unit/recovery_spec.rb
 homepage: https://github.com/tsukasaoishi/fresh_connection
-licenses: 
+licenses:
 - MIT
+metadata: {}
 post_install_message: 
 rdoc_options: []
-
-require_paths: 
+require_paths:
 - lib
-required_ruby_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+required_ruby_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
-required_rubygems_version: !ruby/object:Gem::Requirement 
-  none: false
-  requirements: 
+    - !ruby/object:Gem::Version
+      version: '0'
+required_rubygems_version: !ruby/object:Gem::Requirement
+  requirements:
   - - ">="
-    - !ruby/object:Gem::Version 
-      hash: 3
-      segments: 
-      - 0
-      version: "0"
+    - !ruby/object:Gem::Version
+      version: '0'
 requirements: []
-
 rubyforge_project: 
-rubygems_version: 1.6.2
+rubygems_version: 2.2.0
 signing_key: 
-specification_version: 3
+specification_version: 4
 summary: FreshConnection supports to connect with Mysql slave servers via Load Balancers.
-test_files: []
-
+test_files:
+- spec/database.yml
+- spec/db_schema.sql
+- spec/prepare.rb
+- spec/spec_helper.rb
+- spec/support/active_record_logger.rb
+- spec/unit/access_control_spec.rb
+- spec/unit/ar_base_spec.rb
+- spec/unit/fresh_connection_spec.rb
+- spec/unit/recovery_spec.rb