2016-08-03 00:19:13 +00:00
|
|
|
require 'spec_helper'
|
|
|
|
|
2017-08-31 18:18:35 +00:00
|
|
|
describe Boards::IssuesController do
|
2017-08-02 19:55:11 +00:00
|
|
|
let(:project) { create(:project) }
|
2016-10-06 02:44:44 +00:00
|
|
|
let(:board) { create(:board, project: project) }
|
2016-08-03 00:19:13 +00:00
|
|
|
let(:user) { create(:user) }
|
2016-10-04 20:41:24 +00:00
|
|
|
let(:guest) { create(:user) }
|
2016-08-03 00:19:13 +00:00
|
|
|
|
2016-08-03 01:53:44 +00:00
|
|
|
let(:planning) { create(:label, project: project, name: 'Planning') }
|
|
|
|
let(:development) { create(:label, project: project, name: 'Development') }
|
|
|
|
|
2016-10-06 02:44:44 +00:00
|
|
|
let!(:list1) { create(:list, board: board, label: planning, position: 0) }
|
|
|
|
let!(:list2) { create(:list, board: board, label: development, position: 1) }
|
2016-08-03 01:53:44 +00:00
|
|
|
|
2016-08-03 00:19:13 +00:00
|
|
|
before do
|
2017-12-22 08:18:28 +00:00
|
|
|
project.add_master(user)
|
|
|
|
project.add_guest(guest)
|
2016-08-03 00:19:13 +00:00
|
|
|
end
|
|
|
|
|
2016-08-17 14:55:12 +00:00
|
|
|
describe 'GET index' do
|
2017-01-27 18:21:11 +00:00
|
|
|
let(:johndoe) { create(:user, avatar: fixture_file_upload(File.join(Rails.root, 'spec/fixtures/dk.png'))) }
|
2016-08-03 00:19:13 +00:00
|
|
|
|
2016-10-06 02:44:44 +00:00
|
|
|
context 'with invalid board id' do
|
|
|
|
it 'returns a not found 404 response' do
|
|
|
|
list_issues user: user, board: 999, list: list2
|
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2016-10-06 02:44:44 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-01-27 18:21:11 +00:00
|
|
|
context 'when list id is present' do
|
|
|
|
context 'with valid list id' do
|
|
|
|
it 'returns issues that have the list label applied' do
|
|
|
|
issue = create(:labeled_issue, project: project, labels: [planning])
|
|
|
|
create(:labeled_issue, project: project, labels: [planning])
|
|
|
|
create(:labeled_issue, project: project, labels: [development], due_date: Date.tomorrow)
|
2017-05-04 12:11:15 +00:00
|
|
|
create(:labeled_issue, project: project, labels: [development], assignees: [johndoe])
|
2017-01-27 18:21:11 +00:00
|
|
|
issue.subscribe(johndoe, project)
|
2016-08-03 00:19:13 +00:00
|
|
|
|
2017-01-27 18:21:11 +00:00
|
|
|
list_issues user: user, board: board, list: list2
|
|
|
|
|
|
|
|
parsed_response = JSON.parse(response.body)
|
|
|
|
|
|
|
|
expect(response).to match_response_schema('issues')
|
|
|
|
expect(parsed_response.length).to eq 2
|
2017-02-27 15:45:55 +00:00
|
|
|
expect(development.issues.map(&:relative_position)).not_to include(nil)
|
2017-01-27 18:21:11 +00:00
|
|
|
end
|
2017-09-12 19:49:01 +00:00
|
|
|
|
|
|
|
it 'avoids N+1 database queries' do
|
|
|
|
create(:labeled_issue, project: project, labels: [development])
|
|
|
|
control_count = ActiveRecord::QueryRecorder.new { list_issues(user: user, board: board, list: list2) }.count
|
|
|
|
|
|
|
|
# 25 issues is bigger than the page size
|
|
|
|
# the relative position will ignore the `#make_sure_position_set` queries
|
|
|
|
create_list(:labeled_issue, 25, project: project, labels: [development], assignees: [johndoe], relative_position: 1)
|
|
|
|
|
|
|
|
expect { list_issues(user: user, board: board, list: list2) }.not_to exceed_query_limit(control_count)
|
|
|
|
end
|
2017-01-27 18:21:11 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'with invalid list id' do
|
|
|
|
it 'returns a not found 404 response' do
|
|
|
|
list_issues user: user, board: board, list: 999
|
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2017-01-27 18:21:11 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when list id is missing' do
|
|
|
|
it 'returns opened issues without board labels applied' do
|
|
|
|
bug = create(:label, project: project, name: 'Bug')
|
|
|
|
create(:issue, project: project)
|
|
|
|
create(:labeled_issue, project: project, labels: [planning])
|
|
|
|
create(:labeled_issue, project: project, labels: [development])
|
|
|
|
create(:labeled_issue, project: project, labels: [bug])
|
|
|
|
|
|
|
|
list_issues user: user, board: board
|
|
|
|
|
|
|
|
parsed_response = JSON.parse(response.body)
|
|
|
|
|
|
|
|
expect(response).to match_response_schema('issues')
|
|
|
|
expect(parsed_response.length).to eq 2
|
2016-08-03 00:19:13 +00:00
|
|
|
end
|
|
|
|
end
|
2016-08-03 13:57:24 +00:00
|
|
|
|
2016-08-08 22:03:41 +00:00
|
|
|
context 'with unauthorized user' do
|
2016-08-17 15:15:51 +00:00
|
|
|
before do
|
2017-12-11 14:21:06 +00:00
|
|
|
allow(Ability).to receive(:allowed?).and_call_original
|
2016-08-08 18:55:13 +00:00
|
|
|
allow(Ability).to receive(:allowed?).with(user, :read_project, project).and_return(true)
|
|
|
|
allow(Ability).to receive(:allowed?).with(user, :read_issue, project).and_return(false)
|
2016-08-17 15:15:51 +00:00
|
|
|
end
|
2016-08-08 22:03:41 +00:00
|
|
|
|
2016-10-06 02:44:44 +00:00
|
|
|
it 'returns a forbidden 403 response' do
|
|
|
|
list_issues user: user, board: board, list: list2
|
2016-08-08 22:03:41 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(403)
|
2016-08-08 22:03:41 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2017-01-27 18:21:11 +00:00
|
|
|
def list_issues(user:, board:, list: nil)
|
2016-08-08 22:03:41 +00:00
|
|
|
sign_in(user)
|
|
|
|
|
2017-01-27 18:21:11 +00:00
|
|
|
params = {
|
|
|
|
namespace_id: project.namespace.to_param,
|
2017-02-23 23:55:01 +00:00
|
|
|
project_id: project,
|
2017-01-27 18:21:11 +00:00
|
|
|
board_id: board.to_param,
|
|
|
|
list_id: list.try(:to_param)
|
|
|
|
}
|
|
|
|
|
|
|
|
get :index, params.compact
|
2016-08-03 13:57:24 +00:00
|
|
|
end
|
2016-08-03 00:19:13 +00:00
|
|
|
end
|
2016-08-03 01:53:44 +00:00
|
|
|
|
2016-10-04 20:41:24 +00:00
|
|
|
describe 'POST create' do
|
|
|
|
context 'with valid params' do
|
|
|
|
it 'returns a successful 200 response' do
|
2016-10-06 20:13:33 +00:00
|
|
|
create_issue user: user, board: board, list: list1, title: 'New issue'
|
2016-10-04 20:41:24 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(200)
|
2016-10-04 20:41:24 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'returns the created issue' do
|
2016-10-06 20:13:33 +00:00
|
|
|
create_issue user: user, board: board, list: list1, title: 'New issue'
|
2016-10-04 20:41:24 +00:00
|
|
|
|
|
|
|
expect(response).to match_response_schema('issue')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with invalid params' do
|
|
|
|
context 'when title is nil' do
|
|
|
|
it 'returns an unprocessable entity 422 response' do
|
2016-10-06 20:13:33 +00:00
|
|
|
create_issue user: user, board: board, list: list1, title: nil
|
2016-10-04 20:41:24 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(422)
|
2016-10-04 20:41:24 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when list does not belongs to project board' do
|
|
|
|
it 'returns a not found 404 response' do
|
|
|
|
list = create(:list)
|
|
|
|
|
2016-10-06 20:13:33 +00:00
|
|
|
create_issue user: user, board: board, list: list, title: 'New issue'
|
2016-10-04 20:41:24 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2016-10-04 20:41:24 +00:00
|
|
|
end
|
|
|
|
end
|
2017-08-31 18:18:35 +00:00
|
|
|
|
|
|
|
context 'with invalid board id' do
|
|
|
|
it 'returns a not found 404 response' do
|
|
|
|
create_issue user: user, board: 999, list: list1, title: 'New issue'
|
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2017-08-31 18:18:35 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with invalid list id' do
|
|
|
|
it 'returns a not found 404 response' do
|
|
|
|
create_issue user: user, board: board, list: 999, title: 'New issue'
|
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2017-08-31 18:18:35 +00:00
|
|
|
end
|
|
|
|
end
|
2016-10-04 20:41:24 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
context 'with unauthorized user' do
|
|
|
|
it 'returns a forbidden 403 response' do
|
2016-10-06 20:13:33 +00:00
|
|
|
create_issue user: guest, board: board, list: list1, title: 'New issue'
|
2016-10-04 20:41:24 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(403)
|
2016-10-04 20:41:24 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-10-06 20:13:33 +00:00
|
|
|
def create_issue(user:, board:, list:, title:)
|
2016-10-04 20:41:24 +00:00
|
|
|
sign_in(user)
|
|
|
|
|
2017-08-31 18:18:35 +00:00
|
|
|
post :create, board_id: board.to_param,
|
2016-10-04 20:41:24 +00:00
|
|
|
list_id: list.to_param,
|
2017-08-31 18:18:35 +00:00
|
|
|
issue: { title: title, project_id: project.id },
|
2016-10-04 20:41:24 +00:00
|
|
|
format: :json
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-08-17 14:55:12 +00:00
|
|
|
describe 'PATCH update' do
|
2017-08-31 18:18:35 +00:00
|
|
|
let!(:issue) { create(:labeled_issue, project: project, labels: [planning]) }
|
2016-08-03 01:53:44 +00:00
|
|
|
|
|
|
|
context 'with valid params' do
|
|
|
|
it 'returns a successful 200 response' do
|
2016-10-06 02:44:44 +00:00
|
|
|
move user: user, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
|
2016-08-03 01:53:44 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(200)
|
2016-08-03 01:53:44 +00:00
|
|
|
end
|
|
|
|
|
|
|
|
it 'moves issue to the desired list' do
|
2016-10-06 02:44:44 +00:00
|
|
|
move user: user, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
|
2016-08-03 01:53:44 +00:00
|
|
|
|
|
|
|
expect(issue.reload.labels).to contain_exactly(development)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'with invalid params' do
|
|
|
|
it 'returns a unprocessable entity 422 response for invalid lists' do
|
2016-10-06 02:44:44 +00:00
|
|
|
move user: user, board: board, issue: issue, from_list_id: nil, to_list_id: nil
|
2016-08-03 01:53:44 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(422)
|
2016-08-03 01:53:44 +00:00
|
|
|
end
|
|
|
|
|
2016-10-06 02:44:44 +00:00
|
|
|
it 'returns a not found 404 response for invalid board id' do
|
|
|
|
move user: user, board: 999, issue: issue, from_list_id: list1.id, to_list_id: list2.id
|
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2016-10-06 02:44:44 +00:00
|
|
|
end
|
|
|
|
|
2016-08-03 01:53:44 +00:00
|
|
|
it 'returns a not found 404 response for invalid issue id' do
|
2017-08-31 18:18:35 +00:00
|
|
|
move user: user, board: board, issue: double(id: 999), from_list_id: list1.id, to_list_id: list2.id
|
2016-08-03 01:53:44 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(404)
|
2016-08-03 01:53:44 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-08-08 22:03:41 +00:00
|
|
|
context 'with unauthorized user' do
|
2016-10-06 02:44:44 +00:00
|
|
|
let(:guest) { create(:user) }
|
|
|
|
|
|
|
|
before do
|
2017-12-22 08:18:28 +00:00
|
|
|
project.add_guest(guest)
|
2016-10-06 02:44:44 +00:00
|
|
|
end
|
|
|
|
|
2016-10-04 20:41:24 +00:00
|
|
|
it 'returns a forbidden 403 response' do
|
2016-10-06 02:44:44 +00:00
|
|
|
move user: guest, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
|
2016-08-08 22:03:41 +00:00
|
|
|
|
2017-10-19 18:28:19 +00:00
|
|
|
expect(response).to have_gitlab_http_status(403)
|
2016-08-08 22:03:41 +00:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2016-10-06 02:44:44 +00:00
|
|
|
def move(user:, board:, issue:, from_list_id:, to_list_id:)
|
2016-08-08 22:03:41 +00:00
|
|
|
sign_in(user)
|
|
|
|
|
2016-08-03 01:53:44 +00:00
|
|
|
patch :update, namespace_id: project.namespace.to_param,
|
2017-08-31 18:18:35 +00:00
|
|
|
project_id: project.id,
|
2016-10-06 02:44:44 +00:00
|
|
|
board_id: board.to_param,
|
2017-08-31 18:18:35 +00:00
|
|
|
id: issue.id,
|
2016-08-15 21:39:26 +00:00
|
|
|
from_list_id: from_list_id,
|
|
|
|
to_list_id: to_list_id,
|
2016-08-03 01:53:44 +00:00
|
|
|
format: :json
|
|
|
|
end
|
|
|
|
end
|
2016-08-03 00:19:13 +00:00
|
|
|
end
|