add permissions exec after gem install #17

manifests/gem.pp

@@ -54,6 +54,10 @@
   exec { "rbenv-rehash-${gem}-${ruby_version}":
     command     => "${install_dir}/bin/rbenv rehash",
     refreshonly => true,
+  }~>
+  exec { "rbenv-permissions-${gem}-${ruby_version}":
+    command     => "/bin/chown -R ${rbenv::owner}:${rbenv::group} ${install_dir}/versions/${ruby_version}/lib/ruby/gems && /bin/chmod -R g+w ${install_dir}/versions/${ruby_version}/lib/ruby/gems",
+    refreshonly => true,
   }
 
   Exec { require => Exec["rbenv-install-${ruby_version}"] }

manifests/plugin.pp

@@ -8,18 +8,6 @@
 #   Default: $rbenv::install_dir
 #   This variable is required.
 #
-# [$owner]
-#   This is set when you declare the rbenv class. There is no
-#   need to overrite it when calling the rbenv::build define.
-#   Default: $rbenv::owner
-#   This variable is required.
-#
-# [$group]
-#   This is set when you declare the rbenv class. There is no
-#   need to overrite it when calling the rbenv::build define.
-#   Default: $rbenv::group
-#   This variable is required.
-#
 # === Requires
 #
 # You will need to install the git package on the host system.
@@ -34,8 +22,6 @@
 #
 define rbenv::plugin(
   $install_dir = $rbenv::install_dir,
-  $owner       = $rbenv::owner,
-  $group       = $rbenv::group,
 ) {
   include rbenv
 
@@ -48,7 +34,7 @@
     unless  => "/usr/bin/test -d ${install_dir}/plugins/${plugin[1]}",
   }~>
   exec { "rbenv-permissions-${name}":
-    command     => "/bin/chown -R ${owner}:${group} ${install_dir} && /bin/chmod -R g+w ${install_dir}",
+    command     => "/bin/chown -R ${rbenv::owner}:${rbenv::group} ${install_dir} && /bin/chmod -R g+w ${install_dir}",
     refreshonly => true,
   }
 

spec/defines/gem_spec.rb

@@ -13,10 +13,8 @@
     end
 
     it { should include_class('rbenv') }
-
     it { should contain_exec("gem-install-bundler-2.0.0-p247") }
-
     it { should contain_exec("rbenv-rehash-bundler-2.0.0-p247") }
-
+    it { should contain_exec("rbenv-permissions-bundler-2.0.0-p247") }
   end
 end

spec/defines/plugin_spec.rb

@@ -7,31 +7,11 @@
     let(:params) do
       {
         :install_dir => '/usr/local/rbenv',
-        :owner => 'root',
-        :group => 'adm',
       }
     end
 
     it { should include_class('rbenv') }
-
     it { should contain_exec("install-sstephenson/ruby-build") }
-    #it { should contain_exec("install-ruby-build").with(
-    #  {
-    #    'command' => "/usr/bin/git clone git://github.com/sstephenson/ruby-build.git",
-    #    'cwd' => "/usr/local/rbenv/plugins",
-    #    'onlyif' => "/usr/bin/test -d /usr/local/rbenv/plugins",
-    #    'unless' => "/usr/bin/test -d /usr/local/rbenv/plugins/ruby-build",
-    #  }
-    #)}
-
     it { should contain_exec("rbenv-permissions-sstephenson/ruby-build") }
-    #it { should contain_exec('rbenv-permissions-ruby-build').with(
-    #  {
-    #    'command' => "chown -R root:adm /usr/local/rbenv && chmod -R g+w /usr/local/rbenv",
-    #    'refreshonly' => true,
-    #  }
-    #)}
-
   end
-
 end