Merge pull request #5500 from consuldemocracy/remove_bullet

Remove Bullet from Gemfile

Gemfile

@@ -64,7 +64,6 @@ gem "wicked_pdf", "~> 2.8.0"
 gem "wkhtmltopdf-binary", "~> 0.12.6"
 
 group :development, :test do
-  gem "bullet", "~> 7.1.6"
   gem "debug", "~> 1.9.2"
   gem "factory_bot_rails", "~> 6.4.3"
   gem "faker", "~> 3.3.1"

Gemfile.lock

@@ -109,9 +109,6 @@ GEM
     bing_translator (6.2.0)
       json
     builder (3.2.4)
-    bullet (7.1.6)
-      activesupport (>= 3.0.0)
-      uniform_notifier (~> 1.11)
     cancancan (3.5.0)
     capistrano (3.18.1)
       airbrussh (>= 1.0.0)
@@ -658,7 +655,6 @@ GEM
     uglifier (4.2.0)
       execjs (>= 0.3.0, < 3)
     unicode-display_width (2.5.0)
-    uniform_notifier (1.16.0)
     uri (0.13.0)
     uuidtools (2.2.0)
     version_gem (1.1.3)
@@ -703,7 +699,6 @@ DEPENDENCIES
   audited (~> 5.4.3)
   autoprefixer-rails (~> 10.4.16)
   bing_translator (~> 6.2.0)
-  bullet (~> 7.1.6)
   cancancan (~> 3.5.0)
   capistrano (~> 3.18.1)
   capistrano-bundler (~> 2.1.0)

config/environments/development.rb

@@ -79,15 +79,6 @@ Rails.application.configure do
   logger.formatter = config.log_formatter
   config.logger = ActiveSupport::TaggedLogging.new(logger)
 
-  config.after_initialize do
-    Bullet.enable = true
-    Bullet.bullet_logger = true
-    if ENV["BULLET"]
-      Bullet.rails_logger = true
-      Bullet.add_footer = true
-    end
-  end
-
   # Uncomment if you wish to allow Action Cable access from any origin.
   # config.action_cable.disable_request_forgery_protection = true
 end

config/environments/test.rb

@@ -72,14 +72,6 @@ Rails.application.configure do
   logger.formatter = config.log_formatter
   config.logger = ActiveSupport::TaggedLogging.new(logger)
 
-  config.after_initialize do
-    Bullet.enable = true
-    Bullet.bullet_logger = true
-    if ENV["BULLET"]
-      Bullet.raise = true # raise an error if n+1 query occurs
-    end
-  end
-
   # Allow managing different tenants using the same application
   config.multitenancy = true
 

spec/spec_helper.rb

@@ -69,15 +69,9 @@ RSpec.configure do |config|
   end
 
   config.before(:each, type: :system) do
-    Bullet.start_request
     allow(InvisibleCaptcha).to receive(:timestamp_threshold).and_return(0)
   end
 
-  config.after(:each, type: :system) do
-    Bullet.perform_out_of_channel_notifications if Bullet.notification?
-    Bullet.end_request
-  end
-
   config.before(:each, :admin, type: :system) do
     login_as(create(:administrator).user)
   end