diff_test 0.8.5 → 0.8.7

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 71eb34b2f03e6cfb5c88f0382645ab9991273650ab54bf33ca985dd3d244b631
-  data.tar.gz: 140bc0788f6bb66aa6bf75d1d55934feeba868006593ff4b134d75633900a376
+  metadata.gz: 30a81175a332e93a6bb3791a2457c3f8afd04217a8f48ccb1d1abf48306407b7
+  data.tar.gz: '0891219e6104458312cdc2fbf164218318271349fcc8e1c2b42ade2b55546451'
 SHA512:
-  metadata.gz: 0160b17c695801c363ab06aa98844ac586238df47db23f4fc25a628f49081e2fa76b7abafb49dacf447ad2db3960418d6dd5827442ee4da60da700c76bf77eca
-  data.tar.gz: 707a6088a90723c80715002f2d27cbc87f5833398c4d16b695213efb3af9feaafc8166f68d6cf867d94286e3dacb6d6330701fe8973b5383ee333c053aab0bdd
+  metadata.gz: 3bdc4073b05ac2260f8875e3fb309d27d306507598d1f4d85f5941515ad5878fd2101cfe166fa2ffd3579794180de1bfe5e63b080cc7edf0592202788a0801fe
+  data.tar.gz: aab168ef2a5be65d5f7fb551cc0a7f1e9ac3513539d8b6b6f78e82dbfe0c87d9544bb1a2687a741f459345fff9751d2ae618de1cf58c4cac0904b53346470701

data/README.md

@@ -249,3 +249,10 @@ 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
+
+
+# WRITE test for it setting time from previous successful run

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/integrations/minitest/lifecycle.rb

@@ -55,7 +55,7 @@ module DiffTest
               test_name = self.name
 
               DiffTest::TestExecution.current = execution =  DiffTest::TestExecution.new(test_file_path:, test_name:)
-              should_run = DiffTest::ShouldRunDecider.should_run?(execution.id, system_test: true)
+              should_run = execution.should_run?
               DiffTest::TestSuiteExecution.current.ensure_application_eager_loaded! if should_run
             rescue => e
               DiffTest.error("An error ocurred while trying to create a test execution. Please report this issue.\nException: #{e}")
@@ -67,11 +67,10 @@ module DiffTest
               super
             else
               # Skip raises an exception too, which is handled by `capture_exception`
-              time_it do
-                capture_exceptions do
-                  skip("Impacted files have not changed. Skipping...")
-                end
+              capture_exceptions do
+                skip("Impacted files have not changed. Skipping...")
               end
+              self.time = execution.previous_runtime_s
               ::Minitest::Result.from(self)
             end
           ensure

data/lib/diff_test/should_run_decider.rb

@@ -8,9 +8,10 @@ module DiffTest
     end
 
     def should_run?(test_id, system_test:)
-      return true unless @test_executions_by_test
+      return { should_run: true } unless @test_executions_by_test
 
       previous_test_executions = @test_executions_by_test[test_id] || []
+      previous_runtime_ms = nil
 
       # If there are any previous test executions run whose impacted files are the same as now
       #  Assume its safe to skip
@@ -23,6 +24,8 @@ module DiffTest
           DiffTest::FileHashComputer.compute_relative(impacted_file_path) == file_hash
         end
         next false unless hashes_match
+
+        previous_runtime_ms = previous_test_execution['runtime_ms']
         next true unless system_test
 
         test_suite_execution = @test_suite_executions_by_id[previous_test_execution['test_suite_execution_id'].to_s]
@@ -32,8 +35,13 @@ module DiffTest
         false
       end
 
-      return true if ENV['DIFF_TEST_ALWAYS_RUN'] == '1'
-      !has_previous_execution_with_no_impacted_changes
+      return { should_run: true } if ENV['DIFF_TEST_ALWAYS_RUN'] == '1'
+
+      should_run =  !has_previous_execution_with_no_impacted_changes
+      {
+        should_run:,
+        previous_runtime_ms:,
+      }
     end
 
     def self.current

data/lib/diff_test/test_execution.rb

@@ -13,9 +13,16 @@ module DiffTest
       self.class.set(id, self)
     end
 
+    def should_run_decider_result
+      @should_run_decider_result ||= DiffTest::ShouldRunDecider.should_run?(id, system_test: system_test?)
+    end
+
     def should_run?
-      return @should_run unless @should_run.nil?
-      @should_run = DiffTest::ShouldRunDecider.should_run?(id, system_test: system_test?)
+      should_run_decider_result[:should_run]
+    end
+
+    def previous_runtime_s
+      (should_run_decider_result[:previous_runtime_ms] / 1000.0) if should_run_decider_result[:previous_runtime_ms]
     end
 
     def save_payload
@@ -23,7 +30,7 @@ module DiffTest
         test: id,
         result: @result,
         runtime_ms: runtime_ms,
-        impacted_files: (@impacted_file_tracker.save_payload if passed?),
+        impacted_files: (@impacted_file_tracker.save_payload if successful?),
       }.compact
     end
 
@@ -66,6 +73,14 @@ module DiffTest
       @result == :passed
     end
 
+    def skipped?
+      @result == :skipped
+    end
+
+    def successful?
+      passed? || skipped?
+    end
+
     def runtime_ms
       return 0 if @stopped_at.nil? || @started_at.nil?
       (@stopped_at - @started_at) * 1000

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
 

data/lib/diff_test/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: diff_test
 version: !ruby/object:Gem::Version
-  version: 0.8.5
+  version: 0.8.7
 platform: ruby
 authors:
 - Owais
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-04-20 00:00:00.000000000 Z
+date: 2025-04-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty