mongoid-multitenancy-2 2.0.3

data/spec/inheritance_spec.rb

@@ -0,0 +1,22 @@
+require 'spec_helper'
+
+describe 'Inheritance' do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  before do
+    Mongoid::Multitenancy.current_tenant = client
+  end
+
+  describe 'class' do
+    it 'uses inheritance pattern' do
+      MutableChild.create!(title: 'title X', slug: 'page-x')
+      expect(Mutable.last).to be_a MutableChild
+    end
+
+    it 'keeps options' do
+      expect(AnotherMutableChild.new(title: 'title X', slug: 'page-x')).to be_valid
+    end
+  end
+end

data/spec/mandatory_spec.rb

@@ -0,0 +1,116 @@
+require 'spec_helper'
+
+describe Mandatory do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  let(:another_client) do
+    Account.create!(name: 'another client')
+  end
+
+  let(:item) do
+    Mandatory.new(title: 'title X', slug: 'page-x')
+  end
+
+  it_behaves_like 'a tenantable model'
+  it { is_expected.to validate_tenant_uniqueness_of(:slug) }
+
+  describe '.shared' do
+    it 'is defined' do
+      expect(Mandatory).to respond_to(:shared)
+    end
+  end
+
+  describe '.unshared' do
+    it 'is defined' do
+      expect(Mandatory).to respond_to(:unshared)
+    end
+  end
+
+  describe '.default_scope' do
+    before do
+      Mongoid::Multitenancy.with_tenant(client) { @itemX = Mandatory.create!(title: 'title X', slug: 'article-x') }
+      Mongoid::Multitenancy.with_tenant(another_client) { @itemY = Mandatory.create!(title: 'title Y', slug: 'article-y') }
+    end
+
+    context 'with a current tenant' do
+      before do
+        Mongoid::Multitenancy.current_tenant = another_client
+      end
+
+      it 'filters on the current tenant' do
+        expect(Mandatory.all.to_a).to match_array [@itemY]
+      end
+    end
+
+    context 'without a current tenant' do
+      before do
+        Mongoid::Multitenancy.current_tenant = nil
+      end
+
+      it 'does not filter on any tenant' do
+        expect(Mandatory.all.to_a).to match_array [@itemX, @itemY]
+      end
+    end
+  end
+
+  describe '#delete_all' do
+    before do
+      Mongoid::Multitenancy.with_tenant(client) { @itemX = Mandatory.create!(title: 'title X', slug: 'article-x') }
+      Mongoid::Multitenancy.with_tenant(another_client) { @itemY = Mandatory.create!(title: 'title Y', slug: 'article-y') }
+    end
+
+    context 'with a current tenant' do
+      it 'only deletes the current tenant' do
+        Mongoid::Multitenancy.with_tenant(another_client) { Mandatory.delete_all }
+        expect(Mandatory.all.to_a).to match_array [@itemX]
+      end
+    end
+
+    context 'without a current tenant' do
+      it 'deletes all the items' do
+        Mandatory.delete_all
+        expect(Mandatory.all.to_a).to be_empty
+      end
+    end
+  end
+
+  describe '#valid?' do
+    context 'with a tenant' do
+      before do
+        Mongoid::Multitenancy.current_tenant = client
+      end
+
+      it 'is valid' do
+        expect(item).to be_valid
+      end
+
+      context 'with a uniqueness constraint' do
+        let(:duplicate) do
+          Mandatory.new(title: 'title Y', slug: 'page-x')
+        end
+
+        before do
+          item.save!
+        end
+
+        it 'does not allow duplicates on the same tenant' do
+          expect(duplicate).not_to be_valid
+        end
+
+        it 'allow duplicates on a different same tenant' do
+          Mongoid::Multitenancy.with_tenant(another_client) do
+            expect(duplicate).to be_valid
+          end
+        end
+      end
+    end
+
+    context 'without a tenant' do
+      it 'is not valid' do
+        expect(item).not_to be_valid
+      end
+    end
+  end
+end

data/spec/models/account.rb

@@ -0,0 +1,5 @@
+class Account
+  include Mongoid::Document
+
+  field :name, type: String
+end

data/spec/models/immutable.rb

@@ -0,0 +1,15 @@
+class Immutable
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant(:tenant, class_name: 'Account', immutable: true)
+
+  field :slug, type: String
+  field :title, type: String
+
+  validates_tenant_uniqueness_of :slug
+  validates_presence_of :slug
+  validates_presence_of :title
+
+  index(title: 1)
+end

data/spec/models/indexable.rb

@@ -0,0 +1,10 @@
+class Indexable
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  field :title, type: String
+
+  tenant :tenant, class_name: 'Account', index: true, full_indexes: false
+
+  index(title: 1)
+end

data/spec/models/mandatory.rb

@@ -0,0 +1,15 @@
+class Mandatory
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant(:tenant, class_name: 'Account')
+
+  field :slug, type: String
+  field :title, type: String
+
+  validates_tenant_uniqueness_of :slug
+  validates_presence_of :slug
+  validates_presence_of :title
+
+  index(title: 1)
+end

data/spec/models/mutable.rb

@@ -0,0 +1,15 @@
+class Mutable
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant :tenant, class_name: 'Account', immutable: false, optional: false
+
+  field :slug, type: String
+  field :title, type: String
+
+  validates_tenant_uniqueness_of :slug
+  validates_presence_of :slug
+  validates_presence_of :title
+
+  index(title: 1)
+end

data/spec/models/mutable_child.rb

@@ -0,0 +1,9 @@
+require 'models/mutable'
+
+class MutableChild < Mutable
+  field :random, type: String
+end
+
+class AnotherMutableChild < Mutable
+  field :random, type: String
+end

data/spec/models/no_scopable.rb

@@ -0,0 +1,11 @@
+class NoScopable
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant :tenant, class_name: 'Account', scopes: false
+
+  field :slug, type: String
+  field :title, type: String
+
+  index(title: 1)
+end

data/spec/models/optional.rb

@@ -0,0 +1,15 @@
+class Optional
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant(:tenant, class_name: 'Account', optional: true)
+
+  field :slug, type: String
+  field :title, type: String
+
+  validates_tenant_uniqueness_of :slug
+  validates_presence_of :slug
+  validates_presence_of :title
+
+  index(title: 1)
+end

data/spec/models/optional_exclude.rb

@@ -0,0 +1,15 @@
+class OptionalExclude
+  include Mongoid::Document
+  include Mongoid::Multitenancy::Document
+
+  tenant(:tenant, class_name: 'Account', optional: true)
+
+  field :slug, type: String
+  field :title, type: String
+
+  validates_tenant_uniqueness_of :slug, exclude_shared: true
+  validates_presence_of :slug
+  validates_presence_of :title
+
+  index(title: 1)
+end

data/spec/mongoid-multitenancy_spec.rb

@@ -0,0 +1,37 @@
+require 'spec_helper'
+
+describe Mongoid::Multitenancy do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  let(:another_client) do
+    Account.create!(name: 'another client')
+  end
+
+  before do
+    Mongoid::Multitenancy.current_tenant = client
+  end
+
+  describe '.with_tenant' do
+    it 'changes temporary the current tenant within the block' do
+      Mongoid::Multitenancy.with_tenant(another_client) do
+        expect(Mongoid::Multitenancy.current_tenant).to eq another_client
+      end
+    end
+
+    it 'restores the current tenant after the block' do
+      Mongoid::Multitenancy.with_tenant(another_client) { ; }
+      expect(Mongoid::Multitenancy.current_tenant).to eq client
+    end
+
+    context 'when the block fails' do
+      it 'restores the current tenant' do
+        begin
+          Mongoid::Multitenancy.with_tenant(another_client) { raise StandardError }
+        rescue StandardError; end
+        expect(Mongoid::Multitenancy.current_tenant).to eq client
+      end
+    end
+  end
+end

data/spec/mutable_child_spec.rb

@@ -0,0 +1,46 @@
+require 'spec_helper'
+
+describe MutableChild do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  let(:another_client) do
+    Account.create!(name: 'another client')
+  end
+
+  let(:item) do
+    MutableChild.new(title: 'title X', slug: 'page-x')
+  end
+
+  it_behaves_like 'a tenantable model'
+
+  describe '#valid?' do
+    before do
+      Mongoid::Multitenancy.current_tenant = client
+    end
+
+    context 'when the tenant has not changed' do
+      before do
+        item.save!
+      end
+
+      it 'is valid' do
+        item.title = 'title X (2)'
+        expect(item).to be_valid
+      end
+    end
+
+    context 'when the tenant has changed' do
+      before do
+        item.save!
+        Mongoid::Multitenancy.current_tenant = another_client
+      end
+
+      it 'is valid' do
+        item.tenant = another_client
+        expect(item).to be_valid
+      end
+    end
+  end
+end

data/spec/mutable_spec.rb

@@ -0,0 +1,50 @@
+require 'spec_helper'
+
+describe Mutable do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  let(:another_client) do
+    Account.create!(name: 'another client')
+  end
+
+  let(:item) do
+    Mutable.new(title: 'title X', slug: 'page-x')
+  end
+
+  it_behaves_like 'a tenantable model'
+
+  describe '#valid?' do
+    before do
+      Mongoid::Multitenancy.current_tenant = client
+    end
+
+    after do
+      Mongoid::Multitenancy.current_tenant = nil
+    end
+
+    context 'when the tenant has not changed' do
+      before do
+        item.save!
+      end
+
+      it 'is valid' do
+        item.title = 'title X (2)'
+        expect(item).to be_valid
+      end
+    end
+
+    context 'when the tenant has changed' do
+      before do
+        item.save!
+        Mongoid::Multitenancy.current_tenant = another_client
+      end
+
+      it 'is valid' do
+        item.tenant = another_client
+        expect(item).to be_valid
+      end
+    end
+  end
+end

data/spec/optional_exclude_spec.rb

@@ -0,0 +1,71 @@
+require 'spec_helper'
+
+describe OptionalExclude do
+  let(:client) do
+    Account.create!(name: 'client')
+  end
+
+  let(:another_client) do
+    Account.create!(name: 'another client')
+  end
+
+  let(:item) do
+    OptionalExclude.new(title: 'title X', slug: 'page-x')
+  end
+
+  it_behaves_like 'a tenantable model'
+
+  describe '#valid?' do
+    context 'with a tenant' do
+      before do
+        Mongoid::Multitenancy.current_tenant = client
+      end
+
+      it 'is valid' do
+        expect(item).to be_valid
+      end
+
+      context 'with a uniqueness constraint' do
+        let(:duplicate) do
+          OptionalExclude.new(title: 'title Y', slug: 'page-x')
+        end
+
+        before do
+          item.save!
+        end
+
+        it 'does not allow duplicates on the same tenant' do
+          expect(duplicate).not_to be_valid
+        end
+
+        it 'allow duplicates on a different same tenant' do
+          Mongoid::Multitenancy.with_tenant(another_client) do
+            expect(duplicate).to be_valid
+          end
+        end
+      end
+    end
+
+    context 'without a tenant' do
+      it 'is valid' do
+        expect(item).to be_valid
+      end
+
+      context 'with a uniqueness constraint' do
+        let(:duplicate) do
+          OptionalExclude.new(title: 'title Y', slug: 'page-x')
+        end
+
+        before do
+          item.save!
+        end
+
+        it 'allow duplicates on any client' do
+          Mongoid::Multitenancy.with_tenant(client) do
+            expect(duplicate).to be_valid
+          end
+        end
+      end
+    end
+  end
+end