puppet/ssh
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

Merge pull request #39 from Phil-Friderici/spec_test_fixes

Browse Source 
fixing spec warnings
This commit is contained in:
Garrett Honeycutt 2013-12-19 04:47:16 -08:00
commit 7c378d0e8b
2 changed files with 23 additions and 23 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
manifests/init.pp
View File

@ -63,7 +63,7 @@ class ssh (
$ssh_config_sendenv_xmodifiers_real = $ssh_config_sendenv_xmodifiers $ssh_config_sendenv_xmodifiers_real = $ssh_config_sendenv_xmodifiers
} }
default: { default: {
fail("ssh_config_sendenv_xmodifiers type must be true or false.") fail('ssh_config_sendenv_xmodifiers type must be true or false.')
} }
} }

44
spec/classes/init_spec.rb
View File

@ -9,9 +9,9 @@ describe 'ssh' do
:sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ=='
} }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should_not include_class('common')} it { should_not contain_class('common')}
it { it {
should contain_package('ssh_packages').with({ should contain_package('ssh_packages').with({
@ -92,9 +92,9 @@ describe 'ssh' do
:sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ=='
} }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should_not include_class('common')} it { should_not contain_class('common')}
it { it {
should contain_package('ssh_packages').with({ should contain_package('ssh_packages').with({
@ -176,9 +176,9 @@ describe 'ssh' do
:sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ=='
} }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should_not include_class('common')} it { should_not contain_class('common')}
it { it {
should contain_package('ssh_packages').with({ should contain_package('ssh_packages').with({
@ -260,9 +260,9 @@ describe 'ssh' do
:sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ==' :sshrsakey => 'AAAAB3NzaC1yc2EAAAABIwAAAQEArGElx46pD6NNnlxVaTbp0ZJMgBKCmbTCT3RaeCk0ZUJtQ8wkcwTtqIXmmiuFsynUT0DFSd8UIodnBOPqitimmooAVAiAi30TtJVzADfPScMiUnBJKZajIBkEMkwUcqsfh630jyBvLPE/kyQcxbEeGtbu1DG3monkeymanOBW1AKc5o+cJLXcInLnbowMG7NXzujT3BRYn/9s5vtT1V9cuZJs4XLRXQ50NluxJI7sVfRPVvQI9EMbTS4AFBXUej3yfgaLSV+nPZC/lmJ2gR4t/tKvMFF9m16f8IcZKK7o0rK7v81G/tREbOT5YhcKLK+0wBfR6RsmHzwy4EddZloyLQ=='
} }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should_not include_class('common')} it { should_not contain_class('common')}
it { it {
should contain_package('ssh_packages').with({ should contain_package('ssh_packages').with({
@ -350,7 +350,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/ssh supports osfamilies RedHat, Suse and Debian. Detected osfamily is <C64>./) }.to raise_error(Puppet::Error,/ssh supports osfamilies RedHat, Suse and Debian. Detected osfamily is <C64>./)
end end
end end
@ -462,9 +462,9 @@ describe 'ssh' do
{ :manage_root_ssh_config => 'true' } { :manage_root_ssh_config => 'true' }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should include_class('common')} it { should contain_class('common')}
it { it {
should contain_file('root_ssh_dir').with({ should contain_file('root_ssh_dir').with({
@ -521,7 +521,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_config_port must be a valid number and is set to <22invalid>./) }.to raise_error(Puppet::Error,/sshd_config_port must be a valid number and is set to <22invalid>./)
end end
end end
@ -541,7 +541,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/manage_root_ssh_config is <invalid> and must be \'true\' or \'false\'./) }.to raise_error(Puppet::Error,/manage_root_ssh_config is <invalid> and must be \'true\' or \'false\'./)
end end
end end
@ -560,7 +560,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_password_authentication may be either \'yes\' or \'no\' and is set to <invalid>./) }.to raise_error(Puppet::Error,/sshd_password_authentication may be either \'yes\' or \'no\' and is set to <invalid>./)
end end
end end
@ -579,7 +579,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_allow_tcp_forwarding may be either \'yes\' or \'no\' and is set to <invalid>./) }.to raise_error(Puppet::Error,/sshd_allow_tcp_forwarding may be either \'yes\' or \'no\' and is set to <invalid>./)
end end
end end
@ -598,7 +598,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_x11_forwarding may be either \'yes\' or \'no\' and is set to <invalid>./) }.to raise_error(Puppet::Error,/sshd_x11_forwarding may be either \'yes\' or \'no\' and is set to <invalid>./)
end end
end end
@ -617,7 +617,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_use_pam may be either \'yes\' or \'no\' and is set to <invalid>./) }.to raise_error(Puppet::Error,/sshd_use_pam may be either \'yes\' or \'no\' and is set to <invalid>./)
end end
end end
@ -636,7 +636,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/sshd_client_alive_interval must be an integer and is set to <invalid>./) }.to raise_error(Puppet::Error,/sshd_client_alive_interval must be an integer and is set to <invalid>./)
end end
end end
@ -655,7 +655,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error,/ssh_config_sendenv_xmodifiers type must be true or false./) }.to raise_error(Puppet::Error,/ssh_config_sendenv_xmodifiers type must be true or false./)
end end
end end
@ -689,9 +689,9 @@ describe 'ssh' do
{ :manage_firewall => true } { :manage_firewall => true }
end end
it { should include_class('ssh')} it { should contain_class('ssh')}
it { should_not include_class('common')} it { should_not contain_class('common')}
it { it {
should contain_firewall('22 open port 22 for SSH').with({ should contain_firewall('22 open port 22 for SSH').with({
@ -759,7 +759,7 @@ describe 'ssh' do
it 'should fail' do it 'should fail' do
expect { expect {
should include_class('ssh') should contain_class('ssh')
}.to raise_error(Puppet::Error) }.to raise_error(Puppet::Error)
end end
end end