diff options
author | Markus Roberts <Markus@reality.com> | 2010-11-10 20:56:42 -0800 |
---|---|---|
committer | Markus Roberts <Markus@reality.com> | 2010-11-10 20:56:42 -0800 |
commit | cc5224c1912f2e95663c14ca803700070130fb69 (patch) | |
tree | e3eb6121cc5e6665edf056aef492a2b77855f168 | |
parent | 5f7d0fbb3699cba28d10adac62c69aedfce2df38 (diff) | |
download | puppet-cc5224c1912f2e95663c14ca803700070130fb69.tar.gz puppet-cc5224c1912f2e95663c14ca803700070130fb69.tar.xz puppet-cc5224c1912f2e95663c14ca803700070130fb69.zip |
Maint. fix for test broken by 00eedac5
As per Nigel, this fixes the test broken by commit 00eedac5 in which a :combine
was added in lib but the corresponding change was not made in the test.
-rwxr-xr-x | test/puppet/tc_suidmanager.rb | 19 |
1 files changed, 5 insertions, 14 deletions
diff --git a/test/puppet/tc_suidmanager.rb b/test/puppet/tc_suidmanager.rb index 449f2aef5..eeb56f2c9 100755 --- a/test/puppet/tc_suidmanager.rb +++ b/test/puppet/tc_suidmanager.rb @@ -80,20 +80,11 @@ class TestSUIDManager < Test::Unit::TestCase warn "Cannot run this test on ruby < 1.8.4" else set_exit_status! - - - Puppet::Util.expects(:execute).with( - 'yay', - { :failonfail => false, - :uid => @user.uid, - - :gid => @user.gid } - ).returns('output') - - - output = Puppet::Util::SUIDManager.run_and_capture 'yay', - @user.uid, - @user.gid + Puppet::Util. + expects(:execute). + with('yay',:combine => true, :failonfail => false, :uid => @user.uid, :gid => @user.gid). + returns('output') + output = Puppet::Util::SUIDManager.run_and_capture 'yay', @user.uid, @user.gid assert_equal 'output', output.first assert_kind_of Process::Status, output.last |