require 'spec_helper'

describe Gitlab::Diff::FileCollection::MergeRequestDiff do
  let(:merge_request) { create(:merge_request) }
  let(:subject) { described_class.new(merge_request.merge_request_diff, diff_options: nil) }
  let(:diff_files) { subject.diff_files }

  describe '#diff_files' do
    it 'does not highlight binary files' do
      allow_any_instance_of(Gitlab::Diff::File).to receive(:text?).and_return(false)

      expect_any_instance_of(Gitlab::Diff::File).not_to receive(:highlighted_diff_lines)

      diff_files
    end

    it 'does not highlight files marked as undiffable in .gitattributes' do
      allow_any_instance_of(Gitlab::Diff::File).to receive(:diffable?).and_return(false)

      expect_any_instance_of(Gitlab::Diff::File).not_to receive(:highlighted_diff_lines)

      diff_files
    end
  end

  it_behaves_like 'unfoldable diff' do
    let(:diffable) { merge_request.merge_request_diff }
  end

  it 'uses a different cache key if diff line keys change' do
    mr_diff = described_class.new(merge_request.merge_request_diff, diff_options: nil)
    key = mr_diff.cache_key

    stub_const('Gitlab::Diff::Line::SERIALIZE_KEYS', [:foo])

    expect(mr_diff.cache_key).not_to eq(key)
  end

  it_behaves_like 'diff statistics' do
    let(:collection_default_args) do
      { diff_options: {} }
    end
    let(:diffable) { merge_request.merge_request_diff }
    let(:stub_path) { '.gitignore' }
  end

  it 'returns a valid instance of a DiffCollection' do
    expect(diff_files).to be_a(Gitlab::Git::DiffCollection)
  end
end