(maint) Fix the specs due to changes to backup.
This commit is contained in:
parent
aefbf567c5
commit
eee60088dd
4 changed files with 9 additions and 8 deletions
|
@ -29,7 +29,6 @@ describe 'mysql::server::backup class' do
|
|||
end
|
||||
apply_manifest(pp, :catch_failures => true) do |r|
|
||||
expect(r.stderr).to eq("")
|
||||
expect(r.exit_code).to be_zero
|
||||
end
|
||||
end
|
||||
end
|
||||
|
|
|
@ -97,8 +97,12 @@ describe 'mysql::bindings class' do
|
|||
it { should be_installed }
|
||||
end
|
||||
|
||||
describe package(php_package) do
|
||||
it { should be_installed }
|
||||
# This package is not available out of the box and adding in other repos
|
||||
# is a bit much for the scope of this test.
|
||||
unless fact('osfamily') == 'RedHat'
|
||||
describe package(php_package) do
|
||||
it { should be_installed }
|
||||
end
|
||||
end
|
||||
|
||||
describe package(python_package) do
|
||||
|
|
|
@ -14,10 +14,8 @@ describe 'mysql::db define' do
|
|||
|
||||
# Run it twice and test for idempotency
|
||||
apply_manifest(pp, :catch_failures => true)
|
||||
expect(apply_manifest(pp, :catch_failures => true).exit_code).to be_zero
|
||||
end
|
||||
apply_manifest(pp, :catch_changes => true)
|
||||
|
||||
it 'should have the database' do
|
||||
expect(shell("mysql -e 'show databases;'|grep spec1").exit_code).to be_zero
|
||||
end
|
||||
end
|
||||
|
@ -41,7 +39,7 @@ describe 'mysql::db define' do
|
|||
|
||||
# Run it twice and test for idempotency
|
||||
apply_manifest(pp, :catch_failures => true)
|
||||
expect(apply_manifest(pp, :catch_failures => true).exit_code).to be_zero
|
||||
apply_manifest(pp, :catch_changes => true)
|
||||
end
|
||||
|
||||
it 'should have the table' do
|
||||
|
|
|
@ -254,7 +254,7 @@ describe 'mysql_grant' do
|
|||
EOS
|
||||
|
||||
apply_manifest(pp, :catch_failures => true)
|
||||
expect(apply_manifest(pp, :catch_failures => true).exit_code).to be_zero
|
||||
apply_manifest(pp, :catch_changes => true)
|
||||
end
|
||||
end
|
||||
|
||||
|
|
Loading…
Reference in a new issue