(naïvely) replace git clone/pull execs with vcsrepo

manifests/install.pp

@@ -17,23 +17,18 @@
     require rbenv::dependencies
   }
 
-  exec { "rbenv::checkout ${user}":
-    command => "git clone https://github.com/sstephenson/rbenv.git ${root_path}",
-    user    => $user,
-    group   => $group,
-    creates => $root_path,
-    path    => ['/bin', '/usr/bin', '/usr/sbin'],
-    timeout => 100,
-    cwd     => $home_path,
-    require => Package['git'],
+  vcsrepo { $root_path:
+    source => 'https://github.com/sstephenson/rbenv.git',
+    owern  => $user,
+    group  => $group,
   }
 
   file { "rbenv::rbenvrc ${user}":
     path    => $rbenvrc,
     owner   => $user,
     group   => $group,
     content => template('rbenv/dot.rbenvrc.erb'),
-    require => Exec["rbenv::checkout ${user}"],
+    require => Vcsrepo[$root_path],
   }
 
   exec { "rbenv::shrc ${user}":
@@ -50,6 +45,6 @@
     owner   => $user,
     group   => $group,
     path    => "${root_path}/cache",
-    require => Exec["rbenv::checkout ${user}"]
+    require => Vcsrepo[$root_path]
   }
 }

manifests/plugin.pp

@@ -23,30 +23,15 @@
       path    => $plugins,
       owner   => $user,
       group   => $group,
-      require => Exec["rbenv::checkout ${user}"],
+      require => Vcsrepo[$root_path],
     }
   }
 
-  exec { "rbenv::plugin::checkout ${user} ${plugin_name}":
-    command => "git clone ${source} ${destination}",
-    user    => $user,
+  vcsrepo { $destination:
+    ensure  => latest,
+    source  => $source,
+    owner   => $user,
     group   => $group,
-    creates => $destination,
-    path    => ['/bin', '/usr/bin', '/usr/sbin'],
-    timeout => $timeout,
-    cwd     => $home_path,
     require => File["rbenv::plugins ${user}"],
   }
-
-  exec { "rbenv::plugin::update ${user} ${plugin_name}":
-    command => 'git pull',
-    user    => $user,
-    group   => $group,
-    path    => ['/bin', '/usr/bin', '/usr/sbin'],
-    timeout => $timeout,
-    cwd     => $destination,
-    require => Exec["rbenv::plugin::checkout ${user} ${plugin_name}"],
-    onlyif  => 'git remote update; if [ "$(git rev-parse @{0})" = "$(git rev-parse @{u})" ]; then return 0; else return 1; fi ]',
-  }
-
 }

spec/defines/rbenv__install_spec.rb

@@ -7,8 +7,8 @@
 
   context 'install rbenv' do
     it "clones rbenv from the official repository" do
-      should contain_exec("rbenv::checkout #{user}").
-        with_command("git clone https://github.com/sstephenson/rbenv.git /home/#{user}/.rbenv")
+      should contain_vcsrepo(" /home/#{user}/.rbenv").
+        with_source("https://github.com/sstephenson/rbenv.git")
     end
 
     it "appends in a rc file, a command to include .rbenv/bin folder in PATH env variable" do

spec/defines/rbenv__plugin_spec.rb

@@ -10,26 +10,12 @@
 
   let(:target_path) { "#{dot_rbenv}/plugins/#{plugin_name}" }
 
-  it 'clones repository to the right path' do
-    should contain_exec("rbenv::plugin::checkout #{user} #{plugin_name}").with(
-      :command => "git clone #{source} #{target_path}",
-      :user    => user,
-      :creates => target_path,
+  it 'clones repository to the right path (in the latest version)' do
+    should contain_vcsrepo(target_path).with(
+      :source  => source,
+      :ensure  => :latest,
+      :owner   => user,
       :require => /rbenv::plugins #{user}/,
-      :path    => ['/bin','/usr/bin','/usr/sbin']
-    )
-  end
-
-  it 'pulls the latest plugin changes from their git repos' do
-    should contain_exec("rbenv::plugin::update #{user} #{plugin_name}").with(
-      :command => 'git pull',
-      :user    => user,
-      :cwd     => target_path,
-      :require => /rbenv::plugin::checkout #{user} #{plugin_name}/,
-      :path    => ['/bin','/usr/bin','/usr/sbin'],
-      :onlyif  => 'git remote update; ' \
-                  'if [ "$(git rev-parse @{0})" = "$(git rev-parse @{u})" ]; ' \
-                  'then return 0; else return 1; fi ]'
     )
   end