affected_tests 0.5.0 → 0.6.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dcc672116e5bb9b0e2da893b7210c77f11e5b95e3c64fc4c217edb1f06f2e7d2
-  data.tar.gz: 1f03fe6932d5f970ee5606501412f30cc397163e86b3a543fe1e95c2d04c3dbd
+  metadata.gz: 9f7612b94dce034b4d89f2891cdc08bc1ee7781449b1f99c5896a22654e9c9e1
+  data.tar.gz: c56ce89c20979413b0af7f0bef9566bf19d6d073b01a27d79a98ac57b2d738da
 SHA512:
-  metadata.gz: 2f92a2f7dd2d2f322d27ca90ebeeaffb18d765145d51efcd6b2df571dd95ffe6a67ba27b3a16e25736853cc148f07f3d064fee1b28f843cbfe43150872757d8e
-  data.tar.gz: ac127737a5b1555e35d18eb3ce9b0cde6adfd679b8bb852482934296e0ec371ed9f14027c1614af98e436f52896c59053094be5b6016c296a481f3b45ffd5078
+  metadata.gz: b19b06a01002be005295109ca9cf81c1d8177f40730ade142f9e3a5293fcb91286302398fb06299d5c56e8248615adae0223735302a4bd60ec845467d85af5fd
+  data.tar.gz: f52bf08410c743a5032ebae1fb260a54ab0fa8098d3f76e043855d18d92df8c4bdf27e637b962a45fbda5c4aa5cec56d8954efebe6be201fd989d6040ac0ac28

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    affected_tests (0.3.0)
+    affected_tests (0.5.0)
       rotoscope
 
 GEM

data/README.md

@@ -27,6 +27,7 @@ require "affected_tests"
 require "affected_tests/rspec"
 
 AffectedTests.setup(
+  engine: :rotoscope,
   project_path: File.expand_path("../../", __FILE__),
   test_dir_path: "spec/",
   output_path: "log/affected-tests-map.json",

data/lib/affected_tests/engine/coverage.rb

@@ -0,0 +1,67 @@
+# frozen_string_literal: true
+
+module AffectedTests
+  module Engine
+    class Coverage
+      def initialize(config)
+        @config = config
+
+        ::Coverage.start(lines: true)
+      end
+
+      def start_trace
+        # Clear buffer in coverage
+        ::Coverage.result(stop: false, clear: true)
+      end
+
+      def stop_trace
+        # Do nothing
+      end
+
+      def tracing?
+        ::Coverage.running?
+      end
+
+      def checkpoint(target_test_path)
+        result = ::Coverage.result(stop: false, clear: true)
+        actual_used = result.select do |_file_path, coverage|
+          coverage[:lines].any? { |line| line && line > 0 }
+        end
+        import(target_test_path, actual_used.keys)
+      end
+
+      def import(target_test_path, used_file_paths)
+        all_related_paths = used_file_paths.select do |file_path|
+          @config.target_path?(file_path)
+        end
+
+        formatted_target_test_path = @config.format_path(target_test_path)
+
+        all_related_paths.each do |path|
+          formatted_path = @config.format_path(path)
+
+          next if formatted_path.start_with?(@config.test_dir_path)
+
+          if formatted_path != formatted_target_test_path
+            add(formatted_target_test_path, formatted_path)
+          end
+        end
+      end
+
+      def add(caller, callee)
+        cache[callee] ||= Set.new
+        cache[callee].add(caller)
+      end
+
+      def cache
+        Thread.current[:cache] ||= {}
+      end
+
+      def dump
+        ::Coverage.result(stop: true, clear: true)
+
+        cache.transform_values(&:to_a)
+      end
+    end
+  end
+end

data/lib/affected_tests/engine/rotoscope.rb

@@ -0,0 +1,70 @@
+# frozen_string_literal: true
+
+module AffectedTests
+  module Engine
+    class Rotoscope
+      def initialize(config)
+        @config = config
+
+        @rotoscope = ::Rotoscope.new do |call|
+          next if self == call.receiver
+
+          if call.caller_path && @config.target_path?(call.caller_path)
+            buffer << call.caller_path
+          end
+        end
+      end
+
+      def start_trace
+        @rotoscope.start_trace
+      end
+
+      def stop_trace
+        @rotoscope.stop_trace
+      end
+
+      def tracing?
+        @rotoscope.tracing?
+      end
+
+      def checkpoint(target_test_path)
+        diff = buffer
+        import(target_test_path, diff)
+        buffer.clear
+      end
+
+      def import(target_test_path, result)
+        all_related_paths = result.uniq.map do |caller_path|
+          @config.format_path(caller_path)
+        end
+
+        formatted_target_test_path = @config.format_path(target_test_path)
+
+        all_related_paths.each do |path|
+          next if path.start_with?(@config.test_dir_path)
+
+          if path != formatted_target_test_path
+            add(formatted_target_test_path, path)
+          end
+        end
+      end
+
+      def add(caller, callee)
+        cache[callee] ||= Set.new
+        cache[callee].add(caller)
+      end
+
+      def buffer
+        Thread.current[:buffer] ||= []
+      end
+
+      def cache
+        Thread.current[:cache] ||= {}
+      end
+
+      def dump
+        cache.transform_values(&:to_a)
+      end
+    end
+  end
+end

data/lib/affected_tests/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module AffectedTests
-  VERSION = "0.5.0"
+  VERSION = "0.6.0"
 end

data/lib/affected_tests.rb

@@ -3,75 +3,80 @@
 require "set"
 require "json"
 
-require "rotoscope"
-
 require_relative "affected_tests/version"
 
 module AffectedTests
-  module_function
-
-  def setup(project_path:, test_dir_path:, output_path:, revision: nil)
-    @project_path = project_path
-    @test_dir_path = test_dir_path
-    @output_path = output_path
-    @revision = revision
-    @rotoscope = Rotoscope.new do |call|
-      next if self == call.receiver
+  class Configuration
+    attr_reader :project_path, :bundler_path, :test_dir_path, :output_path, :revision
+
+    def initialize(project_path:, test_dir_path:, output_path:, revision:)
+      @project_path = project_path
+      @test_dir_path = test_dir_path
+      @bundler_path = Bundler.bundle_path.to_s
+      @output_path = output_path
+      @revision = revision
+    end
 
-      if call.caller_path && target_path?(call.caller_path)
-        buffer << call.caller_path
+    def format_path(path)
+      if path&.start_with?(@project_path)
+        path.sub(@project_path + "/", "")
+      else
+        path
       end
     end
-  end
 
-  def start_trace
-    @rotoscope.start_trace
-  end
+    def target_path?(path)
+      return false if path.nil?
 
-  def stop_trace
-    @rotoscope.stop_trace
+      path.start_with?(@project_path) && !path.start_with?(@bundler_path)
+    end
   end
 
-  def import_from_rotoscope(target_test_path, result)
-    all_related_paths = result.uniq.map do |caller_path|
-      format_path(caller_path)
-    end
+  module_function
 
-    formatted_target_test_path = format_path(target_test_path)
+  def setup(engine: :rotoscope, project_path:, test_dir_path:, output_path:, revision: nil)
+    @config = Configuration.new(
+      project_path: project_path,
+      output_path: output_path,
+      revision: revision,
+      test_dir_path: test_dir_path
+    )
 
-    all_related_paths.each do |path|
-      next if path.start_with?(@test_dir_path)
+    @engine = select_engine(engine).new(@config)
+  end
 
-      if path != formatted_target_test_path
-        add(formatted_target_test_path, path)
-      end
+  def select_engine(engine)
+    case engine
+    when :rotoscope
+      require "rotoscope"
+      require "affected_tests/engine/rotoscope"
+      AffectedTests::Engine::Rotoscope
+    when :coverage
+      require "coverage"
+      require "affected_tests/engine/coverage"
+      AffectedTests::Engine::Coverage
+    else
+      raise "Unknown engine: #{engine}"
     end
   end
 
-  def checkpoint(target_test_path)
-    diff = buffer
-    import_from_rotoscope(target_test_path, diff)
-    buffer.clear
+  def start_trace
+    @engine.start_trace
   end
 
-  def dump
-    data = { revision: @revision || build_revision, map: cache.transform_values(&:to_a) }
-    File.write(@output_path, JSON.dump(data))
-  ensure
-    @rotoscope.stop_trace if @rotoscope.tracing?
+  def stop_trace
+    @engine.stop_trace
   end
 
-  def format_path(path)
-    if path&.start_with?(@project_path)
-      path.sub(@project_path + "/", "")
-    else
-      path
-    end
+  def checkpoint(target_test_path)
+    @engine.checkpoint(target_test_path)
   end
 
-  def add(caller, callee)
-    cache[callee] ||= Set.new
-    cache[callee].add(caller)
+  def dump
+    data = { revision: @config.revision || build_revision, map: @engine.dump }
+    File.write(@config.output_path, JSON.dump(data))
+  ensure
+    @engine.stop_trace if @engine.tracing?
   end
 
   def build_revision
@@ -86,22 +91,4 @@ module AffectedTests
       "UNKNOWN"
     end
   end
-
-  def buffer
-    Thread.current[:buffer] ||= []
-  end
-
-  def cache
-    Thread.current[:cache] ||= {}
-  end
-
-  def bundler_path
-    @bundler_path ||= Bundler.bundle_path.to_s
-  end
-
-  def target_path?(path)
-    return false if path.nil?
-
-    path.start_with?(@project_path) && !path.start_with?(bundler_path)
-  end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: affected_tests
 version: !ruby/object:Gem::Version
-  version: 0.5.0
+  version: 0.6.0
 platform: ruby
 authors:
 - Shia
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2022-08-29 00:00:00.000000000 Z
+date: 2022-09-13 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: rotoscope
@@ -39,6 +39,8 @@ files:
 - Rakefile
 - lib/affected_tests.rb
 - lib/affected_tests/differ.rb
+- lib/affected_tests/engine/coverage.rb
+- lib/affected_tests/engine/rotoscope.rb
 - lib/affected_tests/map_merger.rb
 - lib/affected_tests/rspec.rb
 - lib/affected_tests/version.rb