From c042d387caa405226ea3c3d3476c3f54d808fb0e Mon Sep 17 00:00:00 2001
From: Yorick Peterse <yorickpeterse@gmail.com>
Date: Wed, 11 May 2016 23:59:04 +0200
Subject: Added participant specs to various models

This adds specs for the "participants" method of Commit, Issue,
MergeRequest, Note, and Snippet.
---
 spec/models/commit_spec.rb        | 38 +++++++++++++++++++++++++++++++++++++-
 spec/models/issue_spec.rb         | 21 +++++++++++++++++++++
 spec/models/merge_request_spec.rb | 24 ++++++++++++++++++++++++
 spec/models/note_spec.rb          | 10 ++++++++++
 spec/models/snippet_spec.rb       | 27 +++++++++++++++++++++++++++
 5 files changed, 119 insertions(+), 1 deletion(-)

diff --git a/spec/models/commit_spec.rb b/spec/models/commit_spec.rb
index ad47e338a33..3eacddc2e5f 100644
--- a/spec/models/commit_spec.rb
+++ b/spec/models/commit_spec.rb
@@ -1,7 +1,7 @@
 require 'spec_helper'
 
 describe Commit, models: true do
-  let(:project) { create(:project) }
+  let(:project) { create(:project, :public) }
   let(:commit)  { project.commit }
 
   describe 'modules' do
@@ -171,4 +171,40 @@ eos
   describe '#status' do
     # TODO: kamil
   end
+
+  describe '#participants' do
+    let(:user1) { build(:user) }
+    let(:user2) { build(:user) }
+
+    let!(:note1) do
+      create(:note_on_commit,
+             commit_id: commit.id,
+             project: project,
+             note: 'foo')
+    end
+
+    let!(:note2) do
+      create(:note_on_commit,
+             commit_id: commit.id,
+             project: project,
+             note: 'bar')
+    end
+
+    before do
+      allow(commit).to receive(:author).and_return(user1)
+      allow(commit).to receive(:committer).and_return(user2)
+    end
+
+    it 'includes the commit author as the 1st participant' do
+      expect(commit.participants[0]).to eq(commit.author)
+    end
+
+    it 'includes the committer as the 2nd participant' do
+      expect(commit.participants[1]).to eq(commit.committer)
+    end
+
+    it 'includes the authors of the commit notes' do
+      expect(commit.participants).to include(note1.author, note2.author)
+    end
+  end
 end
diff --git a/spec/models/issue_spec.rb b/spec/models/issue_spec.rb
index 8ab00c70f9d..52c7f19abbe 100644
--- a/spec/models/issue_spec.rb
+++ b/spec/models/issue_spec.rb
@@ -231,4 +231,25 @@ describe Issue, models: true do
       expect(issue.to_branch_name).to match /confidential-issue\z/
     end
   end
+
+  describe '#participants' do
+    let(:project) { create(:project, :public) }
+    let(:issue) { create(:issue, project: project) }
+
+    let!(:note1) do
+      create(:note_on_issue, noteable: issue, project: project, note: 'a')
+    end
+
+    let!(:note2) do
+      create(:note_on_issue, noteable: issue, project: project, note: 'b')
+    end
+
+    it 'includes the issue author as the first participant' do
+      expect(issue.participants[0]).to eq(issue.author)
+    end
+
+    it 'includes the authors of the notes' do
+      expect(issue.participants).to include(note1.author, note2.author)
+    end
+  end
 end
diff --git a/spec/models/merge_request_spec.rb b/spec/models/merge_request_spec.rb
index c8578749b21..40c74a32fad 100644
--- a/spec/models/merge_request_spec.rb
+++ b/spec/models/merge_request_spec.rb
@@ -395,4 +395,28 @@ describe MergeRequest, models: true do
       end
     end
   end
+
+  describe '#participants' do
+    let(:project) { create(:project, :public) }
+
+    let(:mr) do
+      create(:merge_request, source_project: project, target_project: project)
+    end
+
+    let!(:note1) do
+      create(:note_on_merge_request, noteable: mr, project: project, note: 'a')
+    end
+
+    let!(:note2) do
+      create(:note_on_merge_request, noteable: mr, project: project, note: 'b')
+    end
+
+    it 'includes the merge request author as the first participant' do
+      expect(mr.participants[0]).to eq(mr.author)
+    end
+
+    it 'includes the authors of the notes' do
+      expect(mr.participants).to include(note1.author, note2.author)
+    end
+  end
 end
diff --git a/spec/models/note_spec.rb b/spec/models/note_spec.rb
index 01ab6154bba..dabc9280f36 100644
--- a/spec/models/note_spec.rb
+++ b/spec/models/note_spec.rb
@@ -284,4 +284,14 @@ describe Note, models: true do
       expect { note.valid? }.to change(note, :line_code).to(nil)
     end
   end
+
+  describe '#participants' do
+    it 'includes the note author' do
+      project = create(:project, :public)
+      issue = create(:issue, project: project)
+      note = create(:note_on_issue, noteable: issue, project: project)
+
+      expect(note.participants).to include(note.author)
+    end
+  end
 end
diff --git a/spec/models/snippet_spec.rb b/spec/models/snippet_spec.rb
index 7a613e360d4..132972ae79c 100644
--- a/spec/models/snippet_spec.rb
+++ b/spec/models/snippet_spec.rb
@@ -87,4 +87,31 @@ describe Snippet, models: true do
       expect(described_class.search_code('FOO')).to eq([snippet])
     end
   end
+
+  describe '#participants' do
+    let(:project) { create(:project, :public) }
+    let(:snippet) { create(:snippet, content: 'foo', project: project) }
+
+    let!(:note1) do
+      create(:note_on_project_snippet,
+             noteable: snippet,
+             project: project,
+             note: 'a')
+    end
+
+    let!(:note2) do
+      create(:note_on_project_snippet,
+             noteable: snippet,
+             project: project,
+             note: 'b')
+    end
+
+    it 'includes the snippet author as the 1st participant' do
+      expect(snippet.participants[0]).to eq(snippet.author)
+    end
+
+    it 'includes the note authors' do
+      expect(snippet.participants).to include(note1.author, note2.author)
+    end
+  end
 end
-- 
cgit v1.2.1