openstack/puppet-nova
Code Issues Proposed changes

Remove remaining usage of absolute class names

Browse Source 
Change-Id: I4855c9505517a4cc6c3ccad48c40105d06a8bcfc
This commit is contained in:
Takashi Kajinami 2020-11-19 20:56:41 +09:00
parent 678c6523c2
commit 7395acbba0
4 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/wsgi/apache_api.pp
View File

@ -136,7 +136,7 @@ class nova::wsgi::apache_api (
include apache::mod::ssl
}
if ! defined(Class[::nova::api]) {
if ! defined(Class[nova::api]) {
fail('::nova::api class must be declared in composition layer.')
}

4
manifests/wsgi/apache_metadata.pp
View File

@ -129,8 +129,8 @@ class nova::wsgi::apache_metadata (
ensure_package => $ensure_package,
}
if ! defined(Class[::nova::metadata]) {
fail('::nova::metadata class must be declared in composition layer.')
if ! defined(Class[nova::metadata]) {
fail('nova::metadata class must be declared in composition layer.')
}
Service <| title == 'httpd' |> { tag +> 'nova-service' }

4
spec/classes/nova_wsgi_apache_api_spec.rb
View File

@ -97,13 +97,13 @@ describe 'nova::wsgi::apache_api' do
)}
end
context 'when ::nova::api is missing in the composition layer' do
context 'when nova::api is missing in the composition layer' do
let :pre_condition do
"include nova"
end
it { should raise_error(Puppet::Error, /::nova::api class must be declared in composition layer./) }
it { should raise_error(Puppet::Error, /nova::api class must be declared in composition layer./) }
end
end

2
spec/classes/nova_wsgi_apache_metadata_spec.rb
View File

@ -99,7 +99,7 @@ describe 'nova::wsgi::apache_metadata' do
"include nova"
end
it { should raise_error(Puppet::Error, /::nova::metadata class must be declared in composition layer./) }
it { should raise_error(Puppet::Error, /nova::metadata class must be declared in composition layer./) }
end
end