diff options
author | Lin Jen-Shin <godfat@godfat.org> | 2017-06-01 21:48:31 +0800 |
---|---|---|
committer | Lin Jen-Shin <godfat@godfat.org> | 2017-06-01 21:48:31 +0800 |
commit | 7da88278c33d20d97cf0eabb76b3117219479d12 (patch) | |
tree | 347f66298605063e36895696472348e0cdff5a0a /spec/models | |
parent | 0ab8c852db118701ae5a1d105c1da74a0b88f60f (diff) | |
download | gitlab-ce-7da88278c33d20d97cf0eabb76b3117219479d12.tar.gz |
Make sure protected can't be null; Test protected!24196-protected-variables
Diffstat (limited to 'spec/models')
-rw-r--r-- | spec/models/ci/variable_spec.rb | 26 |
1 files changed, 24 insertions, 2 deletions
diff --git a/spec/models/ci/variable_spec.rb b/spec/models/ci/variable_spec.rb index 38b869f59ae..077b10227d7 100644 --- a/spec/models/ci/variable_spec.rb +++ b/spec/models/ci/variable_spec.rb @@ -12,11 +12,33 @@ describe Ci::Variable, models: true do it { is_expected.not_to allow_value('foo bar').for(:key) } it { is_expected.not_to allow_value('foo/bar').for(:key) } - before :each do - subject.value = secret_value + describe '.unprotected' do + subject { described_class.unprotected } + + context 'when variable is protected' do + before do + create(:ci_variable, :protected) + end + + it 'returns nothing' do + is_expected.to be_empty + end + end + + context 'when variable is not protected' do + let(:variable) { create(:ci_variable, protected: false) } + + it 'returns the variable' do + is_expected.to contain_exactly(variable) + end + end end describe '#value' do + before do + subject.value = secret_value + end + it 'stores the encrypted value' do expect(subject.encrypted_value).not_to be_nil end |