2019-02-15 15:39:39 +05:30
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'spec_helper'
|
|
|
|
|
|
|
|
describe 'User comments on a diff', :js do
|
|
|
|
include MergeRequestDiffHelpers
|
|
|
|
include RepoHelpers
|
|
|
|
|
2019-07-07 11:18:12 +05:30
|
|
|
def expect_suggestion_has_content(element, expected_changing_content, expected_suggested_content)
|
2019-09-04 21:01:54 +05:30
|
|
|
changing_content = element.all(:css, '.line_holder.old').map { |el| el.text(normalize_ws: true) }
|
|
|
|
suggested_content = element.all(:css, '.line_holder.new').map { |el| el.text(normalize_ws: true) }
|
2019-07-07 11:18:12 +05:30
|
|
|
|
|
|
|
expect(changing_content).to eq(expected_changing_content)
|
|
|
|
expect(suggested_content).to eq(expected_suggested_content)
|
|
|
|
end
|
|
|
|
|
2019-12-21 20:55:43 +05:30
|
|
|
def expect_appliable_suggestions(amount)
|
|
|
|
expect(all('button', text: 'Apply suggestion').size).to eq(amount)
|
|
|
|
end
|
|
|
|
|
2019-02-15 15:39:39 +05:30
|
|
|
let(:project) { create(:project, :repository) }
|
|
|
|
let(:merge_request) do
|
|
|
|
create(:merge_request_with_diffs, source_project: project, target_project: project, source_branch: 'merge-test')
|
|
|
|
end
|
|
|
|
let(:user) { create(:user) }
|
|
|
|
|
|
|
|
before do
|
2019-12-26 22:10:19 +05:30
|
|
|
stub_feature_flags(single_mr_diff_view: false)
|
2019-02-15 15:39:39 +05:30
|
|
|
project.add_maintainer(user)
|
|
|
|
sign_in(user)
|
|
|
|
|
|
|
|
visit(diffs_project_merge_request_path(project, merge_request))
|
|
|
|
end
|
|
|
|
|
2019-12-26 22:10:19 +05:30
|
|
|
it_behaves_like 'rendering a single diff version'
|
|
|
|
|
2019-02-15 15:39:39 +05:30
|
|
|
context 'single suggestion note' do
|
2019-09-04 21:01:54 +05:30
|
|
|
it 'hides suggestion popover' do
|
|
|
|
click_diff_line(find("[id='#{sample_compare.changes[1][:line_code]}']"))
|
|
|
|
|
|
|
|
expect(page).to have_selector('.diff-suggest-popover')
|
|
|
|
|
|
|
|
page.within('.diff-suggest-popover') do
|
|
|
|
click_button 'Got it'
|
|
|
|
end
|
|
|
|
|
|
|
|
expect(page).not_to have_selector('.diff-suggest-popover')
|
|
|
|
end
|
|
|
|
|
2019-02-15 15:39:39 +05:30
|
|
|
it 'suggestion is presented' do
|
|
|
|
click_diff_line(find("[id='#{sample_compare.changes[1][:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
|
|
|
fill_in('note_note', with: "```suggestion\n# change to a comment\n```")
|
|
|
|
click_button('Comment')
|
|
|
|
end
|
|
|
|
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
page.within('.diff-discussions') do
|
|
|
|
expect(page).to have_button('Apply suggestion')
|
|
|
|
expect(page).to have_content('Suggested change')
|
2019-07-07 11:18:12 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
page.within('.md-suggestion-diff') do
|
|
|
|
expected_changing_content = [
|
|
|
|
"6 url = https://github.com/gitlabhq/gitlab-shell.git"
|
|
|
|
]
|
|
|
|
|
|
|
|
expected_suggested_content = [
|
|
|
|
"6 # change to a comment"
|
|
|
|
]
|
|
|
|
|
|
|
|
expect_suggestion_has_content(page, expected_changing_content, expected_suggested_content)
|
2019-02-15 15:39:39 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'suggestion is appliable' do
|
|
|
|
click_diff_line(find("[id='#{sample_compare.changes[1][:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
|
|
|
fill_in('note_note', with: "```suggestion\n# change to a comment\n```")
|
|
|
|
click_button('Comment')
|
|
|
|
end
|
|
|
|
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
page.within('.diff-discussions') do
|
|
|
|
expect(page).not_to have_content('Applied')
|
|
|
|
|
|
|
|
click_button('Apply suggestion')
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
expect(page).to have_content('Applied')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-12-21 20:55:43 +05:30
|
|
|
context 'multiple suggestions in expanded lines' do
|
|
|
|
it 'suggestions are appliable' do
|
|
|
|
diff_file = merge_request.diffs(paths: ['files/ruby/popen.rb']).diff_files.first
|
|
|
|
hash = Digest::SHA1.hexdigest(diff_file.file_path)
|
|
|
|
|
|
|
|
expanded_changes = [
|
|
|
|
{
|
|
|
|
line_code: "#{hash}_1_1",
|
|
|
|
file_path: diff_file.file_path
|
|
|
|
},
|
|
|
|
{
|
|
|
|
line_code: "#{hash}_5_5",
|
|
|
|
file_path: diff_file.file_path
|
|
|
|
}
|
|
|
|
]
|
|
|
|
changes = sample_compare(expanded_changes).changes.last(expanded_changes.size)
|
|
|
|
|
|
|
|
page.within("[id='#{hash}']") do
|
|
|
|
find("button[data-original-title='Show full file']").click
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
click_diff_line(find("[id='#{changes.first[:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
|
|
|
fill_in('note_note', with: "```suggestion\n# change to a comment\n```")
|
|
|
|
click_button('Comment')
|
|
|
|
wait_for_requests
|
|
|
|
end
|
|
|
|
|
|
|
|
click_diff_line(find("[id='#{changes.last[:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
|
|
|
fill_in('note_note', with: "```suggestion\n# 2nd change to a comment\n```")
|
|
|
|
click_button('Comment')
|
|
|
|
wait_for_requests
|
|
|
|
end
|
|
|
|
|
|
|
|
expect_appliable_suggestions(2)
|
|
|
|
end
|
|
|
|
|
|
|
|
# Making sure it's not a Front-end cache.
|
|
|
|
visit(diffs_project_merge_request_path(project, merge_request))
|
|
|
|
|
|
|
|
expect_appliable_suggestions(2)
|
|
|
|
|
|
|
|
page.within("[id='#{hash}']") do
|
|
|
|
all('button', text: 'Apply suggestion').last.click
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
expect(page).to have_content('Applied')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2019-02-15 15:39:39 +05:30
|
|
|
context 'multiple suggestions in a single note' do
|
|
|
|
it 'suggestions are presented' do
|
|
|
|
click_diff_line(find("[id='#{sample_compare.changes[1][:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
2019-07-07 11:18:12 +05:30
|
|
|
fill_in('note_note', with: "```suggestion\n# change to a comment\n```\n```suggestion:-2\n# or that\n# heh\n```")
|
2019-02-15 15:39:39 +05:30
|
|
|
click_button('Comment')
|
|
|
|
end
|
|
|
|
|
|
|
|
wait_for_requests
|
|
|
|
|
|
|
|
page.within('.diff-discussions') do
|
|
|
|
suggestion_1 = page.all(:css, '.md-suggestion-diff')[0]
|
|
|
|
suggestion_2 = page.all(:css, '.md-suggestion-diff')[1]
|
|
|
|
|
2019-07-07 11:18:12 +05:30
|
|
|
suggestion_1_expected_changing_content = [
|
|
|
|
"6 url = https://github.com/gitlabhq/gitlab-shell.git"
|
|
|
|
]
|
|
|
|
suggestion_1_expected_suggested_content = [
|
|
|
|
"6 # change to a comment"
|
|
|
|
]
|
|
|
|
|
|
|
|
suggestion_2_expected_changing_content = [
|
|
|
|
"4 [submodule \"gitlab-shell\"]",
|
|
|
|
"5 path = gitlab-shell",
|
|
|
|
"6 url = https://github.com/gitlabhq/gitlab-shell.git"
|
|
|
|
]
|
|
|
|
suggestion_2_expected_suggested_content = [
|
|
|
|
"4 # or that",
|
|
|
|
"5 # heh"
|
|
|
|
]
|
|
|
|
|
|
|
|
expect_suggestion_has_content(suggestion_1,
|
|
|
|
suggestion_1_expected_changing_content,
|
|
|
|
suggestion_1_expected_suggested_content)
|
|
|
|
|
|
|
|
expect_suggestion_has_content(suggestion_2,
|
|
|
|
suggestion_2_expected_changing_content,
|
|
|
|
suggestion_2_expected_suggested_content)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'multi-line suggestions' do
|
2019-07-31 22:56:46 +05:30
|
|
|
before do
|
2019-07-07 11:18:12 +05:30
|
|
|
click_diff_line(find("[id='#{sample_compare.changes[1][:line_code]}']"))
|
|
|
|
|
|
|
|
page.within('.js-discussion-note-form') do
|
|
|
|
fill_in('note_note', with: "```suggestion:-3+5\n# change to a\n# comment\n# with\n# broken\n# lines\n```")
|
|
|
|
click_button('Comment')
|
|
|
|
end
|
2019-02-15 15:39:39 +05:30
|
|
|
|
2019-07-07 11:18:12 +05:30
|
|
|
wait_for_requests
|
2019-07-31 22:56:46 +05:30
|
|
|
end
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2019-07-31 22:56:46 +05:30
|
|
|
it 'suggestion is presented' do
|
2019-07-07 11:18:12 +05:30
|
|
|
page.within('.diff-discussions') do
|
|
|
|
expect(page).to have_button('Apply suggestion')
|
|
|
|
expect(page).to have_content('Suggested change')
|
|
|
|
end
|
|
|
|
|
|
|
|
page.within('.md-suggestion-diff') do
|
|
|
|
expected_changing_content = [
|
|
|
|
"3 url = git://github.com/randx/six.git",
|
|
|
|
"4 [submodule \"gitlab-shell\"]",
|
|
|
|
"5 path = gitlab-shell",
|
|
|
|
"6 url = https://github.com/gitlabhq/gitlab-shell.git",
|
|
|
|
"7 [submodule \"gitlab-grack\"]",
|
|
|
|
"8 path = gitlab-grack",
|
|
|
|
"9 url = https://gitlab.com/gitlab-org/gitlab-grack.git"
|
|
|
|
]
|
|
|
|
|
|
|
|
expected_suggested_content = [
|
|
|
|
"3 # change to a",
|
|
|
|
"4 # comment",
|
|
|
|
"5 # with",
|
|
|
|
"6 # broken",
|
|
|
|
"7 # lines"
|
|
|
|
]
|
|
|
|
|
|
|
|
expect_suggestion_has_content(page, expected_changing_content, expected_suggested_content)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'suggestion is appliable' do
|
2019-07-31 22:56:46 +05:30
|
|
|
page.within('.diff-discussions') do
|
|
|
|
expect(page).not_to have_content('Applied')
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2019-07-31 22:56:46 +05:30
|
|
|
click_button('Apply suggestion')
|
|
|
|
wait_for_requests
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2019-07-31 22:56:46 +05:30
|
|
|
expect(page).to have_content('Applied')
|
|
|
|
end
|
|
|
|
end
|
2019-07-07 11:18:12 +05:30
|
|
|
|
2019-07-31 22:56:46 +05:30
|
|
|
it 'resolves discussion when applied' do
|
2019-07-07 11:18:12 +05:30
|
|
|
page.within('.diff-discussions') do
|
2019-09-30 21:07:59 +05:30
|
|
|
expect(page).not_to have_content('Unresolve thread')
|
2019-07-07 11:18:12 +05:30
|
|
|
|
|
|
|
click_button('Apply suggestion')
|
|
|
|
wait_for_requests
|
|
|
|
|
2019-09-30 21:07:59 +05:30
|
|
|
expect(page).to have_content('Unresolve thread')
|
2019-02-15 15:39:39 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|