checker 0.0.5 → 0.0.6

data/.travis.yml

@@ -0,0 +1,12 @@
+language: ruby
+rvm:
+  - 1.8.7
+  - 1.9.2
+  - 1.9.3
+  - ree
+notifications:
+  email:
+    recipients:
+      - jacek.jakubik@netguru.pl
+      - tomasz.pewinski@netguru.pl
+

data/CHANGELOG

@@ -1,3 +1,10 @@
+=== version 0.0.6 (released 2012-08-28)
+
+* better support for rvm users
+* adding slim support (thanks to Filip Skokan)
+* better check for files which are staged and modified
+* cleanups, speed and refactor
+
 === version 0.0.5 (released 2012-07-09)
 
 * depencendy message - if executable not found, 

data/Gemfile

@@ -0,0 +1,5 @@
+source 'http://rubygems.org'
+
+gem 'rake'
+gem 'rspec'
+gem 'colorize'

data/Gemfile.lock

@@ -0,0 +1,22 @@
+GEM
+  remote: http://rubygems.org/
+  specs:
+    colorize (0.5.8)
+    diff-lcs (1.1.3)
+    rake (0.9.2.2)
+    rspec (2.11.0)
+      rspec-core (~> 2.11.0)
+      rspec-expectations (~> 2.11.0)
+      rspec-mocks (~> 2.11.0)
+    rspec-core (2.11.0)
+    rspec-expectations (2.11.1)
+      diff-lcs (~> 1.1.3)
+    rspec-mocks (2.11.1)
+
+PLATFORMS
+  ruby
+
+DEPENDENCIES
+  colorize
+  rake
+  rspec

data/README.md

@@ -1,4 +1,4 @@
-# Checker
+# Checker [![Build Status](https://secure.travis-ci.org/netguru/checker.png?branch=master)](http://travis-ci.org/netguru/checker)
 
 A collection of modules for which every is designed to check syntax in files to be commited via git.
 
@@ -34,6 +34,13 @@ fi
 echo ":checkered_flag:" >> $1
 ```
 
+you can also prepend the flag to the first line of commit message, by changing last line with:
+
+``` bash
+text=`echo -n ':checkered_flag: '; cat $1`
+echo $text > $1
+```
+
 #### pre-commit hook
 To just check your source code every time you commit, add to your `.git/hooks/pre-commit` line:
 
@@ -68,12 +75,12 @@ To check only specific filetypes on commit, use `git config` :
 git config checker.check 'ruby, haml, coffeescript'
 ```
 
-Available options are: ruby, haml, pry, coffeescript, sass
+Available options are: ruby, haml, pry, coffeescript, sass, slim
 
 ### Dependencies
 
 For various modules to work you may need to install additional dependencies:
 
 * coffeescript - `npm install -g coffee-script` - see https://github.com/jashkenas/coffee-script/
-* javascript - `npm install -g jshint` - see https://github.com/jshint/node-jshint/#install
-* haml & sass - `gem install haml` `gem install sass`
+* javascript - install jsl binary - see http://www.javascriptlint.com/download.htm
+* haml & sass & slim - `gem install haml sass slim`

data/Rakefile

@@ -1,2 +1,4 @@
 require 'rspec/core/rake_task'
-RSpec::Core::RakeTask.new(:spec)
+RSpec::Core::RakeTask.new(:spec)
+
+task :default  => :spec

data/checker.gemspec

@@ -1,11 +1,14 @@
+# -*- encoding : utf-8 -*-
+require File.expand_path('../lib/checker/version', __FILE__)
+
 Gem::Specification.new do |s|
   s.name        = 'checker'
-  s.version     = '0.0.5'
-  s.date        = '2012-07-11'
+  s.version     = Checker::VERSION
+  s.date        = '2012-08-28'
   s.summary     = "Syntax checker for various files"
   s.description = "A collection of modules which every is designed to check syntax for specific files."
-  s.authors     = ["Jacek Jakubik"]
-  s.email       = 'jacek.jakubik@netguru.pl'
+  s.authors     = ["Jacek Jakubik", "Tomasz Pewiński"]
+  s.email       = ['jacek.jakubik@netguru.pl', 'tomasz.pewinski@netguru.pl']
   s.files       = `git ls-files`.split("\n")
   s.executables = `git ls-files -- bin/*`.split("\n").map{ |f| File.basename(f) }
   s.require_paths = ["lib"]

data/lib/checker.rb

@@ -1,11 +1,15 @@
 require 'colorize'
+require 'digest/md5'
 
 require 'checker/core_ext'
+require 'checker/version'
 
 require "checker/modules/base"
 require "checker/modules/ruby"
 require "checker/modules/haml"
+require "checker/modules/slim"
 require "checker/modules/pry"
 require "checker/modules/coffeescript"
 require "checker/modules/javascript"
 require "checker/modules/sass"
+require "checker/modules/yaml"

data/lib/checker/cli.rb

@@ -21,13 +21,18 @@ module Checker
           exit 1
         end
 
-        checked = []
+        module_instances = []
         files = modified_files
         modules.each do |mod|
           klass = "Checker::Modules::#{mod.downcase.capitalize}".constantize
-          checked << klass.new(files.dup).check
+          module_instances << klass.new(files.dup)
         end
-        exit (checked.all_true? ? 0 : 1)
+
+        files_checked = module_instances.map(&:files_to_check).flatten.uniq
+        puts "[ CHECKER #{Checker::VERSION} - #{files_checked.size} files ]".light_blue
+
+        results = module_instances.map(&:check)
+        exit (results.all_true? ? 0 : 1)
       end
 
       private
@@ -49,7 +54,7 @@ module Checker
       end
 
       def modified_files
-        @modified_files ||= `git status --porcelain | egrep "^(A |M |R ).*" | awk ' { if ($3 == "->") print $4; else print $2 } '`.split
+        @modified_files ||= `git status --porcelain | egrep "^(A|M|R).*" | awk ' { if ($3 == "->") print $4; else print $2 } '`.split
       end
     end
   end

data/lib/checker/modules/base.rb

@@ -2,10 +2,9 @@ module Checker
   module Modules
     class Base < Struct.new(:files)
       def check
+        check_files_existing or return true
         print_module_header
-        prepare_check
         check_executable or return true
-        select_proper_files
         check_all_files
         valid?
       end
@@ -14,19 +13,30 @@ module Checker
         @results.all_true?
       end
 
+      def files_to_check
+        @files_to_check ||= begin
+          if self.class.extensions.any?
+            self.files.select { |f|
+              self.class.extensions.map { |ex| f.ends_with?(".#{ex}") }.any?
+            }
+          else
+            self.files
+          end
+        end
+      end
+
       private
 
       def print_module_header
-        color "[ #{name} ]\n", :light_blue
+        color "[ #{name} - #{files_to_check.size} files ]\n", :light_blue
       end
 
       def dependency_message
         "Executable not found, skipping...\n"
       end
 
-      def prepare_check
-        @files_to_check = []
-        @results = []
+      def check_files_existing
+        files_to_check.any?
       end
 
       def check_executable
@@ -42,21 +52,22 @@ module Checker
         true
       end
 
-      def select_proper_files
-        @files_to_check = self.files
-        if self.class.extensions.any?
-          @files_to_check = @files_to_check.select { |f|
-            self.class.extensions.map { |ex| f.ends_with?(".#{ex}") }.any?
-          }
+      def check_all_files
+        with_checker_cache do
+          @results = files_to_check.map do |file_name|
+            color "  Checking #{file_name}...", :yellow
+            result = check_one_file(file_name)
+            show_status(result)
+            flush_and_forget_output(result)
+            result
+          end
         end
-        @files_to_check
       end
 
-      def check_all_files
-        @results = @files_to_check.map do |file|
-          color "Checking #{file}...", :yellow
-          check_one(file)
-        end
+      def check_one_file file_name
+        checksum = ::Digest::MD5.hexdigest(file_name)
+        checkout_file(file_name, checksum)
+        check_one(checkout_file_name(checksum))
       end
 
       def self.extensions *args
@@ -69,38 +80,47 @@ module Checker
       end
 
       def plain_command(cmd, options = {})
-        exitstatus = system(cmd)
-        show_output(exitstatus, options)
-        exitstatus
+        cmd = parse_command(cmd, options)
+        execute(cmd)
       end
 
-      def command(cmd, options = {})
-        if options[:use_bundler] == true
-          if use_bundler?
-            cmd = "bundle exec #{cmd}"
-          end
-        end
+      def silent_command(cmd, options = {})
+        options = { :output => false }.merge(options)
+        cmd = parse_command(cmd, options)
+        execute(cmd)
+      end
 
-        cmd = rvm_command(cmd) if use_rvm?
-        cmd << " #{options[:append]}" if options[:append]
-        exitstatus = execute(cmd)
-        show_output(exitstatus, options)
-        exitstatus
+      def flush_and_forget_output(success)
+        print @buffer.to_s unless success
+        @buffer = ""
       end
 
-      def show_output(exitstatus, options)
-        unless options[:show_output] == false
-          if exitstatus
-            puts " [OK]".green
-          else
-            puts " [FAIL]".red
-          end
+      def show_status(success)
+        if success
+          puts " [OK]".green
+        else
+          puts " [FAIL]".red
         end
       end
 
-
       def execute(cmd)
-        system(cmd)
+        io = IO.popen(cmd)
+        Process.wait(io.pid)
+        @buffer ||= ""
+        @buffer << io.read
+        success?
+      end
+
+      def success?
+        $? && $?.success?
+      end
+
+      def parse_command command, options
+        options = { :bundler => true, :output => true }.merge(options)
+        command = bundler_command(command) if use_bundler? && options[:bundler]
+        command = rvm_command(command) if use_rvm?
+        command << " > /dev/null" unless options[:output]
+        "#{command} 2>&1"
       end
 
       def color(str, color)
@@ -112,20 +132,41 @@ module Checker
       end
 
       def use_bundler?
-        File.exists?("Gemfile.lock") 
+        File.exists?("Gemfile.lock")
+      end
+
+      def bundler_command(command)
+        "bundle exec #{command}"
       end
 
       def use_rvm?
-        File.exists?(".rvmrc") && File.exists?(rvm_shell)
+        File.exists?(rvm_shell)
+      end
+
+      def rvm_command(command)
+        rvm_version = ENV.fetch('RUBY_VERSION')
+        "#{rvm_shell} '#{rvm_version}' -c '#{command}'"
       end
 
       def rvm_shell
         File.join(ENV.fetch('rvm_path', ''), 'bin/rvm-shell')
       end
 
-      def rvm_command(command)
-        rvm_version = `echo $rvm_ruby_string`.chomp
-        "#{rvm_shell} '#{rvm_version}' -c '#{command}'"
+      def with_checker_cache
+        begin
+          `mkdir .checker-cache`
+          yield if block_given?
+        ensure
+          `rm -rf .checker-cache > /dev/null 2>&1`
+        end
+      end
+
+      def checkout_file file_name, target
+        `git show :0:#{file_name} > #{checkout_file_name(target)} 2>/dev/null`
+      end
+
+      def checkout_file_name target
+        ".checker-cache/#{target}"
       end
     end
   end

data/lib/checker/modules/coffeescript.rb

@@ -8,7 +8,7 @@ module Checker
       end
 
       def check_for_executable
-        command('coffee -v', :show_output => false, :append => ">> /dev/null 2>&1")
+        silent_command('coffee -v', :bundler => false)
       end
 
       def dependency_message

data/lib/checker/modules/haml.rb

@@ -7,6 +7,10 @@ module Checker
         plain_command("haml --check #{file} >> /dev/null")
       end
 
+      def check_for_executable
+        silent_command("haml -v")
+      end
+
       def dependency_message
         str = "Executable not found\n"
         str << "Install haml from rubygems: 'gem install haml'\n"

data/lib/checker/modules/javascript.rb

@@ -8,7 +8,7 @@ module Checker
       end
 
       def check_for_executable
-        command('jsl -help:conf', :show_output => false, :append => ">> /dev/null 2>&1")
+        silent_command('jsl -help:conf', :bundler => false)
       end
 
       def dependency_message
@@ -19,11 +19,8 @@ module Checker
       end
 
       # ignore exit status 1 - warnings
-      def plain_command cmd, options={}
-        system(cmd)
-        exitstatus = $?.exitstatus == 0 || $?.exitstatus == 1
-        show_output(exitstatus, options)
-        exitstatus
+      def success?
+        $? && ($?.exitstatus == 0 || $?.exitstatus == 1)
       end
     end
   end

data/lib/checker/modules/pry.rb

@@ -8,30 +8,11 @@ module Checker
       end
 
       def check_for_binding_pry(file)
-        result = `grep -n "binding\\.pry" #{file}`.chomp
-
-        unless result.empty?
-          puts " pry -> FAIL, ".red
-          puts result
-        else
-          print " pry -> OK, ".green
-        end
-
-        result.empty?
+        !plain_command("grep -n \"binding\\.pry\" #{file}", :bundler => false)
       end
 
       def check_for_binding_remote_pry(file)
-        result = `grep -n "binding\\.remote_pry" #{file}`.chomp
-
-        unless result.empty?
-          puts " remote_pry -> FAIL".red
-          puts result
-        else
-          print " remote_pry -> OK".green
-        end
-        puts ""
-
-        result.empty?
+        !plain_command("grep -n \"binding\\.remote_pry\" #{file}", :bundler => false)
       end
     end
   end

data/lib/checker/modules/ruby.rb

@@ -4,7 +4,7 @@ module Checker
       extensions 'rb'
       private
       def check_one(file)
-        command("ruby -c #{file}", :append => ">> /dev/null")
+        plain_command("ruby -c #{file}", :bundler => false)
       end
     end
   end

data/lib/checker/modules/sass.rb

@@ -4,11 +4,11 @@ module Checker
       extensions 'scss', 'sass'
       private
       def check_one(file)
-        command("sass #{file}", :use_bundler => true, :append => ">> /dev/null")
+        plain_command("sass #{file}")
       end
 
       def check_for_executable
-        command("sass -v", :use_bundler => true, :show_output => false, :append => ">> /dev/null 2>&1")
+        silent_command("sass -v")
       end
 
       def dependency_message

data/lib/checker/modules/slim.rb

@@ -0,0 +1,21 @@
+module Checker
+  module Modules
+    class Slim < Base
+      extensions 'slim'
+      private
+      def check_one(file)
+        plain_command("slimrb --compile #{file} >> /dev/null")
+      end
+
+      def check_for_executable
+        silent_command("slimrb -v")
+      end
+
+      def dependency_message
+        str = "Executable not found\n"
+        str << "Install slim from rubygems: 'gem install slim'\n"
+        str
+      end
+    end
+  end
+end

data/lib/checker/modules/yaml.rb

@@ -0,0 +1,21 @@
+require 'yaml'
+
+module Checker
+  module Modules
+    class Yaml < Base
+      extensions 'yaml', 'yml'
+      private
+      def check_one(file)
+        ret = begin
+          YAML.load_file(file)
+          true
+        rescue Exception => e
+          puts e
+          false
+        end
+        show_status(ret)
+        ret
+      end
+    end
+  end
+end

data/lib/checker/version.rb

@@ -0,0 +1,3 @@
+module Checker
+  VERSION = '0.0.6'
+end

data/spec/checker/cli_spec.rb

@@ -17,13 +17,6 @@ module Checker
 end
 
 describe Checker::CLI do
-  before do
-    @argv_copy = ARGV
-  end
-  after do
-    ARGV = @argv_copy
-  end
-
   context "running without arguments" do
     it "should run checks on modules from git config" do
       ARGV.stub(:size).and_return 0
@@ -35,7 +28,7 @@ describe Checker::CLI do
 
   context "running with argument" do
     it "should run check on modules from argument" do
-      ARGV = ["pry"]
+      stub_const("ARGV", ["pry"])
       Checker::CLI.should_not_receive(:get_modules_to_check)
       Checker::CLI.should_receive(:exit).with(0).and_return true
       Checker::CLI.execute

data/spec/checker/fixtures/pry/with_both.rb

@@ -0,0 +1,4 @@
+binding.pry
+def test
+end
+binding.remote_pry

data/spec/checker/fixtures/pry/with_pry.rb

@@ -0,0 +1,3 @@
+def test
+  binding.pry
+end

data/spec/checker/fixtures/pry/with_pry_remote.rb

@@ -0,0 +1,3 @@
+def test
+  binding.remote_pry
+end

data/spec/checker/fixtures/pry/without_pry.rb

@@ -0,0 +1,2 @@
+def test
+end

data/spec/checker/fixtures/ruby/1.8/bad.rb

@@ -0,0 +1,3 @@
+def test
+  {some: 'random', hash: 'qwe'}
+end

data/spec/checker/fixtures/ruby/1.8/bad2.rb

@@ -0,0 +1,2 @@
+def test
+  {:some => 'random', :hash => 'qwe'}

data/spec/checker/fixtures/ruby/1.8/good.rb

@@ -0,0 +1,3 @@
+def test
+  {:some => 'random', :hash => 'qwe'}
+end

data/spec/checker/fixtures/ruby/1.9/bad.rb

@@ -0,0 +1,2 @@
+def test
+  {:some => 'random', :hash => 'qwe'}

data/spec/checker/fixtures/ruby/1.9/good.rb

@@ -0,0 +1,3 @@
+def test
+  {some: 'random', hash: 'qwe'}
+end

data/spec/checker/fixtures/yaml/bad.yaml

@@ -0,0 +1,18 @@
+defaults: &defaults
+  mobile_ipad: "iPad"
+  mobile_agents:
+    - "AppleWebKit.+Mobile"
+    - "webOS"
+   - "SymbianOS"
+    - "AppleWebKit.+midori"
+  new_search_config:
+    options:
+      suffix: "/search"
+    params:
+      a: "a"
+      b: "b"
+
+test:
+  <<: *defaults
+  discount: 0.12
+  host: test.test

data/spec/checker/fixtures/yaml/good.yaml

@@ -0,0 +1,18 @@
+defaults: &defaults
+  mobile_ipad: "iPad"
+  mobile_agents:
+    - "AppleWebKit.+Mobile"
+    - "webOS"
+    - "SymbianOS"
+    - "AppleWebKit.+midori"
+  new_search_config:
+    options:
+      suffix: "/search"
+    params:
+      a: "a"
+      b: "b"
+
+test:
+  <<: *defaults
+  discount: 0.12
+  host: test.test

data/spec/checker/modules/coffeescript_spec.rb

@@ -0,0 +1,16 @@
+require 'spec_helper'
+
+describe Checker::Modules::Coffeescript do
+  it 'should only check .coffee files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml', 'f.coffee']
+    mod = Checker::Modules::Coffeescript.new(files)
+    mod.stub(:check_for_executable).and_return(true)
+    mod.should_receive(:check_one_file).with('f.coffee')
+    mod.should_not_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('d.yml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check 
+  end
+end

data/spec/checker/modules/haml_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Checker::Modules::Haml do
+  it 'should only check .haml files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml', 'f.coffee', 'g.haml']
+    mod = Checker::Modules::Haml.new(files)
+    mod.stub(:check_for_executable).and_return(true)
+    mod.stub(:check_one_file).and_return(true)
+    mod.should_receive(:check_one_file).with('g.haml')
+    mod.should_not_receive(:check_one_file).with('f.coffee')
+    mod.should_not_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('d.yml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check 
+  end
+end

data/spec/checker/modules/javascript_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Checker::Modules::Javascript do
+  it 'should only check .js files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml', 'f.coffee', 'g.haml', 'h.js']
+    mod = Checker::Modules::Javascript.new(files)
+    mod.stub(:check_for_executable).and_return(true)
+    mod.should_receive(:check_one_file).with('h.js').and_return(true)
+    mod.should_not_receive(:check_one_file).with('g.haml')
+    mod.should_not_receive(:check_one_file).with('f.coffee')
+    mod.should_not_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('d.yml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check 
+  end
+end

data/spec/checker/modules/pry_spec.rb

@@ -0,0 +1,35 @@
+require 'spec_helper'
+
+describe Checker::Modules::Pry do
+  it 'should check all files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yaml', 'e.yml', 'f.coffee']
+    mod = Checker::Modules::Pry.new(files)
+    mod.stub(:check_one).and_return(true)
+    mod.should_receive(:check_one).exactly(6).times
+    mod.check 
+  end
+
+  it "should not find the pry or remote_pry" do
+    files = [fixture("pry", "without_pry.rb")]
+    mod = Checker::Modules::Pry.new(files)
+    mod.check.should be_true
+  end
+
+  it "should find the pry or remote_pry" do
+    files = [fixture("pry", "with_pry.rb")]
+    mod = Checker::Modules::Pry.new(files)
+    mod.check.should be_false
+  end
+
+  it "should find the pry or remote_pry" do
+    files = [fixture("pry", "with_pry_remote.rb")]
+    mod = Checker::Modules::Pry.new(files)
+    mod.check.should be_false
+  end
+
+  it "should find the pry or remote_pry" do
+    files = [fixture("pry", "with_both.rb")]
+    mod = Checker::Modules::Pry.new(files)
+    mod.check.should be_false
+  end
+end

data/spec/checker/modules/ruby_spec.rb

@@ -4,8 +4,46 @@ describe Checker::Modules::Ruby do
   it 'should only check .rb files' do
     files = ['a.rb', 'b.js.erb', 'c.r']
     mod = Checker::Modules::Ruby.new(files)
-    mod.stub(:check_one).and_return(true)
-    mod.should_receive(:check_one).once
+    mod.stub(:check_one_file).and_return(true)
+    mod.should_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
     mod.check 
   end
-end
+
+  # 1.8
+  if RUBY_VERSION < "1.9"
+    it "should pass the syntax check" do
+      files = [fixture("ruby", "1.8/good.rb")]
+      mod = Checker::Modules::Ruby.new(files)
+      mod.check.should be_true
+    end
+
+    it "should not pass the syntax check" do
+      files = [fixture("ruby", "1.8/bad.rb")]
+      mod = Checker::Modules::Ruby.new(files)
+      mod.check.should be_false
+    end
+
+    it "should not pass the syntax check" do
+      files = [fixture("ruby", "1.8/bad2.rb")]
+      mod = Checker::Modules::Ruby.new(files)
+      mod.check.should be_false
+    end
+  end
+
+  if RUBY_VERSION >= "1.9"
+    it "should pass the syntax check" do
+      files = [fixture("ruby", "1.9/good.rb")]
+      mod = Checker::Modules::Ruby.new(files)
+      mod.check.should be_true
+    end
+
+    it "should not pass the syntax check" do
+      files = [fixture("ruby", "1.9/bad.rb")]
+      mod = Checker::Modules::Ruby.new(files)
+      mod.check.should be_false
+    end
+  end
+  # 1.9
+end

data/spec/checker/modules/sass_spec.rb

@@ -0,0 +1,21 @@
+require 'spec_helper'
+
+describe Checker::Modules::Sass do
+  it 'should only check .sass and .scss files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml', 'f.coffee', 'g.haml', 'h.js', 'i.scss', 'j.sass']
+    mod = Checker::Modules::Sass.new(files)
+    mod.stub(:check_for_executable).and_return(true)
+    mod.stub(:check_one_file).and_return(true)
+    mod.should_receive(:check_one_file).with('j.sass')
+    mod.should_receive(:check_one_file).with('i.scss')
+    mod.should_not_receive(:check_one_file).with('h.js')
+    mod.should_not_receive(:check_one_file).with('g.haml')
+    mod.should_not_receive(:check_one_file).with('f.coffee')
+    mod.should_not_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('d.yml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check 
+  end
+end

data/spec/checker/modules/slim_spec.rb

@@ -0,0 +1,18 @@
+require 'spec_helper'
+
+describe Checker::Modules::Slim do
+  it 'should only check .slim files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml', 'f.coffee', 'g.slim']
+    mod = Checker::Modules::Slim.new(files)
+    mod.stub(:check_for_executable).and_return(true)
+    mod.stub(:check_one_file).and_return(true)
+    mod.should_receive(:check_one_file).with('g.slim')
+    mod.should_not_receive(:check_one_file).with('f.coffee')
+    mod.should_not_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('d.yml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check
+  end
+end

data/spec/checker/modules/yaml_spec.rb

@@ -0,0 +1,27 @@
+require 'spec_helper'
+
+describe Checker::Modules::Yaml do
+  it 'should only check .yaml and .yml files' do
+    files = ['a.rb', 'b.js.erb', 'c.r', 'd.yml', 'e.yaml']
+    mod = Checker::Modules::Yaml.new(files)
+    mod.stub(:check_one_file).and_return(true)
+    mod.should_receive(:check_one_file).with('d.yml')
+    mod.should_receive(:check_one_file).with('e.yaml')
+    mod.should_not_receive(:check_one_file).with('a.rb')
+    mod.should_not_receive(:check_one_file).with('b.js.erb')
+    mod.should_not_receive(:check_one_file).with('c.r')
+    mod.check 
+  end
+
+  it "should properly fetch yaml files" do
+    files = [fixture("yaml", "good.yaml")]
+    mod = Checker::Modules::Yaml.new(files)
+    mod.check.should be_true
+  end
+
+  it "should not pass the syntax check" do
+    files = [fixture("yaml", "bad.yaml")]
+    mod = Checker::Modules::Yaml.new(files)
+    mod.check.should be_false
+  end
+end

data/spec/spec_helper.rb

@@ -1,3 +1,25 @@
+require 'digest/md5'
+
 $LOAD_PATH.unshift(File.join(File.dirname(__FILE__), '..', 'lib'))
 
-require 'checker/cli'
+FIXTURES_PATH = File.join(File.dirname(__FILE__), "checker", "fixtures")
+
+def fixture(dir, filename)
+  "#{FIXTURES_PATH}/#{dir}/#{filename}"
+end
+
+def digest filename
+  Digest::MD5.hexdigest(filename)
+end
+
+require 'checker/cli'
+
+module Checker
+  module Modules
+    class Base
+      def checkout_file file_name, target
+        `cp #{file_name} #{checkout_file_name(target)} > /dev/null 2>&1`
+      end
+    end
+  end
+end

metadata

@@ -1,30 +1,38 @@
 --- !ruby/object:Gem::Specification
 name: checker
 version: !ruby/object:Gem::Version
-  version: 0.0.5
+  version: 0.0.6
   prerelease: 
 platform: ruby
 authors:
 - Jacek Jakubik
+- Tomasz Pewiński
 autorequire: 
 bindir: bin
 cert_chain: []
-date: 2012-07-11 00:00:00.000000000 Z
+date: 2012-08-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: colorize
-  requirement: &70291943302500 !ruby/object:Gem::Requirement
+  requirement: !ruby/object:Gem::Requirement
     none: false
     requirements:
-    - - =
+    - - '='
       - !ruby/object:Gem::Version
         version: 0.5.8
   type: :runtime
   prerelease: false
-  version_requirements: *70291943302500
+  version_requirements: !ruby/object:Gem::Requirement
+    none: false
+    requirements:
+    - - '='
+      - !ruby/object:Gem::Version
+        version: 0.5.8
 description: A collection of modules which every is designed to check syntax for specific
   files.
-email: jacek.jakubik@netguru.pl
+email:
+- jacek.jakubik@netguru.pl
+- tomasz.pewinski@netguru.pl
 executables:
 - checker
 - prepare-commit-msg
@@ -32,7 +40,10 @@ extensions: []
 extra_rdoc_files: []
 files:
 - .gitignore
+- .travis.yml
 - CHANGELOG
+- Gemfile
+- Gemfile.lock
 - README.md
 - Rakefile
 - bin/checker
@@ -48,8 +59,29 @@ files:
 - lib/checker/modules/pry.rb
 - lib/checker/modules/ruby.rb
 - lib/checker/modules/sass.rb
+- lib/checker/modules/slim.rb
+- lib/checker/modules/yaml.rb
+- lib/checker/version.rb
 - spec/checker/cli_spec.rb
+- spec/checker/fixtures/pry/with_both.rb
+- spec/checker/fixtures/pry/with_pry.rb
+- spec/checker/fixtures/pry/with_pry_remote.rb
+- spec/checker/fixtures/pry/without_pry.rb
+- spec/checker/fixtures/ruby/1.8/bad.rb
+- spec/checker/fixtures/ruby/1.8/bad2.rb
+- spec/checker/fixtures/ruby/1.8/good.rb
+- spec/checker/fixtures/ruby/1.9/bad.rb
+- spec/checker/fixtures/ruby/1.9/good.rb
+- spec/checker/fixtures/yaml/bad.yaml
+- spec/checker/fixtures/yaml/good.yaml
+- spec/checker/modules/coffeescript_spec.rb
+- spec/checker/modules/haml_spec.rb
+- spec/checker/modules/javascript_spec.rb
+- spec/checker/modules/pry_spec.rb
 - spec/checker/modules/ruby_spec.rb
+- spec/checker/modules/sass_spec.rb
+- spec/checker/modules/slim_spec.rb
+- spec/checker/modules/yaml_spec.rb
 - spec/spec_helper.rb
 homepage: http://github.com/netguru/checker
 licenses: []
@@ -71,7 +103,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements: []
 rubyforge_project: 
-rubygems_version: 1.8.10
+rubygems_version: 1.8.24
 signing_key: 
 specification_version: 3
 summary: Syntax checker for various files