fnichol/chef-user — pull request #6 by nessche (2012-01-11 00:08:17+00:00)
Title: Fixed bugs in resource provider

I started looking into the reason why even though ssh_keygen was set to false, the create action was still creating empty id_dsa and id_dsa.pub empty files (hence the name of the branch). Then I also found other bugs, but I am too lazy to create separate branches for each of them. :)

Change List:
* @create_group was read from new_resource.manage_home rather than new_resource.create_group
* changed order of resource handling in action :remove to have the user being removed last (otherwise action delete on authorized_keys resource was failing because there was no user with the given name)
* changed handling of id_dsa files so that they are not created unless necessary

I tested the code with both ssh_keygen true and false, and for create and remove actions.