build_spec.rb 11.2 KB
Newer Older
D
Douwe Maan 已提交
1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27
# == Schema Information
#
# Table name: builds
#
#  id                 :integer          not null, primary key
#  project_id         :integer
#  status             :string(255)
#  finished_at        :datetime
#  trace              :text
#  created_at         :datetime
#  updated_at         :datetime
#  started_at         :datetime
#  runner_id          :integer
#  commit_id          :integer
#  coverage           :float
#  commands           :text
#  job_id             :integer
#  name               :string(255)
#  deploy             :boolean          default(FALSE)
#  options            :text
#  allow_failure      :boolean          default(FALSE), not null
#  stage              :string(255)
#  trigger_request_id :integer
#

require 'spec_helper'

D
Dmitriy Zaporozhets 已提交
28 29
describe Ci::Build do
  let(:project) { FactoryGirl.create :ci_project }
K
WIP  
Kamil Trzcinski 已提交
30
  let(:gl_project) { FactoryGirl.create :empty_project, gitlab_ci_project: project }
K
Kamil Trzcinski 已提交
31
  let(:commit) { FactoryGirl.create :ci_commit, gl_project: gl_project }
D
Dmitriy Zaporozhets 已提交
32
  let(:build) { FactoryGirl.create :ci_build, commit: commit }
D
Douwe Maan 已提交
33

K
Kamil Trzcinski 已提交
34
  it { is_expected.to validate_presence_of :ref }
D
Douwe Maan 已提交
35

D
Dmitriy Zaporozhets 已提交
36
  it { is_expected.to respond_to :trace_html }
D
Douwe Maan 已提交
37 38

  describe :first_pending do
D
Dmitriy Zaporozhets 已提交
39 40
    let(:first) { FactoryGirl.create :ci_build, commit: commit, status: 'pending', created_at: Date.yesterday }
    let(:second) { FactoryGirl.create :ci_build, commit: commit, status: 'pending' }
D
Douwe Maan 已提交
41
    before { first; second }
D
Dmitriy Zaporozhets 已提交
42
    subject { Ci::Build.first_pending }
D
Douwe Maan 已提交
43

D
Dmitriy Zaporozhets 已提交
44 45
    it { is_expected.to be_a(Ci::Build) }
    it('returns with the first pending build') { is_expected.to eq(first) }
D
Douwe Maan 已提交
46 47 48 49 50 51 52
  end

  describe :create_from do
    before do
      build.status = 'success'
      build.save
    end
D
Dmitriy Zaporozhets 已提交
53
    let(:create_from_build) { Ci::Build.create_from build }
D
Douwe Maan 已提交
54

V
Valery Sizov 已提交
55
    it 'there should be a pending task' do
D
Dmitriy Zaporozhets 已提交
56
      expect(Ci::Build.pending.count(:all)).to eq 0
D
Douwe Maan 已提交
57
      create_from_build
D
Dmitriy Zaporozhets 已提交
58
      expect(Ci::Build.pending.count(:all)).to be > 0
D
Douwe Maan 已提交
59 60 61 62 63 64 65 66 67 68 69 70
    end
  end

  describe :ignored? do
    subject { build.ignored? }

    context 'if build is not allowed to fail' do
      before { build.allow_failure = false }

      context 'and build.status is success' do
        before { build.status = 'success' }

D
Dmitriy Zaporozhets 已提交
71
        it { is_expected.to be_falsey }
D
Douwe Maan 已提交
72 73 74 75 76
      end

      context 'and build.status is failed' do
        before { build.status = 'failed' }

D
Dmitriy Zaporozhets 已提交
77
        it { is_expected.to be_falsey }
D
Douwe Maan 已提交
78 79 80 81 82 83 84 85 86
      end
    end

    context 'if build is allowed to fail' do
      before { build.allow_failure = true }

      context 'and build.status is success' do
        before { build.status = 'success' }

D
Dmitriy Zaporozhets 已提交
87
        it { is_expected.to be_falsey }
D
Douwe Maan 已提交
88 89 90 91 92
      end

      context 'and build.status is failed' do
        before { build.status = 'failed' }

D
Dmitriy Zaporozhets 已提交
93
        it { is_expected.to be_truthy }
D
Douwe Maan 已提交
94 95 96 97 98 99 100
      end
    end
  end

  describe :trace do
    subject { build.trace_html }

D
Dmitriy Zaporozhets 已提交
101
    it { is_expected.to be_empty }
D
Douwe Maan 已提交
102 103 104 105 106

    context 'if build.trace contains text' do
      let(:text) { 'example output' }
      before { build.trace = text }

D
Dmitriy Zaporozhets 已提交
107
      it { is_expected.to include(text) }
K
Kamil Trzcinski 已提交
108
      it { expect(subject.length).to be >= text.length }
D
Douwe Maan 已提交
109
    end
110 111 112 113 114 115 116 117 118 119 120

    context 'if build.trace hides token' do
      let(:token) { 'my_secret_token' }

      before do
        build.project.update_attributes(token: token)
        build.update_attributes(trace: token)
      end

      it { is_expected.to_not include(token) }
    end
D
Douwe Maan 已提交
121 122 123 124 125
  end

  describe :timeout do
    subject { build.timeout }

D
Dmitriy Zaporozhets 已提交
126
    it { is_expected.to eq(commit.project.timeout) }
D
Douwe Maan 已提交
127 128 129
  end

  describe :options do
V
Valery Sizov 已提交
130
    let(:options) do
D
Douwe Maan 已提交
131
      {
V
Valery Sizov 已提交
132 133
        image: "ruby:2.1",
        services: [
D
Douwe Maan 已提交
134 135 136
          "postgres"
        ]
      }
V
Valery Sizov 已提交
137
    end
D
Douwe Maan 已提交
138 139

    subject { build.options }
D
Dmitriy Zaporozhets 已提交
140
    it { is_expected.to eq(options) }
D
Douwe Maan 已提交
141 142 143 144 145
  end

  describe :allow_git_fetch do
    subject { build.allow_git_fetch }

D
Dmitriy Zaporozhets 已提交
146
    it { is_expected.to eq(project.allow_git_fetch) }
D
Douwe Maan 已提交
147 148 149 150 151
  end

  describe :project do
    subject { build.project }

D
Dmitriy Zaporozhets 已提交
152
    it { is_expected.to eq(commit.project) }
D
Douwe Maan 已提交
153 154 155 156 157
  end

  describe :project_id do
    subject { build.project_id }

D
Dmitriy Zaporozhets 已提交
158
    it { is_expected.to eq(commit.project_id) }
D
Douwe Maan 已提交
159 160 161 162 163
  end

  describe :project_name do
    subject { build.project_name }

D
Dmitriy Zaporozhets 已提交
164
    it { is_expected.to eq(project.name) }
D
Douwe Maan 已提交
165 166 167 168 169
  end

  describe :repo_url do
    subject { build.repo_url }

D
Dmitriy Zaporozhets 已提交
170
    it { is_expected.to eq(project.repo_url_with_auth) }
D
Douwe Maan 已提交
171 172 173 174 175 176
  end

  describe :extract_coverage do
    context 'valid content & regex' do
      subject { build.extract_coverage('Coverage 1033 / 1051 LOC (98.29%) covered', '\(\d+.\d+\%\) covered') }

D
Dmitriy Zaporozhets 已提交
177
      it { is_expected.to eq(98.29) }
D
Douwe Maan 已提交
178 179 180 181 182
    end

    context 'valid content & bad regex' do
      subject { build.extract_coverage('Coverage 1033 / 1051 LOC (98.29%) covered', 'very covered') }

D
Dmitriy Zaporozhets 已提交
183
      it { is_expected.to be_nil }
D
Douwe Maan 已提交
184 185 186 187 188
    end

    context 'no coverage content & regex' do
      subject { build.extract_coverage('No coverage for today :sad:', '\(\d+.\d+\%\) covered') }

D
Dmitriy Zaporozhets 已提交
189
      it { is_expected.to be_nil }
D
Douwe Maan 已提交
190 191 192 193 194
    end

    context 'multiple results in content & regex' do
      subject { build.extract_coverage(' (98.39%) covered. (98.29%) covered', '\(\d+.\d+\%\) covered') }

D
Dmitriy Zaporozhets 已提交
195
      it { is_expected.to eq(98.29) }
D
Douwe Maan 已提交
196 197 198 199 200 201 202
    end
  end

  describe :variables do
    context 'returns variables' do
      subject { build.variables }

203 204 205 206 207 208 209 210
      let(:predefined_variables) do
        [
          { key: :CI_BUILD_NAME, value: 'test', public: true },
          { key: :CI_BUILD_STAGE, value: 'stage', public: true },
        ]
      end

      let(:yaml_variables) do
D
Douwe Maan 已提交
211
        [
V
Valery Sizov 已提交
212
          { key: :DB_NAME, value: 'postgres', public: true }
D
Douwe Maan 已提交
213
        ]
V
Valery Sizov 已提交
214
      end
D
Douwe Maan 已提交
215

216 217 218 219 220 221 222 223 224 225 226 227 228 229 230
      before { build.update_attributes(stage: 'stage') }

      it { is_expected.to eq(predefined_variables + yaml_variables) }

      context 'for tag' do
        let(:tag_variable) do
          [
            { key: :CI_BUILD_TAG, value: 'master', public: true }
          ]
        end

        before { build.update_attributes(tag: true) }

        it { is_expected.to eq(tag_variable + predefined_variables + yaml_variables) }
      end
D
Douwe Maan 已提交
231 232

      context 'and secure variables' do
V
Valery Sizov 已提交
233
        let(:secure_variables) do
D
Douwe Maan 已提交
234
          [
V
Valery Sizov 已提交
235
            { key: 'SECRET_KEY', value: 'secret_value', public: false }
D
Douwe Maan 已提交
236
          ]
V
Valery Sizov 已提交
237
        end
D
Douwe Maan 已提交
238 239

        before do
D
Dmitriy Zaporozhets 已提交
240
          build.project.variables << Ci::Variable.new(key: 'SECRET_KEY', value: 'secret_value')
D
Douwe Maan 已提交
241 242
        end

243
        it { is_expected.to eq(predefined_variables + yaml_variables + secure_variables) }
D
Douwe Maan 已提交
244 245

        context 'and trigger variables' do
D
Dmitriy Zaporozhets 已提交
246 247
          let(:trigger) { FactoryGirl.create :ci_trigger, project: project }
          let(:trigger_request) { FactoryGirl.create :ci_trigger_request_with_variables, commit: commit, trigger: trigger }
V
Valery Sizov 已提交
248
          let(:trigger_variables) do
D
Douwe Maan 已提交
249
            [
V
Valery Sizov 已提交
250
              { key: :TRIGGER_KEY, value: 'TRIGGER_VALUE', public: false }
D
Douwe Maan 已提交
251
            ]
V
Valery Sizov 已提交
252
          end
253 254 255 256 257
          let(:predefined_trigger_variable) do
            [
              { key: :CI_BUILD_TRIGGERED, value: 'true', public: true }
            ]
          end
D
Douwe Maan 已提交
258 259 260 261 262

          before do
            build.trigger_request = trigger_request
          end

263
          it { is_expected.to eq(predefined_variables + predefined_trigger_variable + yaml_variables + secure_variables + trigger_variables) }
D
Douwe Maan 已提交
264 265 266 267
        end
      end
    end
  end
K
Kamil Trzcinski 已提交
268 269

  describe :project_recipients do
K
Kamil Trzcinski 已提交
270 271
    let(:pusher_email) { 'pusher@gitlab.test' }
    let(:user) { User.new(notification_email: pusher_email) }
K
Kamil Trzcinski 已提交
272 273 274 275 276 277 278 279 280 281 282 283 284 285 286 287 288 289 290 291 292 293 294 295 296 297 298 299 300 301
    subject { build.project_recipients }

    before do
      build.update_attributes(user: user)
    end

    it 'should return pusher_email as only recipient when no additional recipients are given' do
      project.update_attributes(email_add_pusher: true,
                                email_recipients: '')
      is_expected.to eq([pusher_email])
    end

    it 'should return pusher_email and additional recipients' do
      project.update_attributes(email_add_pusher: true,
                                email_recipients: 'rec1 rec2')
      is_expected.to eq(['rec1', 'rec2', pusher_email])
    end

    it 'should return recipients' do
      project.update_attributes(email_add_pusher: false,
                                email_recipients: 'rec1 rec2')
      is_expected.to eq(['rec1', 'rec2'])
    end

    it 'should return unique recipients only' do
      project.update_attributes(email_add_pusher: true,
                                email_recipients: "rec1 rec1 #{pusher_email}")
      is_expected.to eq(['rec1', pusher_email])
    end
  end
302 303 304 305 306 307 308 309 310 311 312 313 314 315 316 317 318 319 320 321 322 323 324 325 326 327 328 329 330 331 332 333 334 335 336 337 338 339 340 341 342 343 344 345 346 347 348 349 350 351 352 353 354 355 356 357 358 359 360 361 362 363 364 365 366 367 368 369 370 371 372 373 374 375 376 377 378 379 380 381 382 383 384 385 386 387 388 389 390 391 392 393 394 395 396 397 398 399 400 401 402

  describe :can_be_served? do
    let(:runner) { FactoryGirl.create :ci_specific_runner }

    before { build.project.runners << runner }

    context 'runner without tags' do
      it 'can handle builds without tags' do
        expect(build.can_be_served?(runner)).to be_truthy
      end

      it 'cannot handle build with tags' do
        build.tag_list = ['aa']
        expect(build.can_be_served?(runner)).to be_falsey
      end
    end

    context 'runner with tags' do
      before { runner.tag_list = ['bb', 'cc'] }

      it 'can handle builds without tags' do
        expect(build.can_be_served?(runner)).to be_truthy
      end

      it 'can handle build with matching tags' do
        build.tag_list = ['bb']
        expect(build.can_be_served?(runner)).to be_truthy
      end

      it 'cannot handle build with not matching tags' do
        build.tag_list = ['aa']
        expect(build.can_be_served?(runner)).to be_falsey
      end
    end
  end

  describe :any_runners_online? do
    subject { build.any_runners_online? }

    context 'when no runners' do
      it { is_expected.to be_falsey }
    end

    context 'if there are runner' do
      let(:runner) { FactoryGirl.create :ci_specific_runner }

      before do
        build.project.runners << runner
        runner.update_attributes(contacted_at: 1.second.ago)
      end

      it { is_expected.to be_truthy }

      it 'that is inactive' do
        runner.update_attributes(active: false)
        is_expected.to be_falsey
      end

      it 'that is not online' do
        runner.update_attributes(contacted_at: nil)
        is_expected.to be_falsey
      end

      it 'that cannot handle build' do
        expect_any_instance_of(Ci::Build).to receive(:can_be_served?).and_return(false)
        is_expected.to be_falsey
      end

    end
  end

  describe :show_warning? do
    subject { build.show_warning? }

    %w(pending).each do |state|
      context "if commit_status.status is #{state}" do
        before { build.status = state }

        it { is_expected.to be_truthy }

        context "and there are specific runner" do
          let(:runner) { FactoryGirl.create :ci_specific_runner, contacted_at: 1.second.ago }

          before do
            build.project.runners << runner
            runner.save
          end

          it { is_expected.to be_falsey }
        end
      end
    end

    %w(success failed canceled running).each do |state|
      context "if commit_status.status is #{state}" do
        before { build.status = state }

        it { is_expected.to be_falsey }
      end
    end
  end
K
Kamil Trzcinski 已提交
403 404 405 406 407 408 409 410 411 412 413 414 415 416 417

  describe :download_url do
    subject { build.download_url }

    it "should be nil if artifact doesn't exist" do
      build.update_attributes(artifacts_file: nil)
      is_expected.to be_nil
    end

    it 'should be nil if artifact exist' do
      gif = fixture_file_upload(Rails.root + 'spec/fixtures/banana_sample.gif', 'image/gif')
      build.update_attributes(artifacts_file: gif)
      is_expected.to_not be_nil
    end
  end
D
Douwe Maan 已提交
418
end