spec/models/merge_request/metrics_spec.rb


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
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
67
68
69
70
71
72
73
74
75
76
77
78
79
80
81
82
83
84
85
86
87
88
89
90
91
92
93
94
95
96

# frozen_string_literal: true

require 'spec_helper'

RSpec.describe MergeRequest::Metrics do
  describe 'associations' do
    it { is_expected.to belong_to(:merge_request) }
    it { is_expected.to belong_to(:target_project).class_name('Project') }
    it { is_expected.to belong_to(:latest_closed_by).class_name('User') }
    it { is_expected.to belong_to(:merged_by).class_name('User') }
  end

  describe 'scopes' do
    let_it_be(:metrics_1) { create(:merge_request).metrics.tap { |m| m.update!(merged_at: 10.days.ago) } }
    let_it_be(:metrics_2) { create(:merge_request).metrics.tap { |m| m.update!(merged_at: 5.days.ago) } }

    describe '.merged_after' do
      subject { described_class.merged_after(7.days.ago) }

      it 'finds the record' do
        is_expected.to eq([metrics_2])
      end

      it "doesn't include record outside of the filter" do
        is_expected.not_to include([metrics_1])
      end
    end

    describe '.merged_before' do
      subject { described_class.merged_before(7.days.ago) }

      it 'finds the record' do
        is_expected.to eq([metrics_1])
      end

      it "doesn't include record outside of the filter" do
        is_expected.not_to include([metrics_2])
      end
    end

    describe '.by_target_project' do
      let(:target_project) { metrics_1.target_project }

      subject { described_class.by_target_project(target_project) }

      it 'finds metrics record with the associated target project' do
        is_expected.to eq([metrics_1])
      end
    end
  end

  it_behaves_like 'cleanup by a loose foreign key' do
    let!(:merge_request) { create(:merge_request) }
    let!(:parent) { create(:ci_pipeline, project: merge_request.target_project) }
    let!(:model) { merge_request.metrics.tap { |metrics| metrics.update!(pipeline: parent) } }
  end

  describe 'update' do
    let(:merge_request) { create(:merge_request) }
    let(:metrics) { merge_request.metrics }

    before do
      metrics.update!(
        pipeline_id: 1,
        latest_build_started_at: Time.current,
        latest_build_finished_at: Time.current
      )
    end

    context 'when pipeline_id is nullified' do
      before do
        metrics.update!(pipeline_id: nil)
      end

      it 'nullifies build related columns via DB trigger' do
        metrics.reload

        expect(metrics.latest_build_started_at).to be_nil
        expect(metrics.latest_build_finished_at).to be_nil
      end
    end

    context 'when updated but pipeline_id is not nullified' do
      before do
        metrics.update!(latest_closed_at: Time.current)
      end

      it 'does not nullify build related columns' do
        metrics.reload

        expect(metrics.latest_build_started_at).not_to be_nil
        expect(metrics.latest_build_finished_at).not_to be_nil
      end
    end
  end
end