Skip to content

Commit 74d3ff3

Browse files
committed
Stub deprecated_parameter type
1 parent 5a7ed69 commit 74d3ff3

File tree

4 files changed

+49
-7
lines changed

4 files changed

+49
-7
lines changed

Diff for: .fixtures.yml

-2
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,4 @@ fixtures:
99
ruby_task_helper: 'https://[email protected]/puppetlabs/puppetlabs-ruby_task_helper'
1010
stdlib: 'https://github.com/puppetlabs/puppetlabs-stdlib'
1111
facts: 'https://github.com/puppetlabs/puppetlabs-facts'
12-
puppet_enterprise:
13-
repo: [email protected]:puppetlabs/puppetlabs-puppet_enterprise.git
1412

Diff for: spec/classes/init_spec.rb

+16
Original file line numberDiff line numberDiff line change
@@ -3,13 +3,24 @@
33
describe 'pe_databases' do
44
on_supported_os.each do |os, os_facts|
55
context "on #{os}" do
6+
let(:pre_condition) do
7+
<<-PRE_COND
8+
define puppet_enterprise::deprecated_parameter() {}
9+
PRE_COND
10+
end
611
let(:facts) { os_facts }
712

813
it { is_expected.to compile }
914
end
1015
end
1116

1217
context 'with default parameters' do
18+
let(:pre_condition) do
19+
<<-PRE_COND
20+
define puppet_enterprise::deprecated_parameter() {}
21+
PRE_COND
22+
end
23+
1324
it {
1425
is_expected.to contain_class('pe_databases::pg_repack').with(disable_maintenance: false)
1526

@@ -40,6 +51,11 @@
4051
end
4152

4253
context 'when systemd is not the init provider' do
54+
let(:pre_condition) do
55+
<<-PRE_COND
56+
define puppet_enterprise::deprecated_parameter() {}
57+
PRE_COND
58+
end
4359
let(:facts) { { pe_databases: { have_systemd: false } } }
4460

4561
it { is_expected.to contain_notify('pe_databases_systemd_warn') }

Diff for: spec/classes/pg_repack_spec.rb

+21-3
Original file line numberDiff line numberDiff line change
@@ -22,15 +22,27 @@
2222

2323
on_supported_os.each do |os, os_facts|
2424
context "on #{os}" do
25-
let(:pre_condition) { 'include pe_databases' }
25+
let(:pre_condition) do
26+
<<-PRE_COND
27+
define puppet_enterprise::deprecated_parameter() {}
28+
29+
include pe_databases
30+
PRE_COND
31+
end
2632
let(:facts) { os_facts }
2733

2834
it { is_expected.to compile }
2935
end
3036
end
3137

3238
context 'with default parameters' do
33-
let(:pre_condition) { 'include pe_databases' }
39+
let(:pre_condition) do
40+
<<-PRE_COND
41+
define puppet_enterprise::deprecated_parameter() {}
42+
43+
include pe_databases
44+
PRE_COND
45+
end
3446

3547
it {
3648
tables_hash.each do |name, val|
@@ -69,7 +81,13 @@
6981
end
7082

7183
context 'when customizing timers' do
72-
let(:pre_condition) { "class {'pe_databases': facts_tables_repack_timer => 'Tue *-*-* 04:20:00'}" }
84+
let(:pre_condition) do
85+
<<-PRE_COND
86+
define puppet_enterprise::deprecated_parameter() {}
87+
88+
class {'pe_databases': facts_tables_repack_timer => 'Tue *-*-* 04:20:00'}
89+
PRE_COND
90+
end
7391

7492
it {
7593
is_expected.to contain_pe_databases__collect('facts').with(

Diff for: spec/defines/collect_spec.rb

+12-2
Original file line numberDiff line numberDiff line change
@@ -2,7 +2,12 @@
22

33
describe 'pe_databases::collect' do
44
context 'when repacking tables' do
5-
let(:pre_condition) { 'include pe_databases' }
5+
let(:pre_condition) do
6+
<<-PRE_COND
7+
define puppet_enterprise::deprecated_parameter() {}
8+
include pe_databases
9+
PRE_COND
10+
end
611
let(:title) { 'test' }
712
let(:params) do
813
{
@@ -30,7 +35,12 @@
3035
end
3136

3237
context 'when disabling maintenance' do
33-
let(:pre_condition) { 'include pe_databases' }
38+
let(:pre_condition) do
39+
<<-PRE_COND
40+
define puppet_enterprise::deprecated_parameter() {}
41+
include pe_databases
42+
PRE_COND
43+
end
3444
let(:title) { 'test' }
3545
let(:params) do
3646
{

0 commit comments

Comments
 (0)