remove inheritance from NullRecorder to make sure that there are no s…

…ide effects in the future

lib/datadog/ci/test_visibility/null_recorder.rb

@@ -6,11 +6,7 @@ module Datadog
   module CI
     module TestVisibility
       # Special recorder that does not record anything
-      #
-      class NullRecorder < Recorder
-        def initialize(*)
-        end
-
+      class NullRecorder
         def start_test_session(service: nil, tags: {})
           skip_tracing
         end
@@ -57,6 +53,20 @@ def deactivate_test_module
 
         def deactivate_test_suite(test_suite_name)
         end
+
+        private
+
+        def skip_tracing(block = nil)
+          if block
+            block.call(null_span)
+          else
+            null_span
+          end
+        end
+
+        def null_span
+          @null_span ||= NullSpan.new
+        end
       end
     end
   end

sig/datadog/ci/test_visibility/null_recorder.rbs

@@ -1,7 +1,9 @@
 module Datadog
   module CI
     module TestVisibility
-      class NullRecorder < Recorder
+      class NullRecorder
+        @null_span: Datadog::CI::NullSpan
+
         def initialize: (?untyped args) -> void
 
         def trace_test: (String span_name, String test_suite_name, ?service: String?, ?tags: Hash[untyped, untyped]) ?{ (Datadog::CI::Span span) -> untyped } -> untyped
@@ -31,6 +33,12 @@ module Datadog
         def deactivate_test_module: () -> void
 
         def deactivate_test_suite: (String test_suite_name) -> void
+
+        private
+
+        def null_span: () -> Datadog::CI::Span
+
+        def skip_tracing: (?untyped block) -> untyped
       end
     end
   end