Merge pull request #650 from cyberious/master

(maint) Fix tests from #527 merge - Looks like copy tests mismatched logic of when it should expect changes
This commit is contained in:
Morgan Haskel 2015-06-25 16:50:42 -07:00
commit b87da29090

View file

@ -126,7 +126,7 @@ describe 'postgresql_psql', :unless => UNSUPPORTED_PLATFORMS.include?(fact('osfa
EOS
apply_manifest(pp, :catch_failures => true)
apply_manifest(pp, :expect_changes => true)
apply_manifest(pp, :catch_changes => true)
end
it 'should run SQL when the onlyif query returns rows' do
@ -136,12 +136,12 @@ describe 'postgresql_psql', :unless => UNSUPPORTED_PLATFORMS.include?(fact('osfa
db => 'postgres',
psql_user => 'postgres',
command => 'select * from pg_database limit 1',
onlfy => 'select 1 where 1=1',
onlyif => 'select 1 where 1=1',
}
EOS
apply_manifest(pp, :catch_failures => true)
apply_manifest(pp, :catch_changes => true)
apply_manifest(pp, :expect_changes => true)
end
context 'with secure password passing by environment' do