From f1eaab7bf1f2ee14a66682c2e53f4e0700fc8243 Mon Sep 17 00:00:00 2001 From: Stan Hu Date: Sun, 10 Dec 2017 23:41:28 -0800 Subject: Remove the need for destroy and add a comment in the spec --- spec/lib/gitlab/bare_repository_import/importer_spec.rb | 7 +++---- 1 file changed, 3 insertions(+), 4 deletions(-) (limited to 'spec/lib/gitlab/bare_repository_import') diff --git a/spec/lib/gitlab/bare_repository_import/importer_spec.rb b/spec/lib/gitlab/bare_repository_import/importer_spec.rb index cce491fa10e..8a83e446935 100644 --- a/spec/lib/gitlab/bare_repository_import/importer_spec.rb +++ b/spec/lib/gitlab/bare_repository_import/importer_spec.rb @@ -134,12 +134,11 @@ describe Gitlab::BareRepositoryImport::Importer, repository: true do end it 'moves an existing project to the correct path' do - # This is a quick way to get a valid repository instead of copying an existing one + # This is a quick way to get a valid repository instead of copying an + # existing one. Since it's not persisted, the importer will try to + # create the project. project = build(:project, :repository) original_commit_count = project.repository.commit_count - # We need to destroy the model but keep the repository so that the - # importer will attempt to load the repository - project.destroy bare_repo = Gitlab::BareRepositoryImport::Repository.new(project.repository_storage_path, project.repository.path) gitlab_importer = described_class.new(admin, bare_repo) -- cgit v1.2.1