Update tests and fix Finders readme

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

1 files changed, 58 insertions, 0 deletions

diff --git a/spec/finders/issues_finder_spec.rb b/spec/finders/issues_finder_spec.rb
new file mode 100644
index 00000000000..7489e56f423
--- /dev/null
+++ b/spec/finders/issues_finder_spec.rb
@@ -0,0 +1,58 @@
+require 'spec_helper'
+
+describe IssuesFinder do
+  let(:user) { create :user }
+  let(:user2) { create :user }
+  let(:project1) { create(:project) }
+  let(:project2) { create(:project) }
+  let(:issue1) { create(:issue, assignee: user, project: project1) }
+  let(:issue2) { create(:issue, assignee: user, project: project2) }
+  let(:issue3) { create(:issue, assignee: user2, project: project2) }
+
+  before do
+    project1.team << [user, :master]
+    project2.team << [user, :developer]
+    project2.team << [user2, :developer]
+  end
+
+  describe :execute do
+    before :each do
+      issue1
+      issue2
+      issue3
+    end
+
+    it 'should filter by all' do
+      params = { scope: "all", state: 'opened' }
+      issues = IssuesFinder.new.execute(user, params)
+      issues.size.should == 3
+    end
+
+    it 'should filter by assignee' do
+      params = { scope: "assigned-to-me", state: 'opened' }
+      issues = IssuesFinder.new.execute(user, params)
+      issues.size.should == 2
+    end
+
+    it 'should filter by project' do
+      params = { scope: "assigned-to-me", state: 'opened', project_id: project1.id }
+      issues = IssuesFinder.new.execute(user, params)
+      issues.size.should == 1
+    end
+
+    it 'should be empty for unauthorized user' do
+      params = { scope: "all", state: 'opened' }
+      issues = IssuesFinder.new.execute(nil, params)
+      issues.size.should be_zero
+    end
+
+    it 'should not include unauthorized issues' do
+      params = { scope: "all", state: 'opened' }
+      issues = IssuesFinder.new.execute(user2, params)
+      issues.size.should == 2
+      issues.should_not include(issue1)
+      issues.should include(issue2)
+      issues.should include(issue3)
+    end
+  end
+end