Backport EE changes to make test possible when prepending modules

1 files changed, 10 insertions, 6 deletions

diff --git a/spec/services/projects/hashed_storage_migration_service_spec.rb b/spec/services/projects/hashed_storage_migration_service_spec.rb
index 28b6daf217e..466f0b5d7c2 100644
--- a/spec/services/projects/hashed_storage_migration_service_spec.rb
+++ b/spec/services/projects/hashed_storage_migration_service_spec.rb
@@ -6,32 +6,36 @@ describe Projects::HashedStorageMigrationService do
 
   describe '#execute' do
     context 'repository migration' do
+      let(:repository_service) { Projects::HashedStorage::MigrateRepositoryService.new(project, subject.logger) }
+
       it 'delegates migration to Projects::HashedStorage::MigrateRepositoryService' do
-        expect(Projects::HashedStorage::MigrateRepositoryService).to receive(:new).with(project, subject.logger).and_call_original
-        expect_any_instance_of(Projects::HashedStorage::MigrateRepositoryService).to receive(:execute)
+        expect(Projects::HashedStorage::MigrateRepositoryService).to receive(:new).with(project, subject.logger).and_return(repository_service)
+        expect(repository_service).to receive(:execute)
 
         service.execute
       end
 
       it 'does not delegate migration if repository is already migrated' do
         project.storage_version = ::Project::LATEST_STORAGE_VERSION
-        expect_any_instance_of(Projects::HashedStorage::MigrateRepositoryService).not_to receive(:execute)
+        expect(Projects::HashedStorage::MigrateRepositoryService).not_to receive(:new)
 
         service.execute
       end
     end
 
     context 'attachments migration' do
+      let(:attachments_service) { Projects::HashedStorage::MigrateAttachmentsService.new(project, subject.logger) }
+
       it 'delegates migration to Projects::HashedStorage::MigrateRepositoryService' do
-        expect(Projects::HashedStorage::MigrateAttachmentsService).to receive(:new).with(project, subject.logger).and_call_original
-        expect_any_instance_of(Projects::HashedStorage::MigrateAttachmentsService).to receive(:execute)
+        expect(Projects::HashedStorage::MigrateAttachmentsService).to receive(:new).with(project, subject.logger).and_return(attachments_service)
+        expect(attachments_service).to receive(:execute)
 
         service.execute
       end
 
       it 'does not delegate migration if attachments are already migrated' do
         project.storage_version = ::Project::LATEST_STORAGE_VERSION
-        expect_any_instance_of(Projects::HashedStorage::MigrateAttachmentsService).not_to receive(:execute)
+        expect(Projects::HashedStorage::MigrateAttachmentsService).not_to receive(:new)
 
         service.execute
       end