debian-mirror-gitlab/spec/controllers/boards/issues_controller_spec.rb

332 lines
11 KiB
Ruby
Raw Normal View History

2019-07-31 22:56:46 +05:30
# frozen_string_literal: true
2016-09-13 17:45:13 +05:30
require 'spec_helper'
2018-03-17 18:26:18 +05:30
describe Boards::IssuesController do
2019-07-07 11:18:12 +05:30
include ExternalAuthorizationServiceHelpers
2019-03-02 22:35:43 +05:30
let(:project) { create(:project, :private) }
2016-11-03 12:29:30 +05:30
let(:board) { create(:board, project: project) }
2016-09-13 17:45:13 +05:30
let(:user) { create(:user) }
2016-11-03 12:29:30 +05:30
let(:guest) { create(:user) }
2016-09-13 17:45:13 +05:30
let(:planning) { create(:label, project: project, name: 'Planning') }
let(:development) { create(:label, project: project, name: 'Development') }
2016-11-03 12:29:30 +05:30
let!(:list1) { create(:list, board: board, label: planning, position: 0) }
let!(:list2) { create(:list, board: board, label: development, position: 1) }
2016-09-13 17:45:13 +05:30
before do
2018-11-18 11:00:15 +05:30
project.add_maintainer(user)
2018-03-17 18:26:18 +05:30
project.add_guest(guest)
2016-09-13 17:45:13 +05:30
end
2018-11-08 19:23:39 +05:30
describe 'GET index', :request_store do
let(:johndoe) { create(:user, avatar: fixture_file_upload(File.join('spec/fixtures/dk.png'))) }
2016-09-13 17:45:13 +05:30
2016-11-03 12:29:30 +05:30
context 'with invalid board id' do
it 'returns a not found 404 response' do
list_issues user: user, board: 999, list: list2
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(404)
2016-11-03 12:29:30 +05:30
end
end
2017-08-17 22:00:37 +05:30
context 'when list id is present' do
context 'with valid list id' do
2018-12-13 13:39:08 +05:30
let(:group) { create(:group, :private, projects: [project]) }
let(:group_board) { create(:board, group: group) }
let!(:list3) { create(:list, board: group_board, label: development, position: 2) }
let(:sub_group_1) { create(:group, :private, parent: group) }
before do
group.add_maintainer(user)
end
2017-08-17 22:00:37 +05:30
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)
create(:labeled_issue, project: project, labels: [development], assignees: [johndoe])
issue.subscribe(johndoe, project)
2016-09-13 17:45:13 +05:30
2017-08-17 22:00:37 +05:30
list_issues user: user, board: board, list: list2
parsed_response = JSON.parse(response.body)
2019-02-15 15:39:39 +05:30
expect(response).to match_response_schema('entities/issue_boards')
2018-11-18 11:00:15 +05:30
expect(parsed_response['issues'].length).to eq 2
2017-08-17 22:00:37 +05:30
expect(development.issues.map(&:relative_position)).not_to include(nil)
end
2018-03-17 18:26:18 +05:30
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
2018-12-13 13:39:08 +05:30
it 'avoids N+1 database queries when adding a project', :request_store do
create(:labeled_issue, project: project, labels: [development])
control_count = ActiveRecord::QueryRecorder.new { list_issues(user: user, board: group_board, list: list3) }.count
2.times do
p = create(:project, group: group)
create(:labeled_issue, project: p, labels: [development])
end
project_2 = create(:project, group: group)
create(:labeled_issue, project: project_2, labels: [development], assignees: [johndoe])
# because each issue without relative_position must be updated with
# a different value, we have 8 extra queries per issue
expect { list_issues(user: user, board: group_board, list: list3) }.not_to exceed_query_limit(control_count + (2 * 8 - 1))
end
it 'avoids N+1 database queries when adding a subgroup, project, and issue', :nested_groups do
create(:project, group: sub_group_1)
create(:labeled_issue, project: project, labels: [development])
control_count = ActiveRecord::QueryRecorder.new { list_issues(user: user, board: group_board, list: list3) }.count
project_2 = create(:project, group: group)
2.times do
p = create(:project, group: sub_group_1)
create(:labeled_issue, project: p, labels: [development])
end
create(:labeled_issue, project: project_2, labels: [development], assignees: [johndoe])
expect { list_issues(user: user, board: group_board, list: list3) }.not_to exceed_query_limit(control_count + (2 * 8 - 1))
end
2017-08-17 22:00:37 +05:30
end
context 'with invalid list id' do
it 'returns a not found 404 response' do
list_issues user: user, board: board, list: 999
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(404)
2017-08-17 22:00:37 +05:30
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)
2019-02-15 15:39:39 +05:30
expect(response).to match_response_schema('entities/issue_boards')
2018-11-18 11:00:15 +05:30
expect(parsed_response['issues'].length).to eq 2
2016-09-13 17:45:13 +05:30
end
end
context 'with unauthorized user' do
2019-03-02 22:35:43 +05:30
let(:unauth_user) { create(:user) }
2016-09-13 17:45:13 +05:30
2016-11-03 12:29:30 +05:30
it 'returns a forbidden 403 response' do
2019-03-02 22:35:43 +05:30
list_issues user: unauth_user, board: board, list: list2
2016-09-13 17:45:13 +05:30
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(403)
2016-09-13 17:45:13 +05:30
end
end
2019-07-07 11:18:12 +05:30
context 'with external authorization' do
before do
sign_in(user)
enable_external_authorization_service_check
end
it 'returns a 403 for group boards' do
group = create(:group)
group_board = create(:board, group: group)
list_issues(user: user, board: group_board)
expect(response).to have_gitlab_http_status(403)
end
it 'is successful for project boards' do
project_board = create(:board, project: project)
list_issues(user: user, board: project_board)
expect(response).to have_gitlab_http_status(200)
end
end
2017-08-17 22:00:37 +05:30
def list_issues(user:, board:, list: nil)
2016-09-13 17:45:13 +05:30
sign_in(user)
2017-08-17 22:00:37 +05:30
params = {
board_id: board.to_param,
list_id: list.try(:to_param)
}
2018-12-13 13:39:08 +05:30
unless board.try(:parent)&.is_a?(Group)
params[:namespace_id] = project.namespace.to_param
params[:project_id] = project
end
2019-02-15 15:39:39 +05:30
get :index, params: params.compact
2016-11-03 12:29:30 +05:30
end
end
describe 'POST create' do
context 'with valid params' do
it 'returns a successful 200 response' do
create_issue user: user, board: board, list: list1, title: 'New issue'
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(200)
2016-11-03 12:29:30 +05:30
end
it 'returns the created issue' do
create_issue user: user, board: board, list: list1, title: 'New issue'
2019-02-15 15:39:39 +05:30
expect(response).to match_response_schema('entities/issue_board')
2016-11-03 12:29:30 +05:30
end
end
context 'with invalid params' do
context 'when title is nil' do
it 'returns an unprocessable entity 422 response' do
create_issue user: user, board: board, list: list1, title: nil
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(422)
2016-11-03 12:29:30 +05:30
end
end
context 'when list does not belongs to project board' do
it 'returns a not found 404 response' do
list = create(:list)
create_issue user: user, board: board, list: list, title: 'New issue'
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(404)
end
end
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'
expect(response).to have_gitlab_http_status(404)
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'
expect(response).to have_gitlab_http_status(404)
2016-11-03 12:29:30 +05:30
end
end
end
2018-12-13 13:39:08 +05:30
context 'with guest user' do
context 'in open list' do
it 'returns a successful 200 response' do
open_list = board.lists.create(list_type: :backlog)
create_issue user: guest, board: board, list: open_list, title: 'New issue'
2016-11-03 12:29:30 +05:30
2018-12-13 13:39:08 +05:30
expect(response).to have_gitlab_http_status(200)
end
end
context 'in label list' do
it 'returns a forbidden 403 response' do
create_issue user: guest, board: board, list: list1, title: 'New issue'
expect(response).to have_gitlab_http_status(403)
end
2016-11-03 12:29:30 +05:30
end
end
def create_issue(user:, board:, list:, title:)
sign_in(user)
2019-02-15 15:39:39 +05:30
post :create, params: {
board_id: board.to_param,
list_id: list.to_param,
2019-03-02 22:35:43 +05:30
issue: { title: title, project_id: project.id }
2019-02-15 15:39:39 +05:30
},
2016-11-03 12:29:30 +05:30
format: :json
2016-09-13 17:45:13 +05:30
end
end
describe 'PATCH update' do
2018-03-17 18:26:18 +05:30
let!(:issue) { create(:labeled_issue, project: project, labels: [planning]) }
2016-09-13 17:45:13 +05:30
context 'with valid params' do
it 'returns a successful 200 response' do
2016-11-03 12:29:30 +05:30
move user: user, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
2016-09-13 17:45:13 +05:30
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(200)
2016-09-13 17:45:13 +05:30
end
it 'moves issue to the desired list' do
2016-11-03 12:29:30 +05:30
move user: user, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
2016-09-13 17:45:13 +05:30
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-11-03 12:29:30 +05:30
move user: user, board: board, issue: issue, from_list_id: nil, to_list_id: nil
2016-09-13 17:45:13 +05:30
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(422)
2016-09-13 17:45:13 +05:30
end
2016-11-03 12:29:30 +05:30
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
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(404)
2016-11-03 12:29:30 +05:30
end
2016-09-13 17:45:13 +05:30
it 'returns a not found 404 response for invalid issue id' do
2018-03-17 18:26:18 +05:30
move user: user, board: board, issue: double(id: 999), from_list_id: list1.id, to_list_id: list2.id
2016-09-13 17:45:13 +05:30
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(404)
2016-09-13 17:45:13 +05:30
end
end
context 'with unauthorized user' do
let(:guest) { create(:user) }
before do
2018-03-17 18:26:18 +05:30
project.add_guest(guest)
2016-09-13 17:45:13 +05:30
end
2016-11-03 12:29:30 +05:30
it 'returns a forbidden 403 response' do
move user: guest, board: board, issue: issue, from_list_id: list1.id, to_list_id: list2.id
2016-09-13 17:45:13 +05:30
2018-03-17 18:26:18 +05:30
expect(response).to have_gitlab_http_status(403)
2016-09-13 17:45:13 +05:30
end
end
2016-11-03 12:29:30 +05:30
def move(user:, board:, issue:, from_list_id:, to_list_id:)
2016-09-13 17:45:13 +05:30
sign_in(user)
2019-02-15 15:39:39 +05:30
patch :update, params: {
namespace_id: project.namespace.to_param,
project_id: project.id,
board_id: board.to_param,
id: issue.id,
from_list_id: from_list_id,
to_list_id: to_list_id
},
2016-09-13 17:45:13 +05:30
format: :json
end
end
end