openstack/puppet-swift
Code Issues Proposed changes

Merge "set url_base option in staticweb swift middleware"

Browse Source
This commit is contained in:
Jenkins
2016-11-05 15:44:57 +00:00
committed by Gerrit Code Review
parent c7b3299808 239a8e592c
commit 22f32f8ea1
2 changed files with 19 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

11
manifests/proxy/staticweb.pp
View File

@@ -8,6 +8,12 @@
#
# include 'swift::proxy::staticweb'
#
# == Parameters
#
# [*url_base*]
# (optional) The URL scheme and/or the host name used to generate redirects.
# Defaults to $::os_service_default.
#
# == Authors
#
# Mehdi Abaakouk <sileht@sileht.net>
@@ -16,11 +22,14 @@
#
# Copyright 2012 eNovance licensing@enovance.com
#
class swift::proxy::staticweb() {
class swift::proxy::staticweb(
$url_base = $::os_service_default
) {
include ::swift::deps
swift_proxy_config {
'filter:staticweb/use': value => 'egg:swift#staticweb';
'filter:staticweb/url_base': value => $url_base;
}
}

9
spec/classes/swift_proxy_staticweb_spec.rb
View File

@@ -8,4 +8,13 @@ describe 'swift::proxy::staticweb' do
it { is_expected.to contain_swift_proxy_config('filter:staticweb/use').with_value('egg:swift#staticweb') }
describe "when overriding default parameters" do
let :params do
{
:url_base => 'https://www.example.com',
}
end
it { is_expected.to contain_swift_proxy_config('filter:staticweb/url_base').with_value('https://www.example.com') }
end
end