CID-508: add specs for provider_account validation
This commit is contained in:
parent
56b7c914d1
commit
05fa1d323f
@ -29,6 +29,21 @@ RSpec.describe Devops::Model::DeployEnvEc2, type: :model do
|
|||||||
env.validate!
|
env.validate!
|
||||||
}.to raise_error Devops::NonExistingUser
|
}.to raise_error Devops::NonExistingUser
|
||||||
end
|
end
|
||||||
|
|
||||||
|
it "isn't valid if provider_account isn't present" do
|
||||||
|
allow(stubbed_connector).to receive(:provider_accounts) do |provider|
|
||||||
|
if provider == 'ec2'
|
||||||
|
account_name = 'ec2_account'
|
||||||
|
else
|
||||||
|
account_name = 'static_account'
|
||||||
|
end
|
||||||
|
[Provider::ProviderFactory.get_account_class(provider).new('account_name' => account_name)]
|
||||||
|
end
|
||||||
|
env.provider_account = 'static_account'
|
||||||
|
expect(env).not_to be_valid
|
||||||
|
env.provider_account = 'ec2_account'
|
||||||
|
expect(env).to be_valid
|
||||||
|
end
|
||||||
end
|
end
|
||||||
|
|
||||||
describe '#initialize' do
|
describe '#initialize' do
|
||||||
|
|||||||
@ -12,8 +12,22 @@ RSpec.describe Devops::Model::DeployEnvStatic, type: :model do
|
|||||||
end
|
end
|
||||||
|
|
||||||
it_behaves_like 'deploy env'
|
it_behaves_like 'deploy env'
|
||||||
end
|
|
||||||
|
|
||||||
|
it "isn't valid if provider_account isn't present" do
|
||||||
|
allow(stubbed_connector).to receive(:provider_accounts) do |provider|
|
||||||
|
if provider == 'ec2'
|
||||||
|
account_name = 'ec2_account'
|
||||||
|
else
|
||||||
|
account_name = 'static_account'
|
||||||
|
end
|
||||||
|
[Provider::ProviderFactory.get_account_class(provider).new('account_name' => account_name)]
|
||||||
|
end
|
||||||
|
env.provider_account = 'ec2_account'
|
||||||
|
expect(env).not_to be_valid
|
||||||
|
env.provider_account = 'static_account'
|
||||||
|
expect(env).to be_valid
|
||||||
|
end
|
||||||
|
end
|
||||||
|
|
||||||
describe '.create' do
|
describe '.create' do
|
||||||
it 'returns instance of DeployEnvStatic' do
|
it 'returns instance of DeployEnvStatic' do
|
||||||
|
|||||||
Loading…
Reference in New Issue
Block a user