Skip to content

Change Geo::FileDownloadWorker for failed-sync to test the correct scenario

Leandro Gomes requested to merge leandrogs/gitlab:remove-duplicated-tests into master

What does this MR do?

It fixes RSpec/RepeatedExample cop at ee/spec/workers/geo/file_download_dispatch_worker_spec.rb as stated in #212541 (closed)

bundle exec rubocop ee/spec/workers/geo/file_download_dispatch_worker_spec.rb                                                                   
Inspecting 1 file
C

Offenses:

ee/spec/workers/geo/file_download_dispatch_worker_spec.rb:369:7: C: RSpec/RepeatedExample: Don't repeat examples within an example group.
      it 'performs Geo::FileDownloadWorker for unsynced job artifacts' do ...
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
ee/spec/workers/geo/file_download_dispatch_worker_spec.rb:378:7: C: RSpec/RepeatedExample: Don't repeat examples within an example group.
      it 'performs Geo::FileDownloadWorker for failed-sync job artifacts' do ...
      ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^

1 file inspected, 2 offenses detected

Screenshots

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Security

Edited by Leandro Gomes

Merge request reports