diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-10-21 07:08:36 +0000 |
commit | 48aff82709769b098321c738f3444b9bdaa694c6 (patch) | |
tree | e00c7c43e2d9b603a5a6af576b1685e400410dee /spec/features/merge_request/user_edits_assignees_sidebar_spec.rb | |
parent | 879f5329ee916a948223f8f43d77fba4da6cd028 (diff) | |
download | gitlab-ce-48aff82709769b098321c738f3444b9bdaa694c6.tar.gz |
Add latest changes from gitlab-org/gitlab@13-5-stable-eev13.5.0-rc42
Diffstat (limited to 'spec/features/merge_request/user_edits_assignees_sidebar_spec.rb')
-rw-r--r-- | spec/features/merge_request/user_edits_assignees_sidebar_spec.rb | 47 |
1 files changed, 4 insertions, 43 deletions
diff --git a/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb b/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb index affd6f6b7b5..7d55a72c2b1 100644 --- a/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb +++ b/spec/features/merge_request/user_edits_assignees_sidebar_spec.rb @@ -20,7 +20,7 @@ RSpec.describe 'Merge request > User edits assignees sidebar', :js do let(:sidebar_assignee_dropdown_item) { sidebar_assignee_block.find(".dropdown-menu li[data-user-id=\"#{assignee.id}\"]") } let(:sidebar_assignee_dropdown_tooltip) { sidebar_assignee_dropdown_item.find('a')['data-title'] || '' } - context 'when invite_members_version_a experiment is not enabled' do + context 'when user is an owner' do before do stub_const('Autocomplete::UsersFinder::LIMIT', users_find_limit) @@ -52,12 +52,6 @@ RSpec.describe 'Merge request > User edits assignees sidebar', :js do it "shows assignee tooltip '#{expected_tooltip}" do expect(sidebar_assignee_dropdown_tooltip).to eql(expected_tooltip) end - - it 'does not show invite link' do - page.within '.dropdown-menu-user' do - expect(page).not_to have_link('Invite Members') - end - end end end @@ -74,48 +68,15 @@ RSpec.describe 'Merge request > User edits assignees sidebar', :js do end end - context 'when invite_members_version_a experiment is enabled' do + context 'with invite members experiment considerations' do let_it_be(:user) { create(:user) } before do - stub_experiment_for_user(invite_members_version_a: true) sign_in(user) end - context 'when user can not see invite members' do - before do - project.add_developer(user) - visit project_merge_request_path(project, merge_request) - - find('.block.assignee .edit-link').click - - wait_for_requests - end - - it 'does not see link to invite members' do - page.within '.dropdown-menu-user' do - expect(page).not_to have_link('Invite Members') - end - end - end - - context 'when user can see invite members' do - before do - project.add_maintainer(user) - visit project_merge_request_path(project, merge_request) - - find('.block.assignee .edit-link').click - - wait_for_requests - end - - it 'sees link to invite members' do - page.within '.dropdown-menu-user' do - expect(page).to have_link('Invite Members', href: project_project_members_path(project)) - expect(page).to have_selector('[data-track-event="click_invite_members"]') - expect(page).to have_selector("[data-track-label='edit_assignee']") - end - end + include_examples 'issuable invite members experiments' do + let(:issuable_path) { project_merge_request_path(project, merge_request) } end end end |