[v3] Change manual instrumentation approach to not use reverse duck-typing

tracer/src/Datadog.Trace.Manual/Ci/ITest.cs

@@ -69,27 +69,6 @@ public interface ITest
     /// <param name="traits">Traits dictionary</param>
     void SetTraits(Dictionary<string, List<string>> traits);
 
-    /// <summary>
-    /// Set Test parameters
-    /// </summary>
-    /// <param name="parameters">TestParameters instance</param>
-    void SetParameters(TestParameters parameters);
-
-    /// <summary>
-    /// Set benchmark metadata
-    /// </summary>
-    /// <param name="hostInfo">Host info</param>
-    /// <param name="jobInfo">Job info</param>
-    void SetBenchmarkMetadata(BenchmarkHostInfo hostInfo, BenchmarkJobInfo jobInfo);
-
-    /// <summary>
-    /// Add benchmark data
-    /// </summary>
-    /// <param name="measureType">Measure type</param>
-    /// <param name="info">Measure info</param>
-    /// <param name="statistics">Statistics values</param>
-    void AddBenchmarkData(BenchmarkMeasureType measureType, string info, BenchmarkDiscreteStats statistics);
-
     /// <summary>
     /// Close test
     /// </summary>

tracer/src/Datadog.Trace.Manual/Ci/Stubs/NullTest.cs

@@ -9,13 +9,19 @@
 
 namespace Datadog.Trace.Ci.Stubs;
 
-internal class NullTest(ITestSuite testSuite, string name, DateTimeOffset? startDate) : ITest
+internal class NullTest : ITest
 {
-    public string? Name { get; } = name;
+    public static readonly NullTest Instance = new();
 
-    public DateTimeOffset StartTime { get; } = startDate ?? DateTimeOffset.UtcNow;
+    private NullTest()
+    {
+    }
+
+    public string? Name => "Undefined";
+
+    public DateTimeOffset StartTime => default;
 
-    public ITestSuite Suite { get; } = testSuite;
+    public ITestSuite Suite => NullTestSuite.Instance;
 
     public void SetTag(string key, string? value)
     {
@@ -41,18 +47,6 @@ public void SetTraits(Dictionary<string, List<string>> traits)
     {
     }
 
-    public void SetParameters(TestParameters parameters)
-    {
-    }
-
-    public void SetBenchmarkMetadata(BenchmarkHostInfo hostInfo, BenchmarkJobInfo jobInfo)
-    {
-    }
-
-    public void AddBenchmarkData(BenchmarkMeasureType measureType, string info, BenchmarkDiscreteStats statistics)
-    {
-    }
-
     public void Close(TestStatus status)
     {
     }

tracer/src/Datadog.Trace.Manual/Ci/Stubs/NullTestModule.cs

@@ -5,13 +5,19 @@
 
 namespace Datadog.Trace.Ci.Stubs;
 
-internal class NullTestModule(string name, string? framework, DateTimeOffset? startDate) : ITestModule
+internal class NullTestModule : ITestModule
 {
-    public string Name { get; } = name;
+    public static readonly NullTestModule Instance = new();
 
-    public DateTimeOffset StartTime { get; } = startDate ?? DateTimeOffset.UtcNow;
+    private NullTestModule()
+    {
+    }
+
+    public string Name => "Undefined";
+
+    public DateTimeOffset StartTime => default;
 
-    public string? Framework { get; } = framework;
+    public string? Framework => null;
 
     public void SetTag(string key, string? value)
     {
@@ -41,9 +47,7 @@ public void Close(TimeSpan? duration)
 
     public Task CloseAsync(TimeSpan? duration) => Task.CompletedTask;
 
-    public ITestSuite GetOrCreateSuite(string name)
-        => new NullTestSuite(this, name, null);
+    public ITestSuite GetOrCreateSuite(string name) => NullTestSuite.Instance;
 
-    public ITestSuite GetOrCreateSuite(string name, DateTimeOffset? startDate)
-        => new NullTestSuite(this, name, startDate);
+    public ITestSuite GetOrCreateSuite(string name, DateTimeOffset? startDate) => NullTestSuite.Instance;
 }