Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
path: root/spec
diff options
context:
space:
mode:
authorRiyad Preukschas <riyad@informatik.uni-bremen.de>2013-01-15 03:11:35 +0400
committerRiyad Preukschas <riyad@informatik.uni-bremen.de>2013-01-15 03:52:57 +0400
commitd28176b132bdc74055f31e2d62665d5d74ebacb4 (patch)
tree4bc9c8e3778d51020da0e8bb318aa7c3830ee67d /spec
parentf8e1f4a7c9ee2554efcf7376a0cfbefae8956d7b (diff)
Replace commit diff note request specs with spinach features
Diffstat (limited to 'spec')
-rw-r--r--spec/requests/notes_on_commits_spec.rb137
1 files changed, 0 insertions, 137 deletions
diff --git a/spec/requests/notes_on_commits_spec.rb b/spec/requests/notes_on_commits_spec.rb
deleted file mode 100644
index e301f9ac9bf..00000000000
--- a/spec/requests/notes_on_commits_spec.rb
+++ /dev/null
@@ -1,137 +0,0 @@
-require 'spec_helper'
-
-describe "On a commit diff", js: true do
- let!(:project) { create(:project) }
- let!(:commit) { project.repository.commit("bcf03b5de6c33f3869ef70d68cf06e679d1d7f9a") }
-
- before do
- login_as :user
- project.team << [@user, :reporter]
-
- visit project_commit_path(project, commit)
- end
-
- subject { page }
-
- describe "when adding a note" do
- before do
- find("#0_185_185.line_holder .js-add-diff-note-button").trigger("click")
- end
-
- describe "the notes holder" do
- it { should have_css("#0_185_185.line_holder + .js-temp-notes-holder") }
-
- it { within(".js-temp-notes-holder") { should have_css(".new_note") } }
- end
-
- describe "the note form" do
- # set up hidden fields correctly
- it { within(".js-temp-notes-holder") { find("#note_noteable_type").value.should == "Commit" } }
- it { within(".js-temp-notes-holder") { find("#note_noteable_id").value.should == "" } }
- it { within(".js-temp-notes-holder") { find("#note_commit_id").value.should == "bcf03b5de6c33f3869ef70d68cf06e679d1d7f9a" } }
- it { within(".js-temp-notes-holder") { find("#note_line_code").value.should == "0_185_185" } }
-
- # buttons
- it { should have_button("Add Comment") }
- it { should have_css(".js-close-discussion-note-form", text: "Cancel") }
-
- # notification options
- it { should have_unchecked_field("Project team") }
- it { should have_checked_field("Commit author") }
-
- it "shouldn't add a second form for same row" do
- find("#0_185_185.line_holder .js-add-diff-note-button").trigger("click")
-
- should have_css("#0_185_185.line_holder + .js-temp-notes-holder form", count: 1)
- end
-
- it "should be removed when canceled" do
- find(".js-close-discussion-note-form").trigger("click")
-
- should have_no_css(".js-temp-notes-holder")
- end
- end
- end
-
- describe "with muliple note forms" do
- before do
- find("#0_185_185.line_holder .js-add-diff-note-button").trigger("click")
- find("#1_18_17.line_holder .js-add-diff-note-button").trigger("click")
- end
-
- # has two line forms
- 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("#0_185_185.line_holder + .js-temp-notes-holder") do
- fill_in "note[note]", with: "One comment on line 185"
- find(".js-note-preview-button").trigger("click")
- end
- within("#1_18_17.line_holder + .js-temp-notes-holder") do
- fill_in "note[note]", with: "Another comment on line 17"
- find(".js-note-preview-button").trigger("click")
- end
- end
-
- # check if previews were rendered separately
- it { within("#0_185_185.line_holder + .js-temp-notes-holder") { should have_css(".js-note-preview", text: "One comment on line 185") } }
- it { within("#1_18_17.line_holder + .js-temp-notes-holder") { should have_css(".js-note-preview", text: "Another comment on line 17") } }
- end
-
- describe "posting a note" do
- before do
- within("#1_18_17.line_holder + .js-temp-notes-holder") do
- fill_in "note[note]", with: "Another comment on line 17"
- click_button("Add Comment")
- end
- end
-
- # removed form after submit
- it { should have_no_css("#1_18_17.line_holder + .js-temp-notes-holder") }
-
- # added discussion
- it { should have_content("Another comment on line 17") }
- it { should have_css("#1_18_17.line_holder + .notes_holder") }
- it { should have_css("#1_18_17.line_holder + .notes_holder .note", count: 1) }
- it { should have_link("Reply") }
-
- it "should remove last note of a discussion" do
- within("#1_18_17.line_holder + .notes_holder") do
- find(".js-note-delete").trigger("click")
- end
-
- # removed whole discussion
- should_not have_css(".note_holder")
- should have_css("#1_18_17.line_holder + #1_18_18.line_holder")
- end
- end
- end
-
- describe "when replying to a note" do
- before do
- # create first note
- find("#0_184_184.line_holder .js-add-diff-note-button").trigger("click")
- within("#0_184_184.line_holder + .js-temp-notes-holder") do
- fill_in "note[note]", with: "One comment on line 184"
- click_button("Add Comment")
- end
- # create second note
- within("#0_184_184.line_holder + .notes_holder") do
- find(".js-discussion-reply-button").trigger("click")
- fill_in "note[note]", with: "An additional comment in reply"
- click_button("Add Comment")
- end
- end
-
- # inserted note
- it { should have_content("An additional comment in reply") }
- it { within("#0_184_184.line_holder + .notes_holder") { should have_css(".note", count: 2) } }
-
- # removed form after reply
- it { within("#0_184_184.line_holder + .notes_holder") { should have_no_css("form") } }
- it { within("#0_184_184.line_holder + .notes_holder") { should have_link("Reply") } }
- end
-end
- \ No newline at end of file