Optimize Geo specs by using truncation strategy for DB cleaner only when necessary

Closes #4259

spec/ee/spec/finders/geo/attachment_registry_finder_spec.rb

 require 'spec_helper'
 
-# Disable transactions via :truncate method because a foreign table
-# can't see changes inside a transaction of a different connection.
-describe Geo::AttachmentRegistryFinder, :geo, :truncate do
+describe Geo::AttachmentRegistryFinder, :geo do
   include ::EE::GeoHelpers
 
   let(:secondary) { create(:geo_node) }
@@ -28,7 +26,9 @@ describe Geo::AttachmentRegistryFinder, :geo, :truncate do
     stub_current_geo_node(secondary)
   end
 
-  context 'FDW' do
+  # Disable transactions via :truncate method because a foreign table
+  # can't see changes inside a transaction of a different connection.
+  context 'FDW', :truncate do
     before do
       skip('FDW is not configured') if Gitlab::Database.postgresql? && !Gitlab::Geo.fdw?
     end

spec/ee/spec/finders/geo/expire_uploads_finder_spec.rb

 require 'spec_helper'
 
-# Disable transactions via :truncate method because a foreign table
-# can't see changes inside a transaction of a different connection.
-describe Geo::ExpireUploadsFinder, :geo, :truncate do
+describe Geo::ExpireUploadsFinder, :geo do
   let(:project) { create(:project) }
 
-  context 'FDW' do
+  # Disable transactions via :truncate method because a foreign table
+  # can't see changes inside a transaction of a different connection.
+  context 'FDW', :truncate do
     before do
       skip('FDW is not configured') if Gitlab::Database.postgresql? && !Gitlab::Geo.fdw?
     end

spec/ee/spec/finders/geo/project_registry_finder_spec.rb

 require 'spec_helper'
 
-# Disable transactions via :truncate method because a foreign table
-# can't see changes inside a transaction of a different connection.
-describe Geo::ProjectRegistryFinder, :geo, :truncate do
+describe Geo::ProjectRegistryFinder, :geo do
   include ::EE::GeoHelpers
 
   let(:secondary) { create(:geo_node) }
@@ -154,7 +152,9 @@ describe Geo::ProjectRegistryFinder, :geo, :truncate do
     end
   end
 
-  context 'FDW' do
+  # Disable transactions via :truncate method because a foreign table
+  # can't see changes inside a transaction of a different connection.
+  context 'FDW', :truncate do
     before do
       skip('FDW is not configured') if Gitlab::Database.postgresql? && !Gitlab::Geo.fdw?
     end

spec/models/geo_node_status_spec.rb

 require 'spec_helper'
 
-# Disable transactions via :truncate method because a foreign table
-# can't see changes inside a transaction of a different connection.
-describe GeoNodeStatus, :geo, :truncate do
+describe GeoNodeStatus, :geo do
   include ::EE::GeoHelpers
 
   let!(:primary)  { create(:geo_node, :primary) }
@@ -52,7 +50,9 @@ describe GeoNodeStatus, :geo, :truncate do
     end
   end
 
-  describe '#attachments_synced_count' do
+  # Disable transactions via :truncate method because a foreign table
+  # can't see changes inside a transaction of a different connection.
+  describe '#attachments_synced_count', :truncate do
     it 'only counts successful syncs' do
       create_list(:user, 3, avatar: fixture_file_upload(Rails.root + 'spec/fixtures/dk.png', 'image/png'))
       uploads = Upload.all.pluck(:id)
@@ -95,7 +95,7 @@ describe GeoNodeStatus, :geo, :truncate do
     end
   end
 
-  describe '#attachments_failed_count' do
+  describe '#attachments_failed_count', :truncate do
     it 'counts failed avatars, attachment, personal snippets and files' do
       # These two should be ignored
       create(:geo_file_registry, :lfs, :with_file, success: false)
@@ -110,7 +110,7 @@ describe GeoNodeStatus, :geo, :truncate do
     end
   end
 
-  describe '#attachments_synced_in_percentage' do
+  describe '#attachments_synced_in_percentage', :truncate do
     let(:avatar) { fixture_file_upload(Rails.root.join('spec/fixtures/dk.png')) }
     let(:upload_1) { create(:upload, model: group, path: avatar) }
     let(:upload_2) { create(:upload, model: project_1, path: avatar) }

spec/workers/geo/file_download_dispatch_worker_spec.rb

@@ -2,7 +2,7 @@ require 'spec_helper'
 
 # Disable transactions via :truncate method because a foreign table
 # can't see changes inside a transaction of a different connection.
-describe Geo::FileDownloadDispatchWorker, :geo, :truncate do
+describe Geo::FileDownloadDispatchWorker, :geo do
   include ::EE::GeoHelpers
 
   let(:primary)   { create(:geo_node, :primary, host: 'primary-geo-node') }
@@ -171,7 +171,7 @@ describe Geo::FileDownloadDispatchWorker, :geo, :truncate do
     end
   end
 
-  describe 'when PostgreSQL FDW is available', :geo do
+  describe 'when PostgreSQL FDW is available', :geo, :truncate do
     # Skip if FDW isn't activated on this database
     it_behaves_like '#perform', Gitlab::Database.postgresql? && !Gitlab::Geo.fdw?
   end