summaryrefslogtreecommitdiff
path: root/spec/migrations/add_unique_constraint_to_approvals_user_id_and_merge_request_id_spec.rb
blob: 74830ab4ce215552f676ab157bd7b49284ef9144 (plain)
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
# frozen_string_literal: true

require 'spec_helper'
require Rails.root.join('db', 'post_migrate', '20190404143330_add_unique_constraint_to_approvals_user_id_and_merge_request_id.rb')

describe AddUniqueConstraintToApprovalsUserIdAndMergeRequestId do
  let(:migration) { described_class.new }
  let(:namespaces) { table(:namespaces) }
  let(:projects) { table(:projects) }
  let(:merge_requests) { table(:merge_requests) }
  let(:approvals) { table(:approvals) }

  describe '#up' do
    before do
      namespaces.create(id: 1, name: 'ns', path: 'ns')
      projects.create(id: 1, namespace_id: 1)
      merge_requests.create(id: 1, target_branch: 'master', source_branch: 'feature-1', target_project_id: 1)
      merge_requests.create(id: 2, target_branch: 'master', source_branch: 'feature-2', target_project_id: 1)
    end

    it 'deletes duplicate records and keeps the first one' do
      first_approval = approvals.create(id: 1, merge_request_id: 1, user_id: 1)
      approvals.create(id: 2, merge_request_id: 1, user_id: 1)

      migration.up

      expect(approvals.all.to_a).to contain_exactly(first_approval)
    end

    it 'does not delete unique records' do
      unique_approvals = [
        approvals.create(id: 1, merge_request_id: 1, user_id: 1),
        approvals.create(id: 2, merge_request_id: 1, user_id: 2),
        approvals.create(id: 3, merge_request_id: 2, user_id: 1)
      ]

      migration.up

      expect(approvals.all.to_a).to contain_exactly(*unique_approvals)
    end

    it 'creates unique index' do
      migration.up

      expect(migration.index_exists?(:approvals, [:user_id, :merge_request_id], unique: true)).to be_truthy
    end
  end

  describe '#down' do
    it 'removes unique index' do
      migration.up
      migration.down

      expect(migration.index_exists?(:approvals, [:user_id, :merge_request_id], unique: true)).to be_falsey
    end
  end
end