Skip to content
Snippets Groups Projects
Commit 9a425c3f authored by Phil Hughes's avatar Phil Hughes
Browse files

Fixed permissions around create new list button

parent 77a074a7
No related branches found
No related tags found
No related merge requests found
Loading
@@ -29,7 +29,7 @@
Loading
@@ -29,7 +29,7 @@
.pull-right .pull-right
- if controller.controller_name != 'boards' - if controller.controller_name != 'boards'
= render 'shared/sort_dropdown' = render 'shared/sort_dropdown'
- elsif current_user - if can?(current_user, :admin_list, @project)
.dropdown .dropdown
%button.btn.btn-create.js-new-board-list{ type: "button", data: { toggle: "dropdown", labels: labels_filter_path, project_id: @project.try(:id) } } %button.btn.btn-create.js-new-board-list{ type: "button", data: { toggle: "dropdown", labels: labels_filter_path, project_id: @project.try(:id) } }
Create new list Create new list
Loading
Loading
Loading
@@ -3,7 +3,7 @@ require 'rails_helper'
Loading
@@ -3,7 +3,7 @@ require 'rails_helper'
describe 'Issue Boards', feature: true, js: true do describe 'Issue Boards', feature: true, js: true do
include WaitForAjax include WaitForAjax
   
let(:project) { create(:empty_project) } let(:project) { create(:empty_project, :public) }
let(:user) { create(:user) } let(:user) { create(:user) }
let!(:user2) { create(:user) } let!(:user2) { create(:user) }
   
Loading
@@ -561,6 +561,32 @@ describe 'Issue Boards', feature: true, js: true do
Loading
@@ -561,6 +561,32 @@ describe 'Issue Boards', feature: true, js: true do
end end
end end
   
context 'signed out user' do
before do
logout
visit namespace_project_board_path(project.namespace, project)
end
it 'does not show create new list' do
expect(page).not_to have_selector('.js-new-board-list')
end
end
context 'as guest user' do
let(:user_guest) { create(:user) }
before do
project.team << [user_guest, :guest]
logout
login_as(user_guest)
visit namespace_project_board_path(project.namespace, project)
end
it 'does not show create new list' do
expect(page).not_to have_selector('.js-new-board-list')
end
end
def drag_to(list_from_index: 0, card_index: 0, to_index: 0, list_to_index: 0, selector: '.board-list') def drag_to(list_from_index: 0, card_index: 0, to_index: 0, list_to_index: 0, selector: '.board-list')
evaluate_script("simulateDrag({scrollable: document.getElementById('board-app'), from: {el: $('#{selector}').eq(#{list_from_index}).get(0), index: #{card_index}}, to: {el: $('.board-list').eq(#{list_to_index}).get(0), index: #{to_index}}});") evaluate_script("simulateDrag({scrollable: document.getElementById('board-app'), from: {el: $('#{selector}').eq(#{list_from_index}).get(0), index: #{card_index}}, to: {el: $('.board-list').eq(#{list_to_index}).get(0), index: #{to_index}}});")
   
Loading
Loading
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment