Merge pull request #58 from mhaylock/support_plugins_without_prefix

Make prefixing of plugin names optional

Author: jhaals

README.md

@@ -37,6 +37,20 @@ vagrant::plugin { 'vagrant-berkshelf':
 }
 ```
 
+By default, the module will prefix any plugin name that is missing the `vagrant-` prefix. e.g:
+
+```puppet
+vagrant::plugin { 'berkshelf': } # Resolves to vagrant-berkshelf
+```
+
+Some plugins such as [sahara](https://github.com/jedi4ever/sahara) do not use this prefix. You can override the automatic prefix behaviour with the prefix parameter. The usage would look as follows:
+
+```puppet
+vagrant::plugin { 'sahara':
+  prefix => false
+}
+```
+
 Boxes
 --
 

lib/puppet/type/vagrant_plugin.rb

@@ -18,12 +18,6 @@
 
   newparam :name do
     isnamevar
-
-    validate do |value|
-      unless value =~ /\Avagrant-\w+(-\w+)*\z/
-        raise Puppet::Error, "Invalid value for name: #{value}"
-      end
-    end
   end
 
   newparam :user do

manifests/plugin.pp

@@ -10,11 +10,12 @@
   $ensure  = 'present',
   $force   = false,
   $license = undef,
-  $version = latest
+  $version = latest,
+  $prefix  = true
 ) {
   require vagrant
 
-  if $name =~ /^vagrant-/ {
+  if !$prefix or $name =~ /^vagrant-/ {
     $plugin_name = $name
   } else {
     $plugin_name = "vagrant-${name}"

spec/defines/vagrant__plugin_spec.rb

@@ -17,6 +17,18 @@
     it do
       should contain_vagrant_plugin('vagrant-vmware-fusion')
     end
+
+    context 'prefix is false' do
+      let(:params) do
+        {
+          :prefix => false
+        }
+      end
+
+      it do
+        should contain_vagrant_plugin('vmware-fusion')
+      end
+    end
   end
 
   context 'version specified' do