Issue #2452 has been updated by Markus Roberts.

Status changed from Accepted to Ready for Checkin
% Done changed from 70 to 100
Estimated time changed from 0.05 to 0.15

Change tested and patch sent to list.
----------------------------------------
Bug #2452: Faulty tests in spec/unit/provider/ssh_authorized_key/parsed.rb
http://projects.reductivelabs.com/issues/2452

Author: Markus Roberts
Status: Ready for Checkin
Priority: Low
Assigned to: Markus Roberts
Category: testing
Target version: 0.25.0
Complexity: Trivial
Affected version: 0.25.0
Keywords: 


In two tests we see the logic:

          it "should chmod the key file to 0600" do
                File.chmod(0600, 
File.expand_path("~nobody/.ssh/authorized_keys"))
                @provider.flush
          end

...which doesn't in fact test anything.  It should probably be 

          it "should chmod the key file to 0600" do
                File.expect(:chmod).with(0600, 
File.expand_path("~nobody/.ssh/authorized_keys"))
                @provider.flush
          end




-- 
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