Simple model spec changes made possible by new factories

1 files changed, 6 insertions, 18 deletions

diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index c7ad08a1e06..f4b93eea966 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -20,46 +20,34 @@ describe MergeRequest do
     it { MergeRequest.should respond_to :opened }
   end
 
-  it { Factory.create(:merge_request,
-                      author: Factory(:user),
-                      assignee: Factory(:user),
-                      project: Factory.create(:project)).should be_valid }
-
   describe "plus 1" do
-    let(:project) { Factory(:project) }
-    subject {
-      Factory.create(:merge_request,
-                     author: Factory(:user),
-                     assignee: Factory(:user),
-                     project: project)
-    }
+    subject { Factory.create(:merge_request) }
 
     it "with no notes has a 0/0 score" do
       subject.upvotes.should == 0
     end
 
     it "should recognize non-+1 notes" do
-      subject.notes << Factory(:note, note: "No +1 here", project: Factory(:project, path: 'plusone', code: 'plusone'))
+      subject.notes << Factory(:note, note: "No +1 here")
       subject.should have(1).note
       subject.notes.first.upvote?.should be_false
       subject.upvotes.should == 0
     end
 
     it "should recognize a single +1 note" do
-      subject.notes << Factory(:note, note: "+1 This is awesome", project: Factory(:project, path: 'plusone', code: 'plusone'))
+      subject.notes << Factory(:note, note: "+1 This is awesome")
       subject.upvotes.should == 1
     end
 
     it "should recognize a multiple +1 notes" do
-      subject.notes << Factory(:note, note: "+1 This is awesome", project: Factory(:project, path: 'plusone', code: 'plusone'))
-      subject.notes << Factory(:note, note: "+1 I want this", project: Factory(:project, path: 'plustwo', code: 'plustwo'))
+      subject.notes << Factory(:note, note: "+1 This is awesome")
+      subject.notes << Factory(:note, note: "+1 I want this")
       subject.upvotes.should == 2
     end
   end
 
   describe ".search" do
-    let!(:issue) { Factory.create(:issue, title: "Searchable issue",
-                                 project: Factory.create(:project)) }
+    let!(:issue) { Factory.create(:issue, title: "Searchable issue") }
 
     it "matches by title" do
       Issue.search('able').all.should == [issue]