Updated rspec

Author: pramod-mahadev-bhat-1996

lib/rubocop/cop/bugcrowd/prevent_bugsnag_usage.rb

@@ -3,31 +3,21 @@
 module RuboCop
   module Cop
     module Bugcrowd
-      class PreventBugsnagUsage < Cop
+      class PreventBugsnagUsage < RuboCop::Cop::Base
+        extend RuboCop::Cop::AutoCorrector
+
         MSG = 'Avoid using Bugsnag in the codebase. ' \
               'It has been replaced with ErrorNotifierService for error ' \
-              'notification handling. Please use ErrorNotifierService' \
-              'instead.'
-
-        def_node_matcher :bugsnag_usage?, <<-PATTERN
-          (send
-            (const nil? :Bugsnag) :notify ...)
-        PATTERN
+              'notification handling. Please use ErrorNotifierService instead.'
 
-        def_node_matcher :bugsnag_constant?, <<-PATTERN
-          (const nil? :Bugsnag)
+        def_node_matcher :bugsnag_usage?, <<~PATTERN
+          (send (const nil? :Bugsnag) ...)
         PATTERN
 
         def on_send(node)
-          if bugsnag_usage?(node)
-            add_offense(node, message: MSG)
-          end
-        end
+          return unless bugsnag_usage?(node)
 
-        def on_const(node)
-          if bugsnag_constant?(node)
-            add_offense(node, message: MSG)
-          end
+          add_offense(node, message: MSG)
         end
       end
     end

spec/rubocop/cop/bugcrowd/prevent_bugsnag_usage_spec.rb

@@ -1,21 +1,19 @@
 # frozen_string_literal: true
 
-RSpec.describe RuboCop::Cop::Bugcrowd::PreventBugsnagUsage do
+RSpec.describe RuboCop::Cop::Bugcrowd::PreventBugsnagUsage, :config do
   subject(:cop) { described_class.new(config) }
-  let(:config) { RuboCop::Config.new }
 
-  it 'registers an offense when Bugsnag.notify is used' do
+  it 'registers an offense when Bugsnag is used as a constant' do
     expect_offense(<<~RUBY)
-      Bugsnag.notify('Error')
-      ^^^^^^^ Bugcrowd/PreventBugsnagUsage: Avoid using Bugsnag in the codebase. It has been replaced with ErrorNotifierService for error notification handling. Please use ErrorNotifierServiceinstead.
-      ^^^^^^^^^^^^^^^^^^^^^^^ Bugcrowd/PreventBugsnagUsage: Avoid using Bugsnag in the codebase. It has been replaced with ErrorNotifierService for error notification handling. Please use ErrorNotifierServiceinstead.
+      Bugsnag.error('Error')
+      ^^^^^^^^^^^^^^^^^^^^^^ Avoid using Bugsnag in the codebase. It has been replaced with ErrorNotifierService for error notification handling. Please use ErrorNotifierService instead.
     RUBY
   end
 
-  it 'registers an offense when Bugsnag is used as a constant' do
+  it 'registers an offense when Bugsnag.notify is used' do
     expect_offense(<<~RUBY)
-      Bugsnag.error('Error')
-      ^^^^^^^ Bugcrowd/PreventBugsnagUsage: Avoid using Bugsnag in the codebase. It has been replaced with ErrorNotifierService for error notification handling. Please use ErrorNotifierServiceinstead.
+      Bugsnag.notify('Error')
+      ^^^^^^^^^^^^^^^^^^^^^^^ Avoid using Bugsnag in the codebase. It has been replaced with ErrorNotifierService for error notification handling. Please use ErrorNotifierService instead.
     RUBY
   end
 
@@ -24,10 +22,4 @@
       ErrorNotifierService.notify('Error')
     RUBY
   end
-
-  it 'does not register an offense when no Bugsnag is used' do
-    expect_no_offenses(<<~RUBY)
-      SomeOtherService.notify('Error')
-    RUBY
-  end
 end