diff_test 0.8.4 → 0.8.5

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 194963612fbe79dd0506ee1d92e500fca33e8aacf6b5fe38b00cb91741a64367
-  data.tar.gz: 7a8f7a9a4e516cff513ae27247397ac40d35ed697ca5a483aed94a75d60bc668
+  metadata.gz: 71eb34b2f03e6cfb5c88f0382645ab9991273650ab54bf33ca985dd3d244b631
+  data.tar.gz: 140bc0788f6bb66aa6bf75d1d55934feeba868006593ff4b134d75633900a376
 SHA512:
-  metadata.gz: 55bf9771f23900a6f3c0800e090db8dc7884acf9e0432573e25a4e4f75cb5adb20af9f750bb0c04e8a765f91503c4fa10ec559dbb34b77d2802e1a2ffea9c560
-  data.tar.gz: 6ee7fb4f979941883644d0bf27c29832b613c4e90c66f962beaffe0282b1c9d80761e73009d7da0f823f017c44d8d665ec0a9015222986da1bae53e7c019b03e
+  metadata.gz: 0160b17c695801c363ab06aa98844ac586238df47db23f4fc25a628f49081e2fa76b7abafb49dacf447ad2db3960418d6dd5827442ee4da60da700c76bf77eca
+  data.tar.gz: 707a6088a90723c80715002f2d27cbc87f5833398c4d16b695213efb3af9feaafc8166f68d6cf867d94286e3dacb6d6330701fe8973b5383ee333c053aab0bdd

data/lib/diff_test/core_ext.rb

@@ -0,0 +1,11 @@
+class BasicObject
+  def __basic_object?
+    true
+  end
+end
+
+module Kernel
+  def __basic_object?
+    false
+  end
+end

data/lib/diff_test/helper.rb

@@ -66,5 +66,13 @@ module DiffTest
     def committer_email
       @committer_email ||= `git log -1 --pretty=%ae`.strip
     end
+
+    def bm(*name, &block)
+      puts "Starting #{name.join(' ')}------------"
+      start = Time.now
+      result = yield
+      puts "Finished #{name.join(' ')} in #{(Time.now - start).round(2)}s------------"
+      result
+    end
   end
 end

data/lib/diff_test/impacted_file_tracker.rb

@@ -17,7 +17,11 @@ module DiffTest
 
     def save_payload
       impacted_files.each_with_object({}) do |file_path, payload|
-        payload[file_path] = FileHashComputer.compute(file_path)
+        begin
+          payload[file_path] = FileHashComputer.compute(file_path)
+        rescue Errno::ENOENT
+          nil
+        end
       end
     end
 

data/lib/diff_test/integrations/minitest/lifecycle.rb

@@ -7,6 +7,17 @@ module DiffTest
         end
 
         module InstanceMethods
+          def relative_test_file_path(runnable_class, runnable_method)
+            value   = Helper.relative_const_source_path_from_project_root(runnable_class.name) rescue nil
+
+            unbound_method = runnable_class.instance_method(runnable_method) if runnable_method
+            value ||= Helper.relative_path_from_project_root(unbound_method.source_location.first) if unbound_method
+
+            value
+          rescue
+            nil
+          end
+
           def run
             return super unless DiffTest.configuration.enabled?
 
@@ -15,7 +26,7 @@ module DiffTest
                 tests = ::Minitest::Test.runnables.flat_map do |runnable|
                   next if runnable.runnable_methods.empty?
 
-                  test_file_path = Helper.relative_const_source_path_from_project_root(runnable.name) rescue nil
+                  test_file_path = relative_test_file_path(runnable, runnable.runnable_methods.first)
                   next unless test_file_path
 
                   runnable.runnable_methods.map do |test_name|
@@ -39,7 +50,7 @@ module DiffTest
             return super unless DiffTest::TestSuiteExecution.current.valid?
 
             begin
-              test_file_path = Helper.relative_const_source_path_from_project_root(self.class.name) rescue nil
+              test_file_path = relative_test_file_path(self.class, self.name)
               return super if test_file_path.nil?
               test_name = self.name
 

data/lib/diff_test/integrations/rails_js/integration.rb

@@ -13,7 +13,7 @@ module DiffTest
         end
 
         def loaded?
-          supported? && defined?(::Rails)
+          supported? && defined?(::Rails) && defined?(::Rails::Railtie)
         end
 
         def integrate

data/lib/diff_test/test_execution.rb

@@ -23,8 +23,8 @@ module DiffTest
         test: id,
         result: @result,
         runtime_ms: runtime_ms,
-        impacted_files: @impacted_file_tracker.save_payload,
-      }
+        impacted_files: (@impacted_file_tracker.save_payload if passed?),
+      }.compact
     end
 
     def id
@@ -62,6 +62,10 @@ module DiffTest
       @result = :failed
     end
 
+    def passed?
+      @result == :passed
+    end
+
     def runtime_ms
       return 0 if @stopped_at.nil? || @started_at.nil?
       (@stopped_at - @started_at) * 1000

data/lib/diff_test/test_suite_execution.rb

@@ -9,20 +9,40 @@ module DiffTest
     end
 
     def find_or_create_on_server
-      response = DiffTest::ApiClient.post("/test_suite_executions", body: payload)
+      if ENV['DIFF_TEST_SKIP_API']
+        return @id = 'fake-test-suite-execution-id'
+      elsif ENV['DIFF_TEST_SKIP_API_CALL']
+        payload.to_json
+        return @id = 'fake-test-suite-execution-id'
+      end
+
+      response = DiffTest::Helper.bm('create test suite executions') do
+        DiffTest::ApiClient.post("/test_suite_executions", body: payload)
+      end
+
       raise "Failed to create test suite execution: #{response.inspect}" unless response.success?
       raise "Failed to create test suite execution: #{response.inspect}" unless response['id'].present?
+
       @id = response['id']
     end
 
     def previous_compatible_test_executions
-      DiffTest::ApiClient.post("/test_suite_executions/#{id}/previous_compatible_test_executions", body: { tests: @tests })
+      if ENV['DIFF_TEST_SKIP_API']
+        return {}
+      elsif ENV['DIFF_TEST_SKIP_API_CALL']
+        { tests: @tests }.to_json
+        return {}
+      end
+
+      DiffTest::Helper.bm('previous compatible test executions') do
+        DiffTest::ApiClient.post("/test_suite_executions/#{id}/previous_compatible_test_executions", body: { tests: @tests })
+      end
     end
 
     def ensure_application_eager_loaded!
       return if @eager_loaded
       @eager_loaded = true
-      ::Rails.application.eager_load! if defined?(::Rails)
+      ::Rails.application.eager_load! if defined?(::Rails) && ::Rails.respond_to?(:application)
     end
 
 
@@ -64,9 +84,42 @@ module DiffTest
     end
 
     def save_test_execution_results_on_server
+      if ENV['DIFF_TEST_SKIP_API']
+        return
+      elsif ENV['DIFF_TEST_SKIP_API_CALL']
+        @test_executions.map(&:save_payload).to_json
+        return
+      end
+
       raise "Expected to have at least one test execution" if @test_executions.empty?
 
-      response = DiffTest::ApiClient.post("/test_suite_executions/#{id}/test_executions/bulk_create", body: { test_executions: @test_executions.map(&:save_payload) })
+      test_executions = @test_executions.map(&:save_payload)
+
+      impacted_file_hash_and_id_by_path = {}
+
+      test_executions.each do |test_execution|
+        next unless test_execution[:impacted_files]
+
+        test_execution[:impacted_files].each do |file_path, hash|
+          unless impacted_file_hash_and_id_by_path[file_path]
+            impacted_file_hash_and_id_by_path[file_path] = [hash, impacted_file_hash_and_id_by_path.size]
+          end
+        end
+
+        test_execution[:impacted_file_ids] = test_execution[:impacted_files].keys.map { |file_path| impacted_file_hash_and_id_by_path[file_path][1] }
+        test_execution[:impacted_files] = nil
+      end
+
+      impacted_file_path_and_hash_by_id = impacted_file_hash_and_id_by_path.map do |file_path, (hash, id)|
+        [id, [file_path, hash]]
+      end.to_h
+
+      DiffTest::Helper.bm('bulk create test executions') do
+        DiffTest::ApiClient.post("/test_suite_executions/#{id}/test_executions/bulk_create", body: {
+          impacted_file_path_and_hash_by_id: impacted_file_path_and_hash_by_id,
+          test_executions: test_executions
+        })
+      end
     end
 
     def payload

data/lib/diff_test/trackers/ruby_file.rb

@@ -17,7 +17,11 @@ module DiffTest
 
           unless @seen_constants.include?(constant)
             @seen_constants.add(constant)
-            constant = constant.attached_object if constant&.singleton_class?
+
+            if constant&.singleton_class?
+              constant = constant.attached_object if constant&.singleton_class?
+              next if constant.__basic_object?
+            end
 
             path = @@location_by_constant[constant]
 

data/lib/diff_test/version.rb

@@ -1,5 +1,5 @@
 # frozen_string_literal: true
 
 module DiffTest
-  VERSION = "0.8.4"
+  VERSION = "0.8.5"
 end

data/lib/diff_test.rb

@@ -31,6 +31,7 @@ module DiffTest
   def self.integrate!
     return unless configuration.integratable?
 
+    require_relative 'diff_test/core_ext'
     DiffTest::Integrations::RailsJs::Integration.instance.integrate_if_ready
     DiffTest::Integrations::ActiveRecord::Integration.instance.integrate_if_ready
     DiffTest::Integrations::Minitest::Integration.instance.integrate_if_ready

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: diff_test
 version: !ruby/object:Gem::Version
-  version: 0.8.4
+  version: 0.8.5
 platform: ruby
 authors:
 - Owais
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-04-19 00:00:00.000000000 Z
+date: 2025-04-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty
@@ -53,6 +53,7 @@ files:
 - lib/diff_test.rb
 - lib/diff_test/api_client.rb
 - lib/diff_test/configuration.rb
+- lib/diff_test/core_ext.rb
 - lib/diff_test/file_hash_computer.rb
 - lib/diff_test/helper.rb
 - lib/diff_test/impacted_file_tracker.rb