Correct usage of `subject` in specs

4 files changed, 9 insertions, 5 deletions

diff --git a/spec/models/commit_spec.rb b/spec/models/commit_spec.rb
index 11cc7762ce4..2cc23efb986 100644
--- a/spec/models/commit_spec.rb
+++ b/spec/models/commit_spec.rb
@@ -69,8 +69,9 @@ eos
   end
 
   it_behaves_like 'a mentionable' do
-    let(:subject) { commit }
     let(:mauthor) { create :user, email: commit.author_email }
+    subject { commit }
+
     let(:backref_text) { "commit #{subject.id}" }
     let(:set_mentionable_text) { ->(txt){ subject.stub(safe_message: txt) } }
 
diff --git a/spec/models/issue_spec.rb b/spec/models/issue_spec.rb
index 087e40c3d84..20d823b40e5 100644
--- a/spec/models/issue_spec.rb
+++ b/spec/models/issue_spec.rb
@@ -56,7 +56,8 @@ describe Issue do
   end
 
   it_behaves_like 'an editable mentionable' do
-    let(:subject) { create :issue, project: mproject }
+    subject { create(:issue, project: project) }
+
     let(:backref_text) { "issue ##{subject.iid}" }
     let(:set_mentionable_text) { ->(txt){ subject.description = txt } }
   end
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index d40503d791c..3fcd063efe8 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -116,12 +116,13 @@ describe MergeRequest do
   end
 
   it_behaves_like 'an editable mentionable' do
-    let(:subject) { create :merge_request, source_project: mproject, target_project: mproject }
+    subject { create(:merge_request, source_project: project, target_project: project) }
+
     let(:backref_text) { "merge request !#{subject.iid}" }
     let(:set_mentionable_text) { ->(txt){ subject.title = txt } }
   end
 
   it_behaves_like 'a Taskable' do
-    let(:subject) { create :merge_request, :simple }
+    subject { create :merge_request, :simple }
   end
 end
diff --git a/spec/models/note_spec.rb b/spec/models/note_spec.rb
index a7bf5081d5b..e7e6717e775 100644
--- a/spec/models/note_spec.rb
+++ b/spec/models/note_spec.rb
@@ -629,8 +629,9 @@ describe Note do
   end
 
   it_behaves_like 'an editable mentionable' do
+    subject { create :note, noteable: issue, project: project }
+
     let(:issue) { create :issue, project: project }
-    let(:subject) { create :note, noteable: issue, project: project }
     let(:backref_text) { issue.gfm_reference }
     let(:set_mentionable_text) { ->(txt) { subject.note = txt } }
   end