diff options
author | Felipe Artur <felipefac@gmail.com> | 2016-06-06 11:30:17 -0300 |
---|---|---|
committer | Felipe Artur <felipefac@gmail.com> | 2016-06-06 13:06:21 -0300 |
commit | 9264203103bbd2b4f46ce777304f210b07765c43 (patch) | |
tree | ae409366398ca3882b0ed9e55c37708a7c27fee4 /spec/lib | |
parent | 8450fe3074140b86e7d2e5a85d2cdb65051906ee (diff) | |
download | gitlab-ce-9264203103bbd2b4f46ce777304f210b07765c43.tar.gz |
change add_concurrent_index function argumentsissue_3359
Diffstat (limited to 'spec/lib')
-rw-r--r-- | spec/lib/gitlab/database/migration_helpers_spec.rb | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/spec/lib/gitlab/database/migration_helpers_spec.rb b/spec/lib/gitlab/database/migration_helpers_spec.rb index 35ade7a2be0..83ddabe6b0b 100644 --- a/spec/lib/gitlab/database/migration_helpers_spec.rb +++ b/spec/lib/gitlab/database/migration_helpers_spec.rb @@ -16,14 +16,21 @@ describe Gitlab::Database::MigrationHelpers, lib: true do end context 'using PostgreSQL' do - it 'creates the index concurrently' do - expect(Gitlab::Database).to receive(:postgresql?).and_return(true) + before { expect(Gitlab::Database).to receive(:postgresql?).and_return(true) } + it 'creates the index concurrently' do expect(model).to receive(:add_index). with(:users, :foo, algorithm: :concurrently) model.add_concurrent_index(:users, :foo) end + + it 'creates unique index concurrently' do + expect(model).to receive(:add_index). + with(:users, :foo, { algorithm: :concurrently, unique: true }) + + model.add_concurrent_index(:users, :foo, unique: true) + end end context 'using MySQL' do @@ -31,7 +38,7 @@ describe Gitlab::Database::MigrationHelpers, lib: true do expect(Gitlab::Database).to receive(:postgresql?).and_return(false) expect(model).to receive(:add_index). - with(:users, :foo) + with(:users, :foo, {}) model.add_concurrent_index(:users, :foo) end |