datadog-ci 1.39.0 → 1.41.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 18d2efca66b50be67262fdc2ea7adda5a8c0c90a05b6753c441a9dd229372dfe
-  data.tar.gz: 80cb89011890b2617933cac62889e6b18979e8a97a34cfb5298aea43b1afa34b
+  metadata.gz: de15dc03a13226b87763dc2942794ffd12ec4f5b9b19f0430d93b03216f20da6
+  data.tar.gz: 2eaab3c59263e48c75dcdd237aba4db7bdac0e8578069b4e628866cdcd155a34
 SHA512:
-  metadata.gz: d39ff56e9a2fb16615bf40c58621b6c3e9a7197c6c4e63f9568866afe162a4d9bd65778243b41b5e0a3d3127b58bc91dd5bcb26205c215e95c56f4c9c3e4f446
-  data.tar.gz: c3220f31a0f5aabbca8eb905800ed902ae68726567800cf40de2cee33651e7f58c1e72763239a498cc3d44270a13d81698a7172d2e4b3f602d36c6119adf08bb
+  metadata.gz: 25e04a8a9fc1f6f904aa3922a21154aa6f1212a39605fae8f21a18ec43f4541c9399fa6405d29f65896e9ffd9a3dd50cc4dc3cccf5662f1ab7ecd075b67e86cc
+  data.tar.gz: 961c8f33c689df07500a4bb47a5a76aebf5885c4d7fb5752536977af4def6f1a82be2329a6f0570ef96f89670d54f218a6ffa184bca83e0fefc47230dfd68ab2

data/CHANGELOG.md

@@ -1,5 +1,19 @@
 ## [Unreleased]
 
+## [1.41.0] - 2026-10-05
+
+### Fixed
+
+* Stop retries when no test session is active ([#645][])
+* Fix Minitest sessions with no executed tests ([#636][])
+
+## [1.40.0] - 2026-09-28
+
+### Changed
+
+* Knapsack Pro: mark successful empty sessions as skipped ([#634][])
+* RSpec: mark successful zero-example sessions as skipped ([#633][])
+
 ## [1.39.0] - 2026-09-21
 
 ### Added
@@ -723,7 +737,9 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 
 - Ruby versions < 2.7 no longer supported ([#8][])
 
-[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.39.0...main
+[Unreleased]: https://github.com/DataDog/datadog-ci-rb/compare/v1.41.0...main
+[1.41.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.40.0...v1.41.0
+[1.40.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.39.0...v1.40.0
 [1.39.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.38.1...v1.39.0
 [1.38.1]: https://github.com/DataDog/datadog-ci-rb/compare/v1.38.0...v1.38.1
 [1.38.0]: https://github.com/DataDog/datadog-ci-rb/compare/v1.37.0...v1.38.0
@@ -1022,4 +1038,8 @@ Currently test suite level visibility is not used by our instrumentation: it wil
 [#612]: https://github.com/DataDog/datadog-ci-rb/issues/612
 [#615]: https://github.com/DataDog/datadog-ci-rb/issues/615
 [#617]: https://github.com/DataDog/datadog-ci-rb/issues/617
-[#619]: https://github.com/DataDog/datadog-ci-rb/issues/619
+[#619]: https://github.com/DataDog/datadog-ci-rb/issues/619
+[#633]: https://github.com/DataDog/datadog-ci-rb/issues/633
+[#634]: https://github.com/DataDog/datadog-ci-rb/issues/634
+[#636]: https://github.com/DataDog/datadog-ci-rb/issues/636
+[#645]: https://github.com/DataDog/datadog-ci-rb/issues/645

data/lib/datadog/ci/contrib/knapsack/runner.rb

@@ -38,7 +38,16 @@ module Datadog
                 result = super
               ensure
                 [test_module, test_session].each do |span|
-                  _dd_cleanup { (result == 0) ? span.passed! : span.failed! }
+                  _dd_cleanup do
+                    if result != 0
+                      span.failed!
+                    elsif !test_tracing_component.any_tests_started?
+                      span.skipped!(reason: "No tests were executed")
+                      span.set_tag(CI::Ext::Test::TAG_SESSION_EMPTY_REASON, "zero_tests")
+                    else
+                      span.passed!
+                    end
+                  end
                 end
                 _dd_cleanup { test_module.finish }
                 _dd_cleanup { test_session.finish }

data/lib/datadog/ci/contrib/minitest/reporter.rb

@@ -23,12 +23,17 @@ module Datadog
 
               return res if active_test_session.nil? || active_test_module.nil?
 
-              if passed?
-                active_test_module.passed!
-                active_test_session.passed!
-              else
+              if !passed?
                 active_test_module.failed!
                 active_test_session.failed!
+              elsif !test_tracing_component.any_tests_started?
+                active_test_module.skipped!(reason: "No tests were executed")
+                active_test_session.skipped!(reason: "No tests were executed")
+                active_test_module.set_tag(CI::Ext::Test::TAG_SESSION_EMPTY_REASON, "zero_tests")
+                active_test_session.set_tag(CI::Ext::Test::TAG_SESSION_EMPTY_REASON, "zero_tests")
+              else
+                active_test_module.passed!
+                active_test_session.passed!
               end
 
               active_test_module.finish
@@ -39,6 +44,10 @@ module Datadog
 
             private
 
+            def test_tracing_component
+              Datadog.send(:components).test_tracing
+            end
+
             def datadog_configuration
               Datadog.configuration.ci[:minitest]
             end

data/lib/datadog/ci/contrib/rspec/runner.rb

@@ -48,6 +48,11 @@ module Datadog
               if result != 0
                 test_module.failed!
                 test_session.failed!
+              elsif !test_tracing_component.any_tests_started?
+                test_module.skipped!(reason: "No tests were executed")
+                test_session.skipped!(reason: "No tests were executed")
+                test_module.set_tag(CI::Ext::Test::TAG_SESSION_EMPTY_REASON, "zero_tests")
+                test_session.set_tag(CI::Ext::Test::TAG_SESSION_EMPTY_REASON, "zero_tests")
               else
                 test_module.passed!
                 test_session.passed!

data/lib/datadog/ci/ext/test.rb

@@ -13,6 +13,7 @@ module Datadog
         TAG_FRAMEWORK_VERSION = "test.framework_version"
         TAG_NAME = "test.name"
         TAG_SKIP_REASON = "test.skip_reason"
+        TAG_SESSION_EMPTY_REASON = "test.session.empty_reason"
 
         # Status is the result of a single test run
         # See the [Datadog::CI::Ext::Test::Status] module for the list of possible values of this tag

data/lib/datadog/ci/test_retries/component.rb

@@ -147,11 +147,17 @@ module Datadog
         end
 
         def should_retry?
+          return false unless test_tracing.active_test_session
+
           !!current_retry_driver&.should_retry?
         end
 
         private
 
+        def test_tracing
+          Datadog.send(:components).test_tracing
+        end
+
         def current_retry_driver
           Thread.current[FIBER_LOCAL_CURRENT_RETRY_DRIVER_KEY]
         end

data/lib/datadog/ci/test_tracing/component.rb

@@ -162,8 +162,16 @@ module Datadog
           @context.active_test
         end
 
+        # Includes skipped tests: a session is empty only if no tests were observed.
+        def any_tests_started?
+          maybe_remote_context.any_tests_started?
+        end
+
         def active_test_session
           maybe_remote_context.active_test_session
+        rescue => e
+          Datadog.logger.error("Failed to get active test session: #{e.class}: #{e.message}")
+          nil
         end
 
         def active_test_module
@@ -286,6 +294,8 @@ module Datadog
         end
 
         def on_test_started(test)
+          maybe_remote_context.record_test_started
+
           # Sometimes test suite is not being assigned correctly.
           # Fix it by fetching the one single running test suite from the process context.
           #

data/lib/datadog/ci/test_tracing/context.rb

@@ -40,10 +40,12 @@ module Datadog
           @mutex = Mutex.new
 
           @tests_skipped_by_tia_count = 0
+          @any_tests_started = false
         end
 
         def start_test_session(service: nil, tags: {})
           @process_context.fetch_or_activate_test_session do
+            @mutex.synchronize { @any_tests_started = false }
             tracer_span = start_datadog_tracer_span(
               "test.session", build_tracing_span_options(service, Ext::AppTypes::TYPE_TEST_SESSION)
             )
@@ -191,6 +193,14 @@ module Datadog
           @process_context.deactivate_test_suite!(test_suite_name)
         end
 
+        def any_tests_started?
+          @mutex.synchronize { @any_tests_started }
+        end
+
+        def record_test_started
+          @mutex.synchronize { @any_tests_started = true }
+        end
+
         def incr_tests_skipped_by_tia_count
           @mutex.synchronize { @tests_skipped_by_tia_count += 1 }
         end

data/lib/datadog/ci/test_tracing/null_component.rb

@@ -47,6 +47,10 @@ module Datadog
         def active_test
         end
 
+        def any_tests_started?
+          false
+        end
+
         def active_test_session
         end
 

data/lib/datadog/ci/version.rb

@@ -4,7 +4,7 @@ module Datadog
   module CI
     module VERSION
       MAJOR = 1
-      MINOR = 39
+      MINOR = 41
       PATCH = 0
       PRE = nil
       BUILD = nil

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: datadog-ci
 version: !ruby/object:Gem::Version
-  version: 1.39.0
+  version: 1.41.0
 platform: ruby
 authors:
 - Datadog, Inc.
@@ -327,6 +327,7 @@ files:
 homepage: https://github.com/DataDog/datadog-ci-rb
 licenses:
 - BSD-3-Clause
+- Apache-2.0
 metadata:
   allowed_push_host: https://rubygems.org
   changelog_uri: https://github.com/DataDog/datadog-ci-rb/blob/main/CHANGELOG.md