activerecord-multi-tenant 1.2.0 → 2.4.0

data/spec/activerecord-multi-tenant/model_extensions_spec.rb

@@ -1,3 +1,5 @@
+# frozen_string_literal: true
+
 require 'spec_helper'
 
 describe MultiTenant do
@@ -10,11 +12,11 @@ describe MultiTenant do
   end
 
   describe 'is_scoped_as_tenant should return the correct value when true' do
-    it {expect(Project.respond_to?(:scoped_by_tenant?)).to eq(true)}
+    it { expect(Project.respond_to?(:scoped_by_tenant?)).to eq(true) }
   end
 
   describe 'is_scoped_as_tenant should return the correct value when false' do
-    it {expect(UnscopedModel.respond_to?(:scoped_by_tenant?)).to eq(false)}
+    it { expect(UnscopedModel.respond_to?(:scoped_by_tenant?)).to eq(false) }
   end
 
   context 'immutability' do
@@ -43,17 +45,17 @@ describe MultiTenant do
       @account = Account.create! name: 'foo'
       MultiTenant.current_tenant = @account
     end
-    it {expect(Project.new.account_id).to eq(@account.id)}
+    it { expect(Project.new.account_id).to eq(@account.id) }
     it 'should handle partial selects' do
       project = Project.create!
-      expect{project = Project.select(:name).find(project.id)}.not_to raise_error
+      expect { project = Project.select(:name).find(project.id) }.not_to raise_error
       expect(project.account_id).to eq(@account.id)
     end
   end
 
   describe 'Handles custom partition_key on tenant model' do
     before do
-      @account  = Account.create! name: 'foo'
+      @account = Account.create! name: 'foo'
       MultiTenant.current_tenant = @account
       @custom_partition_key_task = CustomPartitionKeyTask.create! name: 'foo'
     end
@@ -70,6 +72,72 @@ describe MultiTenant do
     it { expect(@partition_key_not_model_task.non_model_id).to be 77 }
   end
 
+  describe 'Tenant model with a nonstandard class name' do
+    let(:account_klass) do
+      Class.new(ActiveRecord::Base) do
+        self.table_name = 'account'
+
+        def self.name
+          'UserAccount'
+        end
+
+        multi_tenant(:account)
+      end
+    end
+    it 'does not register the tenant model' do
+      expect(MultiTenant).not_to receive(:register_multi_tenant_model)
+      account_klass
+    end
+  end
+
+  describe 'Changes table_name after multi_tenant called' do
+    before do
+      account_klass.has_many(:posts, anonymous_class: post_klass)
+      post_klass.belongs_to(:account, anonymous_class: account_klass)
+
+      @account1 = account_klass.create! name: 'foo'
+      @account2 = account_klass.create! name: 'bar'
+
+      @post1 = @account1.posts.create! name: 'foobar'
+      @post2 = @account2.posts.create! name: 'baz'
+
+      MultiTenant.current_tenant = @account1
+      @posts = post_klass.all
+    end
+
+    let(:account_klass) do
+      Class.new(Account) do
+        def self.name
+          'Account'
+        end
+      end
+    end
+
+    let(:post_klass) do
+      Class.new(ActiveRecord::Base) do
+        self.table_name = 'unknown'
+
+        multi_tenant(:account)
+
+        self.table_name = 'posts'
+
+        def self.name
+          'Post'
+        end
+      end
+    end
+
+    it { expect(@posts.length).to eq(1) }
+    it { expect(@posts).to eq([@post1]) }
+  end
+
+  describe 'inspect method filters senstive column values' do
+    it 'filters senstive value' do
+      account = Account.new(name: 'foo', password: 'baz')
+      expect(account.inspect).to eq '#<Account id: nil, name: nil, subdomain: nil, domain: nil, password: [FILTERED]>'
+    end
+  end
+
   # Scoping models
   describe 'Project.all should be scoped to the current tenant if set' do
     before do
@@ -112,16 +180,16 @@ describe MultiTenant do
     end
   end
 
-  describe "It should be possible to use aliased associations" do
+  describe 'It should be possible to use aliased associations' do
     before do
       @account = Account.create! name: 'baz'
       MultiTenant.current_tenant = @account
     end
 
-    it { expect(AliasedTask.create(:name => 'foo', :project_alias => @project2).valid?).to eq(true) }
+    it { expect(AliasedTask.create(name: 'foo', project_alias: @project2).valid?).to eq(true) }
   end
 
-  describe "It should be possible to use associations with partition_key from polymorphic" do
+  describe 'It should be possible to use associations with partition_key from polymorphic' do
     before do
       @account = Account.create!(name: 'foo')
       MultiTenant.current_tenant = @account
@@ -144,6 +212,22 @@ describe MultiTenant do
       end
     end
 
+    it 'handles belongs_to with optional: true' do
+      record = OptionalSubTask.create(sub_task_id: sub_task.id)
+      expect(record.reload.sub_task).to eq(sub_task)
+      expect(record.account_id).to eq(nil)
+    end
+
+    it 'handles changing tenant from nil to a value' do
+      record = OptionalSubTask.create(sub_task_id: sub_task.id)
+      expect(record.reload.sub_task).to eq(sub_task)
+      expect(record.account_id).to eq(nil)
+
+      record.account = account
+      record.save!
+      expect(record.reload.account_id).to eq(account.id)
+    end
+
     it 'handles has_many through' do
       MultiTenant.with(account) do
         expect(project.sub_tasks).to eq [sub_task]
@@ -162,7 +246,7 @@ describe MultiTenant do
       MultiTenant.with(account) do
         sub_task
         manager
-        expect(Project.eager_load([{manager: :project}, {tasks: :project}]).first).to eq project
+        expect(Project.eager_load([{ manager: :project }, { tasks: :project }]).first).to eq project
       end
     end
   end
@@ -173,7 +257,8 @@ describe MultiTenant do
 
     it 'rewrites sub-selects correctly' do
       MultiTenant.with(account) do
-        expect(Project.where(id: Project.where(id: project.id)).where(id: Project.where(id: project.id)).first).to eq project
+        expect(Project.where(id: Project.where(id: project.id))
+                      .where(id: Project.where(id: project.id)).first).to eq project
       end
     end
   end
@@ -202,70 +287,132 @@ describe MultiTenant do
   end
 
   describe 'non-STI Subclass of abstract Multi Tenant Model' do
-    let(:tenant_id_1) { 42 }
-    let(:tenant_id_2) { 314158 }
+    let(:tenant_id1) { 42 }
+    let(:tenant_id2) { 314_158 }
     let(:name) { 'fooname' }
-    let(:subclass_task_1) do
-      MultiTenant.with(tenant_id_1) { SubclassTask.create! name: name }
+    let(:subclass_task1) do
+      MultiTenant.with(tenant_id1) { SubclassTask.create! name: name }
     end
-    let(:subclass_task_2) do
-      MultiTenant.with(tenant_id_2) { SubclassTask.create! name: name }
+    let(:subclass_task2) do
+      MultiTenant.with(tenant_id2) { SubclassTask.create! name: name }
     end
 
     before do
-      subclass_task_1
-      subclass_task_2
+      subclass_task1
+      subclass_task2
     end
 
     it 'injects tenant_id on create' do
-      expect(subclass_task_1.non_model_id).to be tenant_id_1
-      expect(subclass_task_2.non_model_id).to be tenant_id_2
+      expect(subclass_task1.non_model_id).to be tenant_id1
+      expect(subclass_task2.non_model_id).to be tenant_id2
     end
 
     it 'rewrites query' do
-      MultiTenant.with(tenant_id_1) do
+      MultiTenant.with(tenant_id1) do
         expect(SubclassTask.where(name: name).count).to eq 1
-        expect(SubclassTask.where(name: name).first).to eq subclass_task_1
+        expect(SubclassTask.where(name: name).first).to eq subclass_task1
       end
-      MultiTenant.with(tenant_id_2) do
+      MultiTenant.with(tenant_id2) do
         expect(SubclassTask.where(name: name).count).to eq 1
-        expect(SubclassTask.where(name: name).first).to eq subclass_task_2
+        expect(SubclassTask.where(name: name).first).to eq subclass_task2
+      end
+    end
+  end
+
+  # Joins
+  describe 'joins for models' do
+    context 'for models with where condition in associations' do
+      let(:account) { Account.create!(name: 'Account 1') }
+
+      it 'should add tenant condition to the queries when tenant is set' do
+        expected_join_sql = <<-SQL.strip
+          SELECT "comments".*#{' '}
+          FROM "comments"#{' '}
+          INNER JOIN "tasks" ON "tasks"."id" = "comments"."commentable_id"#{' '}
+          AND "comments"."commentable_type" = 'Task' AND "tasks"."account_id" = 1#{' '}
+          WHERE "comments"."account_id" = 1
+        SQL
+
+        MultiTenant.with(account) do
+          expect(format_sql(Comment.joins(:task).to_sql)).to eq(format_sql(expected_join_sql))
+        end
+      end
+
+      it 'should add tenant condition to the queries when tenant is not set' do
+        MultiTenant.without do
+          expected_join_sql = <<-SQL.strip
+            SELECT "comments".*#{' '}
+            FROM "comments"#{' '}
+            INNER JOIN "tasks" ON "tasks"."id" = "comments"."commentable_id"#{' '}
+            AND "comments"."commentable_type" = 'Task' AND "comments"."account_id" = "tasks"."account_id"
+          SQL
+          expect(format_sql(Comment.joins(:task).to_sql)).to eq(format_sql(expected_join_sql))
+        end
+      end
+    end
+
+    context 'for models with default associations' do
+      let(:account) { Account.create!(name: 'Account 1') }
+
+      it 'should add tenant condition to the queries when tenant is set' do
+        expected_join_sql = <<-SQL.strip
+          SELECT "projects".*#{' '}
+          FROM "projects"#{' '}
+          INNER JOIN "tasks" ON "tasks"."project_id" = "projects"."id"#{' '}
+          AND "tasks"."account_id" = 1#{' '}
+          WHERE "projects"."account_id" = 1
+        SQL
+
+        MultiTenant.with(account) do
+          expect(format_sql(Project.joins(:tasks).to_sql)).to eq(format_sql(expected_join_sql))
+        end
+      end
+
+      it 'should add tenant condition to the queries when tenant is not set' do
+        MultiTenant.without do
+          expected_join_sql = <<-SQL.strip
+            SELECT "projects".*
+            FROM "projects"
+            INNER JOIN "tasks" ON "tasks"."project_id" = "projects"."id"
+            AND "projects"."account_id" = "tasks"."account_id"
+          SQL
+          expect(format_sql(Project.joins(:tasks).to_sql)).to eq(format_sql(expected_join_sql))
+        end
       end
     end
   end
 
   # ::with
-  describe "::with" do
-    it "should set current_tenant to the specified tenant inside the block" do
-      @account = Account.create!(:name => 'baz')
+  describe '::with' do
+    it 'should set current_tenant to the specified tenant inside the block' do
+      @account = Account.create!(name: 'baz')
 
       MultiTenant.with(@account) do
         expect(MultiTenant.current_tenant).to eq(@account)
       end
     end
 
-    it "should reset current_tenant to the previous tenant once exiting the block" do
-      @account1 = Account.create!(:name => 'foo')
-      @account2 = Account.create!(:name => 'bar')
+    it 'should reset current_tenant to the previous tenant once exiting the block' do
+      @account1 = Account.create!(name: 'foo')
+      @account2 = Account.create!(name: 'bar')
 
       MultiTenant.current_tenant = @account1
       MultiTenant.with @account2 do
-
       end
 
       expect(MultiTenant.current_tenant).to eq(@account1)
     end
 
-    it "should return the value of the block" do
-      @account1 = Account.create!(:name => 'foo')
-      @account2 = Account.create!(:name => 'bar')
+    it 'should return the value of the block' do
+      @account1 = Account.create!(name: 'foo')
+      @account2 = Account.create!(name: 'bar')
 
       MultiTenant.current_tenant = @account1
       value = MultiTenant.with @account2 do
-        "something"
+        'something'
       end
 
-      expect(value).to eq "something"
+      expect(value).to eq 'something'
     end
 
     it 'supports reload inside the block' do
@@ -280,9 +427,9 @@ describe MultiTenant do
   end
 
   # ::without
-  describe "::without" do
-    it "should unset current_tenant inside the block" do
-      @account = Account.create!(:name => 'baz')
+  describe '::without' do
+    it 'should unset current_tenant inside the block' do
+      @account = Account.create!(name: 'baz')
 
       MultiTenant.current_tenant = @account
       MultiTenant.without do
@@ -290,39 +437,25 @@ describe MultiTenant do
       end
     end
 
-    it "should reset current_tenant to the previous tenant once exiting the block" do
-      @account1 = Account.create!(:name => 'foo')
+    it 'should reset current_tenant to the previous tenant once exiting the block' do
+      @account1 = Account.create!(name: 'foo')
 
       MultiTenant.current_tenant = @account1
       MultiTenant.without do
-
       end
 
       expect(MultiTenant.current_tenant).to eq(@account1)
     end
 
-    it "should return the value of the block" do
-      @account1 = Account.create!(:name => 'foo')
+    it 'should return the value of the block' do
+      @account1 = Account.create!(name: 'foo')
 
       MultiTenant.current_tenant = @account1
       value = MultiTenant.without do
-        "something"
+        'something'
       end
 
-      expect(value).to eq "something"
-    end
-  end
-
-  describe '.with_lock' do
-    it 'supports with_lock blocks inside the block' do
-      @account = Account.create!(name: 'foo')
-
-      MultiTenant.with @account do
-        project = @account.projects.create!(name: 'project')
-        project.with_lock do
-          expect(project.name).to eq 'project'
-        end
-      end
+      expect(value).to eq 'something'
     end
   end
 
@@ -347,13 +480,20 @@ describe MultiTenant do
     end
   end
 
-  it "applies the team_id conditions in the where clause" do
-    option1 = <<-sql.strip
-      SELECT "sub_tasks".* FROM "sub_tasks" INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id" AND "sub_tasks"."account_id" = "tasks"."account_id" WHERE "tasks"."project_id" = 1 AND "sub_tasks"."account_id" = 1 AND "tasks"."account_id" = 1
-    sql
-    option2 = <<-sql.strip
-      SELECT "sub_tasks".* FROM "sub_tasks" INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id" AND "sub_tasks"."account_id" = "tasks"."account_id" WHERE "sub_tasks"."account_id" = 1 AND "tasks"."project_id" = 1 AND "tasks"."account_id" = 1
-    sql
+  it 'applies the team_id conditions in the where clause' do
+    option1 = <<-SQL.strip
+      SELECT "sub_tasks".*#{' '}
+      FROM "sub_tasks"#{' '}
+      INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id" AND "tasks"."account_id" = "sub_tasks"."account_id"#{' '}
+      WHERE "tasks"."project_id" = 1 AND "sub_tasks"."account_id" = 1 AND "tasks"."account_id" = 1
+    SQL
+    option2 = <<-SQL.strip
+      SELECT "sub_tasks".*#{' '}
+      FROM "sub_tasks"#{' '}
+      INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id"#{' '}
+      AND "tasks"."account_id" = "sub_tasks"."account_id"#{' '}
+      WHERE "sub_tasks"."account_id" = 1 AND "tasks"."project_id" = 1 AND "tasks"."account_id" = 1
+    SQL
 
     account1 = Account.create! name: 'Account 1'
 
@@ -361,27 +501,38 @@ describe MultiTenant do
       project1 = Project.create! name: 'Project 1'
       task1 = Task.create! name: 'Task 1', project: project1
       subtask1 = SubTask.create! task: task1
-      expect(project1.sub_tasks.to_sql).to eq(option1).or(eq(option2))
+      expect(format_sql(project1.sub_tasks.to_sql))
+        .to eq(format_sql(option1)).or(eq(format_sql(option2)))
       expect(project1.sub_tasks).to include(subtask1)
     end
 
     MultiTenant.without do
-      expected_sql = <<-sql
-                        SELECT "sub_tasks".* FROM "sub_tasks" INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id" AND "sub_tasks"."account_id" = "tasks"."account_id" WHERE "tasks"."project_id" = 1
-                     sql
+      expected_sql = <<-SQL
+        SELECT "sub_tasks".*#{' '}
+        FROM "sub_tasks"#{' '}
+        INNER JOIN "tasks" ON "sub_tasks"."task_id" = "tasks"."id"#{' '}
+        AND "tasks"."account_id" = "sub_tasks"."account_id"#{' '}
+        WHERE "tasks"."project_id" = 1
+      SQL
 
       project = Project.first
-      expect(project.sub_tasks.to_sql).to eq(expected_sql.strip)
+      expect(format_sql(project.sub_tasks.to_sql)).to eq(format_sql(expected_sql.strip))
     end
   end
 
-  it "tests joins between distributed and reference table" do
-    option1 = <<-sql.strip
-      SELECT "categories".* FROM "categories" INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id" WHERE "project_categories"."project_id" = 1 AND "project_categories"."account_id" = 1
-    sql
-    option2 = <<-sql.strip
-      SELECT "categories".* FROM "categories" INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id" WHERE "project_categories"."account_id" = 1 AND "project_categories"."project_id" = 1
-    sql
+  it 'tests joins between distributed and reference table' do
+    option1 = <<-SQL.strip
+      SELECT "categories".*#{' '}
+      FROM "categories"#{' '}
+      INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id"#{' '}
+      WHERE "project_categories"."project_id" = 1 AND "project_categories"."account_id" = 1
+    SQL
+    option2 = <<-SQL.strip
+      SELECT "categories".*#{' '}
+      FROM "categories"#{' '}
+      INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id"#{' '}
+      WHERE "project_categories"."account_id" = 1 AND "project_categories"."project_id" = 1
+    SQL
 
     account1 = Account.create! name: 'Account 1'
     category1 = Category.create! name: 'Category 1'
@@ -390,47 +541,72 @@ describe MultiTenant do
       project1 = Project.create! name: 'Project 1'
       projectcategory = ProjectCategory.create! name: 'project cat 1', project: project1, category: category1
 
-      expect(project1.categories.to_sql).to eq(option1).or(eq(option2))
+      expect(format_sql(project1.categories.to_sql))
+        .to eq(format_sql(option1)).or(eq(format_sql(option2)))
       expect(project1.categories).to include(category1)
       expect(project1.project_categories).to include(projectcategory)
     end
 
     MultiTenant.without do
-      expected_sql = <<-sql
-                     SELECT "categories".* FROM "categories" INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id" WHERE "project_categories"."project_id" = 1
-                     sql
+      expected_sql = <<-SQL
+        SELECT "categories".*#{' '}
+        FROM "categories"#{' '}
+        INNER JOIN "project_categories" ON "categories"."id" = "project_categories"."category_id"#{' '}
+        WHERE "project_categories"."project_id" = 1
+      SQL
 
       project = Project.first
-      expect(project.categories.to_sql).to eq(expected_sql.strip)
+      expect(format_sql(project.categories.to_sql))
+        .to eq(format_sql(expected_sql.strip))
       expect(project.categories).to include(category1)
 
-      expected_sql = <<-sql
-                        SELECT "projects".* FROM "projects" INNER JOIN "project_categories" ON "project_categories"."project_id" = "projects"."id" AND "project_categories"."account_id" = "projects"."account_id" INNER JOIN "categories" ON "categories"."id" = "project_categories"."category_id" WHERE "projects"."account_id" = 1
-                    sql
+      expected_sql = <<-SQL
+        SELECT "projects".* FROM "projects"#{' '}
+        INNER JOIN "project_categories" ON "project_categories"."project_id" = "projects"."id"#{' '}
+        AND "projects"."account_id" = "project_categories"."account_id"#{' '}
+        INNER JOIN "categories" ON "categories"."id" = "project_categories"."category_id"#{' '}
+        WHERE "projects"."account_id" = 1
+      SQL
 
-      expect(Project.where(account_id: 1).joins(:categories).to_sql).to eq(expected_sql.strip)
+      expect(format_sql(Project.where(account_id: 1).joins(:categories).to_sql))
+        .to eq(format_sql(expected_sql.strip))
       project = Project.where(account_id: 1).joins(:categories).first
       expect(project.categories).to include(category1)
     end
   end
 
-
-  it "test eager_load" do
+  it 'test eager_load' do
     account1 = Account.create! name: 'Account 1'
     category1 = Category.create! name: 'Category 1'
 
-    option1 = <<-sql.strip
-      SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2, "categories"."id" AS t1_r0, "categories"."name" AS t1_r1 FROM "projects" LEFT OUTER JOIN "project_categories" ON "project_categories"."project_id" = "projects"."id" AND "project_categories"."account_id" = 1 AND "projects"."account_id" = 1 LEFT OUTER JOIN "categories" ON "categories"."id" = "project_categories"."category_id" AND "project_categories"."account_id" = 1 WHERE "projects"."account_id" = 1
-    sql
-    option2 = <<-sql.strip
-      SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2, "categories"."id" AS t1_r0, "categories"."name" AS t1_r1 FROM "projects" LEFT OUTER JOIN "project_categories" ON "project_categories"."account_id" = 1 AND "project_categories"."project_id" = "projects"."id" AND "projects"."account_id" = 1 LEFT OUTER JOIN "categories" ON "categories"."id" = "project_categories"."category_id" AND "project_categories"."account_id" = 1 WHERE "projects"."account_id" = 1
-    sql
+    option1 = <<-SQL.strip
+      SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2,
+      "categories"."id" AS t1_r0, "categories"."name" AS t1_r1
+      FROM "projects"
+      LEFT OUTER JOIN "project_categories"
+      ON "project_categories"."project_id" = "projects"."id" AND "project_categories"."account_id" = 1
+      AND "projects"."account_id" = 1#{' '}
+      LEFT OUTER JOIN "categories" ON "categories"."id" = "project_categories"."category_id"
+      AND "project_categories"."account_id" = 1
+      WHERE "projects"."account_id" = 1
+    SQL
+    option2 = <<-SQL.strip
+      SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2,#{' '}
+      "categories"."id" AS t1_r0, "categories"."name" AS t1_r1#{' '}
+      FROM "projects"#{' '}
+      LEFT OUTER JOIN "project_categories"#{' '}
+      ON "project_categories"."account_id" = 1#{' '}
+      AND "project_categories"."project_id" = "projects"."id" AND "projects"."account_id" = 1#{' '}
+      LEFT OUTER JOIN "categories" ON "categories"."id" = "project_categories"."category_id"#{' '}
+      AND "project_categories"."account_id" = 1 WHERE "projects"."account_id" = 1
+    SQL
 
     MultiTenant.with(account1) do
       project1 = Project.create! name: 'Project 1'
       projectcategory = ProjectCategory.create! name: 'project cat 1', project: project1, category: category1
 
-      expect(Project.eager_load(:categories).to_sql).to eq(option1).or(eq(option2))
+      expect(format_sql(Project.eager_load(:categories).to_sql))
+        .to eq(format_sql(option1)).or(eq(format_sql(option2)))
 
       project = Project.eager_load(:categories).first
       expect(project.categories).to include(category1)
@@ -438,92 +614,120 @@ describe MultiTenant do
     end
 
     MultiTenant.without do
-      expected_sql = <<-sql
-                     SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2, "categories"."id" AS t1_r0, "categories"."name" AS t1_r1 FROM "projects" LEFT OUTER JOIN "project_categories" ON "project_categories"."project_id" = "projects"."id" AND "project_categories"."account_id" = "projects"."account_id" LEFT OUTER JOIN "categories" ON "categories"."id" = "project_categories"."category_id" WHERE "projects"."account_id" = 1
-                     sql
-
-      expect(Project.where(account_id: 1).eager_load(:categories).to_sql).to eq(expected_sql.strip)
+      expected_sql = <<-SQL
+        SELECT "projects"."id" AS t0_r0, "projects"."account_id" AS t0_r1, "projects"."name" AS t0_r2,#{' '}
+        "categories"."id" AS t1_r0, "categories"."name" AS t1_r1#{' '}
+        FROM "projects" LEFT OUTER JOIN "project_categories"#{' '}
+        ON "project_categories"."project_id" = "projects"."id" AND "projects"."account_id" = "project_categories"."account_id"#{' '}
+        LEFT OUTER JOIN "categories"#{' '}
+        ON "categories"."id" = "project_categories"."category_id"#{' '}
+        WHERE "projects"."account_id" = 1
+      SQL
+
+      expect(format_sql(Project.where(account_id: 1).eager_load(:categories).to_sql))
+        .to eq(format_sql(expected_sql.strip))
 
       project = Project.where(account_id: 1).eager_load(:categories).first
       expect(project.categories).to include(category1)
-
     end
   end
 
-  it "test raw SQL joins" do
+  it 'test raw SQL joins' do
     account1 = Account.create! name: 'Account 1'
     category1 = Category.create! name: 'Category 1'
 
     MultiTenant.with(account1) do
-      option1 = <<-sql.strip
-        SELECT "tasks".* FROM "tasks" INNER JOIN "projects" ON "projects"."id" = "tasks"."project_id" AND "projects"."account_id" = 1 LEFT JOIN project_categories pc ON project.category_id = pc.id WHERE "tasks"."account_id" = 1
-      sql
-      option2 = <<-sql.strip
-        SELECT "tasks".* FROM "tasks" INNER JOIN "projects" ON "projects"."account_id" = 1 AND "projects"."id" = "tasks"."project_id" LEFT JOIN project_categories pc ON project.category_id = pc.id WHERE "tasks"."account_id" = 1
-      sql
+      option1 = <<-SQL.strip
+        SELECT "tasks".* FROM "tasks"
+        INNER JOIN "projects" ON "projects"."id" = "tasks"."project_id" AND "projects"."account_id" = 1
+        LEFT JOIN project_categories pc ON project.category_id = pc.id#{' '}
+        WHERE "tasks"."account_id" = 1
+      SQL
+      option2 = <<-SQL.strip
+        SELECT "tasks".* FROM "tasks"
+        INNER JOIN "projects" ON "projects"."account_id" = 1#{' '}
+        AND "projects"."id" = "tasks"."project_id"
+        LEFT JOIN project_categories pc ON project.category_id = pc.id#{' '}
+        WHERE "tasks"."account_id" = 1
+      SQL
 
       project1 = Project.create! name: 'Project 1'
-      projectcategory = ProjectCategory.create! name: 'project cat 1', project: project1, category: category1
+      ProjectCategory.create! name: 'project cat 1', project: project1, category: category1
 
       project1.tasks.create! name: 'baz'
-      expect(Task.joins(:project).joins('LEFT JOIN project_categories pc ON project.category_id = pc.id').to_sql).to eq(option1).or(eq(option2))
+      expect(
+        format_sql(
+          Task.joins(:project).joins('LEFT JOIN project_categories pc ON project.category_id = pc.id').to_sql
+        )
+      ).to eq(format_sql(option1)).or(eq(format_sql(option2)))
     end
 
     MultiTenant.without do
-      expected_sql = <<-sql
-                     SELECT "tasks".* FROM "tasks" INNER JOIN "projects" ON "projects"."id" = "tasks"."project_id" AND "projects"."account_id" = "tasks"."account_id" LEFT JOIN project_categories pc ON project.category_id = pc.id WHERE "tasks"."account_id" = 1
-                     sql
-
-      expect(Task.where(account_id: 1).joins(:project).joins('LEFT JOIN project_categories pc ON project.category_id = pc.id').to_sql).to eq(expected_sql.strip)
-
+      expected_sql = <<-SQL.strip
+        SELECT "tasks".* FROM "tasks"
+        INNER JOIN "projects" ON "projects"."id" = "tasks"."project_id"
+        AND "tasks"."account_id" = "projects"."account_id"
+        LEFT JOIN project_categories pc ON project.category_id = pc.id
+        WHERE "tasks"."account_id" = 1
+      SQL
+
+      expect(format_sql(Task.where(account_id: 1).joins(:project)
+                 .joins('LEFT JOIN project_categories pc ON project.category_id = pc.id')
+                 .to_sql)).to eq(format_sql(expected_sql.strip))
     end
-
   end
 
-  it "only applies clauses when a tenant is set" do
+  it 'only applies clauses when a tenant is set' do
     account = Account.create! name: 'Account 1'
     project = Project.create! name: 'Project 1', account: account
     project2 = Project.create! name: 'Project 2', account: Account.create!(name: 'Account2')
 
     MultiTenant.with(account) do
-      option1 = <<-sql.strip
-        SELECT "projects".* FROM "projects" WHERE "projects"."account_id" = #{account.id} AND "projects"."id" = $1 LIMIT $2
-      sql
-      option2 = <<-sql.strip
-        SELECT "projects".* FROM "projects" WHERE "projects"."id" = $1 AND "projects"."account_id" = #{account.id} LIMIT $2
-      sql
-      option3 = <<-sql.strip
-        SELECT  "projects".* FROM "projects" WHERE "projects"."id" = $1 AND "projects"."account_id" = #{account.id} LIMIT $2
-      sql
+      option1 = <<-SQL.strip
+        SELECT "projects".* FROM "projects"#{' '}
+        WHERE "projects"."account_id" = #{account.id} AND "projects"."id" = $1 LIMIT $2
+      SQL
+      option2 = <<-SQL.strip
+        SELECT "projects".* FROM "projects"#{' '}
+        WHERE "projects"."id" = $1 AND "projects"."account_id" = #{account.id} LIMIT $2
+      SQL
+      option3 = <<-SQL.strip
+        SELECT  "projects".* FROM "projects"
+        WHERE "projects"."id" = $1
+        AND "projects"."account_id" = #{account.id} LIMIT $2
+      SQL
 
       # Couldn't make the following line pass for some reason, so came up with an uglier alternative
-      # expect(Project).to receive(:find_by_sql).with(eq(option1).or(eq(option2)).or(eq(option3)), any_args).and_call_original
+      # expect(Project).to receive(:find_by_sql).with(eq(option1).
+      # or(eq(option2)).or(eq(option3)), any_args).and_call_original
       expect(Project).to receive(:find_by_sql).and_wrap_original do |m, *args|
-        expect(args[0]).to(eq(option1).or(eq(option2)).or(eq(option3)))
-        m.call(args[0], args[1], preparable:args[2][:preparable])
+        expect(format_sql(args[0])).to(eq(format_sql(option1))
+                                               .or(eq(format_sql(option2))).or(eq(format_sql(option3))))
+        m.call(args[0], args[1], preparable: args[2][:preparable])
       end
       expect(Project.find(project.id)).to eq(project)
     end
 
     MultiTenant.without do
-      option1 = <<-sql.strip
-        SELECT "projects".* FROM "projects" WHERE "projects"."id" = $1 LIMIT $2
-      sql
-      option2 = <<-sql.strip
-        SELECT  "projects".* FROM "projects" WHERE "projects"."id" = $1 LIMIT $2
-      sql
+      option1 = <<-SQL.strip
+        SELECT "projects".* FROM "projects"#{' '}
+        WHERE "projects"."id" = $1 LIMIT $2
+      SQL
+      option2 = <<-SQL.strip
+        SELECT  "projects".* FROM "projects"#{' '}
+        WHERE "projects"."id" = $1 LIMIT $2
+      SQL
 
       # Couldn't make the following line pass for some reason, so came up with an uglier alternative
       # expect(Project).to receive(:find_by_sql).with(eq(option1).or(eq(option2)), any_args).and_call_original
       expect(Project).to receive(:find_by_sql).and_wrap_original do |m, *args|
-        expect(args[0]).to(eq(option1).or(eq(option2)))
-        m.call(args[0], args[1], preparable:args[2][:preparable])
+        expect(format_sql(args[0])).to(eq(format_sql(option1)).or(eq(format_sql(option2))))
+        m.call(args[0], args[1], preparable: args[2][:preparable])
       end
       expect(Project.find(project2.id)).to eq(project2)
     end
   end
 
-
   describe 'with unsaved association' do
     before do
       @account = Account.create!(name: 'reflection tenant')
@@ -537,13 +741,13 @@ describe MultiTenant do
     end
   end
 
-  it "test value of RETURNING insert in table with no pkey" do
+  it 'test value of RETURNING insert in table with no pkey' do
     account1 = Account.create(name: 'test1')
 
     MultiTenant.with(account1) do
-      allowed_place = AllowedPlace.create! name: 'something1'
+      AllowedPlace.create! name: 'something1'
 
-      project = Project.create! name: 'Project 1'
+      Project.create! name: 'Project 1'
     end
   end
 end