diff_test 0.8.9 → 0.8.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0eb6d31f88ec71845c1da318d944b46bcb0d7c6b550b0bb9c936a4236cd4e802
-  data.tar.gz: 879e45fa217f327c095c4f1bccac3d7308df661a3735ba3a549cdc8550c4360c
+  metadata.gz: d36a2feba28342c37ee92d44d615a9466f17824cffc14992c4031901d3035c2b
+  data.tar.gz: 75996211c6e3f6ba9942c201e3c12ec5b2c0031ebbb4052e11a82ebf3b7a2307
 SHA512:
-  metadata.gz: d935e9d4ebce09151f96d998b9aa7cfcd4adc3f8a5e45622b29e1e57a093184b39e98321392eab581b37c2a7b51f8927bbb99fb4df447896ced3f3e409868b3f
-  data.tar.gz: cf82343110711249ec65a56486a559ae899614e886f49faceea1c35020abc58758f438a87969db609f866171abe5228421e7e23f12fcd9170689cbe1c8e40580
+  metadata.gz: 54fed90750ebe5ab28d7cf81568401f5db7db5b9fd341538eef0f23403255cf1891c3a3795696d207c59b15fbb5f6b965063a8c5a5d968e54722e36b48b66fba
+  data.tar.gz: bfd6d8d252b80a9769465e6879c36d91613c3998fa9f0f71d0fbfcde5c79a83e9da9fb40a392a74bf572eaf0fe1b6f3d73a39ab9c7ffc7dcd196b9cce95dc60c

data/README.md

@@ -253,3 +253,6 @@ test/services/zoom_webhook_event_processor_test.rb
 
 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/helper.rb

@@ -74,5 +74,9 @@ module DiffTest
       puts "Finished #{name.join(' ')} in #{(Time.now - start).round(2)}s------------"
       result
     end
+
+    def log(message)
+      puts "DiffTest #{Time.current.strftime('%H:%M:%S')}: #{message}"
+    end
   end
 end

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

@@ -4,6 +4,16 @@ module DiffTest
       module Lifecycle
         def self.included(base)
           base.prepend(InstanceMethods)
+
+          require 'get_process_mem'
+          Thread.new do
+            mem = GetProcessMem.new
+            loop do
+              # `mem.mb` returns the resident set size as a Float in megabytes
+              DiffTest::Helper.log("Memory: #{format('%8.2f MB', mem.mb)}")
+              sleep 1
+            end
+          end
         end
 
         module InstanceMethods
@@ -55,7 +65,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 +77,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/integrations/rails_js/annotator.rb

@@ -34,7 +34,7 @@ module DiffTest
           insert_at_first_non_comment_line_str(content, annotation)
         end
 
-        UNANNOTATE_REGEX = /(\/\/|#) Automatically added by DiffTest.*?(\/\/|#) Automatically added by DiffTest\n/m
+        UNANNOTATE_REGEX = /(\/\/|#) Automatically added by DiffTest.*?(\/\/|#) Automatically added by DiffTest\n\n?/m
         def self.unannotate(content)
           content.gsub!(UNANNOTATE_REGEX, '')
           content

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
 
@@ -32,6 +39,7 @@ module DiffTest
     end
 
     def start
+      DiffTest::Helper.log("TestExecution.start: #{id}")
       @impacted_file_tracker.start
       @started_at = Process.clock_gettime(Process::CLOCK_MONOTONIC)
     end
@@ -39,6 +47,7 @@ module DiffTest
     def stop
       @impacted_file_tracker.stop
       @stopped_at = Process.clock_gettime(Process::CLOCK_MONOTONIC)
+      DiffTest::Helper.log("TestExecution.stop: #{id}")
       finish
     end
 
@@ -66,6 +75,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/version.rb

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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: diff_test
 version: !ruby/object:Gem::Version
-  version: 0.8.9
+  version: 0.8.11
 platform: ruby
 authors:
 - Owais
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-04-28 00:00:00.000000000 Z
+date: 2025-05-10 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: httparty