update gems, rubocop

Author: kirillshevch

.rubocop.yml

@@ -2,18 +2,18 @@ AllCops:
   DisplayCopNames:    true
   DisplayStyleGuide:  true
   StyleGuideCopsOnly: true
-  TargetRubyVersion:  2.5
+  TargetRubyVersion:  3.4
 
-Metrics/LineLength:
+Layout/LineLength:
   Max: 120
 
-Documentation:
+Style/Documentation:
   Enabled: false
 
-FrozenStringLiteralComment:
+Style/FrozenStringLiteralComment:
   Enabled: false
 
-Style/FileName:
+Naming/FileName:
   Enabled: false
 
 Metrics/ModuleLength:

Gemfile.lock

@@ -9,79 +9,88 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (7.1.3)
+    activesupport (8.0.2.1)
       base64
+      benchmark (>= 0.3)
       bigdecimal
-      concurrent-ruby (~> 1.0, >= 1.0.2)
+      concurrent-ruby (~> 1.0, >= 1.3.1)
       connection_pool (>= 2.2.5)
       drb
       i18n (>= 1.6, < 2)
+      logger (>= 1.4.2)
       minitest (>= 5.1)
-      mutex_m
-      tzinfo (~> 2.0)
-    ast (2.4.2)
-    base64 (0.2.0)
-    bigdecimal (3.1.6)
-    byebug (11.1.3)
-    concurrent-ruby (1.2.3)
-    connection_pool (2.4.1)
-    diff-lcs (1.5.1)
-    drb (2.2.0)
-      ruby2_keywords
-    dry-configurable (1.1.0)
-      dry-core (~> 1.0, < 2)
+      securerandom (>= 0.3)
+      tzinfo (~> 2.0, >= 2.0.5)
+      uri (>= 0.13.1)
+    ast (2.4.3)
+    base64 (0.3.0)
+    benchmark (0.4.1)
+    bigdecimal (3.2.2)
+    byebug (12.0.0)
+    concurrent-ruby (1.3.5)
+    connection_pool (2.5.3)
+    diff-lcs (1.6.2)
+    drb (2.2.3)
+    dry-configurable (1.3.0)
+      dry-core (~> 1.1)
       zeitwerk (~> 2.6)
-    dry-core (1.0.1)
+    dry-core (1.1.0)
       concurrent-ruby (~> 1.0)
+      logger
       zeitwerk (~> 2.6)
-    i18n (1.14.1)
+    i18n (1.14.7)
       concurrent-ruby (~> 1.0)
-    json (2.7.1)
-    language_server-protocol (3.17.0.3)
-    minitest (5.22.2)
-    mutex_m (0.2.0)
-    parallel (1.24.0)
-    parser (3.3.0.5)
+    json (2.13.2)
+    language_server-protocol (3.17.0.5)
+    lint_roller (1.1.0)
+    logger (1.7.0)
+    minitest (5.25.5)
+    parallel (1.27.0)
+    parser (3.3.9.0)
       ast (~> 2.4.1)
       racc
-    racc (1.7.3)
+    prism (1.4.0)
+    racc (1.8.1)
     rainbow (3.1.1)
-    rake (13.1.0)
-    regexp_parser (2.9.0)
-    rexml (3.2.6)
-    rspec (3.13.0)
+    rake (13.3.0)
+    regexp_parser (2.11.2)
+    rspec (3.13.1)
       rspec-core (~> 3.13.0)
       rspec-expectations (~> 3.13.0)
       rspec-mocks (~> 3.13.0)
-    rspec-core (3.13.0)
+    rspec-core (3.13.5)
       rspec-support (~> 3.13.0)
-    rspec-expectations (3.13.0)
+    rspec-expectations (3.13.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-mocks (3.13.0)
+    rspec-mocks (3.13.5)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.13.0)
-    rspec-support (3.13.0)
-    rubocop (1.60.2)
+    rspec-support (3.13.5)
+    rubocop (1.79.2)
       json (~> 2.3)
-      language_server-protocol (>= 3.17.0)
+      language_server-protocol (~> 3.17.0.2)
+      lint_roller (~> 1.1.0)
       parallel (~> 1.10)
       parser (>= 3.3.0.2)
       rainbow (>= 2.2.2, < 4.0)
-      regexp_parser (>= 1.8, < 3.0)
-      rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.30.0, < 2.0)
+      regexp_parser (>= 2.9.3, < 3.0)
+      rubocop-ast (>= 1.46.0, < 2.0)
       ruby-progressbar (~> 1.7)
-      unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.30.0)
-      parser (>= 3.2.1.0)
+      unicode-display_width (>= 2.4.0, < 4.0)
+    rubocop-ast (1.46.0)
+      parser (>= 3.3.7.2)
+      prism (~> 1.4)
     ruby-progressbar (1.13.0)
-    ruby2_keywords (0.0.5)
+    securerandom (0.4.1)
     slack_hook (0.1.2)
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unicode-display_width (2.5.0)
-    zeitwerk (2.6.13)
+    unicode-display_width (3.1.5)
+      unicode-emoji (~> 4.0, >= 4.0.4)
+    unicode-emoji (4.0.4)
+    uri (1.0.3)
+    zeitwerk (2.7.3)
 
 PLATFORMS
   ruby

lib/query_track.rb

@@ -18,7 +18,7 @@ class << self
 
     def logger
       @logger ||= Logger.new($stdout).tap do |log|
-        log.progname = self.name
+        log.progname = name
       end
     end
   end

lib/query_track/event_processor.rb

@@ -13,11 +13,11 @@ def call
 
       return if under_filter?(caller)
 
-      if duration_seconds > QueryTrack::Settings.config.duration
-        QueryTrack::Notifications::Slack.new(event.payload[:sql], duration_seconds).call
-        QueryTrack::Notifications::Log.new(event.payload[:sql], duration_seconds).call
-        QueryTrack::Notifications::Custom.new(event.payload[:sql], duration_seconds).call
-      end
+      return unless duration_seconds > QueryTrack::Settings.config.duration
+
+      QueryTrack::Notifications::Slack.new(event.payload[:sql], duration_seconds).call
+      QueryTrack::Notifications::Log.new(event.payload[:sql], duration_seconds).call
+      QueryTrack::Notifications::Custom.new(event.payload[:sql], duration_seconds).call
     end
 
     private
@@ -32,7 +32,7 @@ def duration_seconds
       event.duration / 1000
     end
 
-    def under_filter?(trace)
+    def under_filter?(_trace)
       QueryTrack::Filters.new(caller).call
     end
   end

lib/query_track/filters.rb

@@ -7,8 +7,8 @@ def initialize(full_trace)
     end
 
     def call
-      QueryTrack::Settings.config.filters.find do |filter| 
-        full_trace.select { |v| v =~ %r{#{filter}} }[0]
+      QueryTrack::Settings.config.filters.find do |filter|
+        full_trace.select { |v| v =~ /#{filter}/ }[0]
       end
     end
   end

spec/query_track/event_processor_spec.rb

@@ -1,13 +1,13 @@
 RSpec.describe QueryTrack::EventProcessor do
   let(:payload) do ['sql.active_record', Time.now, Time.now, '1e07319',
-    {
-      sql: "SET client_min_messages TO 'warning'",
-      name: 'SCHEMA',
-      binds: [],
-      type_casted_binds: [],
-      statement_name: nil,
-      connection_id: 1
-    }
+                    {
+                      sql: "SET client_min_messages TO 'warning'",
+                      name: 'SCHEMA',
+                      binds: [],
+                      type_casted_binds: [],
+                      statement_name: nil,
+                      connection_id: 1
+                    }
   ] end
 
   let!(:event) { ActiveSupport::Notifications::Event.new(*payload) }
@@ -21,7 +21,7 @@
       subject.call
     end
   end
-  
+
   context 'processing' do
     before do
       QueryTrack::Settings.configure do |config|

spec/query_track/notifications/slack_spec.rb

@@ -13,7 +13,7 @@
     end
 
     it 'should post notification to slack' do
-      #allow(SlackHook::Incoming).to receive(:new).and_return(slack_stub)
+      # allow(SlackHook::Incoming).to receive(:new).and_return(slack_stub)
       expect(slack_stub).not_to receive(:post)
       subject.call
     end