style: upgrade rubocop

palkan · Sep 15, 2023 · 4f0d9e5 · 4f0d9e5
1 parent acf61f9
commit 4f0d9e5
Show file tree

Hide file tree

Showing 3 changed files with 14 additions and 14 deletions.
diff --git a/lib/logidze/utils/check_pending.rb b/lib/logidze/utils/check_pending.rb
@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "./function_definitions"
-require_relative "./pending_migration_error"
+require_relative "function_definitions"
+require_relative "pending_migration_error"
 
 module Logidze
   module Utils

diff --git a/spec/dummy/app/models/comment.rb b/spec/dummy/app/models/comment.rb
@@ -2,5 +2,5 @@
 
 class Comment < ActiveRecord::Base
   has_logidze ignore_log_data: false
-  belongs_to :article
+  belongs_to :article, touch: true
 end
diff --git a/spec/generators/model_generator_spec.rb b/spec/generators/model_generator_spec.rb
@@ -52,11 +52,11 @@ class User < ActiveRecord::Base
         is_expected.to exist
         is_expected.to contain "ActiveRecord::Migration[#{ar_version}]"
         is_expected.to contain "add_column :users, :log_data, :jsonb"
-        is_expected.to contain(/create trigger "logidze_on_#{full_table_name('users')}"/i)
-        is_expected.to contain(/before update or insert on "#{full_table_name('users')}" for each row/i)
+        is_expected.to contain(/create trigger "logidze_on_#{full_table_name("users")}"/i)
+        is_expected.to contain(/before update or insert on "#{full_table_name("users")}" for each row/i)
         is_expected.to contain(/execute procedure logidze_logger\(null, 'updated_at'\);/i)
-        is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name('users')}" on "#{full_table_name('users')}"/i)
-        is_expected.not_to contain(/update "#{full_table_name('users')}"/i)
+        is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name("users")}" on "#{full_table_name("users")}"/i)
+        is_expected.not_to contain(/update "#{full_table_name("users")}"/i)
 
         expect(file("app/models/user.rb")).to contain "has_logidze"
       end
@@ -120,7 +120,7 @@ class User < ActiveRecord::Base
 
         it "creates backfill query" do
           is_expected.to exist
-          is_expected.to contain(/update "#{full_table_name('users')}" as t/i)
+          is_expected.to contain(/update "#{full_table_name("users")}" as t/i)
           is_expected.to contain(/set log_data = logidze_snapshot\(to_jsonb\(t\), 'updated_at'\);/i)
         end
       end
@@ -131,10 +131,10 @@ class User < ActiveRecord::Base
         it "creates migration with trigger" do
           is_expected.to exist
           is_expected.not_to contain "add_column :users, :log_data, :jsonb"
-          is_expected.to contain(/create trigger "logidze_on_#{full_table_name('users')}"/i)
-          is_expected.to contain(/before update or insert on "#{full_table_name('users')}" for each row/i)
+          is_expected.to contain(/create trigger "logidze_on_#{full_table_name("users")}"/i)
+          is_expected.to contain(/before update or insert on "#{full_table_name("users")}" for each row/i)
           is_expected.to contain(/execute procedure logidze_logger\(null, 'updated_at'\);/i)
-          is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name('users')}" on "#{full_table_name('users')}"/i)
+          is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name("users")}" on "#{full_table_name("users")}"/i)
           is_expected.not_to contain "remove_column :users, :log_data"
         end
       end
@@ -150,8 +150,8 @@ class User < ActiveRecord::Base
         it "creates migration with drop and create trigger" do
           is_expected.to exist
           is_expected.not_to contain "add_column :users, :log_data, :jsonb"
-          is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name('users')}" on "#{full_table_name('users')}"/i)
-          is_expected.to contain(/before update or insert on "#{full_table_name('users')}" for each row/i)
+          is_expected.to contain(/drop trigger if exists "logidze_on_#{full_table_name("users")}" on "#{full_table_name("users")}"/i)
+          is_expected.to contain(/before update or insert on "#{full_table_name("users")}" for each row/i)
           is_expected.to contain "raise ActiveRecord::IrreversibleMigration"
         end
 
@@ -224,7 +224,7 @@ class User < ActiveRecord::Base
 
         it "use after trigger" do
           is_expected.to exist
-          is_expected.to contain(/after update or insert on "#{full_table_name('users')}" for each row/i)
+          is_expected.to contain(/after update or insert on "#{full_table_name("users")}" for each row/i)
           is_expected.to contain(/execute procedure logidze_logger_after\(null, 'updated_at'\);/i)
         end