openstack/puppet-heat
Code Issues Proposed changes

Do not test details of oslo::cache

Browse Source 
... but test interface of oslo::cache, to make these test cases robust
for any change in oslo::cache.

Change-Id: Ic84202156f487e1b4605b665d02a1ebbecc509aa
This commit is contained in:
Takashi Kajinami 2021-08-23 14:10:01 +09:00
parent 7cc1cf4a25
commit b362f4e026
1 changed files with 46 additions and 46 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

92
spec/classes/heat_cache_spec.rb
View File

@ -3,32 +3,34 @@ require 'spec_helper'
describe 'heat::cache' do
let :params do
{ }
{}
end
shared_examples_for 'heat-cache' do
shared_examples_for 'heat::cache' do
context 'with default parameters' do
it 'configures cache' do
is_expected.to contain_heat_config('cache/config_prefix').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/expiration_time').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/backend').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/backend_argument').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/proxies').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/enabled').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/debug_cache_backend').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_servers').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_dead_retry').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_socket_timeout').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_pool_maxsize').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_pool_unused_timeout').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/memcache_pool_connection_get_timeout').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/tls_enabled').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/tls_cafile').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/tls_certfile').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/tls_keyfile').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/tls_allowed_ciphers').with_value('<SERVICE DEFAULT>')
is_expected.to contain_oslo__cache('heat_config').with_manage_backend_package(true)
is_expected.to contain_oslo__cache('heat_config').with(
:config_prefix => '<SERVICE DEFAULT>',
:expiration_time => '<SERVICE DEFAULT>',
:backend => '<SERVICE DEFAULT>',
:backend_argument => '<SERVICE DEFAULT>',
:proxies => '<SERVICE DEFAULT>',
:enabled => '<SERVICE DEFAULT>',
:debug_cache_backend => '<SERVICE DEFAULT>',
:memcache_servers => '<SERVICE DEFAULT>',
:memcache_dead_retry => '<SERVICE DEFAULT>',
:memcache_socket_timeout => '<SERVICE DEFAULT>',
:memcache_pool_maxsize => '<SERVICE DEFAULT>',
:memcache_pool_unused_timeout => '<SERVICE DEFAULT>',
:memcache_pool_connection_get_timeout => '<SERVICE DEFAULT>',
:tls_enabled => '<SERVICE DEFAULT>',
:tls_cafile => '<SERVICE DEFAULT>',
:tls_certfile => '<SERVICE DEFAULT>',
:tls_keyfile => '<SERVICE DEFAULT>',
:tls_allowed_ciphers => '<SERVICE DEFAULT>',
:manage_backend_package => true,
)
is_expected.to contain_heat_config('constraint_validation_cache/caching').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('constraint_validation_cache/expiration_time').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('service_extension_cache/caching').with_value('<SERVICE DEFAULT>')
@ -42,7 +44,7 @@ describe 'heat::cache' do
let :params do
{
:config_prefix => 'prefix',
:expiration_time => '3600',
:expiration_time => 3600,
:backend => 'oslo_cache.memcache_pool',
:proxies => ['proxy01:8888', 'proxy02:8888'],
:enabled => true,
@ -54,10 +56,6 @@ describe 'heat::cache' do
:memcache_pool_unused_timeout => '120',
:memcache_pool_connection_get_timeout => '360',
:tls_enabled => false,
:tls_cafile => nil,
:tls_certfile => nil,
:tls_keyfile => nil,
:tls_allowed_ciphers => nil,
:manage_backend_package => false,
:constraint_validation_caching => true,
:constraint_validation_expiration_time => '3600',
@ -69,25 +67,27 @@ describe 'heat::cache' do
end
it 'configures cache' do
is_expected.to contain_heat_config('cache/config_prefix').with_value('prefix')
is_expected.to contain_heat_config('cache/expiration_time').with_value('3600')
is_expected.to contain_heat_config('cache/backend').with_value('oslo_cache.memcache_pool')
is_expected.to contain_heat_config('cache/backend_argument').with_value('<SERVICE DEFAULT>')
is_expected.to contain_heat_config('cache/proxies').with_value('proxy01:8888,proxy02:8888')
is_expected.to contain_heat_config('cache/enabled').with_value('true')
is_expected.to contain_heat_config('cache/debug_cache_backend').with_value('false')
is_expected.to contain_heat_config('cache/memcache_servers').with_value('memcached01:11211,memcached02:11211')
is_expected.to contain_heat_config('cache/memcache_dead_retry').with_value('60')
is_expected.to contain_heat_config('cache/memcache_socket_timeout').with_value('300.0')
is_expected.to contain_heat_config('cache/memcache_pool_maxsize').with_value('10')
is_expected.to contain_heat_config('cache/memcache_pool_unused_timeout').with_value('120')
is_expected.to contain_heat_config('cache/memcache_pool_connection_get_timeout').with_value('360')
is_expected.to contain_heat_config('cache/tls_enabled').with_value('false')
is_expected.to contain_heat_config('cache/tls_cafile').with_value('nil')
is_expected.to contain_heat_config('cache/tls_certfile').with_value('nil')
is_expected.to contain_heat_config('cache/tls_keyfile').with_value('nil')
is_expected.to contain_heat_config('cache/tls_allowed_ciphers').with_value('nil')
is_expected.to contain_oslo__cache('heat_config').with_manage_backend_package(false)
is_expected.to contain_oslo__cache('heat_config').with(
:config_prefix => 'prefix',
:expiration_time => 3600,
:backend => 'oslo_cache.memcache_pool',
:backend_argument => '<SERVICE DEFAULT>',
:proxies => ['proxy01:8888', 'proxy02:8888'],
:enabled => true,
:debug_cache_backend => false,
:memcache_servers => ['memcached01:11211', 'memcached02:11211'],
:memcache_dead_retry => '60',
:memcache_socket_timeout => '300.0',
:memcache_pool_maxsize => '10',
:memcache_pool_unused_timeout => '120',
:memcache_pool_connection_get_timeout => '360',
:tls_enabled => false,
:tls_cafile => '<SERVICE DEFAULT>',
:tls_certfile => '<SERVICE DEFAULT>',
:tls_keyfile => '<SERVICE DEFAULT>',
:tls_allowed_ciphers => '<SERVICE DEFAULT>',
:manage_backend_package => false,
)
is_expected.to contain_heat_config('constraint_validation_cache/caching').with_value('true')
is_expected.to contain_heat_config('constraint_validation_cache/expiration_time').with_value('3600')
is_expected.to contain_heat_config('service_extension_cache/caching').with_value('true')
@ -106,7 +106,7 @@ describe 'heat::cache' do
facts.merge!(OSDefaults.get_facts())
end
it_configures 'heat-cache'
it_configures 'heat::cache'
end
end