diff options
author | Phil Hughes <me@iamphill.com> | 2016-08-17 14:05:34 +0100 |
---|---|---|
committer | Phil Hughes <me@iamphill.com> | 2016-08-17 17:17:39 +0100 |
commit | 9a425c3f9232a552f18e19c8a36b4c23acc9ac80 (patch) | |
tree | 5915a0d07be373474a666dee8edae143060ed1e1 /spec/features/boards | |
parent | 77a074a7201cf934fc255f9fcd59f9fd2d01ad57 (diff) | |
download | gitlab-ce-9a425c3f9232a552f18e19c8a36b4c23acc9ac80.tar.gz |
Fixed permissions around create new list button
Diffstat (limited to 'spec/features/boards')
-rw-r--r-- | spec/features/boards/boards_spec.rb | 28 |
1 files changed, 27 insertions, 1 deletions
diff --git a/spec/features/boards/boards_spec.rb b/spec/features/boards/boards_spec.rb index cf20ca90b21..2864784559b 100644 --- a/spec/features/boards/boards_spec.rb +++ b/spec/features/boards/boards_spec.rb @@ -3,7 +3,7 @@ require 'rails_helper' describe 'Issue Boards', feature: true, js: true do include WaitForAjax - let(:project) { create(:empty_project) } + let(:project) { create(:empty_project, :public) } let(:user) { create(:user) } let!(:user2) { create(:user) } @@ -561,6 +561,32 @@ describe 'Issue Boards', feature: true, js: true do 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') 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}}});") |