delta_test 0.2.0 → 1.0.0

data/lib/delta_test/git.rb

@@ -2,87 +2,168 @@ require 'open3'
 require 'shellwords'
 
 module DeltaTest
-  module Git
-    class << self
-
-      ###
-      # Check if in git managed directory
-      #
-      # @return {Boolean}
-      ###
-      def git_repo?
-        o, e, s = exec(%q{git rev-parse --is-inside-work-tree}) rescue []
-        !!s && s.success?
-      end
-
-      ###
-      # Get root directory of git
-      #
-      # @return {String}
-      ###
-      def root_dir
-        o, e, s = exec(%q{git rev-parse --show-toplevel})
-        s.success? ? o.strip : nil
-      end
-
-      ###
-      # Get commit id from rev name
-      #
-      # @params {String} rev - e.g., branch name
-      #
-      # @return {String}
-      ###
-      def rev_parse(rev)
-        o, e, s = exec(%q{git rev-parse %s}, rev)
-        s.success? ? o.strip : nil
-      end
-
-      ###
-      # Compare two rev names by their commit ids
-      #
-      # @params {String} r1
-      # @params {String} r2
-      #
-      # @return {Boolean}
-      ###
-      def same_commit?(r1, r2)
-        rev_parse(r1) == rev_parse(r2)
-      end
-
-      ###
-      # Get file list from git index
-      #
-      # @return {Array<String>}
-      ###
-      def ls_files
-        o, e, s = exec(%q{git ls-files -z})
-        s.success? ? o.split("\x0") : []
-      end
-
-      ###
-      # Get list of modified files in diff
-      #
-      # @params {String} base
-      # @params {String} head
-      #
-      # @return {Array<String>}
-      ###
-      def changed_files(base = 'master', head = 'HEAD')
-        o, e, s = exec(%q{git --no-pager diff --name-only -z %s %s}, base, head)
-        s.success? ? o.split("\x0") : []
-      end
-
-
-    private
-
-      ###
-      # Util for executing command
-      ###
-      def exec(command, *args)
-        args = args.map { |a| Shellwords.escape(a) }
-        Open3.capture3(command % args, chdir: DeltaTest.config.base_path)
-      end
+  class Git
 
+    attr_reader :dir
+
+    def initialize(dir)
+      @dir = Pathname.new(dir)
+    end
+
+    ###
+    # Check if in git managed directory
+    #
+    # @return {Boolean}
+    ###
+    def git_repo?
+      _, _, s = exec(%q{rev-parse --is-inside-work-tree}) rescue []
+      !!s && s.success?
+    end
+
+    ###
+    # Get root directory of git
+    #
+    # @return {String}
+    ###
+    def root_dir
+      o, _, s = exec(%q{rev-parse --show-toplevel})
+      s.success? ? o.strip : nil
+    end
+
+    ###
+    # Get commit id from rev name
+    #
+    # @params {String} rev - e.g., branch name
+    #
+    # @return {String}
+    ###
+    def rev_parse(rev)
+      o, _, s = exec(%q{rev-parse %s}, rev)
+      s.success? ? o.strip : nil
+    end
+
+    ###
+    # Compare two rev names by their commit ids
+    #
+    # @params {String} r1
+    # @params {String} r2
+    #
+    # @return {Boolean}
+    ###
+    def same_commit?(r1, r2)
+      rev_parse(r1) == rev_parse(r2)
+    end
+
+    ###
+    # Get file list from git index
+    #
+    # @return {Array<String>}
+    ###
+    def ls_files(path: '.')
+      o, _, s = exec(%q{ls-files -z %s}, path)
+      s.success? ? o.split("\x0") : []
+    end
+
+    ###
+    # Get list of modified files in diff
+    #
+    # @params {String} base
+    # @params {String} head
+    #
+    # @return {Array<String>}
+    ###
+    def changed_files(base = 'master', head = 'HEAD', path: '.')
+      o, _, s = exec(%q{diff --name-only -z %s %s %s}, base, head, path)
+      s.success? ? o.split("\x0") : []
+    end
+
+    ###
+    # Get list of hashes for the last N commits
+    #
+    # @params {Integer} n
+    #
+    # @return {Array<String>}
+    ###
+    def ls_hashes(n)
+      o, _, s = exec(%q{log -z -n %d --format='%%H'}, n.to_i)
+      s.success? ? o.split("\x0") : []
+    end
+
+    ###
+    # Get url for the remote origin
+    #
+    # @return {String}
+    ###
+    def remote_url
+      o, _, s = exec(%q{config --get remote.origin.url})
+      s.success? ? o.strip : nil
+    end
+
+    ###
+    # Check if it has a remote origin
+    #
+    # @return {String}
+    ###
+    def has_remote?
+      !!remote_url rescue false
+    end
+
+    ###
+    # Pull from the origin master
+    #
+    # @return {Boolean}
+    ###
+    def pull
+      _, _, s = exec(%q{pull origin master})
+      s.success?
+    end
+
+    ###
+    # Push to the origin master
+    #
+    # @return {Boolean}
+    ###
+    def push
+      _, _, s = exec(%q{push origin master})
+      s.success?
+    end
+
+    ###
+    # Add a path to the index
+    #
+    # @return {Boolean}
+    ###
+    def add(path)
+      _, _, s = exec(%q{add %s}, path.to_s)
+      s.success?
+    end
+
+    ###
+    # Create commit
+    #
+    # @return {Boolean}
+    ###
+    def commit(message)
+      _, _, s = exec(%q{commit -m %s}, message.to_s)
+      s.success?
+    end
+
+    ###
+    # Util for executing command
+    ###
+    def exec(subcommand, *args)
+      command = [
+        'git',
+        '--git-dir=%s',
+        '--no-pager',
+        subcommand,
+      ].join(' ')
+
+      args.unshift(@dir.join('.git'))
+      args.map! { |a| a.is_a?(String) ? Shellwords.escape(a) : a }
+
+      Open3.capture3(command % args, chdir: @dir)
     end
+
   end
 end

data/lib/delta_test/profiler.rb

@@ -1,5 +1,13 @@
 module DeltaTest
   module Profiler
 
+    # These methods from native ext:
+    #
+    # .clean!
+    # .start!
+    # .stop!
+    # .running?
+    # .last_result
+
   end
 end

data/lib/delta_test/related_spec_list.rb

@@ -6,20 +6,25 @@ require_relative 'dependencies_table'
 module DeltaTest
   class RelatedSpecList
 
-    attr_reader *%i[
+    attr_reader(*%i[
+      git
       table
       changed_files
-    ]
+    ])
+
+    def initialize
+      @git = Git.new(DeltaTest.config.base_path)
+    end
 
     ###
     # Load table from the file
     ###
-    def load_table!
-      unless File.exist?(DeltaTest.config.table_file_path)
-        raise TableNotFoundError.new(DeltaTest.config.table_file_path)
+    def load_table!(table_file_path)
+      unless File.exist?(table_file_path)
+        raise TableNotFoundError.new(table_file_path)
       end
 
-      @table = DependenciesTable.load(DeltaTest.config.table_file_path)
+      @table = DependenciesTable.load(table_file_path)
     end
 
     ###
@@ -28,12 +33,12 @@ module DeltaTest
     # @params {String} base
     # @params {String} head
     ###
-    def retrive_changed_files!(base, head)
-      unless Git.git_repo?
+    def retrive_changed_files!(commit)
+      unless @git.git_repo?
         raise NotInGitRepositoryError
       end
 
-      @changed_files = Git.changed_files(base, head)
+      @changed_files = @git.changed_files(commit)
     end
 
     ###

data/lib/delta_test/stats.rb

@@ -0,0 +1,41 @@
+require_relative 'git'
+
+module DeltaTest
+  class Stats
+
+    attr_reader :base_git
+    attr_reader :stats_git
+
+    TABLE_FILENAME = 'table.marshal'
+
+    def initialize(head: false)
+      @base_git  = Git.new(DeltaTest.config.base_path)
+      @stats_git = Git.new(DeltaTest.config.stats_path)
+
+      @base_commit = @base_git.rev_parse('HEAD') if head
+    end
+
+    def base_commit
+      return @base_commit if defined?(@base_commit)
+
+      indexes = @stats_git.ls_files
+        .map { |f| f.split('/').take(2).join('') }
+        .to_set
+
+      @base_commit = @base_git.ls_hashes(DeltaTest.config.stats_life)
+        .find { |h| indexes.include?(h) }
+    end
+
+    def commit_dir
+      return unless base_commit
+      return @commit_dir if defined?(@commit_dir)
+
+      @commit_dir = DeltaTest.config.stats_path.join(*base_commit.unpack('A2A*'))
+    end
+
+    def table_file_path
+      commit_dir && commit_dir.join(TABLE_FILENAME)
+    end
+
+  end
+end

data/lib/delta_test/version.rb

@@ -1,7 +1,7 @@
 module DeltaTest
 
-  MAJOR    = 0
-  MINOR    = 2
+  MAJOR    = 1
+  MINOR    = 0
   REVISION = 0
 
   VERSION = [MAJOR, MINOR, REVISION].compact.join('.')

data/lib/delta_test.rb

@@ -19,18 +19,19 @@ module DeltaTest
 
     attr_reader :config
 
-    attr_writer *%i[
+    attr_writer(*%i[
       active
       verbose
-    ]
+    ])
 
-    def setup
-      @config = Configuration.new
-      @config.auto_configure! if active?
+    def config
+      @config ||= Configuration.new.tap do |c|
+        c.auto_configure! if active?
+      end
     end
 
     def configure(&block)
-      @config.update(&block)
+      config.update(&block)
     end
 
     def active?
@@ -44,10 +45,14 @@ module DeltaTest
     end
 
     def log(*args)
-      puts *args if verbose?
+      puts(*args) if verbose?
+    end
+
+    def tester_id
+      return @tester_id if @tester_id
+      t = Time.now
+      @tester_id = 'p%dt%dn%d' % [$$, t.to_i, t.nsec]
     end
 
   end
 end
-
-DeltaTest.setup

data/spec/lib/delta_test/cli/command_base_spec.rb

@@ -0,0 +1,164 @@
+require 'delta_test/cli/command_base'
+
+describe DeltaTest::CLI::CommandBase do
+
+  let(:command_base) { DeltaTest::CLI::CommandBase.new([]) }
+
+  describe '#invoke!' do
+
+    it 'should raise a not implemented error' do
+      expect {
+        command_base.invoke!
+      }.to raise_error
+    end
+
+  end
+
+  describe '#invoke' do
+
+    context 'verbose=false' do
+
+      it 'should call `invoke!`' do
+        allow(DeltaTest).to receive(:verbose?).and_return(false)
+        expect(command_base).to receive(:invoke!).and_return(nil)
+        command_base.invoke
+      end
+
+      it 'should exit with an error message' do
+        expect(command_base).to receive(:exit_with_message)
+        command_base.invoke
+      end
+
+    end
+
+    context 'verbose=true' do
+
+      it 'should call `invoke!`' do
+        allow(DeltaTest).to receive(:verbose?).and_return(false)
+        expect(command_base).to receive(:invoke!).and_return(nil)
+        command_base.invoke!
+      end
+
+      it 'should raise an error' do
+        expect {
+          command_base.invoke
+        }.to raise_error
+      end
+
+    end
+
+  end
+
+  describe '#parse_options!' do
+
+    it 'should parse short options' do
+      args = ['-a', '-b']
+      options = command_base.parse_options!(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['a']).to be(true)
+      expect(options['b']).to be(true)
+    end
+
+    it 'should parse long options' do
+      args = ['--long-a', '--long-b']
+      options = command_base.parse_options!(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['long-a']).to be(true)
+      expect(options['long-b']).to be(true)
+    end
+
+    it 'should parse long options with value' do
+      args = ['--long-a=value-of-a', '--long-b=value-of-b']
+      options = command_base.parse_options!(args)
+
+      expect(args).to be_empty
+      expect(options).to be_a(Hash)
+      expect(options['long-a']).to eq('value-of-a')
+      expect(options['long-b']).to eq('value-of-b')
+    end
+
+    it 'should not parse options after once non-option args appears' do
+      args = ['-a', '--long-a', 'non-option', '--long-b=value-of-b']
+      options = command_base.parse_options!(args)
+
+      expect(args).to eq(['non-option', '--long-b=value-of-b'])
+      expect(options).to be_a(Hash)
+      expect(options['a']).to be(true)
+      expect(options['long-a']).to be(true)
+      expect(options['long-b']).to be_nil
+    end
+
+    describe 'Defaults' do
+
+      it 'should set default options' do
+        args = []
+        options = command_base.parse_options!(args)
+
+        expect(options).to be_a(Hash)
+
+        expect(options['force-run']).to eq(false)
+        expect(options['verbose']).to eq(false)
+      end
+
+      it 'should be able to overwrite default options' do
+        args = ['--force-run', '--verbose']
+        options = command_base.parse_options!(args)
+
+        expect(options).to be_a(Hash)
+
+        expect(options['force-run']).to eq(true)
+        expect(options['verbose']).to eq(true)
+      end
+
+    end
+
+  end
+
+  describe '#exit_with_message' do
+
+    let(:message)        { 'a message' }
+    let(:message_regexp) { /a message/ }
+
+    context 'With status code of zero' do
+
+      let(:status)  { 0 }
+
+      it 'should print a message to stdout and exit' do
+        expect {
+          begin
+            command_base.exit_with_message(status, message)
+          rescue SystemExit => e
+            expect(e.status).to eq(status)
+          end
+        }.to output(message_regexp).to_stdout
+      end
+
+    end
+
+    context 'With status code of non-zero' do
+
+      let(:status)  { 1 }
+
+      it 'should print a message to stderr and exit' do
+        expect {
+          begin
+            command_base.exit_with_message(status, message)
+          rescue SystemExit => e
+            expect(e.status).to eq(status)
+          end
+        }.to output(message_regexp).to_stderr
+      end
+
+    end
+
+  end
+
+  describe '#bundler_enabled?' do
+
+  end
+
+end

data/spec/lib/delta_test/cli/exec_command_spec.rb

@@ -0,0 +1,128 @@
+require 'delta_test/cli/exec_command'
+
+describe DeltaTest::CLI::ExecCommand do
+
+  let(:args) { ['bundle', 'exec', 'rspec'] }
+  let(:command) { DeltaTest::CLI::ExecCommand.new(args) }
+
+  let(:table) do
+    {
+      'spec/foo_spec.rb' => ['lib/foo.rb']
+    }
+  end
+
+  let(:related_spec_files) do
+    [
+      'spec/foo_spec.rb',
+    ]
+  end
+
+  let(:base_commit) { '1111111111111111111111111111111111111111' }
+
+  before do
+    allow(command.list).to receive(:load_table!).and_return(nil)
+    allow(command.list).to receive(:table).and_return(table)
+    allow(command.list).to receive(:retrive_changed_files!).and_return(nil)
+    allow(command.list).to receive(:related_spec_files).and_return(related_spec_files)
+
+    allow(command.stats).to receive(:base_commit).and_return(base_commit)
+  end
+
+  describe '#profile_mode?' do
+
+    it 'should return false if a base commit exists' do
+      expect(command.profile_mode?).to be(false)
+    end
+
+    it 'should return true if a base commit does not exist' do
+      allow(command.stats).to receive(:base_commit).and_return(nil)
+      expect(command.profile_mode?).to be(true)
+    end
+
+    it 'should be able to override by an instance variable' do
+      expect(command.profile_mode?).to be(false)
+      command.instance_variable_set(:@profile_mode, true)
+      expect(command.profile_mode?).to be(true)
+    end
+
+    context 'with --force-run' do
+
+      let(:args) { ['--force-run', 'bundle', 'exec', 'rspec'] }
+
+      it 'should always return false' do
+        allow(command.stats).to receive(:base_commit).and_return(nil)
+        expect(command.profile_mode?).to be(false)
+      end
+
+    end
+
+  end
+
+  describe '#invoke!' do
+
+    before do
+      allow(command).to receive(:exec_with_data).and_return(nil)
+    end
+
+    context 'Full tests' do
+
+      before do
+        allow(command).to receive(:profile_mode?).and_return(true)
+      end
+
+      it 'should run script with a flag' do
+        expect(command.list).not_to receive(:related_spec_files)
+
+        _args = ['%s=%s' % [DeltaTest::ACTIVE_FLAG, true], *args].join(' ')
+        expect(command).to receive(:exec_with_data).with(_args, nil)
+
+        expect {
+          command.invoke!
+        }.not_to raise_error
+      end
+
+    end
+
+    context 'Partial tests' do
+
+      before do
+        allow(command).to receive(:profile_mode?).and_return(false)
+      end
+
+      context 'Any related files' do
+
+        it 'should run script with related spec files' do
+          expect(command.list).to receive(:related_spec_files)
+
+          _args = ['cat', '|', 'xargs', *args].join(' ')
+          expect(command).to receive(:exec_with_data).with(_args, related_spec_files)
+
+          expect {
+            command.invoke!
+          }.not_to raise_error
+        end
+
+      end
+
+      context 'No related files' do
+
+        let(:related_spec_files) { [] }
+
+        it 'should not run script and exit with a message' do
+          expect(command.list).to receive(:related_spec_files)
+
+          expect {
+            begin
+              command.invoke!
+            rescue SystemExit => e
+              expect(e.status).to eq(0)
+            end
+          }.to output(/Nothing/).to_stdout
+        end
+
+      end
+
+    end
+
+  end
+end

data/spec/lib/delta_test/cli/help_command_spec.rb

@@ -0,0 +1,17 @@
+require 'delta_test/cli/help_command'
+
+describe DeltaTest::CLI::HelpCommand do
+
+  let(:command) { DeltaTest::CLI::HelpCommand.new([]) }
+
+  describe '#invoke!' do
+
+    it 'should print gem version' do
+      expect {
+        command.invoke!
+      }.to output(/usage/).to_stdout
+    end
+
+  end
+
+end