Skip to content
Snippets Groups Projects
Commit f8eba56b authored by James Fargher's avatar James Fargher
Browse files

Refactor RepositoryService disconnect_alternates spec

This test should be checking that the stub is called correctly like
every other test in this file does. The actual functionality is tested
further up the stack on Git::Repository.
parent a873c0f9
No related branches found
No related tags found
1 merge request!93862Refactor RepositoryService disconnect_alternates spec
......@@ -276,32 +276,12 @@
end
describe '#disconnect_alternates' do
let(:project) { create(:project, :repository) }
let(:repository) { project.repository }
let(:repository_path) { File.join(TestEnv.repos_path, repository.relative_path) }
let(:pool_repository) { create(:pool_repository) }
let(:object_pool) { pool_repository.object_pool }
let(:object_pool_service) { Gitlab::GitalyClient::ObjectPoolService.new(object_pool) }
before do
object_pool_service.create(repository) # rubocop:disable Rails/SaveBang
object_pool_service.link_repository(repository)
end
it 'deletes the alternates file' do
repository.disconnect_alternates
alternates_file = File.join(repository_path, "objects", "info", "alternates")
expect(File.exist?(alternates_file)).to be_falsey
end
context 'when called twice' do
it "doesn't raise an error" do
repository.disconnect_alternates
it 'sends a disconnect_git_alternates message' do
expect_any_instance_of(Gitaly::ObjectPoolService::Stub)
.to receive(:disconnect_git_alternates)
.with(gitaly_request_with_path(storage_name, relative_path), kind_of(Hash))
expect { repository.disconnect_alternates }.not_to raise_error
end
client.disconnect_alternates
end
end
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment