Merge pull request #33 from willdurand/#27-check-config-parameter-bef…

…ore-puppet-run

check whether the configuration parameter is already applied be…

manifests/config/entry.pp

@@ -38,10 +38,19 @@
     default => "${cmd_template} --unset ${entry}"
   }
 
+  $unless = $ensure ? {
+    present => "/usr/bin/test `${cmd_template} ${entry}` = ${value}",
+
+    # NOTE: in this case the parameter will be resetted to the default value so it cannot be checked properly
+    # when to execute the command to remove the command or not.
+    default => undef,
+  }
+
   exec { "composer-config-entry-${entry}-${user}-${ensure}":
     command     => $cmd,
     user        => $user,
     require     => Class['::composer'],
     environment => "HOME=${home_dir}",
+    unless      => $unless
   }
 }

spec/defines/composer_config_entry_spec.rb

@@ -15,7 +15,8 @@
     it { should contain_exec('composer-config-entry-\'process-timeout\'-vagrant-present') \
       .with_command('/usr/local/bin/composer config -g \'process-timeout\' 500') \
       .with_user('vagrant') \
-      .with_environment('HOME=/home/vagrant')
+      .with_environment('HOME=/home/vagrant') \
+      .with_unless('/usr/bin/test `/usr/local/bin/composer config -g \'process-timeout\'` = 500') \
     }
   end