diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2018-07-03 08:21:08 +0000 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2018-07-03 08:21:08 +0000 |
commit | a65fec0b0b36879960d92cef337d0f3094a7275a (patch) | |
tree | 4c4cf0ff8ed9a7b240fd3b80fb8e781030d70fc5 /spec/lib | |
parent | 275fbf24b1810e2fbef92b6599d5372855b97b46 (diff) | |
parent | 26a8472d6d0aa1eb40285105a0f55f2f7d439897 (diff) | |
download | gitlab-ce-a65fec0b0b36879960d92cef337d0f3094a7275a.tar.gz |
Merge branch 'jprovazn-extra-line' into 'master'
Don't add bottom 'match' line for deleted files
Closes #48160
See merge request gitlab-org/gitlab-ce!20255
Diffstat (limited to 'spec/lib')
-rw-r--r-- | spec/lib/gitlab/diff/file_spec.rb | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/spec/lib/gitlab/diff/file_spec.rb b/spec/lib/gitlab/diff/file_spec.rb index 5dfbb8e71f8..ebeb05d6e02 100644 --- a/spec/lib/gitlab/diff/file_spec.rb +++ b/spec/lib/gitlab/diff/file_spec.rb @@ -26,6 +26,21 @@ describe Gitlab::Diff::File do end end + describe '#diff_lines_for_serializer' do + it 'includes bottom match line if not in the end' do + expect(diff_file.diff_lines_for_serializer.last.type).to eq('match') + end + + context 'when deleted' do + let(:commit) { project.commit('d59c60028b053793cecfb4022de34602e1a9218e') } + let(:diff_file) { commit.diffs.diff_file_with_old_path('files/js/commit.js.coffee') } + + it 'does not include bottom match line' do + expect(diff_file.diff_lines_for_serializer.last.type).not_to eq('match') + end + end + end + describe '#mode_changed?' do it { expect(diff_file.mode_changed?).to be_falsey } end |