feat: apply RuboCop auto correct

Author: Mattia Roccoberton

lib/tasks/active_storage_db_tasks.rake

@@ -19,12 +19,12 @@ end
 namespace :asdb do
   desc 'ActiveStorageDB: list attachments ordered by blob id desc'
   task list: [:environment] do |_t, _args|
-    query = ::ActiveStorage::Blob.order(id: :desc).limit(100)
+    query = ActiveStorage::Blob.order(id: :desc).limit(100)
     digits = query.ids.inject(0) { |ret, id| size = id.to_s.size; [size, ret].max }
 
-    ::ActiveStorage::Tasks.print_blob_header(digits: digits)
+    ActiveStorage::Tasks.print_blob_header(digits: digits)
     query.each do |blob|
-      ::ActiveStorage::Tasks.print_blob(blob, digits: digits)
+      ActiveStorage::Tasks.print_blob(blob, digits: digits)
     end
   end
 
@@ -34,7 +34,7 @@ namespace :asdb do
     destination = args[:destination]&.strip || Dir.pwd
     abort('Required arguments: source blob id, destination path') if blob_id.blank? || destination.blank?
 
-    blob = ::ActiveStorage::Blob.find_by(id: blob_id)
+    blob = ActiveStorage::Blob.find_by(id: blob_id)
     abort('Source file not found') unless blob
 
     destination = "#{destination}/#{blob.filename}" if Dir.exist?(destination)
@@ -50,12 +50,12 @@ namespace :asdb do
     filename = args[:filename]&.strip
     abort('Required arguments: filename') if filename.blank?
 
-    blobs = ::ActiveStorage::Blob.where('filename LIKE ?', "%#{filename}%").order(id: :desc)
+    blobs = ActiveStorage::Blob.where('filename LIKE ?', "%#{filename}%").order(id: :desc)
     if blobs.any?
       digits = blobs.ids.inject(0) { |ret, id| size = id.to_s.size; [size, ret].max }
-      ::ActiveStorage::Tasks.print_blob_header(digits: digits)
+      ActiveStorage::Tasks.print_blob_header(digits: digits)
       blobs.each do |blob|
-        ::ActiveStorage::Tasks.print_blob(blob, digits: digits)
+        ActiveStorage::Tasks.print_blob(blob, digits: digits)
       end
     else
       puts 'No results'

spec/rails_helper.rb

@@ -20,7 +20,7 @@
   add_filter %r{^/spec/}
   add_filter %r{^/vendor/}
 
-  case ENV['RAILS']
+  case ENV.fetch('RAILS', nil)
   when '6.0' then add_filter /_rails61|_rails70/
   when '6.1' then add_filter /_rails60|_rails70/
   when '7.0' then add_filter /_rails60|_rails61/
@@ -49,7 +49,7 @@
 
 RSpec.configure do |config|
   config.filter_rails_from_backtrace!
-  config.fixture_path = "#{::Rails.root}/spec/fixtures"
+  config.fixture_path = "#{Rails.root}/spec/fixtures"
   config.infer_spec_type_from_file_location!
   config.use_transactional_fixtures = true
 

spec/requests/file_controller_spec.rb

@@ -29,7 +29,7 @@ def create_blob_before_direct_upload(byte_size:, checksum:, filename: 'hello.txt
     }
   end
   let(:host) { "#{url_options[:protocol]}#{url_options[:host]}:#{url_options[:port]}" }
-  let(:engine_url_helpers) { ::ActiveStorageDB::Engine.routes.url_helpers }
+  let(:engine_url_helpers) { ActiveStorageDB::Engine.routes.url_helpers }
 
   before do
     if ActiveStorage::Current.respond_to? :url_options
@@ -142,7 +142,7 @@ def create_blob_before_direct_upload(byte_size:, checksum:, filename: 'hello.txt
       let(:invalid_file) { create(:active_storage_db_file, data: 'Some other data') }
 
       before do
-        allow(::ActiveStorageDB::File).to receive(:find_by).and_return(invalid_file)
+        allow(ActiveStorageDB::File).to receive(:find_by).and_return(invalid_file)
       end
 
       it 'fails to upload' do

spec/requests/file_url_spec.rb

@@ -2,7 +2,7 @@
 
 RSpec.describe 'File URL', type: :request do
   let(:app_url_helpers) { Rails.application.routes.url_helpers }
-  let(:engine_url_helpers) { ::ActiveStorageDB::Engine.routes.url_helpers }
+  let(:engine_url_helpers) { ActiveStorageDB::Engine.routes.url_helpers }
 
   it 'has the service URL in the engine URL helpers' do
     expect(app_url_helpers).to respond_to :active_storage_db_path

spec/service/active_storage/service/db_service_spec.rb

@@ -57,8 +57,8 @@
       let!(:db_file3) { create(:active_storage_db_file, ref: 'key3', data: 'third file') }
 
       it 'composes the source files' do
-        expect { compose }.to change { ::ActiveStorageDB::File.where(ref: 'dest_key').count }.by(1)
-        expect(compose).to be_kind_of ::ActiveStorageDB::File
+        expect { compose }.to change { ActiveStorageDB::File.where(ref: 'dest_key').count }.by(1)
+        expect(compose).to be_a ActiveStorageDB::File
         expect(compose.data).to eq [db_file1.data, db_file2.data, db_file3.data].join
       end
     end
@@ -154,7 +154,7 @@
 
   describe '.upload' do
     it 'uploads the data' do
-      expect(upload).to be_kind_of ActiveStorageDB::File
+      expect(upload).to be_a ActiveStorageDB::File
     ensure
       service.delete(key)
     end
@@ -163,7 +163,7 @@
       let(:upload) { service.upload(key, StringIO.new(fixture_data), checksum: checksum) }
 
       it 'uploads the data' do
-        expect(upload).to be_kind_of ActiveStorageDB::File
+        expect(upload).to be_a ActiveStorageDB::File
       ensure
         service.delete(key)
       end