diff options
author | Stan Hu <stanhu@gmail.com> | 2015-12-22 19:30:32 +0000 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2015-12-22 19:30:32 +0000 |
commit | 3b61dc47ec377b857b66f9b2a81a183276163f4e (patch) | |
tree | b125241794a7d7143628dd685b1e8682c8586cc1 /spec/helpers | |
parent | 1cf45407d3be9e9767d27e55b6f3e2e0d8dea36f (diff) | |
parent | 66a8281fddf8e4f9d39265a6bfb38e8a6a8309c2 (diff) | |
download | gitlab-ce-3b61dc47ec377b857b66f9b2a81a183276163f4e.tar.gz |
Merge branch 'fix_leave_project_display' into 'master'
Fix and test leave project display
The 'Leave this project' display on project page was broken. It was calling the method incorrectly, which returned `nil` and displayed nothing.
This fixes the display and also adds 2 different tests for it.
cc/ @jvanbaarsen @stanhu Please review and merge if you think it looks good. Thank you!
See merge request !2166
Diffstat (limited to 'spec/helpers')
-rw-r--r-- | spec/helpers/projects_helper_spec.rb | 10 |
1 files changed, 10 insertions, 0 deletions
diff --git a/spec/helpers/projects_helper_spec.rb b/spec/helpers/projects_helper_spec.rb index f2efb528aeb..53207767581 100644 --- a/spec/helpers/projects_helper_spec.rb +++ b/spec/helpers/projects_helper_spec.rb @@ -53,6 +53,16 @@ describe ProjectsHelper do end end + describe 'user_max_access_in_project' do + let(:project) { create(:project) } + let(:user) { create(:user) } + before do + project.team.add_user(user, Gitlab::Access::MASTER) + end + + it { expect(helper.user_max_access_in_project(user.id, project)).to eq('Master') } + end + describe "readme_cache_key" do let(:project) { create(:project) } |