diff options
author | Shinya Maeda <shinya@gitlab.com> | 2018-05-01 14:13:12 +0900 |
---|---|---|
committer | Shinya Maeda <shinya@gitlab.com> | 2018-05-01 14:13:12 +0900 |
commit | 54695563ca3b37f09b625c4c6798e41a0e629c0b (patch) | |
tree | e47a628dcc5bcd969a4e1680ab96e8bb49efd4d3 /spec/features/projects | |
parent | d5344617a8b57e4d6d15f22ad3d09d5af82100fe (diff) | |
parent | 8b3db0d64beb204dfa722b81cfc32843e19a8eae (diff) | |
download | gitlab-ce-54695563ca3b37f09b625c4c6798e41a0e629c0b.tar.gz |
Merge branch 'master' into live-trace-v2
Diffstat (limited to 'spec/features/projects')
-rw-r--r-- | spec/features/projects/members/master_manages_access_requests_spec.rb | 45 |
1 files changed, 3 insertions, 42 deletions
diff --git a/spec/features/projects/members/master_manages_access_requests_spec.rb b/spec/features/projects/members/master_manages_access_requests_spec.rb index 1f4eec0a317..3ac6ca4fc86 100644 --- a/spec/features/projects/members/master_manages_access_requests_spec.rb +++ b/spec/features/projects/members/master_manages_access_requests_spec.rb @@ -1,47 +1,8 @@ require 'spec_helper' feature 'Projects > Members > Master manages access requests' do - let(:user) { create(:user) } - let(:master) { create(:user) } - let(:project) { create(:project, :public, :access_requestable) } - - background do - project.request_access(user) - project.add_master(master) - sign_in(master) - end - - scenario 'master can see access requests' do - visit project_project_members_path(project) - - expect_visible_access_request(project, user) - end - - scenario 'master can grant access' do - visit project_project_members_path(project) - - expect_visible_access_request(project, user) - - perform_enqueued_jobs { click_on 'Grant access' } - - expect(ActionMailer::Base.deliveries.last.to).to eq [user.notification_email] - expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.full_name} project was granted" - end - - scenario 'master can deny access' do - visit project_project_members_path(project) - - expect_visible_access_request(project, user) - - perform_enqueued_jobs { click_on 'Deny access' } - - expect(ActionMailer::Base.deliveries.last.to).to eq [user.notification_email] - expect(ActionMailer::Base.deliveries.last.subject).to match "Access to the #{project.full_name} project was denied" - end - - def expect_visible_access_request(project, user) - expect(project.requesters.exists?(user_id: user)).to be_truthy - expect(page).to have_content "Users requesting access to #{project.name} 1" - expect(page).to have_content user.name + it_behaves_like 'Master manages access requests' do + let(:entity) { create(:project, :public, :access_requestable) } + let(:members_page_path) { project_project_members_path(entity) } end end |