From 2e497d84380907ad61225d358024ac1805da85e1 Mon Sep 17 00:00:00 2001
From: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
Date: Mon, 25 Aug 2014 12:19:52 +0300
Subject: [PATCH] Prevent project stars duplication when fork project

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>
---
 app/services/projects/fork_service.rb         |  7 +++++-
 .../{ => projects}/fork_service_spec.rb       | 23 ++++++++-----------
 2 files changed, 15 insertions(+), 15 deletions(-)
 rename spec/services/{ => projects}/fork_service_spec.rb (74%)

diff --git a/app/services/projects/fork_service.rb b/app/services/projects/fork_service.rb
index 2f1c7b18aa0..66f0a02f0af 100644
--- a/app/services/projects/fork_service.rb
+++ b/app/services/projects/fork_service.rb
@@ -7,7 +7,12 @@ module Projects
     end
 
     def execute
-      project = @from_project.dup
+      project_params = {
+        visibility_level: @from_project.visibility_level,
+        description: @from_project.description,
+      }
+
+      project = Project.new(project_params)
       project.name = @from_project.name
       project.path = @from_project.path
       project.namespace = current_user.namespace
diff --git a/spec/services/fork_service_spec.rb b/spec/services/projects/fork_service_spec.rb
similarity index 74%
rename from spec/services/fork_service_spec.rb
rename to spec/services/projects/fork_service_spec.rb
index b6573095dbd..0edc3a8e807 100644
--- a/spec/services/fork_service_spec.rb
+++ b/spec/services/projects/fork_service_spec.rb
@@ -5,44 +5,40 @@ describe Projects::ForkService do
     before do
       @from_namespace = create(:namespace)
       @from_user = create(:user, namespace: @from_namespace )
-      @from_project = create(:project, creator_id: @from_user.id, namespace: @from_namespace)
+      @from_project = create(:project, creator_id: @from_user.id,
+                             namespace: @from_namespace, star_count: 107,
+                             description: 'wow such project')
       @to_namespace = create(:namespace)
       @to_user = create(:user, namespace: @to_namespace)
     end
 
     context 'fork project' do
+      describe "successfully creates project in the user namespace" do
+        let(:to_project) { fork_project(@from_project, @to_user) }
 
-      it "successfully creates project in the user namespace" do
-        @to_project = fork_project(@from_project, @to_user)
-
-        @to_project.owner.should == @to_user
-        @to_project.namespace.should == @to_user.namespace
+        it { to_project.owner.should == @to_user }
+        it { to_project.namespace.should == @to_user.namespace }
+        it { to_project.star_count.should be_zero }
+        it { to_project.description.should == @from_project.description }
       end
     end
 
     context 'fork project failure' do
-
       it "fails due to transaction failure" do
-        # make the mock gitlab-shell fail
         @to_project = fork_project(@from_project, @to_user, false)
-
         @to_project.errors.should_not be_empty
         @to_project.errors[:base].should include("Fork transaction failed.")
       end
-
     end
 
     context 'project already exists' do
-
       it "should fail due to validation, not transaction failure" do
         @existing_project = create(:project, creator_id: @to_user.id, name: @from_project.name, namespace: @to_namespace)
         @to_project = fork_project(@from_project, @to_user)
-
         @existing_project.persisted?.should be_true
         @to_project.errors[:base].should include("Invalid fork destination")
         @to_project.errors[:base].should_not include("Fork transaction failed.")
       end
-
     end
   end
 
@@ -53,5 +49,4 @@ describe Projects::ForkService do
     context.stub(gitlab_shell: shell)
     context.execute
   end
-
 end