delta_test 0.1.0 → 0.2.0

data/lib/delta_test/profiler.rb

@@ -0,0 +1,5 @@
+module DeltaTest
+  module Profiler
+
+  end
+end

data/lib/delta_test/related_spec_list.rb

@@ -37,27 +37,86 @@ module DeltaTest
     end
 
     ###
-    # Calculate related spec files
+    # Return if full tests are related
     #
-    # @return {Set<String>}
+    # @return {Boolean}
     ###
-    def related_spec_files
-      spec_files = Set.new
+    def full_tests?
+      return false unless @changed_files
+
+      @full_tests ||= if DeltaTest.config.full_test_patterns.empty?
+          false
+        else
+          Utils.files_grep(
+            @changed_files,
+            DeltaTest.config.full_test_patterns
+          ).any?
+        end
+    end
+
+    ###
+    # Dependent spec files
+    #
+    # @return {Set|Nil}
+    ###
+    def dependents
+      return nil unless @changed_files && @table
+
+      return @dependents if @dependents
+      @dependents = Set.new
 
       @table.each do |spec_file, dependencies|
-        related = @changed_files.include?(spec_file) \
+        dependent = @changed_files.include?(spec_file) \
           || (dependencies & @changed_files).any?
 
-        spec_files << spec_file if related
+        @dependents << spec_file if dependent
       end
 
+      @dependents
+    end
+
+    ###
+    # Custom spec files
+    #
+    # @return {Set|Nil}
+    ###
+    def customs
+      return nil unless @changed_files
+
+      return @customs if @customs
+      @customs = Set.new
+
       DeltaTest.config.custom_mappings.each do |spec_file, patterns|
         if Utils.files_grep(@changed_files, patterns).any?
-          spec_files << spec_file
+          @customs << spec_file
         end
       end
 
-      spec_files
+      @customs
+    end
+
+    ###
+    # Full spec files
+    #
+    # @return {Set|Nil}
+    ###
+    def full
+      return nil unless @table
+
+      @full ||= Set.new(@table.keys)
+    end
+
+    ###
+    # Calculate related spec files
+    #
+    # @return {Set<String>}
+    ###
+    def related_spec_files
+      if full_tests?
+        full
+      else
+        dependents | customs
+      end
     end
 
   end

data/lib/delta_test/spec_helpers.rb

@@ -4,19 +4,21 @@ module DeltaTest
   module SpecHelpers
 
     ###
-    # Setup generator and hook analyzer on contexts
+    # Setup generator and hook profiler on contexts
     ###
     def use_delta_test(example)
-      $delta_test_generator ||= DeltaTest::Generator.new
-      $delta_test_generator.setup!
+      generator = DeltaTest::GeneratorSingleton.instance
+      generator.setup!
 
-      example.before(:context) do
-        $delta_test_generator.start!(example.metadata[:file_path])
+      example.before(:all) do
+        generator.start!(example.file_path)
       end
 
-      example.after(:context) do
-        $delta_test_generator.stop!
+      example.after(:all) do
+        generator.stop!
       end
+
+      generator.hook_on_exit
     end
 
     ###

data/lib/delta_test/version.rb

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

data/spec/lib/delta_test/cli_spec.rb

@@ -154,7 +154,7 @@ describe DeltaTest::CLI do
 
   end
 
-  describe '#run_full_tests?' do
+  describe '#profile_mode?' do
 
     let(:map) do
       {
@@ -179,7 +179,7 @@ describe DeltaTest::CLI do
       end
 
       it 'should return true' do
-        expect(cli.run_full_tests?).to be(true)
+        expect(cli.profile_mode?).to be(true)
       end
 
     end
@@ -194,7 +194,7 @@ describe DeltaTest::CLI do
       end
 
       it 'should return false' do
-        expect(cli.run_full_tests?).to be(false)
+        expect(cli.profile_mode?).to be(false)
       end
 
     end
@@ -306,7 +306,7 @@ describe DeltaTest::CLI do
       context 'Full tests' do
 
         before do
-          expect(cli).to receive(:run_full_tests?).with(no_args).and_return(true)
+          allow(cli).to receive(:profile_mode?).with(no_args).and_return(true)
         end
 
         it 'should run script with a flag' do
@@ -325,7 +325,7 @@ describe DeltaTest::CLI do
       context 'Partial tests' do
 
         before do
-          expect(cli).to receive(:run_full_tests?).with(no_args).and_return(false)
+          allow(cli).to receive(:profile_mode?).with(no_args).and_return(false)
         end
 
         context 'Any related files' do
@@ -365,6 +365,26 @@ describe DeltaTest::CLI do
 
     end
 
+    describe '#do_clear' do
+
+      before do
+        allow(Open3).to receive(:capture3).and_return(nil)
+      end
+
+      let(:table_file_path) { '/path/to/table' }
+
+      it 'should remove table files' do
+        allow(DeltaTest.config).to receive(:table_file_path).and_return(table_file_path)
+        expect(DeltaTest.config).to receive(:table_file_path).with('')
+        expect(Open3).to receive(:capture3).with('rm %s*' % table_file_path)
+
+        expect {
+          cli.do_clear
+        }.not_to raise_error
+      end
+
+    end
+
     describe '#do_help' do
 
       it 'should print help' do
@@ -394,6 +414,7 @@ describe DeltaTest::CLI do
         'list'  => 'do_list',
         'table' => 'do_table',
         'exec'  => 'do_exec',
+        'clear' => 'do_clear',
         'help'  => 'do_help',
         '-v'    => 'do_version',
       }

data/spec/lib/delta_test/configuration_spec.rb

@@ -218,6 +218,18 @@ describe DeltaTest::Configuration do
         expect(configuration.table_file_path).to eq(Pathname.new('/somewhere/table_file'))
       end
 
+      context 'With part' do
+
+        it 'should return a path with a part extension' do
+          configuration.base_path  = '/base_path'
+          configuration.table_file = 'somewhere/table_file'
+
+          configuration.precalculate!
+          expect(configuration.table_file_path(1)).to eq(Pathname.new('/base_path/somewhere/table_file.part-1'))
+        end
+
+      end
+
     end
 
   end

data/spec/lib/delta_test/dependencies_table_spec.rb

@@ -53,6 +53,41 @@ describe DeltaTest::DependenciesTable do
 
   end
 
+  describe '#reverse_merge!' do
+
+    let(:one_table)   { DeltaTest::DependenciesTable.new }
+    let(:other_table) { DeltaTest::DependenciesTable.new }
+
+    it 'should raise an error if other is not an instance of DependenciesTable' do
+      expect {
+        one_table.reverse_merge!({})
+      }.to raise_error(TypeError)
+    end
+
+    it 'should merge other table into self' do
+      one_table[:foo] << 1
+      one_table[:foo] << 2
+      one_table[:bar] << 3
+      one_table[:one] << 4
+
+      other_table[:foo] << 10
+      other_table[:foo] << 20
+      other_table[:bar] << 30
+      other_table[:other] << 40
+
+      expect {
+        one_table.reverse_merge!(other_table)
+      }.not_to raise_error
+
+      expect(one_table.keys).to eq(one_table.keys | other_table.keys)
+      expect(one_table[:foo]).to eq(Set[1, 2, 10, 20])
+      expect(one_table[:bar]).to eq(Set[3, 30])
+      expect(one_table[:one]).to eq(Set[4])
+      expect(one_table[:other]).to eq(Set[40])
+    end
+
+  end
+
   describe '#without_default_proc' do
 
     it 'should reset default_proc temporary inside a block' do

data/spec/lib/delta_test/generator_spec.rb

@@ -31,14 +31,14 @@ describe DeltaTest::Generator do
 
   after do
     DeltaTest.active = false
-    RubyProf.stop if RubyProf.running?
+    DeltaTest::Profiler.clean!
   end
 
   describe '#setup!' do
 
     it 'should setup a generator' do
       expect {
-        generator.setup!(false)  # disable tearadown
+        generator.setup!
       }.not_to raise_error
 
       expect(generator).to be_respond_to(:table)
@@ -50,17 +50,17 @@ describe DeltaTest::Generator do
   describe '#start!' do
 
     before do
-      generator.setup!(false)  # disable tearadown
+      generator.setup!
     end
 
-    it 'should start ruby-prof' do
-      expect(RubyProf.running?).to be(false)
+    it 'should start the profiler' do
+      expect(DeltaTest::Profiler.running?).to be(false)
 
       expect {
         generator.start!(spec_file)
       }.not_to raise_error
 
-      expect(RubyProf.running?).to be(true)
+      expect(DeltaTest::Profiler.running?).to be(true)
     end
 
     describe '#current_spec_file' do
@@ -84,15 +84,15 @@ describe DeltaTest::Generator do
   describe '#stop!' do
 
     before do
-      generator.setup!(false)  # disable tearadown
+      generator.setup!
     end
 
-    it 'should stop ruby-prof' do
-      expect(RubyProf.running?).to be(false)
+    it 'should stop the profiler' do
+      expect(DeltaTest::Profiler.running?).to be(false)
       generator.start!(spec_file)
-      expect(RubyProf.running?).to be(true)
+      expect(DeltaTest::Profiler.running?).to be(true)
       generator.stop!
-      expect(RubyProf.running?).to be(false)
+      expect(DeltaTest::Profiler.running?).to be(false)
     end
 
     it 'should unset current_spec_file' do
@@ -108,7 +108,7 @@ describe DeltaTest::Generator do
   describe '#table' do
 
     before do
-      generator.setup!(false)  # disable tearadown
+      generator.setup!
     end
 
     it 'should return a set of source files' do
@@ -168,15 +168,15 @@ describe DeltaTest::Generator do
     context 'When `setup!` is called' do
 
       before do
-        generator.setup!(false)  # disable tearadown
+        generator.setup!
       end
 
-      it 'should stop ruby-prof if running' do
-        expect(RubyProf.running?).to be(false)
+      it 'should stop the profiler if running' do
+        expect(DeltaTest::Profiler.running?).to be(false)
         generator.start!(spec_file)
-        expect(RubyProf.running?).to be(true)
+        expect(DeltaTest::Profiler.running?).to be(true)
         generator.teardown!
-        expect(RubyProf.running?).to be(false)
+        expect(DeltaTest::Profiler.running?).to be(false)
       end
 
       it 'should save the table into a file' do
@@ -198,4 +198,21 @@ describe DeltaTest::Generator do
 
   end
 
+  describe '#hook_on_exit' do
+
+    before do
+      allow(Kernel).to receive(:at_exit).and_return(nil)
+    end
+
+    it 'should call at_exit' do
+      # FIXME
+      # expect(Kernel).to receive(:at_exit)
+
+      expect {
+        generator.hook_on_exit
+      }.not_to raise_error
+    end
+
+  end
+
 end

data/spec/lib/delta_test/profiler_spec.rb

@@ -0,0 +1,121 @@
+require 'delta_test/profiler'
+
+describe DeltaTest::Profiler do
+
+  after do
+    DeltaTest::Profiler.clean!
+  end
+
+  describe '::start!' do
+
+    it 'should start profiler' do
+      expect(DeltaTest::Profiler.running?).to be(false)
+
+      expect {
+        DeltaTest::Profiler.start!
+      }.not_to raise_error
+
+      expect(DeltaTest::Profiler.running?).to be(true)
+    end
+
+  end
+
+  describe '::stop!' do
+
+    it 'should not raise error if the profiler not yet started' do
+      expect {
+        DeltaTest::Profiler.stop!
+      }.not_to raise_error
+    end
+
+    it 'should set result' do
+      expect(DeltaTest::Profiler.last_result).to be_a(Array)
+
+      DeltaTest::Profiler.start!
+
+      expect {
+        DeltaTest::Profiler.stop!
+      }.not_to raise_error
+
+      expect(DeltaTest::Profiler.last_result).to be_a(Array)
+      expect(DeltaTest::Profiler.last_result).not_to be_empty
+    end
+
+  end
+
+  describe '::last_result' do
+
+    it 'should retrun an array if not yet started' do
+      files = DeltaTest::Profiler.last_result
+      expect(files).to be_a(Array)
+    end
+
+    it 'should return nil if running' do
+      DeltaTest::Profiler.start!
+
+      files = DeltaTest::Profiler.last_result
+      expect(files).to be_nil
+    end
+
+    it 'should return an array of source files' do
+      DeltaTest::Profiler.start!
+      DeltaTest::Profiler.stop!
+      files = DeltaTest::Profiler.last_result
+      expect(files).to be_a(Array)
+    end
+
+    describe 'Source files' do
+
+      context 'Instantiated class in a file' do
+
+        it 'should not include the file' do
+          DeltaTest::Profiler.start!
+          Sample::Alpha.new
+          DeltaTest::Profiler.stop!
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/alpha.rb'))
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/beta.rb'))
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/gamma.rb'))
+        end
+
+      end
+
+      context 'Called some instance methods of a class in the file' do
+
+        it 'should include the file' do
+          DeltaTest::Profiler.start!
+          Sample::Alpha.new.alpha
+          DeltaTest::Profiler.stop!
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/alpha.rb'))
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/beta.rb'))
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/gamma.rb'))
+        end
+
+      end
+
+      context 'Called methods that uses extarnal classes' do
+
+        it 'should include a extarnal file' do
+          DeltaTest::Profiler.start!
+          Sample::Alpha.new.beta
+          DeltaTest::Profiler.stop!
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/alpha.rb'))
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/beta.rb'))
+          expect(DeltaTest::Profiler.last_result).not_to include(fixture_path('sample/gamma.rb'))
+        end
+
+        it 'should include extarnal files even if nested' do
+          DeltaTest::Profiler.start!
+          Sample::Alpha.new.beta_gamma
+          DeltaTest::Profiler.stop!
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/alpha.rb'))
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/beta.rb'))
+          expect(DeltaTest::Profiler.last_result).to include(fixture_path('sample/gamma.rb'))
+        end
+
+      end
+
+    end
+
+  end
+
+end