BigW Consortium Gitlab

issues_helper_spec.rb 5.65 KB
Newer Older
Andrew8xx8 committed
1 2 3
require "spec_helper"

describe IssuesHelper do
4
  let(:project) { create(:project) }
Andrew8xx8 committed
5 6
  let(:issue) { create :issue, project: project }
  let(:ext_project) { create :redmine_project }
Andrew8xx8 committed
7

8
  describe "url_for_issue" do
9
    let(:issues_url) { ext_project.external_issue_tracker.issues_url}
10
    let(:ext_expected) { issues_url.gsub(':id', issue.iid.to_s).gsub(':project_id', ext_project.id.to_s) }
11
    let(:int_expected) { polymorphic_path([@project.namespace, @project, issue]) }
Andrew8xx8 committed
12

13
    it "returns internal path if used internal tracker" do
Andrew8xx8 committed
14
      @project = project
15

16
      expect(url_for_issue(issue.iid)).to match(int_expected)
Andrew8xx8 committed
17 18
    end

19
    it "returns path to external tracker" do
Andrew8xx8 committed
20 21
      @project = ext_project

22
      expect(url_for_issue(issue.iid)).to match(ext_expected)
23 24 25 26 27 28
    end

    it "returns path to internal issue when internal option passed" do
      @project = ext_project

      expect(url_for_issue(issue.iid, ext_project, internal: true)).to match(int_expected)
Andrew8xx8 committed
29 30
    end

31
    it "returns empty string if project nil" do
Andrew8xx8 committed
32 33
      @project = nil

34
      expect(url_for_issue(issue.iid)).to eq ""
Andrew8xx8 committed
35
    end
36

37 38 39 40 41 42 43 44 45 46 47 48
    it 'returns an empty string if issue_url is invalid' do
      expect(project).to receive_message_chain('issues_tracker.issue_url') { 'javascript:alert("foo");' }

      expect(url_for_issue(issue.iid, project)).to eq ''
    end

    it 'returns an empty string if issue_path is invalid' do
      expect(project).to receive_message_chain('issues_tracker.issue_path') { 'javascript:alert("foo");' }

      expect(url_for_issue(issue.iid, project, only_path: true)).to eq ''
    end

49 50 51
    describe "when external tracker was enabled and then config removed" do
      before do
        @project = ext_project
52
        allow(Gitlab.config).to receive(:issues_tracker).and_return(nil)
53 54
      end

55
      it "returns external path" do
56
        expect(url_for_issue(issue.iid)).to match(ext_expected)
57 58
      end
    end
Andrew8xx8 committed
59
  end
60

61
  describe '#award_user_list' do
62 63 64
    it "returns a comma-separated list of the first X users" do
      user = build_stubbed(:user, name: 'Joe')
      awards = Array.new(3, build_stubbed(:award_emoji, user: user))
65

66 67
      expect(award_user_list(awards, nil, limit: 3))
        .to eq('Joe, Joe, and Joe')
68 69
    end

70
    it "displays the current user's name as 'You'" do
71 72
      user = build_stubbed(:user, name: 'Joe')
      award = build_stubbed(:award_emoji, user: user)
73

74 75
      expect(award_user_list([award], user)).to eq('You')
      expect(award_user_list([award], nil)).to eq 'Joe'
76 77
    end

78 79 80 81
    it "truncates lists" do
      user = build_stubbed(:user, name: 'Jane')
      awards = Array.new(5, build_stubbed(:award_emoji, user: user))

82 83
      expect(award_user_list(awards, nil, limit: 3))
        .to eq('Jane, Jane, Jane, and 2 more.')
84 85
    end

86 87 88 89 90 91
    it "displays the current user in front of other users" do
      current_user = build_stubbed(:user)
      my_award = build_stubbed(:award_emoji, user: current_user)
      award = build_stubbed(:award_emoji, user: build_stubbed(:user, name: 'Jane'))
      awards = Array.new(5, award).push(my_award)

92 93
      expect(award_user_list(awards, current_user, limit: 2))
        .to eq("You, Jane, and 4 more.")
94 95 96
    end
  end

97
  describe '#award_state_class' do
98
    let!(:upvote) { create(:award_emoji) }
Valery Sizov committed
99

100 101
    it "returns disabled string for unauthenticated user" do
      expect(award_state_class(AwardEmoji.all, nil)).to eq("disabled")
Valery Sizov committed
102 103 104
    end

    it "returns active string for author" do
105
      expect(award_state_class(AwardEmoji.all, upvote.user)).to eq("active")
Valery Sizov committed
106 107
    end
  end
Valery Sizov committed
108

109
  describe "awards_sort" do
Valery Sizov committed
110
    it "sorts a hash so thumbsup and thumbsdown are always on top" do
Valery Sizov committed
111
      data = { "thumbsdown" => "some value", "lifter" => "some value", "thumbsup" => "some value" }
Douwe Maan committed
112
      expect(awards_sort(data).keys).to eq(%w(thumbsup thumbsdown lifter))
Valery Sizov committed
113 114
    end
  end
115

116
  describe "milestone_options" do
117
    it "gets closed milestone from current issue" do
118 119 120 121 122
      closed_milestone = create(:closed_milestone, project: project)
      milestone1       = create(:milestone, project: project)
      milestone2       = create(:milestone, project: project)
      issue.update_attributes(milestone_id: closed_milestone.id)

123
      options = milestone_options(issue)
124

125 126 127 128 129
      expect(options).to have_selector('option[selected]', text: closed_milestone.title)
      expect(options).to have_selector('option', text: milestone1.title)
      expect(options).to have_selector('option', text: milestone2.title)
    end
  end
130 131 132 133 134 135

  describe "#link_to_discussions_to_resolve" do
    describe "passing only a merge request" do
      let(:merge_request) { create(:merge_request) }

      it "links just the merge request" do
136
        expected_path = project_merge_request_path(merge_request.project, merge_request)
137 138 139 140 141 142 143 144 145 146 147 148

        expect(link_to_discussions_to_resolve(merge_request, nil)).to include(expected_path)
      end

      it "containst the reference to the merge request" do
        expect(link_to_discussions_to_resolve(merge_request, nil)).to include(merge_request.to_reference)
      end
    end

    describe "when passing a discussion" do
      let(:diff_note) {  create(:diff_note_on_merge_request) }
      let(:merge_request) { diff_note.noteable }
Douwe Maan committed
149
      let(:discussion) { diff_note.to_discussion }
150 151 152 153 154 155 156 157 158 159 160 161

      it "links to the merge request with first note if a single discussion was passed" do
        expected_path = Gitlab::UrlBuilder.build(diff_note)

        expect(link_to_discussions_to_resolve(merge_request, discussion)).to include(expected_path)
      end

      it "contains both the reference to the merge request and a mention of the discussion" do
        expect(link_to_discussions_to_resolve(merge_request, discussion)).to include("#{merge_request.to_reference} (discussion #{diff_note.id})")
      end
    end
  end
Andrew8xx8 committed
162
end