diff options
Diffstat (limited to 'spec/models/diff_note_spec.rb')
-rw-r--r-- | spec/models/diff_note_spec.rb | 48 |
1 files changed, 24 insertions, 24 deletions
diff --git a/spec/models/diff_note_spec.rb b/spec/models/diff_note_spec.rb index fda00a693f0..41781b167ba 100644 --- a/spec/models/diff_note_spec.rb +++ b/spec/models/diff_note_spec.rb @@ -1,4 +1,4 @@ -require 'spec_helper' +require "spec_helper" describe DiffNote do include RepoHelpers @@ -84,40 +84,40 @@ describe DiffNote do end end - describe '#create_diff_file callback' do + describe "#create_diff_file callback" do let(:noteable) { create(:merge_request) } let(:project) { noteable.project } - context 'merge request' do + context "merge request" do let!(:diff_note) { create(:diff_note_on_merge_request, project: project, noteable: noteable) } - it 'creates a diff note file' do + it "creates a diff note file" do expect(diff_note.reload.note_diff_file).to be_present end - it 'does not create diff note file if it is a reply' do + it "does not create diff note file if it is a reply" do expect { create(:diff_note_on_merge_request, noteable: noteable, in_reply_to: diff_note) } .not_to change(NoteDiffFile, :count) end end - context 'commit' do + context "commit" do let!(:diff_note) { create(:diff_note_on_commit, project: project) } - it 'creates a diff note file' do + it "creates a diff note file" do expect(diff_note.reload.note_diff_file).to be_present end - it 'does not create diff note file if it is a reply' do + it "does not create diff note file if it is a reply" do expect { create(:diff_note_on_commit, in_reply_to: diff_note) } .not_to change(NoteDiffFile, :count) end end end - describe '#diff_file', :clean_gitlab_redis_shared_state do - context 'when note_diff_file association exists' do - it 'returns persisted diff file data' do + describe "#diff_file", :clean_gitlab_redis_shared_state do + context "when note_diff_file association exists" do + it "returns persisted diff file data" do diff_file = subject.diff_file expect(diff_file.diff.to_hash.with_indifferent_access) @@ -125,8 +125,8 @@ describe DiffNote do end end - context 'when the discussion was created in the diff' do - it 'returns correct diff file' do + context "when the discussion was created in the diff" do + it "returns correct diff file" do diff_file = subject.diff_file expect(diff_file.old_path).to eq(position.old_path) @@ -135,7 +135,7 @@ describe DiffNote do end end - context 'when discussion is outdated or not created in the diff' do + context "when discussion is outdated or not created in the diff" do let(:diff_refs) { project.commit(sample_commit.id).diff_refs } let(:position) do Gitlab::Diff::Position.new( @@ -147,7 +147,7 @@ describe DiffNote do ) end - it 'returns the correct diff file' do + it "returns the correct diff file" do diff_file = subject.diff_file expect(diff_file.old_path).to eq(position.old_path) @@ -156,8 +156,8 @@ describe DiffNote do end end - context 'note diff file creation enqueuing' do - it 'enqueues CreateNoteDiffFileWorker if it is the first note of a discussion' do + context "note diff file creation enqueuing" do + it "enqueues CreateNoteDiffFileWorker if it is the first note of a discussion" do subject.note_diff_file.destroy! expect(CreateNoteDiffFileWorker).to receive(:perform_async).with(subject.id) @@ -165,7 +165,7 @@ describe DiffNote do subject.reload.diff_file end - it 'does not enqueues CreateNoteDiffFileWorker if not first note of a discussion' do + it "does not enqueues CreateNoteDiffFileWorker if not first note of a discussion" do mr = create(:merge_request) diff_note = create(:diff_note_on_merge_request, project: mr.project, noteable: mr) reply_diff_note = create(:diff_note_on_merge_request, in_reply_to: diff_note) @@ -283,7 +283,7 @@ describe DiffNote do end end - describe '#created_at_diff?' do + describe "#created_at_diff?" do let(:diff_refs) { project.commit(sample_commit.id).diff_refs } let(:position) do Gitlab::Diff::Position.new( @@ -318,17 +318,17 @@ describe DiffNote do end end - describe '#supports_suggestion?' do - context 'when noteable does not support suggestions' do - it 'returns false' do + describe "#supports_suggestion?" do + context "when noteable does not support suggestions" do + it "returns false" do allow(subject.noteable).to receive(:supports_suggestion?) { false } expect(subject.supports_suggestion?).to be(false) end end - context 'when line is not suggestible' do - it 'returns false' do + context "when line is not suggestible" do + it "returns false" do allow_any_instance_of(Gitlab::Diff::Line).to receive(:suggestible?) { false } expect(subject.supports_suggestion?).to be(false) |