2013-01-14 18:11:35 -05:00
|
|
|
module SharedDiffNote
|
|
|
|
include Spinach::DSL
|
2014-07-31 11:07:24 -04:00
|
|
|
include RepoHelpers
|
2013-01-14 18:11:35 -05:00
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I cancel the diff comment' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2013-04-11 03:45:18 -04:00
|
|
|
find(".js-close-discussion-note-form").click
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I delete a diff comment' do
|
2013-04-11 03:45:18 -04:00
|
|
|
find('.note').hover
|
|
|
|
find(".js-note-delete").click
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I haven\'t written any diff comment text' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
fill_in "note[note]", with: ""
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I leave a diff comment like "Typo, please fix"' do
|
2015-06-18 22:13:32 -04:00
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-01-21 04:08:12 -05:00
|
|
|
page.within("form[id$='#{sample_commit.line_code}']") do
|
2015-06-18 22:13:32 -04:00
|
|
|
fill_in "note[note]", with: "Typo, please fix"
|
|
|
|
find(".js-comment-button").trigger("click")
|
|
|
|
sleep 0.05
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-06-05 18:24:05 -04:00
|
|
|
step 'I leave a diff comment in a parallel view on the left side like "Old comment"' do
|
|
|
|
click_parallel_diff_line(sample_commit.line_code, 'old')
|
2016-01-21 04:08:12 -05:00
|
|
|
page.within("#{diff_file_selector} form[id$='#{sample_commit.line_code}']") do
|
2015-06-05 18:24:05 -04:00
|
|
|
fill_in "note[note]", with: "Old comment"
|
|
|
|
find(".js-comment-button").trigger("click")
|
|
|
|
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-01-21 04:08:12 -05:00
|
|
|
page.within("#{diff_file_selector} form[id$='#{sample_commit.line_code}']") do
|
2015-06-05 18:24:05 -04:00
|
|
|
fill_in "note[note]", with: "New comment"
|
|
|
|
find(".js-comment-button").trigger("click")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I preview a diff comment text like "Should fix it :smile:"' do
|
2015-06-18 22:13:32 -04:00
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-01-21 04:08:12 -05:00
|
|
|
page.within("form[id$='#{sample_commit.line_code}']") do
|
2015-06-18 22:13:32 -04:00
|
|
|
fill_in "note[note]", with: "Should fix it :smile:"
|
|
|
|
find('.js-md-preview-button').click
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I preview another diff comment text like "DRY this up"' do
|
2015-06-18 22:13:32 -04:00
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.del_line_code)
|
2013-04-11 03:45:18 -04:00
|
|
|
|
2016-01-21 04:08:12 -05:00
|
|
|
page.within("form[id$='#{sample_commit.del_line_code}']") do
|
2015-06-18 22:13:32 -04:00
|
|
|
fill_in "note[note]", with: "DRY this up"
|
|
|
|
find('.js-md-preview-button').click
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I open a diff comment form' do
|
2015-06-18 22:13:32 -04:00
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I open another diff comment form' do
|
2015-06-18 22:13:32 -04:00
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.del_line_code)
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I write a diff comment like ":-1: I don\'t like this"' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
fill_in "note[note]", with: ":-1: I don\'t like this"
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-12-03 03:30:30 -05:00
|
|
|
step 'I write a diff comment like ":smile:"' do
|
|
|
|
page.within(diff_file_selector) do
|
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
|
2016-01-21 04:08:12 -05:00
|
|
|
page.within("form[id$='#{sample_commit.line_code}']") do
|
2015-12-03 03:30:30 -05:00
|
|
|
fill_in 'note[note]', with: ':smile:'
|
|
|
|
click_button('Add Comment')
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I submit the diff comment' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
click_button("Add Comment")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should not see the diff comment form' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).not_to have_css("form.new_note")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-10-17 00:36:52 -04:00
|
|
|
step 'The diff comment preview tab should say there is nothing to do' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2014-10-17 00:36:52 -04:00
|
|
|
find('.js-md-preview-button').click
|
|
|
|
expect(find('.js-md-preview')).to have_content('Nothing to preview.')
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should not see the diff comment text field' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2014-09-29 17:58:01 -04:00
|
|
|
expect(find('.js-note-text')).not_to be_visible
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should only see one diff form' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_css("form.new_note", count: 1)
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a diff comment form with ":-1: I don\'t like this"' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_field("note[note]", with: ":-1: I don\'t like this")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a diff comment saying "Typo, please fix"' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within("#{diff_file_selector} .note") do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_content("Typo, please fix")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-06-05 18:24:05 -04:00
|
|
|
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")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a discussion reply button' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_button('Reply')
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a temporary diff comment form' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_css(".js-temp-notes-holder form.new_note")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see add a diff comment button' do
|
2015-12-16 14:55:48 -05:00
|
|
|
expect(page).to have_css('.js-add-diff-note-button')
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see an empty diff comment form' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_field("note[note]", with: "")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see the cancel comment button' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within("#{diff_file_selector} form") do
|
2015-06-12 00:44:13 -04:00
|
|
|
expect(page).to have_css(".js-close-discussion-note-form", text: "Cancel")
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see the diff comment preview' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within("#{diff_file_selector} form") do
|
2014-10-20 22:53:17 -04:00
|
|
|
expect(page).to have_css('.js-md-preview', visible: true)
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-10-17 00:36:52 -04:00
|
|
|
step 'I should see the diff comment write tab' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2014-10-20 22:53:17 -04:00
|
|
|
expect(page).to have_css('.js-md-write-button', visible: true)
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-10-17 00:36:52 -04:00
|
|
|
step 'The diff comment preview tab should display rendered Markdown' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2014-10-17 00:36:52 -04:00
|
|
|
find('.js-md-preview-button').click
|
2014-10-20 22:53:17 -04:00
|
|
|
expect(find('.js-md-preview')).to have_css('img.emoji', visible: true)
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see two separate previews' do
|
2015-06-12 01:36:00 -04:00
|
|
|
page.within(diff_file_selector) do
|
2014-10-20 22:53:17 -04:00
|
|
|
expect(page).to have_css('.js-md-preview', visible: true, count: 2)
|
2014-10-18 19:24:12 -04:00
|
|
|
expect(page).to have_content('Should fix it')
|
|
|
|
expect(page).to have_content('DRY this up')
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
2014-06-24 13:22:33 -04:00
|
|
|
|
2015-12-03 03:30:30 -05:00
|
|
|
step 'I should see a diff comment with an emoji image' do
|
|
|
|
page.within("#{diff_file_selector} .note") do
|
|
|
|
expect(page).to have_xpath("//img[@alt=':smile:']")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2015-06-05 18:24:05 -04:00
|
|
|
step 'I click side-by-side diff button' do
|
2015-09-03 16:15:21 -04:00
|
|
|
find('#parallel-diff-btn').trigger('click')
|
2015-06-05 18:24:05 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
step 'I see side-by-side diff button' do
|
|
|
|
expect(page).to have_content "Side-by-side"
|
|
|
|
end
|
|
|
|
|
2014-06-24 13:22:33 -04:00
|
|
|
def diff_file_selector
|
2015-06-18 22:13:32 -04:00
|
|
|
'.diff-file:nth-of-type(1)'
|
2014-07-31 11:07:24 -04:00
|
|
|
end
|
|
|
|
|
|
|
|
def click_diff_line(code)
|
2014-09-19 09:26:20 -04:00
|
|
|
find("button[data-line-code='#{code}']").click
|
2014-06-24 13:22:33 -04:00
|
|
|
end
|
2015-06-05 18:24:05 -04:00
|
|
|
|
|
|
|
def click_parallel_diff_line(code, line_type)
|
|
|
|
find("button[data-line-code='#{code}'][data-line-type='#{line_type}']").trigger('click')
|
|
|
|
end
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|