diff options
author | Brett Walker <bwalker@gitlab.com> | 2018-05-10 10:00:32 -0500 |
---|---|---|
committer | Brett Walker <bwalker@gitlab.com> | 2018-05-10 10:00:32 -0500 |
commit | 42d27f0b43df544bab2ad5bc4e082728d86c7388 (patch) | |
tree | 213e6f0c028d8f60fb4f68c22b710ac4102fbb7d /spec | |
parent | 8279df2c055abecfd51ef79b87fbcef5da37c618 (diff) | |
download | gitlab-ce-42d27f0b43df544bab2ad5bc4e082728d86c7388.tar.gz |
only issue a warning if column doesn't exist
Diffstat (limited to 'spec')
-rw-r--r-- | spec/models/concerns/sha_attribute_spec.rb | 10 |
1 files changed, 6 insertions, 4 deletions
diff --git a/spec/models/concerns/sha_attribute_spec.rb b/spec/models/concerns/sha_attribute_spec.rb index 592feddf1dc..0d3beb6a6e3 100644 --- a/spec/models/concerns/sha_attribute_spec.rb +++ b/spec/models/concerns/sha_attribute_spec.rb @@ -36,24 +36,26 @@ describe ShaAttribute do end context 'when the table does not exist' do - it 'allows the attribute to be added' 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.sha_attribute(:name) end end context 'when the column does not exist' do - it 'raises ArgumentError' 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).not_to receive(:attribute) + expect(model).to receive(:attribute) + expect(model).to receive(:warn) - expect { model.sha_attribute(:no_name) }.to raise_error(ArgumentError) + model.sha_attribute(:no_name) end end |