1 files changed, 13 insertions, 44 deletions
diff --git a/spec/controllers/projects_controller_spec.rb b/spec/controllers/projects_controller_spec.rb
index 71bc49787cc..89bb35de8fc 100644
--- a/spec/controllers/projects_controller_spec.rb
+++ b/spec/controllers/projects_controller_spec.rb
@@ -7,56 +7,25 @@ describe ProjectsController do
   let(:jpg)     { fixture_file_upload(Rails.root + 'spec/fixtures/rails_sample.jpg', 'image/jpg') }
   let(:txt)     { fixture_file_upload(Rails.root + 'spec/fixtures/doc_sample.txt', 'text/plain') }
 
-  describe "POST #upload_image" do
-    before do
-      sign_in(user)
-      project.team << [user, :developer]
-    end
-
-    context "without params['markdown_img']" do
-      it "returns an error" do
-        post :upload_image, id: project.to_param, format: :json
-        expect(response.status).to eq(422)
-      end
-    end
-
-    context "with invalid file" do
-      before do
-        post :upload_image, id: project.to_param, markdown_img: txt, format: :json
-      end
-
-      it "returns an error" do
-        expect(response.status).to eq(422)
-      end
-    end
-
-    context "with valid file" do
-      before do
-        post :upload_image, id: project.to_param, markdown_img: jpg, format: :json
-      end
-
-      it "returns a content with original filename and new link." do
-        expect(response.body).to match "\"alt\":\"rails_sample\""
-        expect(response.body).to match "\"url\":\"http://test.host/uploads/#{project.path_with_namespace}"
-      end
-    end
-  end
-
   describe "POST #toggle_star" do
     it "toggles star if user is signed in" do
       sign_in(user)
-      expect(user.starred?(public_project)).to be_false
-      post :toggle_star, id: public_project.to_param
-      expect(user.starred?(public_project)).to be_true
-      post :toggle_star, id: public_project.to_param
-      expect(user.starred?(public_project)).to be_false
+      expect(user.starred?(public_project)).to be_falsey
+      post(:toggle_star, namespace_id: public_project.namespace.to_param,
+           id: public_project.to_param)
+      expect(user.starred?(public_project)).to be_truthy
+      post(:toggle_star, namespace_id: public_project.namespace.to_param,
+           id: public_project.to_param)
+      expect(user.starred?(public_project)).to be_falsey
     end
 
     it "does nothing if user is not signed in" do
-      post :toggle_star, id: public_project.to_param
-      expect(user.starred?(public_project)).to be_false
-      post :toggle_star, id: public_project.to_param
-      expect(user.starred?(public_project)).to be_false
+      post(:toggle_star, namespace_id: project.namespace.to_param,
+           id: public_project.to_param)
+      expect(user.starred?(public_project)).to be_falsey
+      post(:toggle_star, namespace_id: project.namespace.to_param,
+           id: public_project.to_param)
+      expect(user.starred?(public_project)).to be_falsey
     end
   end
 end