Merge pull request #43 from scalp42/notify_statsd_dependencies

only install StatsD dependencies when git updated

recipes/install.rb

@@ -24,9 +24,11 @@
   repository node['statsd']['repo']
   revision node['statsd']['version']
   action :sync
+  notifies :run, 'execute[install StatsD dependencies]', :immediately
 end
 
-execute 'install dependencies' do
+execute 'install StatsD dependencies' do
   command 'npm install -d'
   cwd node['statsd']['path']
+  action :nothing
 end

spec/unit/recipes/default_spec.rb

@@ -9,6 +9,9 @@
   cached(:upstart) do
     chef_run.template('/etc/init/statsd.conf')
   end
+  cached(:git) do
+    chef_run.git(node['statsd']['path'])
+  end
 
   %w(nodejs git).each do |recipe|
     it "includes the #{recipe} recipe" do
@@ -24,8 +27,12 @@
     )
   end
 
-  it 'installs statsd dependencies' do
-    expect(chef_run).to run_execute('npm install -d') \
+  it 'notifies to install statsd dependencies' do
+    expect(git).to notify('execute[install StatsD dependencies]').to(:run).immediately
+  end
+
+  it 'only installs statsd dependencies if git notified' do
+    expect(chef_run).to_not run_execute('npm install -d') \
     .with(cwd: node['statsd']['path'])
   end