tenancy 0.2.0 → 1.0.0

data/spec/lib/scoping/mongoid_spec.rb

@@ -0,0 +1,179 @@
+require "spec_helper"
+
+if defined?(Mongoid)
+  describe "Tenancy::Scoping::Mongoid", mongoid: true do
+    let(:camyp)     { Mongo::Portal.create(domain_name: "yp.com.kh") }
+    let(:panpages)  { Mongo::Portal.create(domain_name: "panpages.com") }
+    let(:listing)   { Mongo::Listing.create(name: "Listing 1", portal_id: camyp.id) }
+
+    describe Mongo::Listing do
+      it { should belong_to(:portal).of_type(Mongo::Portal) }
+
+      it { should validate_presence_of(:portal) }
+
+      it { should validate_uniqueness_of(:name).scoped_to(:portal_id).case_insensitive }
+
+      it "have default_scope with :portal_id field" do
+        Mongo::Portal.current = camyp
+
+        expect(Mongo::Listing.where(nil).selector).to eq({"is_active"=>true, "portal_id"=>Mongo::Portal.current_id})
+      end
+
+      it "doesn't have default_scope when it doesn't have current portal" do
+        Mongo::Portal.current = nil
+
+        expect(Mongo::Listing.where(nil).selector).to eq({"is_active"=>true})
+      end
+    end
+
+    describe Mongo::Communication do
+      it { should belong_to(:portal) }
+
+      it { should validate_presence_of(:portal) }
+
+      it { should belong_to(:listing) }
+
+      it { should validate_presence_of(:listing) }
+
+      it { should validate_uniqueness_of(:value).scoped_to(:portal_id, :listing_id) }
+
+      it "have default_scope with :portal_id field" do
+        Mongo::Portal.current  = camyp
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.where(nil).selector).to eq({"is_active"=>true, "portal_id"=>Mongo::Portal.current_id, "listing_id"=>Mongo::Listing.current_id})
+      end
+
+      it "doesn't have default_scope when it doesn't have current portal and listing" do
+        Mongo::Portal.current  = nil
+        Mongo::Listing.current = nil
+
+        expect(Mongo::Communication.where(nil).selector).to eq({"is_active"=>true})
+      end
+    end
+
+    describe Mongo::ExtraCommunication do
+      it { should belong_to(:portal).of_type(Mongo::Portal) }
+
+      it { should belong_to(:listing).of_type(Mongo::Listing) }
+
+      it "raise exception when passing two resources and options" do
+        expect { Mongo::ExtraCommunication.scope_to(:portal, :listing, class_name: "Mongo::Listing") }.to raise_error(ArgumentError)
+      end
+
+      it "uses the correct scope" do
+        listing2 = Mongo::Listing.create(name: "Name 2", portal: camyp)
+
+        Mongo::Portal.current = camyp
+        Mongo::Listing.current = listing2
+
+        extra_communication = Mongo::ExtraCommunication.new
+        expect(extra_communication.listing_id).to eq(listing2.id)
+        expect(extra_communication.portal_id).to eq(camyp.id)
+      end
+    end
+
+    describe "belongs_to method override" do
+      before(:each) { Mongo::Portal.current = camyp }
+
+      it "reload belongs_to when passes true" do
+        listing.portal.domain_name = "abc.com"
+
+        expect(listing.portal(true).object_id).not_to eq(Mongo::Portal.current.object_id)
+      end
+
+      it "doesn't reload belongs_to" do
+        listing.portal.domain_name = "abc.com"
+
+        expect(listing.portal.object_id).to eq(Mongo::Portal.current.object_id)
+      end
+
+      it "returns different object" do
+        listing.portal_id = panpages.id
+
+        expect(listing.portal.object_id).not_to eq(Mongo::Portal.current.object_id)
+      end
+
+      it "doesn't touch db" do
+        current_listing = listing
+
+        Mongo::Portal.store_in session: ""
+        expect(current_listing.portal.object_id).to eq(Mongo::Portal.current.object_id)
+        Mongo::Portal.store_in session: "default"
+      end
+    end
+
+    describe "#shard_key_selector override" do
+      before(:each) { Mongo::Portal.current = camyp }
+
+      it "returns with current_portal" do
+        expect(Mongo::Communication.new.shard_key_selector).to eq({"portal_id"=>camyp.id})
+      end
+
+      it "returns with current_listing" do
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.new.shard_key_selector).to eq({"portal_id"=>camyp.id, "listing_id"=>listing.id})
+      end
+
+      it "returns without :current_portal and :current_listing" do
+        Mongo::Portal.current  = nil
+        Mongo::Listing.current = nil
+
+        expect(Mongo::Communication.new.shard_key_selector).to eq({})
+      end
+    end
+
+    describe "#tenant_scope" do
+      before(:each) { Mongo::Portal.current = camyp }
+
+      it "scopes only :current_portal" do
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.tenant_scope(:portal).selector).to eq({"is_active"=>true, "portal_id"=>Mongo::Portal.current_id})
+      end
+
+      it "scopes only :current_listing" do
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.tenant_scope(:listing).selector).to eq({"is_active"=>true, "listing_id"=>Mongo::Listing.current_id})
+      end
+
+      it "scopes only :current_listing and :current_portal" do
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.tenant_scope(:listing, :portal).selector).to eq(Mongo::Communication.where(nil).selector)
+      end
+
+      it "scopes nothing" do
+        Mongo::Listing.current = listing
+
+        expect(Mongo::Communication.tenant_scope(nil).selector).to eq({"is_active"=>true})
+      end
+    end
+
+    if ::Mongoid::VERSION.start_with?("3.1.")
+      describe "getter method override" do
+        before(:each) { Mongo::Portal.current = camyp }
+
+        it "returns the current portal_id" do
+          expect(Mongo::Listing.new.portal_id).to eq(camyp.id)
+        end
+
+        it "#portal_id returns nil" do
+          Mongo::Portal.current = nil
+
+          expect(Mongo::Listing.new.portal_id).to eq(nil)
+        end
+
+        it "returns its value for existing" do
+          listing = Mongo::Listing.create(name: "abcdef")
+          expect(listing.portal_id).to eq(camyp.id)
+
+          Mongo::Portal.current = nil
+          expect(listing.portal_id).to eq(camyp.id)
+        end
+      end
+    end
+  end
+end

data/spec/lib/shoulda_matchers_spec.rb

@@ -1,20 +1,44 @@
-require 'spec_helper'
-require 'tenancy/matchers'
+require "spec_helper"
+require "tenancy/matchers"
 
-describe Portal do
-  it { should be_a_tenant }
-end
+if defined?(ActiveRecord)
+  describe Portal do
+    it { should be_a_tenant }
+  end
 
-describe Listing do
-  it { should be_a_tenant }
-end
+  describe Listing do
+    it { should be_a_tenant }
+  end
 
-describe ExtraCommunication do
-  let(:camyp) { Portal.create(domain_name: 'yp.com.kh') }
-  before      { Portal.current = camyp }
+  describe ExtraCommunication do
+    let(:camyp) { Portal.create(domain_name: "yp.com.kh") }
+    before      { Portal.current = camyp }
 
-  it { should have_scope_to(:portal) }
-  it { should have_scope_to(:portal).class_name('Portal') }
-  it { should have_scope_to(:listing) }
-  it { should have_scope_to(:listing).class_name('Listing') }
+    it { should have_scope_to(:portal) }
+    it { should have_scope_to(:portal).class_name("Portal") }
+    it { should have_scope_to(:listing) }
+    it { should have_scope_to(:listing).class_name("Listing") }
+  end
 end
+
+if defined?(Mongoid)
+  module Mongo
+    describe Portal do
+      it { should be_a_tenant }
+    end
+
+    describe Listing do
+      it { should be_a_tenant }
+    end
+
+    describe ExtraCommunication do
+      let(:camyp) { Portal.create(domain_name: "yp.com.kh") }
+      before      { Portal.current = camyp }
+
+      it { should have_scope_to(:portal) }
+      it { should have_scope_to(:portal).of_type(Mongo::Portal) }
+      it { should have_scope_to(:listing) }
+      it { should have_scope_to(:listing).of_type(Mongo::Listing) }
+    end
+  end
+end

data/spec/spec_helper.rb

@@ -1,17 +1,68 @@
+require "simplecov"
+require "coveralls"
 require "codeclimate-test-reporter"
-CodeClimate::TestReporter.start
 
-require 'tenancy'
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  Coveralls::SimpleCov::Formatter,
+  SimpleCov::Formatter::HTMLFormatter,
+  CodeClimate::TestReporter::Formatter
+]
+
+SimpleCov.start do
+  add_filter "/spec/"
+end
+
+require "pry"
+require "database_cleaner"
+require "logger"
+require "tenancy"
 
 # active_record
-load File.dirname(__FILE__) + '/support/schema.rb'
-load File.dirname(__FILE__) + '/support/models.rb'
+if Gem.loaded_specs["activerecord"]
+  load File.dirname(__FILE__) + "/support/active_record/schema.rb"
+  load File.dirname(__FILE__) + "/support/active_record/models.rb"
+  require "shoulda-matchers"
+end
 
-require 'pry'
-require 'shoulda-matchers'
+# mongoid
+if Gem.loaded_specs["mongoid"]
+  load File.dirname(__FILE__) + "/support/mongoid/connection.rb"
+  load File.dirname(__FILE__) + "/support/mongoid/models.rb"
+  require "mongoid-rspec"
+end
 
 RSpec.configure do |config|
   config.filter_run focus: true
   config.run_all_when_everything_filtered = true
   config.treat_symbols_as_metadata_keys_with_true_values = true
+
+  config.before(:suite) do
+    DatabaseCleaner.strategy = :truncation
+  end
+  config.include Mongoid::Matchers, mongoid: true if defined?(Mongoid)
+
+  config.around(:each) do |example|
+    if example.metadata[:log]
+      if defined?(ActiveRecord)
+        ActiveRecord::Base.logger = Logger.new(STDOUT)
+      end
+      if defined?(Mongoid)
+        Mongoid.logger.level = Logger::DEBUG
+        Moped.logger.level = Logger::DEBUG
+      end
+    end
+    DatabaseCleaner.start
+    RequestStore.clear!
+
+    example.run
+
+    if defined?(ActiveRecord)
+      ActiveRecord::Base.logger = nil
+    end
+    if defined?(Mongoid)
+      Mongoid.logger.level = Logger::INFO
+      Moped.logger.level = Logger::INFO
+    end
+    DatabaseCleaner.clean
+  end
 end

data/spec/support/{models.rb → active_record/models.rb}

@@ -7,7 +7,7 @@ class Listing < ActiveRecord::Base
   include Tenancy::ResourceScope
 
   default_scope -> { where(is_active: true) }
-  scope_to :portal
+  scope_to      :portal
   validates_uniqueness_in_scope :name, case_sensitive: false
 end
 
@@ -15,13 +15,13 @@ class Communication < ActiveRecord::Base
   include Tenancy::ResourceScope
 
   default_scope -> { where(is_active: true) }
-  scope_to :portal, :listing
+  scope_to      :portal, :listing
   validates_uniqueness_in_scope :value
 end
 
 class ExtraCommunication < ActiveRecord::Base
   include Tenancy::ResourceScope
 
-  scope_to :portal, class_name: 'Portal'
-  scope_to :listing, class_name: 'Listing'
-end
+  scope_to :portal, class_name: "Portal"
+  scope_to :listing, class_name: "Listing"
+end

data/spec/support/{schema.rb → active_record/schema.rb}

@@ -1,7 +1,6 @@
-require 'active_record'
-require 'logger'
+require "active_record"
 
-ActiveRecord::Base.establish_connection(adapter: 'sqlite3', database: 'spec/test.sqlite3')
+ActiveRecord::Base.establish_connection(adapter: "sqlite3", database: "spec/test.sqlite3")
 ActiveRecord::Migration.verbose = false
 
 ActiveRecord::Schema.define do
@@ -35,4 +34,4 @@ ActiveRecord::Schema.define do
     t.references :portal
     t.timestamps
   end
-end
+end

data/spec/support/mongoid/connection.rb

@@ -0,0 +1,3 @@
+require "mongoid"
+
+Mongoid.load! File.dirname(__FILE__) + "/mongoid.yml", :test

data/spec/support/mongoid/models.rb

@@ -0,0 +1,44 @@
+module Mongo
+  class Portal
+    include Mongoid::Document
+    include Tenancy::Resource
+
+    field :domain_name, type: String
+  end
+
+  class Listing
+    include Mongoid::Document
+    include Tenancy::Resource
+    include Tenancy::ResourceScope
+
+    field         :name, type: String
+
+    default_scope -> { where(is_active: true) }
+    scope_to      :portal, class_name: "Mongo::Portal"
+    validates_uniqueness_in_scope \
+                  :name, case_sensitive: false
+  end
+
+  class Communication
+    include Mongoid::Document
+    include Tenancy::ResourceScope
+
+    field         :value, type: String
+
+    default_scope -> { where(is_active: true) }
+    scope_to      :portal, class_name: "Mongo::Portal"
+    scope_to      :listing, class_name: "Mongo::Listing"
+    validates_uniqueness_in_scope \
+                  :value
+  end
+
+  class ExtraCommunication
+    include Mongoid::Document
+    include Tenancy::ResourceScope
+
+    field    :value, type: String
+
+    scope_to :portal, class_name: "Mongo::Portal"
+    scope_to :listing, class_name: "Mongo::Listing"
+  end
+end

data/spec/support/mongoid/mongoid.yml

@@ -0,0 +1,6 @@
+test:
+  sessions:
+    default:
+      database: tenancy_mongoid_test
+      hosts:
+        - localhost:27017

data/tenancy.gemspec

@@ -1,29 +1,26 @@
 # coding: utf-8
-lib = File.expand_path('../lib', __FILE__)
+lib = File.expand_path("../lib", __FILE__)
 $LOAD_PATH.unshift(lib) unless $LOAD_PATH.include?(lib)
-require 'tenancy/version'
+require "tenancy/version"
 
 Gem::Specification.new do |spec|
   spec.name          = "tenancy"
   spec.version       = Tenancy::VERSION
-  spec.authors       = ["chamnap"]
+  spec.authors       = ["Chamnap Chhorn"]
   spec.email         = ["chamnapchhorn@gmail.com"]
-  spec.description   = %q{A simple multitenancy with activerecord through scoping}
-  spec.summary       = %q{A simple multitenancy with activerecord through scoping}
+  spec.description   = %q{A simple multitenancy with activerecord/mongoid through scoping}
+  spec.summary       = %q{A simple multitenancy with activerecord/mongoid through scoping}
   spec.homepage      = "https://github.com/yoolk/tenancy"
   spec.license       = "MIT"
 
+  spec.required_ruby_version     = ">= 1.9.3"
+  spec.required_rubygems_version = ">= 1.8.11"
+
   spec.files         = `git ls-files`.split($/)
   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 "rspec", "~> 2.12.0"
-  spec.add_development_dependency "shoulda", "~> 3.5.0"
-  spec.add_development_dependency "pry", "~> 0.9.12"
-  spec.add_development_dependency "sqlite3", "~> 1.3.7"
-  spec.add_development_dependency "rake"
-
-  spec.add_dependency "activerecord", ">= 3.2.13"
+  spec.add_dependency "activesupport", ">= 3.2"
   spec.add_dependency "request_store", "~> 1.0.5"
-end
+end