last_hit 0.1.11 → 0.1.12

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA1:
-  metadata.gz: 4ca55a8ac1827aeceb584041a728b3032d9cbe2e
-  data.tar.gz: 61f9de1b8b5419b644b4785a6cc2107401fc5d02
+  metadata.gz: fec9a8651a90d144637c9c7e9dbe53dd9b4466e3
+  data.tar.gz: df1afbcd62db73a48163686141df6cba08184b88
 SHA512:
-  metadata.gz: f9f01225c1a23f0f4ece3e0c96fb159f0644118bb185f7c7bc5f7ef38d7cde71c0f6dc12a6f62f3dcaf1f614a1b5331f46485db0b4ed94ef06047c4287d41da6
-  data.tar.gz: fdd805e580e3ef64e8033d846ac448db15d7aa0db3dd24ee35aebde723c03bc9326b6e3a0b6f98f3ad809f237398c82d974d21016476eff3c7dd0ca3230df1ab
+  metadata.gz: 2f940eae70c68c66a661a17e064ace5f445fdfed06e7819793e1a62d900a346a780a6fdc2214b2063155ec91e3d2a75c58181ed82a8d239784f9115afd2072b5
+  data.tar.gz: 5f9ccb5c34cde694c1a560df469b00012fa5bfea36f98a73a5a08a92237612888806c8713293178a2e748e5786270cb8220037808ef1d9cd14d28a2d5a33a17d

data/LICENSE.txt

@@ -1,4 +1,4 @@
-Copyright (c) 2014 TODO: Write your name
+Copyright (c) 2014 Duc Le
 
 MIT License
 

data/README.md

@@ -1,6 +1,7 @@
 # LastHit
 
 You finish your features, push code to test servers and they fails after centuries wating? This gem will reduce your waiting time by running only relevant tests in your current branch before pushing. Then you can fix failed tests right before pushing them to the test server.
+Currently, this gem works with GIT and rspec. In the future, the gem will support other Version Controls and other test frameworks.
 
 ## Installation
 
@@ -54,8 +55,6 @@ last_hit all_tests -b develop
 #=> 3 examples, 0 failures
 ```
 
-`develop` is your base branch
-
 ## Contributing
 
 1. Fork it ( https://github.com/[my-github-username]/last_hit/fork )

data/lib/last_hit/checker.rb

@@ -0,0 +1,35 @@
+require 'last_hit/file_filter'
+
+class LastHit
+  module Checker
+    def self.check(files)
+      test_files = FileFilter::Spec.filter(files)
+      ruby_files = FileFilter::Ruby.filter(files)
+      results = filter(ruby_files, test_files)
+      if results.length > 0
+        $stdout.puts 'There are some changed files without tests'
+        results.each { |r| $stdout.puts r }
+      end
+    end
+
+    private
+
+    def self.filter(ruby_files, test_files)
+      ruby_files.select do |file|
+        not_contain?(file, test_files)
+      end
+    end
+
+    def self.not_contain?(file, test_files)
+      test_name = get_test_name(file)
+      test_files.each do |test_file|
+        return false if test_file.include?(test_name)
+      end
+      true
+    end
+
+    def self.get_test_name(file)
+      file.gsub('.rb', '_spec.rb').split('/').last(2).join('/')
+    end
+  end
+end

data/lib/last_hit/cli.rb

@@ -40,5 +40,13 @@ test_command: 'bundle exec rspec'
       Cli.load_config(options[:config])
       LastHit.new.all_tests(options[:base])
     end
+
+    desc 'check', 'Check if any tests missing for modified files'
+    method_option :base, aliases: '-b', desc: 'Base branch for compare', type: :string
+    method_option :config, aliases: '-C', desc: 'Path to the config file', type: :string, default: CONFIG_PATH
+    def check
+      Cli.load_config(options[:config])
+      LastHit.new.check(options[:base])
+    end
   end
 end

data/lib/last_hit/file_filter/ruby.rb

@@ -0,0 +1,22 @@
+class LastHit
+  module FileFilter
+    class Ruby
+      RUBY_REGEX = /\A.+_spec.rb\z/
+
+      class << self
+        def filter(files)
+          files.select { |file| match?(file) }
+        end
+
+        def match?(file)
+          return false if Spec.match?(file)
+          ['app', 'lib'].each do |dr|
+            regex = /#{dr}\/.+\.rb/
+            return true if file =~ regex
+          end
+          false
+        end
+      end
+    end
+  end
+end

data/lib/last_hit/file_filter/spec.rb

@@ -5,7 +5,11 @@ class LastHit
 
       class << self
         def filter(files)
-          files.select { |file| file =~ SPEC_REGEX }
+          files.select { |file| match?(file) }
+        end
+
+        def match?(file)
+          file =~ SPEC_REGEX
         end
       end
     end

data/lib/last_hit/file_filter.rb

@@ -1,4 +1,5 @@
 require 'last_hit/file_filter/spec'
+require 'last_hit/file_filter/ruby'
 
 class LastHit
   module FileFilter

data/lib/last_hit/version.rb

@@ -1,3 +1,3 @@
 class LastHit
-  VERSION = "0.1.11"
+  VERSION = "0.1.12"
 end

data/lib/last_hit.rb

@@ -5,6 +5,7 @@ require 'last_hit/rc_adapter'
 require 'last_hit/file_filter'
 
 require 'last_hit/test_handler'
+require 'last_hit/checker'
 
 require 'last_hit/cli'
 
@@ -20,10 +21,20 @@ class LastHit
     process(files)
   end
 
+  def check(base_branch)
+    base_branch = Configure.base_branch if base_branch.nil?
+    files = RcAdapter::Git.current_branch_files(base_branch)
+    check(files)
+  end
+
   private
 
   def process(files)
     tests = FileFilter::Spec.filter(files)
     TestHandler.run(tests)
   end
+
+  def check(files)
+    Checker.check(files)
+  end
 end

data/spec/last_hit/checker_spec.rb

@@ -0,0 +1,26 @@
+require 'spec_helper'
+
+describe LastHit::Checker do
+  describe '.filter' do
+    let(:checker) { LastHit::Checker }
+    let(:ruby_files) { [
+      'app/models/user.rb',
+      'app/models/post.rb',
+      'lib/signin.rb',
+      'lib/publish.rb'
+    ] }
+    let(:test_files) { [
+      'spec/models/user_spec.rb',
+      'spec/lib/signin_spec.rb'
+    ] }
+    let(:no_test_files) { [
+      'app/models/post.rb',
+      'lib/publish.rb'
+    ] }
+
+    it 'return ruby files that do not have tests' do
+      results = checker.send(:filter, ruby_files, test_files)
+      expect(results).to match_array(no_test_files)
+    end
+  end
+end

data/spec/last_hit/file_filter/ruby_spec.rb

@@ -0,0 +1,20 @@
+require 'spec_helper'
+
+describe LastHit::FileFilter::Ruby do
+  describe '.filter' do
+    let(:files) { [
+      'app/models/user.rb',
+      'service/author.rb',
+      'lib/post.rb',
+      'spec/service/user_spec.rb'
+    ] }
+    let(:ruby_files) { [
+      'app/models/user.rb',
+      'lib/post.rb'
+    ] }
+
+    it 'return files with extension is spec' do
+      expect(LastHit::FileFilter::Ruby.filter(files)).to match_array(ruby_files)
+    end
+  end
+end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: last_hit
 version: !ruby/object:Gem::Version
-  version: 0.1.11
+  version: 0.1.12
 platform: ruby
 authors:
 - Duc Le
@@ -84,16 +84,20 @@ files:
 - bin/setup
 - last_hit.gemspec
 - lib/last_hit.rb
+- lib/last_hit/checker.rb
 - lib/last_hit/cli.rb
 - lib/last_hit/configure.rb
 - lib/last_hit/file_filter.rb
+- lib/last_hit/file_filter/ruby.rb
 - lib/last_hit/file_filter/spec.rb
 - lib/last_hit/rc_adapter.rb
 - lib/last_hit/rc_adapter/git.rb
 - lib/last_hit/test_handler.rb
 - lib/last_hit/version.rb
-- spec/file_filter/spec_spec.rb
-- spec/rc_adapter/git_spec.rb
+- spec/last_hit/checker_spec.rb
+- spec/last_hit/file_filter/ruby_spec.rb
+- spec/last_hit/file_filter/spec_spec.rb
+- spec/last_hit/rc_adapter/git_spec.rb
 - spec/spec_helper.rb
 homepage: https://github.com/lmduc/last_hit
 licenses:
@@ -121,6 +125,8 @@ specification_version: 4
 summary: Help you to reduce the time waiting for the test servers by running some
   relevant tests before pushing
 test_files:
-- spec/file_filter/spec_spec.rb
-- spec/rc_adapter/git_spec.rb
+- spec/last_hit/checker_spec.rb
+- spec/last_hit/file_filter/ruby_spec.rb
+- spec/last_hit/file_filter/spec_spec.rb
+- spec/last_hit/rc_adapter/git_spec.rb
 - spec/spec_helper.rb