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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-07-31 11:07:24 -04:00
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
within("#{diff_file_selector} form[rel$='#{sample_commit.line_code}']") do
|
2013-01-14 18:11:35 -05:00
|
|
|
fill_in "note[note]", with: "Typo, please fix"
|
|
|
|
find(".js-comment-button").trigger("click")
|
2013-01-15 07:28:04 -05:00
|
|
|
sleep 0.05
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I preview a diff comment text like "Should fix it :smile:"' do
|
2014-07-31 11:07:24 -04:00
|
|
|
click_diff_line(sample_commit.line_code)
|
|
|
|
within("#{diff_file_selector} form[rel$='#{sample_commit.line_code}']") do
|
2013-01-14 18:11:35 -05:00
|
|
|
fill_in "note[note]", with: "Should fix it :smile:"
|
2014-10-17 00:10:50 -04:00
|
|
|
find('.js-md-preview-button').click
|
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
|
2014-07-31 11:07:24 -04:00
|
|
|
click_diff_line(sample_commit.del_line_code)
|
2013-04-11 03:45:18 -04:00
|
|
|
|
2014-07-31 11:07:24 -04:00
|
|
|
within("#{diff_file_selector} form[rel$='#{sample_commit.del_line_code}']") do
|
2013-01-14 18:11:35 -05:00
|
|
|
fill_in "note[note]", with: "DRY this up"
|
2014-10-17 00:10:50 -04:00
|
|
|
find('.js-md-preview-button').click
|
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
|
2014-07-31 11:07:24 -04:00
|
|
|
click_diff_line(sample_commit.line_code)
|
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
|
2014-07-31 11:07:24 -04:00
|
|
|
click_diff_line(sample_commit.del_line_code)
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I submit the diff comment' do
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should_not have_css("form.new_note")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-10-17 00:36:52 -04:00
|
|
|
step 'The diff comment preview tab should say there is nothing to do' do
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_css("form.new_note", count: 1)
|
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_field("note[note]", with: ":-1: I don\'t like this")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a diff comment saying "Typo, please fix"' do
|
2014-06-24 13:22:33 -04:00
|
|
|
within("#{diff_file_selector} .note") do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_content("Typo, please fix")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see a discussion reply button' do
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2014-09-19 09:26:20 -04:00
|
|
|
page.should 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
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_css(".js-temp-notes-holder form.new_note")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see add a diff comment button' do
|
2014-09-29 17:58:01 -04:00
|
|
|
page.should have_css('.js-add-diff-note-button', visible: true)
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
within(diff_file_selector) do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_field("note[note]", with: "")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see the cancel comment button' do
|
2014-06-24 13:22:33 -04:00
|
|
|
within("#{diff_file_selector} form") do
|
2013-01-14 18:11:35 -05:00
|
|
|
page.should have_css(".js-close-discussion-note-form", text: "Cancel")
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
2014-09-21 17:40:22 -04:00
|
|
|
step 'I should see the diff comment preview' do
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
2014-06-24 13:22:33 -04:00
|
|
|
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
|
|
|
|
|
|
|
def diff_file_selector
|
2014-07-31 11:07:24 -04:00
|
|
|
".diff-file:nth-of-type(1)"
|
|
|
|
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
|
2013-01-14 18:11:35 -05:00
|
|
|
end
|