Add NpmInstall post-{commit,checkout,merge,rewrite} hooks

Author: jawshooah

config/default.yml

@@ -527,6 +527,16 @@ PostCheckout:
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -554,6 +564,16 @@ PostCommit:
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -572,6 +592,16 @@ PostMerge:
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'
@@ -590,6 +620,16 @@ PostRewrite:
     quiet: true
     required_executable: 'ctags'
 
+  NpmInstall:
+    enabled: false
+    description: 'Installing NPM dependencies'
+    requires_files: true
+    required_executable: 'npm'
+    flags: ['install']
+    include:
+      - 'package.json'
+      - 'npm-shrinkwrap.json'
+
   SubmoduleStatus:
     enabled: false
     description: 'Checking submodule status'

lib/overcommit/hook/post_checkout/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostCheckout
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

lib/overcommit/hook/post_commit/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostCommit
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

lib/overcommit/hook/post_merge/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostMerge
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

lib/overcommit/hook/post_rewrite/npm_install.rb

@@ -0,0 +1,11 @@
+require 'overcommit/hook/shared/npm_install'
+
+module Overcommit::Hook::PostRewrite
+  # Runs `npm install` when a change is detected in the repository's
+  # dependencies.
+  #
+  # @see {Overcommit::Hook::Shared::NpmInstall}
+  class NpmInstall < Base
+    include Overcommit::Hook::Shared::NpmInstall
+  end
+end

spec/overcommit/hook/post_checkout/npm_install_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Overcommit::Hook::PostCheckout::NpmInstall do
+  let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
+  let(:context) { double('context') }
+  subject { described_class.new(config, context) }
+
+  let(:result) { double('result') }
+
+  before do
+    subject.stub(:execute).and_return(result)
+  end
+
+  context 'when npm install exits successfully' do
+    before do
+      result.stub(:success?).and_return(true)
+    end
+
+    it { should pass }
+  end
+
+  context 'when npm install exits unsuccessfully' do
+    before do
+      result.stub(success?: false, stderr: "npm ERR! install Couldn't read dependencies")
+    end
+
+    it { should fail_hook }
+  end
+end

spec/overcommit/hook/post_commit/npm_install_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Overcommit::Hook::PostCommit::NpmInstall do
+  let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
+  let(:context) { double('context') }
+  subject { described_class.new(config, context) }
+
+  let(:result) { double('result') }
+
+  before do
+    subject.stub(:execute).and_return(result)
+  end
+
+  context 'when npm install exits successfully' do
+    before do
+      result.stub(:success?).and_return(true)
+    end
+
+    it { should pass }
+  end
+
+  context 'when npm install exits unsuccessfully' do
+    before do
+      result.stub(success?: false, stderr: "npm ERR! install Couldn't read dependencies")
+    end
+
+    it { should fail_hook }
+  end
+end

spec/overcommit/hook/post_merge/npm_install_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Overcommit::Hook::PostMerge::NpmInstall do
+  let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
+  let(:context) { double('context') }
+  subject { described_class.new(config, context) }
+
+  let(:result) { double('result') }
+
+  before do
+    subject.stub(:execute).and_return(result)
+  end
+
+  context 'when npm install exits successfully' do
+    before do
+      result.stub(:success?).and_return(true)
+    end
+
+    it { should pass }
+  end
+
+  context 'when npm install exits unsuccessfully' do
+    before do
+      result.stub(success?: false, stderr: "npm ERR! install Couldn't read dependencies")
+    end
+
+    it { should fail_hook }
+  end
+end

spec/overcommit/hook/post_rewrite/npm_install_spec.rb

@@ -0,0 +1,29 @@
+require 'spec_helper'
+
+describe Overcommit::Hook::PostRewrite::NpmInstall do
+  let(:config)  { Overcommit::ConfigurationLoader.default_configuration }
+  let(:context) { double('context') }
+  subject { described_class.new(config, context) }
+
+  let(:result) { double('result') }
+
+  before do
+    subject.stub(:execute).and_return(result)
+  end
+
+  context 'when npm install exits successfully' do
+    before do
+      result.stub(:success?).and_return(true)
+    end
+
+    it { should pass }
+  end
+
+  context 'when npm install exits unsuccessfully' do
+    before do
+      result.stub(success?: false, stderr: "npm ERR! install Couldn't read dependencies")
+    end
+
+    it { should fail_hook }
+  end
+end