2014-09-02 18:07:02 +05:30
|
|
|
module SharedDiffNote
|
|
|
|
include Spinach::DSL
|
|
|
|
include RepoHelpers
|
2017-09-10 17:25:29 +05:30
|
|
|
include WaitForRequests
|
2017-08-17 22:00:37 +05:30
|
|
|
|
|
|
|
after do
|
2017-09-10 17:25:29 +05:30
|
|
|
wait_for_requests if javascript_test?
|
2017-08-17 22:00:37 +05:30
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I cancel the diff comment' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2014-09-02 18:07:02 +05:30
|
|
|
find(".js-close-discussion-note-form").click
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I delete a diff comment' do
|
2014-09-02 18:07:02 +05:30
|
|
|
find('.note').hover
|
|
|
|
find(".js-note-delete").click
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I haven\'t written any diff comment text' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2014-09-02 18:07:02 +05:30
|
|
|
fill_in "note[note]", with: ""
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I leave a diff comment like "Typo, please fix"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-08-24 12:49:21 +05:30
|
|
|
page.within("form[data-line-code='#{sample_commit.line_code}']") do
|
2015-09-11 14:41:01 +05:30
|
|
|
fill_in "note[note]", with: "Typo, please fix"
|
2016-08-24 12:49:21 +05:30
|
|
|
find(".js-comment-button").click
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
step 'I leave a diff comment in a parallel view on the left side like "Old comment"' do
|
2016-08-24 12:49:21 +05:30
|
|
|
click_parallel_diff_line(sample_commit.del_line_code, 'old')
|
|
|
|
page.within("#{diff_file_selector} form[data-line-code='#{sample_commit.del_line_code}']") do
|
2015-09-11 14:41:01 +05:30
|
|
|
fill_in "note[note]", with: "Old comment"
|
2016-08-24 12:49:21 +05:30
|
|
|
find(".js-comment-button").click
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
step 'I leave a diff comment in a parallel view on the right side like "New comment"' do
|
|
|
|
click_parallel_diff_line(sample_commit.line_code, 'new')
|
2016-08-24 12:49:21 +05:30
|
|
|
page.within("#{diff_file_selector} form[data-line-code='#{sample_commit.line_code}']") do
|
2015-09-11 14:41:01 +05:30
|
|
|
fill_in "note[note]", with: "New comment"
|
2016-08-24 12:49:21 +05:30
|
|
|
find(".js-comment-button").click
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I preview a diff comment text like "Should fix it :smile:"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-08-24 12:49:21 +05:30
|
|
|
page.within("form[data-line-code='#{sample_commit.line_code}']") do
|
2015-09-11 14:41:01 +05:30
|
|
|
fill_in "note[note]", with: "Should fix it :smile:"
|
|
|
|
find('.js-md-preview-button').click
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I preview another diff comment text like "DRY this up"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.del_line_code)
|
2014-09-02 18:07:02 +05:30
|
|
|
|
2016-08-24 12:49:21 +05:30
|
|
|
page.within("form[data-line-code='#{sample_commit.del_line_code}']") do
|
2015-09-11 14:41:01 +05:30
|
|
|
fill_in "note[note]", with: "DRY this up"
|
|
|
|
find('.js-md-preview-button').click
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I open a diff comment form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I open another diff comment form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.del_line_code)
|
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I write a diff comment like ":-1: I don\'t like this"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2014-09-02 18:07:02 +05:30
|
|
|
fill_in "note[note]", with: ":-1: I don\'t like this"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-12-23 02:04:40 +05:30
|
|
|
step 'I write a diff comment like ":smile:"' do
|
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-08-24 12:49:21 +05:30
|
|
|
page.within("form[data-line-code='#{sample_commit.line_code}']") do
|
2015-12-23 02:04:40 +05:30
|
|
|
fill_in 'note[note]', with: ':smile:'
|
2016-06-02 11:05:42 +05:30
|
|
|
click_button('Comment')
|
2015-12-23 02:04:40 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I submit the diff comment' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2016-06-02 11:05:42 +05:30
|
|
|
click_button("Comment")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should not see the diff comment form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
expect(page).not_to have_css("form.new_note")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'The diff comment preview tab should say there is nothing to do' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2015-04-26 12:48:37 +05:30
|
|
|
find('.js-md-preview-button').click
|
|
|
|
expect(find('.js-md-preview')).to have_content('Nothing to preview.')
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should not see the diff comment text field' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(find('.js-note-text')).not_to be_visible
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should only see one diff form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2016-06-02 11:05:42 +05:30
|
|
|
expect(page).to have_css("form.new-note", count: 1)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see a diff comment form with ":-1: I don\'t like this"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
expect(page).to have_field("note[note]", with: ":-1: I don\'t like this")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see a diff comment saying "Typo, please fix"' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within("#{diff_file_selector} .note") do
|
|
|
|
expect(page).to have_content("Typo, please fix")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
step 'I should see a diff comment on the left side saying "Old comment"' do
|
|
|
|
page.within("#{diff_file_selector} .notes_content.parallel.old") do
|
|
|
|
expect(page).to have_content("Old comment")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
step 'I should see a diff comment on the right side saying "New comment"' do
|
|
|
|
page.within("#{diff_file_selector} .notes_content.parallel.new") do
|
|
|
|
expect(page).to have_content("New comment")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see a discussion reply button' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2016-06-02 11:05:42 +05:30
|
|
|
expect(page).to have_button('Reply...')
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see a temporary diff comment form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2016-06-02 11:05:42 +05:30
|
|
|
expect(page).to have_css(".js-temp-notes-holder form.new-note")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see an empty diff comment form' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
|
|
|
expect(page).to have_field("note[note]", with: "")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see the cancel comment button' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within("#{diff_file_selector} form") do
|
|
|
|
expect(page).to have_css(".js-close-discussion-note-form", text: "Cancel")
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see the diff comment preview' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within("#{diff_file_selector} form") do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(page).to have_css('.js-md-preview', visible: true)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see the diff comment write tab' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(page).to have_css('.js-md-write-button', visible: true)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'The diff comment preview tab should display rendered Markdown' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2015-04-26 12:48:37 +05:30
|
|
|
find('.js-md-preview-button').click
|
2017-08-17 22:00:37 +05:30
|
|
|
expect(find('.js-md-preview')).to have_css('gl-emoji', visible: true)
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-04-26 12:48:37 +05:30
|
|
|
step 'I should see two separate previews' do
|
2015-09-11 14:41:01 +05:30
|
|
|
page.within(diff_file_selector) do
|
2015-04-26 12:48:37 +05:30
|
|
|
expect(page).to have_css('.js-md-preview', visible: true, count: 2)
|
|
|
|
expect(page).to have_content('Should fix it')
|
|
|
|
expect(page).to have_content('DRY this up')
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-12-23 02:04:40 +05:30
|
|
|
step 'I should see a diff comment with an emoji image' do
|
|
|
|
page.within("#{diff_file_selector} .note") do
|
2017-08-17 22:00:37 +05:30
|
|
|
expect(page).to have_xpath("//gl-emoji[@data-name='smile']")
|
2015-12-23 02:04:40 +05:30
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-09-11 14:41:01 +05:30
|
|
|
step 'I click side-by-side diff button' do
|
2018-03-17 18:26:18 +05:30
|
|
|
find('#parallel-diff-btn').click
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
step 'I see side-by-side diff button' do
|
|
|
|
expect(page).to have_content "Side-by-side"
|
|
|
|
end
|
|
|
|
|
2014-09-02 18:07:02 +05:30
|
|
|
def diff_file_selector
|
2015-09-11 14:41:01 +05:30
|
|
|
'.diff-file:nth-of-type(1)'
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
|
|
|
|
|
|
|
def click_diff_line(code)
|
2018-03-17 18:26:18 +05:30
|
|
|
find(".line_holder[id='#{code}'] button").click
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|
2015-09-11 14:41:01 +05:30
|
|
|
|
|
|
|
def click_parallel_diff_line(code, line_type)
|
2018-03-17 18:26:18 +05:30
|
|
|
find(".line_holder.parallel td[id='#{code}']").find(:xpath, 'preceding-sibling::*[1][self::td]').hover
|
|
|
|
find(".line_holder.parallel button[data-line-code='#{code}']").click
|
2015-09-11 14:41:01 +05:30
|
|
|
end
|
2014-09-02 18:07:02 +05:30
|
|
|
end
|