240 lines
8.2 KiB
Ruby
240 lines
8.2 KiB
Ruby
require 'spec_helper'
|
|
|
|
describe "On a merge request", js: true do
|
|
let!(:project) { create(:project_with_code) }
|
|
let!(:merge_request) { create(:merge_request, source_project: project, target_project: project) }
|
|
let!(:note) { create(:note_on_merge_request_with_attachment, project: project) }
|
|
|
|
before do
|
|
login_as :user
|
|
project.team << [@user, :master]
|
|
|
|
visit project_merge_request_path(project, merge_request)
|
|
end
|
|
|
|
subject { page }
|
|
|
|
describe "the note form" do
|
|
it 'should be valid' do
|
|
should have_css(".js-main-target-form", visible: true, count: 1)
|
|
find(".js-main-target-form input[type=submit]").value.should == "Add Comment"
|
|
within(".js-main-target-form") { should_not have_link("Cancel") }
|
|
within(".js-main-target-form") { should have_css(".js-note-preview-button", visible: false) }
|
|
end
|
|
|
|
describe "with text" do
|
|
before do
|
|
within(".js-main-target-form") do
|
|
fill_in "note[note]", with: "This is awesome"
|
|
end
|
|
end
|
|
|
|
it 'should have enable submit button and preview button' do
|
|
within(".js-main-target-form") { should_not have_css(".js-comment-button[disabled]") }
|
|
within(".js-main-target-form") { should have_css(".js-note-preview-button", visible: true) }
|
|
end
|
|
end
|
|
|
|
describe "with preview" do
|
|
before do
|
|
within(".js-main-target-form") do
|
|
fill_in "note[note]", with: "This is awesome"
|
|
find(".js-note-preview-button").trigger("click")
|
|
end
|
|
end
|
|
|
|
it 'should have text and visible edit button' do
|
|
within(".js-main-target-form") { should have_css(".js-note-preview", text: "This is awesome", visible: true) }
|
|
within(".js-main-target-form") { should have_css(".js-note-preview-button", visible: false) }
|
|
within(".js-main-target-form") { should have_css(".js-note-edit-button", visible: true) }
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "when posting a note" do
|
|
before do
|
|
within(".js-main-target-form") do
|
|
fill_in "note[note]", with: "This is awsome!"
|
|
find(".js-note-preview-button").trigger("click")
|
|
click_button "Add Comment"
|
|
end
|
|
end
|
|
|
|
it 'should be added and form reset' do
|
|
should have_content("This is awsome!")
|
|
within(".js-main-target-form") { should have_no_field("note[note]", with: "This is awesome!") }
|
|
within(".js-main-target-form") { should have_css(".js-note-preview", visible: false) }
|
|
within(".js-main-target-form") { should have_css(".js-note-text", visible: true) }
|
|
end
|
|
end
|
|
|
|
describe "when editing a note", js: true do
|
|
it "should contain the hidden edit form" do
|
|
within("#note_#{note.id}") { should have_css(".note-edit-form", visible: false) }
|
|
end
|
|
|
|
describe "editing the note" do
|
|
before do
|
|
find('.note').hover
|
|
find(".js-note-edit").click
|
|
end
|
|
|
|
it "should show the note edit form and hide the note body" do
|
|
within("#note_#{note.id}") do
|
|
find(".note-edit-form", visible: true).should be_visible
|
|
find(".note-text", visible: false).should_not be_visible
|
|
end
|
|
end
|
|
|
|
it "should reset the edit note form textarea with the original content of the note if cancelled" do
|
|
find('.note').hover
|
|
find(".js-note-edit").click
|
|
|
|
within(".note-edit-form") do
|
|
fill_in "note[note]", with: "Some new content"
|
|
find(".btn-cancel").click
|
|
find(".js-note-text", visible: false).text.should == note.note
|
|
end
|
|
end
|
|
|
|
it "appends the edited at time to the note" do
|
|
find('.note').hover
|
|
find(".js-note-edit").click
|
|
|
|
within(".note-edit-form") do
|
|
fill_in "note[note]", with: "Some new content"
|
|
find(".btn-save").click
|
|
end
|
|
|
|
within("#note_#{note.id}") do
|
|
should have_css(".note-last-update small")
|
|
find(".note-last-update small").text.should match(/Edited just now/)
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "deleting an attachment" do
|
|
before do
|
|
find('.note').hover
|
|
find(".js-note-edit").click
|
|
end
|
|
|
|
it "shows the delete link" do
|
|
within(".note-attachment") do
|
|
should have_css(".js-note-attachment-delete")
|
|
end
|
|
end
|
|
|
|
it "removes the attachment div and resets the edit form" do
|
|
find(".js-note-attachment-delete").click
|
|
should_not have_css(".note-attachment")
|
|
find(".note-edit-form", visible: false).should_not be_visible
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "On a merge request diff", js: true, focus: true do
|
|
let!(:project) { create(:source_project_with_code) }
|
|
let!(:merge_request) { create(:merge_request_with_diffs, source_project: project, target_project: project) }
|
|
|
|
before do
|
|
login_as :user
|
|
project.team << [@user, :master]
|
|
visit diffs_project_merge_request_path(project, merge_request)
|
|
end
|
|
|
|
|
|
subject { page }
|
|
|
|
describe "when adding a note" do
|
|
before do
|
|
find('a[data-line-code="4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185"]').click
|
|
end
|
|
|
|
describe "the notes holder" do
|
|
it { should have_css(".js-temp-notes-holder") }
|
|
|
|
it { within(".js-temp-notes-holder") { should have_css(".new_note") } }
|
|
end
|
|
|
|
describe "the note form" do
|
|
it 'should be valid' do
|
|
within(".js-temp-notes-holder") { find("#note_noteable_type").value.should == "MergeRequest" }
|
|
within(".js-temp-notes-holder") { find("#note_noteable_id").value.should == merge_request.id.to_s }
|
|
within(".js-temp-notes-holder") { find("#note_commit_id").value.should == "" }
|
|
within(".js-temp-notes-holder") { find("#note_line_code").value.should == "4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185" }
|
|
should have_css(".js-close-discussion-note-form", text: "Cancel")
|
|
end
|
|
|
|
it "shouldn't add a second form for same row" do
|
|
find('a[data-line-code="4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185"]').click
|
|
|
|
should have_css("tr[id='4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185'] + .js-temp-notes-holder form", count: 1)
|
|
end
|
|
|
|
it "should be removed when canceled" do
|
|
within(".file form[rel$='4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185']") do
|
|
find(".js-close-discussion-note-form").trigger("click")
|
|
end
|
|
|
|
should have_no_css(".js-temp-notes-holder")
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "with muliple note forms" do
|
|
let!(:project) { create(:source_project_with_code) }
|
|
let!(:merge_request) { create(:merge_request_with_diffs, source_project: project, target_project: project) }
|
|
|
|
before do
|
|
find('a[data-line-code="4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185"]').click
|
|
find('a[data-line-code="342e16cbbd482ac2047dc679b2749d248cc1428f_18_17"]').click
|
|
end
|
|
|
|
it { should have_css(".js-temp-notes-holder", count: 2) }
|
|
|
|
describe "previewing them separately" do
|
|
before do
|
|
# add two separate texts and trigger previews on both
|
|
within("tr[id='4735dfc552ad7bf15ca468adc3cad9d05b624490_185_185'] + .js-temp-notes-holder") do
|
|
fill_in "note[note]", with: "One comment on line 185"
|
|
find(".js-note-preview-button").trigger("click")
|
|
end
|
|
within("tr[id='342e16cbbd482ac2047dc679b2749d248cc1428f_18_17'] + .js-temp-notes-holder") do
|
|
fill_in "note[note]", with: "Another comment on line 17"
|
|
find(".js-note-preview-button").trigger("click")
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "posting a note" do
|
|
before do
|
|
within("tr[id='342e16cbbd482ac2047dc679b2749d248cc1428f_18_17'] + .js-temp-notes-holder") do
|
|
fill_in "note[note]", with: "Another comment on line 17"
|
|
click_button("Add Comment")
|
|
end
|
|
end
|
|
|
|
it do
|
|
within("tr[id='342e16cbbd482ac2047dc679b2749d248cc1428f_18_17'] + .js-temp-notes-holder") do
|
|
should have_no_css(".js-temp-notes-holder")
|
|
end
|
|
end
|
|
|
|
it 'should be added as discussion' do
|
|
should have_content("Another comment on line 17")
|
|
should have_css(".notes_holder")
|
|
should have_css(".notes_holder .note", count: 1)
|
|
should have_link("Reply")
|
|
end
|
|
end
|
|
end
|
|
end
|
|
|
|
describe "On merge request discussion", js: true do
|
|
describe "with merge request diff note"
|
|
describe "with commit note"
|
|
describe "with commit diff note"
|
|
end
|