merge_requests_controller_spec.rb 2.77 KB
Newer Older
1 2 3
require 'spec_helper'

describe MergeRequestsController do
4
  let(:project) { create(:project_with_code) }
5
  let(:user)    { create(:user) }
6
  let(:merge_request) { create(:merge_request_with_diffs, project: project, target_branch: "bcf03b5d~3", source_branch: "bcf03b5d") }
7 8 9

  before do
    sign_in(user)
Dmitriy Zaporozhets's avatar
Dmitriy Zaporozhets committed
10
    project.team << [user, :master]
11
    MergeRequestsController.any_instance.stub(validates_merge_request: true)
12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 31 32 33 34 35 36 37 38 39 40 41 42 43 44 45 46 47 48 49 50 51 52 53 54 55 56 57 58 59 60 61 62 63 64 65 66
  end

  describe "#show" do
    shared_examples "export as" do |format|
      it "should generally work" do
        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response).to be_success
      end

      it "should generate it" do
        MergeRequest.any_instance.should_receive(:"to_#{format}")

        get :show, project_id: project.code, id: merge_request.id, format: format
      end

      it "should render it" do
        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response.body).to eq(merge_request.send(:"to_#{format}"))
      end

      it "should not escape Html" do
        MergeRequest.any_instance.stub(:"to_#{format}").and_return('HTML entities &<>" ')

        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response.body).to_not include('&amp;')
        expect(response.body).to_not include('&gt;')
        expect(response.body).to_not include('&lt;')
        expect(response.body).to_not include('&quot;')
      end
    end

    describe "as diff" do
      include_examples "export as", :diff
      let(:format) { :diff }

      it "should really only be a git diff" do
        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response.body).to start_with("diff --git")
      end
    end

    describe "as patch" do
      include_examples "export as", :patch
      let(:format) { :patch }

      it "should really be a git email patch with commit" do
        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response.body[0..100]).to start_with("From #{merge_request.commits.last.id}")
      end

67 68 69
      # TODO: fix or remove
      #it "should contain as many patches as there are commits" do
        #get :show, project_id: project.code, id: merge_request.id, format: format
70

71 72 73 74 75 76
        #patch_count = merge_request.commits.count
        #merge_request.commits.each_with_index do |commit, patch_num|
          #expect(response.body).to match(/^From #{commit.id}/)
          #expect(response.body).to match(/^Subject: \[PATCH #{patch_num}\/#{patch_count}\]/)
        #end
      #end
77 78 79 80 81 82 83 84 85

      it "should contain git diffs" do
        get :show, project_id: project.code, id: merge_request.id, format: format

        expect(response.body).to match(/^diff --git/)
      end
    end
  end
end