2019-12-11 13:08:10 -05:00
|
|
|
# frozen_string_literal: true
|
|
|
|
|
|
|
|
require 'spec_helper'
|
|
|
|
|
2020-06-24 14:09:03 -04:00
|
|
|
RSpec.describe Sha256Attribute do
|
2019-12-11 13:08:10 -05:00
|
|
|
let(:model) { Class.new { include Sha256Attribute } }
|
|
|
|
|
|
|
|
before do
|
|
|
|
columns = [
|
|
|
|
double(:column, name: 'name', type: :text),
|
|
|
|
double(:column, name: 'sha256', type: :binary)
|
|
|
|
]
|
|
|
|
|
|
|
|
allow(model).to receive(:columns).and_return(columns)
|
|
|
|
end
|
|
|
|
|
|
|
|
describe '#sha_attribute' do
|
|
|
|
context 'when in non-production' do
|
|
|
|
before do
|
|
|
|
stub_rails_env('development')
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the table exists' do
|
|
|
|
before do
|
|
|
|
allow(model).to receive(:table_exists?).and_return(true)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'defines a SHA attribute for a binary column' do
|
|
|
|
expect(model).to receive(:attribute)
|
|
|
|
.with(:sha256, an_instance_of(Gitlab::Database::Sha256Attribute))
|
|
|
|
|
|
|
|
model.sha256_attribute(:sha256)
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'raises ArgumentError when the column type is not :binary' do
|
|
|
|
expect { model.sha256_attribute(:name) }.to raise_error(ArgumentError)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the table does not exist' do
|
|
|
|
it 'allows the attribute to be added and issues a warning' do
|
|
|
|
allow(model).to receive(:table_exists?).and_return(false)
|
|
|
|
|
|
|
|
expect(model).not_to receive(:columns)
|
|
|
|
expect(model).to receive(:attribute)
|
|
|
|
expect(model).to receive(:warn)
|
|
|
|
|
|
|
|
model.sha256_attribute(:name)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when the column does not exist' do
|
|
|
|
it 'allows the attribute to be added and issues a warning' do
|
|
|
|
allow(model).to receive(:table_exists?).and_return(true)
|
|
|
|
|
|
|
|
expect(model).to receive(:columns)
|
|
|
|
expect(model).to receive(:attribute)
|
|
|
|
expect(model).to receive(:warn)
|
|
|
|
|
|
|
|
model.sha256_attribute(:no_name)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when other execeptions are raised' do
|
|
|
|
it 'logs and re-rasises the error' do
|
|
|
|
allow(model).to receive(:table_exists?).and_raise(ActiveRecord::NoDatabaseError.new('does not exist'))
|
|
|
|
|
|
|
|
expect(model).not_to receive(:columns)
|
|
|
|
expect(model).not_to receive(:attribute)
|
|
|
|
expect(Gitlab::AppLogger).to receive(:error)
|
|
|
|
|
|
|
|
expect { model.sha256_attribute(:name) }.to raise_error(ActiveRecord::NoDatabaseError)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
|
|
|
|
context 'when in production' do
|
|
|
|
before do
|
|
|
|
stub_rails_env('production')
|
|
|
|
end
|
|
|
|
|
|
|
|
it 'defines a SHA attribute' do
|
|
|
|
expect(model).not_to receive(:table_exists?)
|
|
|
|
expect(model).not_to receive(:columns)
|
|
|
|
expect(model).to receive(:attribute).with(:sha256, an_instance_of(Gitlab::Database::Sha256Attribute))
|
|
|
|
|
|
|
|
model.sha256_attribute(:sha256)
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|
|
|
|
end
|