Chnage the way how merge request diff is initialized and saved

Signed-off-by: Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com>

1 files changed, 3 insertions, 9 deletions

diff --git a/spec/models/merge_request_diff_spec.rb b/spec/models/merge_request_diff_spec.rb
index 0a55515e8c6..f2954a39b08 100644
--- a/spec/models/merge_request_diff_spec.rb
+++ b/spec/models/merge_request_diff_spec.rb
@@ -1,15 +1,6 @@
 require 'spec_helper'
 
 describe MergeRequestDiff, models: true do
-  describe 'initialize new object' do
-    subject { build(:merge_request).merge_request_diffs.build }
-
-    it { expect(subject).to be_valid }
-    it { expect(subject.head_commit_sha).to eq('5937ac0a7beb003549fc5fd26fc247adbce4a52e') }
-    it { expect(subject.base_commit_sha).to eq('ae73cb07c9eeaf35924a10f713b364d32b2dd34f') }
-    it { expect(subject.start_commit_sha).to eq('0b4bc9a49b562e85de7cc9e834518ea6828729b9') }
-  end
-
   describe 'create new record' do
     subject { create(:merge_request).merge_request_diff }
 
@@ -17,6 +8,9 @@ describe MergeRequestDiff, models: true do
     it { expect(subject).to be_persisted }
     it { expect(subject.commits.count).to eq(5) }
     it { expect(subject.diffs.count).to eq(8) }
+    it { expect(subject.head_commit_sha).to eq('5937ac0a7beb003549fc5fd26fc247adbce4a52e') }
+    it { expect(subject.base_commit_sha).to eq('ae73cb07c9eeaf35924a10f713b364d32b2dd34f') }
+    it { expect(subject.start_commit_sha).to eq('0b4bc9a49b562e85de7cc9e834518ea6828729b9') }
   end
 
   describe '#diffs' do