Issue #2290 has been updated by Markus Roberts.

Status changed from Accepted to Ready for Checkin

The core problem was that spec/unit/util/stoarage.rb wasn't cleaning up after 
itself and the webrick test was assuming it had a clean slate on entry.  If 
spec/unit/util/storage.rb was run by "rake spec" before 
spec/integration/network/server/webrick.rb without any intervening tests which 
did a Puppet.settings.clear, the the webrick test would change the permissions 
on /tmp.

<pre>
--- a/spec/unit/util/storage.rb
+++ b/spec/unit/util/storage.rb
@@ -12,6 +12,10 @@ describe Puppet::Util::Storage do
         Puppet[:statedir] = Dir.tmpdir()
     end
 
+    after(:all) do
+        Puppet.settings.clear
+    end
+
     before(:each) do
         Puppet::Util::Storage.clear()
     end
</pre>


Branch at http://github.com/MarkusQ/puppet/tree/ticket/master/2290

----------------------------------------
Bug #2290: Puppet specs chmod /tmp from 1777 to 1755
http://projects.reductivelabs.com/issues/2290

Author: James Turnbull
Status: Ready for Checkin
Priority: Normal
Assigned to: Markus Roberts
Category: testing
Target version: 0.25.0
Complexity: Unknown
Affected version: 0.24.8
Keywords: 


The chmod works but we don't then revert the change.  This breaks a lot of 
other stuff on the host running the tests.


-- 
You have received this notification because you have either subscribed to it, 
or are involved in it.
To change your notification preferences, please click here: 
http://reductivelabs.com/redmine/my/account

--~--~---------~--~----~------------~-------~--~----~
You received this message because you are subscribed to the Google Groups 
"Puppet Bugs" group.
To post to this group, send email to [email protected]
To unsubscribe from this group, send email to 
[email protected]
For more options, visit this group at 
http://groups.google.com/group/puppet-bugs?hl=en
-~----------~----~----~----~------~----~------~--~---

Reply via email to