git-hooks 0.1.0 → 0.2.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: cdc0af42b64037b04523c40d8d46608f9c290165
-  data.tar.gz: 6f61b8bc2caa7870e2313c69b8f0e5871e2b97d2
+  metadata.gz: 75855100ba5dd50b3b7f661a2ac7d812bd192fae
+  data.tar.gz: 1fd3d8ffd9087ddbb106fc807164c08008a25818
 SHA512:
-  metadata.gz: 9df358874679ddeeda3a1543cc07e5061ed6b7fdbba18f56784bd96d7aa2457bb5390a77c9ff3522c352f5e6a40c9b477ff87b3d7fdbf38b3236fcba3e7bd91d
-  data.tar.gz: a934f38495f94adb396a399f91d4f4c557663a62d2f7e650f9857f22f0cb48a45c27f570226b04751dda1a66340545fea0a7daa56b1b9576abb39bccccbceb86
+  metadata.gz: 6d6dbb403c9cdd29e97ebe8882fcb45e312adcaa2cc1f917b209b23aa7fd7aabeeaa6893799cc7df723dc84927ee9b8153af34280ab02dcedd29846231e5baad
+  data.tar.gz: 2bd1ab2bce9efbfe3a84302fea4d73fc657e95195ed20a03dfad7b8b9f5f4026a567e49afe999031c23a2d43598af8737fe1d76cf04a307fedd94929e94e11db

data/.gitignore

@@ -20,3 +20,4 @@ tmp
 *.o
 *.a
 mkmf.log
+coverage

data/.travis.yml

@@ -0,0 +1,5 @@
+language: ruby
+rvm:
+  - '2.0.0'
+  - '2.1.0'
+script: bundle exec rspec

data/CHANGELOG

@@ -0,0 +1,24 @@
+0.2.0
+
+* Add whitespace validation (by @rranelli)
+* Handle Errno::EEXIST on hook installation
+* Add whitespace validation (by @rranelli)
+* Add all the bagdes
+* Add PreCommit::PreventDebugger
+
+0.1.0
+
+* Add configuration command on cli
+
+0.1.0.pre
+
+ * Add tests for each pre-commit
+ * Include configurations
+ * Fix rubocop offences
+ * Rewrite pre-commits to extract RspecExecutor and Git
+ * Add pre-commit hook existence validator
+ * Add .git_hooks.yml to configure hooks
+
+0.0.2
+
+* Rubocop, Prevent master and Rspec pre-commits

data/README.md

@@ -1,6 +1,12 @@
+[![Version](http://allthebadges.io/stupied4ever/git-hooks/badge_fury.png)](http://allthebadges.io/stupied4ever/git-hooks/badge_fury)
+[![Dependencies](http://allthebadges.io/stupied4ever/git-hooks/gemnasium.png)](http://allthebadges.io/stupied4ever/git-hooks/gemnasium)
+[![Build Status](http://allthebadges.io/stupied4ever/git-hooks/travis.png)](http://allthebadges.io/stupied4ever/git-hooks/travis)
+[![Coverage](http://allthebadges.io/stupied4ever/git-hooks/coveralls.png)](http://allthebadges.io/stupied4ever/git-hooks/coveralls)
+[![Code Climate](http://allthebadges.io/stupied4ever/git-hooks/code_climate.png)](http://allthebadges.io/stupied4ever/git-hooks/code_climate)
+
 # GitHooks
 
-Some usefull ruby git hooks.
+Some usefull git hooks, it's written on ruby but can be used for other languages.
 
 ## Installation
 
@@ -19,36 +25,36 @@ Or install it yourself as:
 ## Usage
 ### Install git_hooks on project.
 
-```
-$ git_hooks install pre-commit
+```bash
+$ git_hooks install pre-commit [--force]
 ```
 
 ### Create configuration file
 
-Create a ```.git_hooks.yml``` on project root.
+Create a `.git_hooks.yml` on project root.
 
 ```bash
-$ git_hooks configure
+$ git_hooks init
 ```
 
-
 By now you will find only some simple hooks to:
 
  - Prevent commit on master.
  - Prevent commit with rubocop offences.
  - prevent commit with broken rspec tests.
+ - prevent commit with debugger
 
 ### Ensure hooks existence
 
-To ensure that hooks exists on ```.git/hooks```, include on your application
-start up (probably  ```config/environments/development.rb``` or
-```config/environments/test.rb```)
+To ensure that hooks exists on `.git/hooks`, include on your application
+start up (probably  `config/environments/development.rb` or
+`config/environments/test.rb`)
 
 ```ruby
 GitHooks.validate_hooks!
 ```
 
-This will force ```git_hooks``` installation before your application start.
+This will force `git_hooks` installation before your application start.
 
 ## Contributing
 

data/git-hooks.gemspec

@@ -24,5 +24,7 @@ Gem::Specification.new do |spec|
   spec.add_dependency 'equalizer', '~> 0.0.9'
 
   spec.add_development_dependency 'bundler', '~> 1.6'
+  spec.add_development_dependency 'coveralls', '>= 0.7.0'
   spec.add_development_dependency 'rspec', '~> 3.0'
+  spec.add_development_dependency 'pry'
 end

data/git_hooks.yml.example

@@ -1,5 +1,6 @@
 ---
 pre_commits:
+- GitHooks::PreCommit::PreventDebugger
 - GitHooks::PreCommit::PreventMaster
 - GitHooks::PreCommit::Rubocop
 - GitHooks::PreCommit::Rspec

data/lib/git-hooks.rb

@@ -6,8 +6,10 @@ require 'git'
 require_relative 'git_hooks/configurations'
 require_relative 'git_hooks/git'
 require_relative 'git_hooks/config_file'
+require_relative 'git_hooks/installer'
 require_relative 'git_hooks/rspec_executor'
 require_relative 'git_hooks/rubocop_validator'
+require_relative 'git_hooks/trailing_whitespace_validator'
 
 require_relative 'git_hooks/exceptions'
 require_relative 'git_hooks/pre_commit'
@@ -30,19 +32,7 @@ module GitHooks
     end
 
     def base_path
-      File.absolute_path(File.join(File.expand_path(__FILE__), '..', '..'))
-    end
-
-    def hook_installed?(hook)
-      hook_file = File.join(Dir.pwd, '.git', 'hooks', hook)
-      real_hook_file = File.join(base_path, HOOK_SAMPLE_FILE)
-
-      return false unless File.symlink?(hook_file)
-      File.realpath(hook_file) == real_hook_file
-    end
-
-    def install_hook(hook)
-      File.symlink(real_hook_template_path, ".git/hooks/#{hook}")
+      File.expand_path('../..', __FILE__)
     end
 
     def validate_hooks!
@@ -52,8 +42,8 @@ module GitHooks
     private
 
     def valid_pre_commit_hook?
-      return true if configurations.pre_commits.empty?
-      hook_installed?(PRE_COMMIT)
+      configurations.pre_commits.empty? ||
+        Installer.new(PRE_COMMIT).installed?
     end
 
     def real_hook_template_path

data/lib/git_hooks/cli.rb

@@ -5,22 +5,23 @@ require_relative '../git-hooks'
 module GitHooks
   class CLI < Thor
     desc 'install HOOK', 'Install some hook'
+    option :force, type: :boolean
     long_desc <<-LONGDESC
       Install some hook:
 
-      > $ git_hooks install pre-commit
+      $ git_hooks install pre-commit
     LONGDESC
     def install(hook)
-      GitHooks.install_hook(hook)
+      GitHooks::Installer.new(hook).install(options[:force])
     end
 
-    desc 'Create configuration file', 'Create a configuration file'
+    desc 'Init GitHooks on current folder', 'Create a configuration file'
     long_desc <<-LONGDESC
       Create a configuration file base on git_hooks.yml.examle
 
-      > $ git_hooks configure
+      $ git_hooks init
     LONGDESC
-    def configure
+    def init
       example_file = File.expand_path(
         '../../../git_hooks.yml.example', __FILE__
       )

data/lib/git_hooks/config_file.rb

@@ -7,7 +7,7 @@ module GitHooks
     end
 
     def pre_commits
-      content['pre_commits'] || []
+      content.fetch('pre_commits') { [] }
     end
 
     def content

data/lib/git_hooks/configurations.rb

@@ -1,20 +1,29 @@
 require 'forwardable'
 
 module GitHooks
-  class Configurations
+  class Configurations < SimpleDelegator
     include Equalizer.new(:config_file, :git_repository)
-    extend Forwardable
 
     attr_reader :config_file, :git_repository
 
-    def_delegator :config_file, :pre_commits
-
     def initialize(
-      config_file: ConfigFile.new('.git_hooks.yml'),
-      git_repository: Git.new('.')
+      config_file: default_config_file,
+      git_repository: default_git_repository
     )
       @config_file = config_file
       @git_repository = git_repository
+
+      super(config_file)
+    end
+
+    private
+
+    def default_config_file
+      ConfigFile.new('.git_hooks.yml')
+    end
+
+    def default_git_repository
+      Git.new('.')
     end
   end
 end

data/lib/git_hooks/exceptions.rb

@@ -1 +1,2 @@
 require_relative 'exceptions/missing_hooks'
+require_relative 'exceptions/unknown_hook_present'

data/lib/git_hooks/exceptions/unknown_hook_present.rb

@@ -0,0 +1,9 @@
+module GitHooks
+  module Exceptions
+    class UnknowHookPresent < RuntimeError
+      def initialize(hook)
+        super "There is a unknown #{hook} hook. If you are sure, use --force."
+      end
+    end
+  end
+end

data/lib/git_hooks/installer.rb

@@ -0,0 +1,33 @@
+module GitHooks
+  class Installer
+    attr_accessor :hook
+
+    HOOK_SAMPLE_FILE = 'hook.sample'
+
+    def initialize(hook)
+      @hook = hook
+    end
+
+    def install(force = false)
+      return true if installed?
+
+      FileUtils.symlink(
+        hook_template_path, File.join('.git', 'hooks', hook), force: force
+      )
+    rescue Errno::EEXIST
+      raise GitHooks::Exceptions::UnknowHookPresent, hook
+    end
+
+    def installed?
+      hook_file = File.join(Dir.pwd, '.git', 'hooks', hook)
+
+      File.symlink?(hook_file) && File.realpath(hook_file) == hook_template_path
+    end
+
+    private
+
+    def hook_template_path
+      File.join(GitHooks.base_path, HOOK_SAMPLE_FILE)
+    end
+  end
+end

data/lib/git_hooks/pre_commit.rb

@@ -1,6 +1,8 @@
+require_relative 'pre_commit/prevent_debugger'
 require_relative 'pre_commit/prevent_master'
 require_relative 'pre_commit/rspec'
 require_relative 'pre_commit/rubocop'
+require_relative 'pre_commit/trailing_whitespace'
 
 module GitHooks
   module PreCommit

data/lib/git_hooks/pre_commit/prevent_debugger.rb

@@ -0,0 +1,36 @@
+module GitHooks
+  module PreCommit
+    class PreventDebugger
+      attr_reader :git_repository
+
+      def self.validate
+        new(GitHooks.configurations.git_repository).validate
+      end
+
+      def initialize(git_repository)
+        @git_repository = git_repository
+      end
+
+      def validate
+        abort 'Prevented to commit with debugger' if any_debugger?
+      end
+
+      private
+
+      def any_debugger?
+        git_repository.added_or_modified.any? do |file_name|
+          File.read(file_name) =~ debugger_regex
+        end
+      end
+
+      def debugger_regex
+        Regexp.union(
+          %w(
+            binding.pry binding.remote_pry
+            save_and_open_page debugger
+          )
+        )
+      end
+    end
+  end
+end

data/lib/git_hooks/pre_commit/trailing_whitespace.rb

@@ -0,0 +1,32 @@
+module GitHooks
+  module PreCommit
+    class TrailingWhitespace
+      attr_reader :git_repository, :trailing_whitespace_validator
+
+      def self.validate
+        new(GitHooks.git_repository, TrailingWhitespaceValidator.new).validate
+      end
+
+      def initialize(git_repository, trailing_whitespace_validator)
+        @git_repository = git_repository
+        @trailing_whitespace_validator = trailing_whitespace_validator
+      end
+
+      def validate
+        abort 'Check presence of trailling space' if offences?
+      end
+
+      private
+
+      def changed_files
+        git_repository
+          .added_or_modified
+          .select { |file| File.extname(file) == '.rb' }
+      end
+
+      def offences?
+        trailing_whitespace_validator.errors?(changed_files)
+      end
+    end
+  end
+end

data/lib/git_hooks/trailing_whitespace_validator.rb

@@ -0,0 +1,12 @@
+module GitHooks
+  class TrailingWhitespaceValidator
+    def errors?(files)
+      files
+        .map { |file| File.open(file) }
+        .map(&:read)
+        .grep(/ +$/)
+        .compact
+        .any?
+    end
+  end
+end

data/lib/git_hooks/version.rb

@@ -1,3 +1,3 @@
 module GitHooks
-  VERSION = '0.1.0'
+  VERSION = '0.2.0'
 end

data/spec/fixtures/git_hooks.yml

@@ -0,0 +1,4 @@
+---
+pre_commits:
+- foo
+- bar

data/spec/git_hooks/config_file_spec.rb

@@ -2,28 +2,24 @@ module GitHooks
   describe ConfigFile do
     subject(:config) { described_class.new(path) }
 
-    let(:path) { '' }
+    let(:path) { 'some-not-existent-file' }
     let(:content) { { 'pre_commits' => pre_commits } }
     let(:pre_commits) { %w(foo bar) }
 
-    context 'when the given file does not exists' do
-      subject(:config) { -> { described_class.new(path) } }
-
-      let(:path) { 'some-not-existent-file' }
-
-      it { is_expected.to_not raise_error }
-    end
-
     describe '#pre_commits' do
       subject { config.pre_commits }
 
-      before do
-        allow(YAML).to receive(:load_file).and_return(content)
-      end
+      let(:path) { fixture_path('git_hooks.yml') }
 
       it 'has the pre commits specified on hook file' do
         is_expected.to eq(%w(foo bar))
       end
+
+      context 'when the given file does not exist' do
+        let(:path) { 'some-not-existent-file' }
+
+        it { is_expected.to eq([]) }
+      end
     end
   end
 end

data/spec/git_hooks/installer_spec.rb

@@ -0,0 +1,107 @@
+describe GitHooks::Installer do
+  subject(:installer) { described_class.new(hook) }
+
+  let(:hook) { 'pre-commit' }
+
+  let(:hook_real_path) do
+    File.join(GitHooks.base_path, 'hook.sample')
+  end
+
+  let(:absolute_path) do
+    File.join(GitHooks.base_path, '.git', 'hooks', hook)
+  end
+
+  let(:git_hook_path) { ".git/hooks/#{hook}" }
+
+  describe '.install' do
+    subject(:install) { installer.install }
+
+    before do
+      allow(installer).to receive(:installed?).and_return(false)
+      allow(FileUtils).to receive(:symlink).and_return(true)
+    end
+
+    it 'creates symlink' do
+      expect(FileUtils)
+        .to receive(:symlink)
+        .with(hook_real_path, git_hook_path, force: false)
+
+      install
+    end
+
+    it { is_expected.to be_truthy }
+
+    context 'when there is a unknown hook' do
+      before do
+        allow(FileUtils).to receive(:symlink).and_raise(Errno::EEXIST)
+      end
+
+      it 'raises an error' do
+        expect { install }.to raise_error(
+          GitHooks::Exceptions::UnknowHookPresent
+        )
+      end
+    end
+
+    context 'when there is a unknown hook but I want to force installation' do
+      subject(:install) { installer.install true }
+
+      it 'creates symlink with force option' do
+        expect(FileUtils)
+          .to receive(:symlink)
+          .with(hook_real_path, git_hook_path, force: true)
+
+        install
+      end
+
+      it { is_expected.to be_truthy }
+    end
+
+    context 'when hook is already installed' do
+      before do
+        allow(installer).to receive(:installed?).and_return(true)
+      end
+
+      it 'does not create symlink' do
+        expect(FileUtils)
+          .to_not receive(:symlink)
+
+        install
+      end
+
+      it { is_expected.to be_truthy }
+    end
+  end
+
+  describe '.installed?' do
+    subject(:installed?) { installer.installed? }
+
+    let(:symlink?) { true }
+
+    before do
+      allow(File).to receive(:symlink?).and_return(symlink?)
+      allow(File).to receive(:realpath).and_return(hook_real_path)
+    end
+
+    it 'validates file is a symlink' do
+      expect(File).to receive(:symlink?).with(absolute_path)
+      installed?
+    end
+
+    it { is_expected.to be_truthy }
+
+    context 'when file is not a symlink' do
+      let(:symlink?) { false }
+
+      it { is_expected.to be_falsy }
+    end
+
+    context 'when is not the GitHooks file' do
+      before do
+        allow(File).to receive(:realpath).and_return('/tmp/foo')
+      end
+
+      it { is_expected.to be_falsy }
+    end
+  end
+end

data/spec/git_hooks/pre_commit/prevent_debugger_spec.rb

@@ -0,0 +1,72 @@
+module GitHooks
+  module PreCommit
+    describe PreventDebugger do
+      subject(:prevent_debugger) { described_class.new git_repository }
+
+      let(:git_repository) do
+        instance_double(GitHooks::Git, added_or_modified: files)
+      end
+
+      let(:files) { ['some_file.rb'] }
+
+      describe '#validate' do
+        subject(:validate) { -> { prevent_debugger.validate } }
+
+        let(:file_content) { 'this is a big file without' }
+
+        before do
+          allow(File)
+            .to receive(:read)
+            .with(files.first)
+            .and_return(file_content)
+
+          allow($stderr).to receive(:write).and_return('')
+        end
+
+        it { is_expected.to_not raise_error }
+
+        %w(
+          binding.pry
+          binding.remote_pry
+          save_and_open_page
+          debugger
+        ).each do |debugger_string|
+          context "when has #{debugger_string}" do
+            let(:file_content) { "this is a big file with #{debugger_string}" }
+
+            it { is_expected.to raise_error(SystemExit) }
+          end
+        end
+      end
+
+      describe '.validate' do
+        subject(:validate) { described_class.validate }
+
+        let(:prevent_debugger) { instance_double(PreventDebugger) }
+        let(:git) { instance_double(GitHooks::Git) }
+
+        let(:configurations) do
+          instance_double(Configurations, git_repository: git)
+        end
+
+        before do
+          allow(GitHooks).to receive(:configurations).and_return(configurations)
+          allow(described_class).to receive(:new).and_return(prevent_debugger)
+          allow(prevent_debugger).to receive(:validate).and_return(nil)
+        end
+
+        it 'creates object with GitHooks.git_repository' do
+          expect(PreventDebugger).to receive(:new).with(git)
+
+          validate
+        end
+
+        it 'validates' do
+          expect(prevent_debugger).to receive(:validate)
+
+          validate
+        end
+      end
+    end
+  end
+end

data/spec/git_hooks/pre_commit/trailing_whitespace_spec.rb

@@ -0,0 +1,82 @@
+module GitHooks
+  module PreCommit
+    describe TrailingWhitespace do
+      subject(:whitespace) do
+        described_class.new(git_repository, whitespace_validator)
+      end
+
+      let(:git_repository) do
+        instance_double(GitHooks::Git, added_or_modified: added_or_modified)
+      end
+
+      let(:whitespace_validator) do
+        instance_double(GitHooks::TrailingWhitespaceValidator, errors?: errors?)
+      end
+
+      let(:added_or_modified) { [] }
+      let(:errors?) { false }
+
+      describe '#validate' do
+        subject(:validate) { whitespace.validate  }
+
+        let(:added_or_modified) { [ruby_file, 'foo.txt'] }
+        let(:ruby_file) { 'foo.rb' }
+
+        it 'only validates ruby files' do
+          expect(whitespace_validator)
+            .to receive(:errors?)
+            .with([ruby_file])
+          validate
+        end
+
+        context 'with errors' do
+          let(:errors?) { true }
+
+          before do
+            allow($stderr).to receive(:write).and_return('')
+          end
+
+          it { expect(-> { validate }).to raise_error(SystemExit) }
+        end
+
+        context 'without errors' do
+          let(:errors?) { false }
+
+          it { expect(-> { validate }).to_not raise_error }
+        end
+      end
+
+      describe '.validate' do
+        subject(:validate) { described_class.validate }
+
+        let(:whitespace) { instance_double(TrailingWhitespace) }
+        let(:git) { instance_double(Git) }
+        let(:whitespace_validator) do
+          instance_double(TrailingWhitespaceValidator)
+        end
+
+        before do
+          allow(described_class).to receive(:new).and_return(whitespace)
+          allow(whitespace).to receive(:validate).and_return(nil)
+
+          allow(GitHooks).to receive(:git_repository).and_return(git)
+          allow(TrailingWhitespaceValidator).to receive(:new)
+            .and_return(whitespace_validator)
+        end
+
+        it 'creates object with GitHooks.git_repository' do
+          expect(TrailingWhitespace).to receive(:new)
+            .with(git, whitespace_validator)
+
+          validate
+        end
+
+        it 'validates' do
+          expect(whitespace).to receive(:validate)
+
+          validate
+        end
+      end
+    end
+  end
+end

data/spec/git_hooks/trailing_space_validator.rb

@@ -0,0 +1,39 @@
+module GitHooks
+  describe TrailingSpaceValidator do
+    subject(:validator) { described_class.new }
+
+    let(:file_names) { %w(a.rb b.rb) }
+    let(:files) { [file] * 2 }
+    let(:file) { instance_double(File) }
+
+    before do
+      allow(File).to receive(:open).and_return(file)
+      allow(file).to receive(:read).and_return('I no have no whitespace!')
+    end
+
+    describe '#errors?' do
+      subject(:errors?) { validator.errors?(files) }
+
+      context 'with files' do
+        context 'when files has no offences' do
+
+          it { is_expected.to be_falsy }
+        end
+
+        context 'when some file has offences' do
+          before do
+            allow(file).to receive(:read).and_return('My text editor sucks   ')
+          end
+
+          it { is_expected.to be_truthy }
+        end
+      end
+
+      context 'with empty files array' do
+        let(:files) { [] }
+
+        it { is_expected.to be_falsy }
+      end
+    end
+  end
+end

data/spec/git_hooks_spec.rb

@@ -2,34 +2,39 @@ describe GitHooks do
   describe '#validate_hooks!' do
     subject { -> { described_class.validate_hooks! } }
 
-    before { GitHooks.configurations = configs }
+    before do
+      allow(GitHooks::Installer)
+        .to receive(:new)
+        .with(GitHooks::PRE_COMMIT)
+        .and_return(installer)
 
-    let(:configs) do
-      instance_double(GitHooks::Configurations, pre_commits: pre_commits)
+      GitHooks.configurations = configs
     end
 
-    let(:pre_commits) { [] }
+    let(:configs) do
+      GitHooks::Configurations.new(config_file: config_file)
+    end
 
-    it { is_expected.to_not raise_error }
+    let(:config_file) do
+      GitHooks::ConfigFile.new(fixture_path('git_hooks.yml'))
+    end
 
-    context 'with pre-commit hooks configured' do
-      let(:pre_commits) { %w(foo bar) }
-      let(:hook_installed?) { true }
+    let(:installed?) { true }
+    let(:installer) { instance_double(GitHooks::Installer) }
 
-      before do
-        allow(GitHooks).to receive(:hook_installed?).and_return(hook_installed?)
-      end
+    before do
+      allow(installer).to receive(:installed?).and_return(installed?)
+    end
 
-      it { is_expected.to_not raise_error }
+    it { is_expected.to_not raise_error }
 
-      context 'but without pre-commit installed' do
-        let(:hook_installed?) { false }
+    context 'but without pre-commit installed' do
+      let(:installed?) { false }
 
-        let(:message) { 'Please install pre-commit hook.' }
+      let(:message) { 'Please install pre-commit hook.' }
 
-        it do
-          is_expected.to raise_error(GitHooks::Exceptions::MissingHook, message)
-        end
+      it do
+        is_expected.to raise_error(GitHooks::Exceptions::MissingHook, message)
       end
     end
   end
@@ -49,26 +54,27 @@ describe GitHooks do
   describe '.configurations' do
     subject(:configurations) { described_class.configurations }
 
-    before do
-      GitHooks.configurations = nil
-      allow(GitHooks::Configurations).to receive(:new).and_return(configs)
-    end
+    before { GitHooks.configurations = nil }
 
     let(:configs) { instance_double(GitHooks::Configurations) }
 
     it 'creates with default params' do
-      expect(GitHooks::Configurations).to receive(:new).with(no_args)
+      expect(GitHooks::Configurations)
+        .to receive(:new)
+        .with(no_args)
+        .and_return(configs)
 
       is_expected.to eq(configs)
     end
   end
 
   describe 'configurations=' do
-    subject(:set_configurations) { described_class.configurations = configs }
+    subject(:set_configurations) { GitHooks.configurations = configs }
 
     let(:configs) { instance_double(GitHooks::Configurations) }
 
     before do
+      GitHooks.configurations = nil
       allow(GitHooks::Configurations).to receive(:new).and_return(nil)
     end
 
@@ -76,65 +82,6 @@ describe GitHooks do
       expect { set_configurations }.to change {
         GitHooks.configurations
       }.to(configs)
-
-      is_expected.to eq(configs)
-    end
-  end
-
-  describe '.install_hook' do
-    subject(:install_hook) { described_class.install_hook(hook) }
-
-    let(:hook) { 'pre-commit' }
-    let(:hook_real_path) { File.join(GitHooks.base_path, 'hook.sample') }
-    let(:git_hook_path) { ".git/hooks/#{hook}" }
-
-    before do
-      allow(File).to receive(:symlink).and_return(true)
-    end
-
-    it 'creates symlink' do
-      expect(File)
-        .to receive(:symlink)
-        .with(hook_real_path, git_hook_path)
-      install_hook
-    end
-
-    it { is_expected.to be_truthy }
-  end
-
-  describe '.hook_installed?' do
-    subject(:installed?) { described_class.hook_installed?(hook) }
-
-    let(:hook) { 'pre-commit' }
-    let(:hook_real_path) { File.join(GitHooks.base_path, 'hook.sample') }
-    let(:symlink?) { true }
-
-    let(:absolute_path) { File.join(GitHooks.base_path, '.git', 'hooks', hook) }
-
-    before do
-      allow(File).to receive(:symlink?).and_return(symlink?)
-      allow(File).to receive(:realpath).and_return(hook_real_path)
-    end
-
-    it 'validates file is a symlink' do
-      expect(File).to receive(:symlink?).with(absolute_path)
-      installed?
-    end
-
-    it { is_expected.to be_truthy }
-
-    context 'when file is not a symlink' do
-      let(:symlink?) { false }
-
-      it { is_expected.to be_falsy }
-    end
-
-    context 'when is not the GitHooks file' do
-      before do
-        allow(File).to receive(:realpath).and_return('/tmp/foo')
-      end
-
-      it { is_expected.to be_falsy }
     end
   end
 end

data/spec/spec_helper.rb

@@ -1 +1,15 @@
+require 'simplecov'
+require 'coveralls'
+
+SimpleCov.formatter = SimpleCov::Formatter::MultiFormatter[
+  SimpleCov::Formatter::HTMLFormatter,
+  Coveralls::SimpleCov::Formatter
+]
+SimpleCov.start
+
 require_relative '../lib/git-hooks'
+
+def fixture_path(filename)
+  return '' if filename == ''
+  File.join(File.absolute_path(File.dirname(__FILE__)), 'fixtures', filename)
+end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: git-hooks
 version: !ruby/object:Gem::Version
-  version: 0.1.0
+  version: 0.2.0
 platform: ruby
 authors:
 - Rafael da Silva Almeida
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2014-07-22 00:00:00.000000000 Z
+date: 2014-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rubocop
@@ -80,6 +80,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '1.6'
+- !ruby/object:Gem::Dependency
+  name: coveralls
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.7.0
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: 0.7.0
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -94,6 +108,20 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '3.0'
+- !ruby/object:Gem::Dependency
+  name: pry
+  requirement: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
+  type: :development
+  prerelease: false
+  version_requirements: !ruby/object:Gem::Requirement
+    requirements:
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '0'
 description: It stores git hooks and force git hooks installation.
 email:
 - eusou@rafaelalmeida.net
@@ -106,6 +134,8 @@ files:
 - ".rspec"
 - ".rubocop.yml"
 - ".rubocop_todo.yml"
+- ".travis.yml"
+- CHANGELOG
 - Gemfile
 - README.md
 - bin/git_hooks
@@ -118,22 +148,32 @@ files:
 - lib/git_hooks/configurations.rb
 - lib/git_hooks/exceptions.rb
 - lib/git_hooks/exceptions/missing_hooks.rb
+- lib/git_hooks/exceptions/unknown_hook_present.rb
 - lib/git_hooks/git.rb
+- lib/git_hooks/installer.rb
 - lib/git_hooks/pre_commit.rb
+- lib/git_hooks/pre_commit/prevent_debugger.rb
 - lib/git_hooks/pre_commit/prevent_master.rb
 - lib/git_hooks/pre_commit/rspec.rb
 - lib/git_hooks/pre_commit/rubocop.rb
+- lib/git_hooks/pre_commit/trailing_whitespace.rb
 - lib/git_hooks/rspec_executor.rb
 - lib/git_hooks/rubocop_validator.rb
+- lib/git_hooks/trailing_whitespace_validator.rb
 - lib/git_hooks/version.rb
+- spec/fixtures/git_hooks.yml
 - spec/git_hooks/config_file_spec.rb
 - spec/git_hooks/configurations_spec.rb
 - spec/git_hooks/git_spec.rb
+- spec/git_hooks/installer_spec.rb
+- spec/git_hooks/pre_commit/prevent_debugger_spec.rb
 - spec/git_hooks/pre_commit/prevent_master_spec.rb
 - spec/git_hooks/pre_commit/rspec_spec.rb
 - spec/git_hooks/pre_commit/rubocop_spec.rb
+- spec/git_hooks/pre_commit/trailing_whitespace_spec.rb
 - spec/git_hooks/rspec_executor_spec.rb
 - spec/git_hooks/rubocop_validator_spec.rb
+- spec/git_hooks/trailing_space_validator.rb
 - spec/git_hooks_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/stupied4ever/ruby-git-hooks
@@ -161,14 +201,19 @@ signing_key:
 specification_version: 4
 summary: Help to keep git hooks organized.
 test_files:
+- spec/fixtures/git_hooks.yml
 - spec/git_hooks/config_file_spec.rb
 - spec/git_hooks/configurations_spec.rb
 - spec/git_hooks/git_spec.rb
+- spec/git_hooks/installer_spec.rb
+- spec/git_hooks/pre_commit/prevent_debugger_spec.rb
 - spec/git_hooks/pre_commit/prevent_master_spec.rb
 - spec/git_hooks/pre_commit/rspec_spec.rb
 - spec/git_hooks/pre_commit/rubocop_spec.rb
+- spec/git_hooks/pre_commit/trailing_whitespace_spec.rb
 - spec/git_hooks/rspec_executor_spec.rb
 - spec/git_hooks/rubocop_validator_spec.rb
+- spec/git_hooks/trailing_space_validator.rb
 - spec/git_hooks_spec.rb
 - spec/spec_helper.rb
 has_rdoc: