Commit fe8f24a5 authored by Lin Jen-Shin's avatar Lin Jen-Shin

Permission is somehow cached. Don't add if we don't want

parent d0d0b92c
...@@ -86,10 +86,11 @@ RSpec.describe Projects::MergeRequests::DiffsController do ...@@ -86,10 +86,11 @@ RSpec.describe Projects::MergeRequests::DiffsController do
let(:project) { create(:project, :repository) } let(:project) { create(:project, :repository) }
let(:user) { create(:user) } let(:user) { create(:user) }
let(:maintainer) { true }
let(:merge_request) { create(:merge_request_with_diffs, target_project: project, source_project: project) } let(:merge_request) { create(:merge_request_with_diffs, target_project: project, source_project: project) }
before do before do
project.add_maintainer(user) project.add_maintainer(user) if maintainer
sign_in(user) sign_in(user)
end end
...@@ -383,8 +384,9 @@ RSpec.describe Projects::MergeRequests::DiffsController do ...@@ -383,8 +384,9 @@ RSpec.describe Projects::MergeRequests::DiffsController do
end end
context 'when the user cannot view the merge request' do context 'when the user cannot view the merge request' do
let(:maintainer) { false }
before do before do
project.team.truncate
diff_for_path(old_path: existing_path, new_path: existing_path) diff_for_path(old_path: existing_path, new_path: existing_path)
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment