diff options
| author | Rémy Coutable <remy@rymai.me> | 2016-06-20 12:36:59 +0200 |
|---|---|---|
| committer | Rémy Coutable <remy@rymai.me> | 2016-06-20 12:36:59 +0200 |
| commit | 909a0ff3ace1eb82a4296764777a552779c39839 (patch) | |
| tree | ab1056b0443b64d5f95eeacaef92b0172c51df94 /spec/controllers | |
| parent | bf05ca88eefb38bb52fb6e31622f9c1a795965e3 (diff) | |
| download | gitlab-ce-909a0ff3ace1eb82a4296764777a552779c39839.tar.gz | |
Fix and remove duplicate specs18755-fix-destroy-project-causes-post_decline_request-to-be-executed
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'spec/controllers')
| -rw-r--r-- | spec/controllers/groups/group_members_controller_spec.rb | 4 | ||||
| -rw-r--r-- | spec/controllers/projects/project_members_controller_spec.rb | 6 |
2 files changed, 2 insertions, 8 deletions
diff --git a/spec/controllers/groups/group_members_controller_spec.rb b/spec/controllers/groups/group_members_controller_spec.rb index 8798d709f30..c8601341d54 100644 --- a/spec/controllers/groups/group_members_controller_spec.rb +++ b/spec/controllers/groups/group_members_controller_spec.rb @@ -118,9 +118,7 @@ describe Groups::GroupMembersController do it 'cannot removes himself from the group' do delete :leave, group_id: group - expect(response).to redirect_to(group_path(group)) - expect(response).to set_flash[:alert].to "You can not leave the \"#{group.name}\" group. Transfer or delete the group." - expect(group.users).to include user + expect(response.status).to eq(403) end end diff --git a/spec/controllers/projects/project_members_controller_spec.rb b/spec/controllers/projects/project_members_controller_spec.rb index 0a8fe5e04c3..e5e750c855f 100644 --- a/spec/controllers/projects/project_members_controller_spec.rb +++ b/spec/controllers/projects/project_members_controller_spec.rb @@ -171,11 +171,7 @@ describe Projects::ProjectMembersController do delete :leave, namespace_id: project.namespace, project_id: project - expect(response).to redirect_to( - namespace_project_path(project.namespace, project) - ) - expect(response).to set_flash[:alert].to "You can not leave the \"#{project.human_name}\" project. Transfer or delete the project." - expect(project.users).to include user + expect(response.status).to eq(403) end end |
