diff options
author | Rémy Coutable <remy@rymai.me> | 2017-08-01 20:03:07 +0200 |
---|---|---|
committer | Rémy Coutable <remy@rymai.me> | 2018-01-18 15:07:57 +0100 |
commit | 799cd471099e032a60fbdb1ba6841e617f6c122f (patch) | |
tree | e18e25e7ec82f9fb35bad04890e57c21a69097af /spec/features/merge_request/user_sees_system_notes_spec.rb | |
parent | 50a649530a53a04d2044f9e4c5502586a7ed0c59 (diff) | |
download | gitlab-ce-799cd471099e032a60fbdb1ba6841e617f6c122f.tar.gz |
Improve MR feature specs and reduce duplicationrc/improve-mr-feature-specs
Signed-off-by: Rémy Coutable <remy@rymai.me>
Diffstat (limited to 'spec/features/merge_request/user_sees_system_notes_spec.rb')
-rw-r--r-- | spec/features/merge_request/user_sees_system_notes_spec.rb | 31 |
1 files changed, 31 insertions, 0 deletions
diff --git a/spec/features/merge_request/user_sees_system_notes_spec.rb b/spec/features/merge_request/user_sees_system_notes_spec.rb new file mode 100644 index 00000000000..a00a682757d --- /dev/null +++ b/spec/features/merge_request/user_sees_system_notes_spec.rb @@ -0,0 +1,31 @@ +require 'rails_helper' + +describe 'Merge request > User sees system notes' do + let(:public_project) { create(:project, :public, :repository) } + let(:private_project) { create(:project, :private, :repository) } + let(:user) { private_project.creator } + let(:issue) { create(:issue, project: private_project) } + let(:merge_request) { create(:merge_request, source_project: public_project, source_branch: 'markdown') } + let!(:note) { create(:note_on_merge_request, :system, noteable: merge_request, project: public_project, note: "mentioned in #{issue.to_reference(public_project)}") } + + context 'when logged-in as a member of the private project' do + before do + private_project.add_developer(user) + sign_in(user) + end + + it 'shows the system note' do + visit project_merge_request_path(public_project, merge_request) + + expect(page).to have_css('.system-note') + end + end + + context 'when not logged-in' do + it 'hides the system note' do + visit project_merge_request_path(public_project, merge_request) + + expect(page).not_to have_css('.system-note') + end + end +end |