diff_test 0.8.4 → 0.8.6

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 194963612fbe79dd0506ee1d92e500fca33e8aacf6b5fe38b00cb91741a64367
-  data.tar.gz: 7a8f7a9a4e516cff513ae27247397ac40d35ed697ca5a483aed94a75d60bc668
+  metadata.gz: d912f83be5f069f6801db49f64088b2300053e75927b9b15994d3a69630954ff
+  data.tar.gz: 1b7f27298ad240ad3d4f2f9f75257de1ebadc47ffdd7da7656f72318524bd36f
 SHA512:
-  metadata.gz: 55bf9771f23900a6f3c0800e090db8dc7884acf9e0432573e25a4e4f75cb5adb20af9f750bb0c04e8a765f91503c4fa10ec559dbb34b77d2802e1a2ffea9c560
-  data.tar.gz: 6ee7fb4f979941883644d0bf27c29832b613c4e90c66f962beaffe0282b1c9d80761e73009d7da0f823f017c44d8d665ec0a9015222986da1bae53e7c019b03e
+  metadata.gz: ac0dba39aa22af19f73911ae8e70d2ba0acb17484f720d39f8f18d355ccb62cd50b4b3d077cb478119f1464eea0a2bdb69c87135d63f76dc02900b61e33ac74d
+  data.tar.gz: c50115b9407f7904675e987e930485932cc72a9da2a148361f9ae05e7c996c35171606e98b338ff4533799edfa083282b3634f8455dd9aa6246de24720e4424c

data/README.md

@@ -249,3 +249,7 @@ test/services/zoom_webhook_event_processor_test.rb
 
 /Users/owaiskhan/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/diff_test-0.8.0/lib/diff_test/test_suite_execution.rb:43: warning: mismatched indentations at 'end' with 'def' at 41
 /Users/owaiskhan/.rbenv/versions/3.3.1/lib/ruby/gems/3.3.0/gems/diff_test-0.8.0/lib/diff_test/test_suite_execution.rb:69: warning: assigned but unused variable - response
+
+
+overhead of const TracePoint and file path prefix native
+method_added/const_added

data/lib/diff_test/api_client.rb

@@ -5,30 +5,55 @@ module DiffTest
     include HTTParty
 
     def self.get(path, options = {}, &block)
-      set_api_key_in_headers(options)
+      process_options(options)
       super
     end
 
     def self.post(path, options = {}, &block)
-      set_api_key_in_headers(options)
+      process_options(options)
       super
     end
 
     def self.put(path, options = {}, &block)
-      set_api_key_in_headers(options)
+      process_options(options)
       super
     end
 
     def self.patch(path, options = {}, &block)
-      set_api_key_in_headers(options)
+      process_options(options)
       super
     end
 
     def self.delete(path, options = {}, &block)
-      set_api_key_in_headers(options)
+      process_options(options)
       super
     end
 
+    def self.process_options(options)
+      jsonify_body(options)
+      gzip_body(options)
+      set_api_key_in_headers(options)
+    end
+
+    def self.jsonify_body(options)
+      return unless options[:body] && options[:body].is_a?(Hash)
+      options[:body] = options[:body].to_json
+      options[:headers] ||= {}
+      options[:headers]['Content-Type'] = 'application/json'
+    end
+
+    def self.gzip_body(options)
+      return unless options[:body]
+
+      buffer = StringIO.new
+      gz = Zlib::GzipWriter.new(buffer)
+      gz.write(options[:body])
+      gz.close
+      options[:body] = buffer.string
+      options[:headers] ||= {}
+      options[:headers]['Content-Encoding'] = 'gzip'
+    end
+
     def self.set_api_key_in_headers(options)
       base_uri DiffTest.configuration.base_api_url
       options[:headers] ||= {}

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

@@ -6,9 +6,10 @@ module DiffTest
 
       def initialize
         super
+        do_line_check = ENV['DIFF_TEST_SKIP_LINE_CHECK'] != '1'
         @seen_constants = Set.new
         @tracepoint = TracePoint.new(:call, :b_call) do |event|
-          if DiffTest::Helper.file_in_project?(event.path)
+          if do_line_check && DiffTest::Helper.file_in_project?(event.path)
             record(DiffTest::Helper.relative_path_from_project_root(event.path))
           end
 
@@ -17,7 +18,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.6"
 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.6
 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