kindredgroup / puppet-forge-server

Private Puppet forge server supports local files and both v1 and v3 API proxies
69 stars 44 forks source link

Return an error if file already exists on upload #8

Closed c10l closed 9 years ago

c10l commented 9 years ago

This is a simple patch that will return an error when trying to upload a module that already exists on the module_dir.

i11 commented 9 years ago

Great stuff! Would you mind including a simple unit test to stress added feature? Thanks!

i11 commented 9 years ago

Just pushed initial unit test for directory backend class. It would be great if you could just update it with the upload method test.

https://github.com/unibet/puppet-forge-server/blob/master/spec/unit/backends/directory_spec.rb

c10l commented 9 years ago

Added. Let me know if you want me to add or change anything. Thanks for the gem, it's serving me very well! :)

i11 commented 9 years ago

Thanks a lot! Merging.