Skip to content

Conversation

@duncanpharvey
Copy link
Contributor

Summary of changes

Update Datadog.Serverless.Compat to v1.2.0 for Datadog.AzureFunctions.

Reason for change

New Serverless Compatibility Layer package available.

Implementation details

Test coverage

Other details

@dd-trace-dotnet-ci-bot
Copy link

Execution-Time Benchmarks Report ⏱️

Execution-time results for samples comparing This PR (8108) and master.

✅ No regressions detected - check the details below

Full Metrics Comparison

FakeDbCommand

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration68.23 ± (68.26 - 68.58) ms68.55 ± (68.55 - 68.80) ms+0.5%✅⬆️
.NET Framework 4.8 - Bailout
duration72.14 ± (72.05 - 72.29) ms72.43 ± (72.32 - 72.61) ms+0.4%✅⬆️
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1015.80 ± (1017.16 - 1023.60) ms1020.41 ± (1028.37 - 1039.32) ms+0.5%✅⬆️
.NET Core 3.1 - Baseline
process.internal_duration_ms21.78 ± (21.75 - 21.81) ms21.90 ± (21.87 - 21.93) ms+0.6%✅⬆️
process.time_to_main_ms78.80 ± (78.62 - 78.98) ms78.95 ± (78.81 - 79.10) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed10.90 ± (10.89 - 10.90) MB10.93 ± (10.92 - 10.93) MB+0.3%✅⬆️
runtime.dotnet.threads.count12 ± (12 - 12)12 ± (12 - 12)+0.0%
.NET Core 3.1 - Bailout
process.internal_duration_ms21.78 ± (21.76 - 21.81) ms21.71 ± (21.69 - 21.73) ms-0.3%
process.time_to_main_ms79.79 ± (79.70 - 79.87) ms80.25 ± (80.15 - 80.35) ms+0.6%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed10.93 ± (10.93 - 10.93) MB10.96 ± (10.96 - 10.97) MB+0.3%✅⬆️
runtime.dotnet.threads.count13 ± (13 - 13)13 ± (13 - 13)+0.0%
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms239.21 ± (235.54 - 242.87) ms252.37 ± (249.47 - 255.27) ms+5.5%✅⬆️
process.time_to_main_ms474.58 ± (474.10 - 475.07) ms476.75 ± (476.26 - 477.25) ms+0.5%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed48.58 ± (48.56 - 48.60) MB48.68 ± (48.65 - 48.71) MB+0.2%✅⬆️
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)+0.6%✅⬆️
.NET 6 - Baseline
process.internal_duration_ms20.65 ± (20.62 - 20.67) ms20.62 ± (20.60 - 20.64) ms-0.1%
process.time_to_main_ms68.36 ± (68.22 - 68.50) ms68.57 ± (68.44 - 68.70) ms+0.3%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed10.61 ± (10.60 - 10.61) MB10.63 ± (10.63 - 10.64) MB+0.3%✅⬆️
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%
.NET 6 - Bailout
process.internal_duration_ms20.58 ± (20.55 - 20.60) ms20.47 ± (20.45 - 20.48) ms-0.5%
process.time_to_main_ms69.19 ± (69.12 - 69.25) ms69.20 ± (69.13 - 69.27) ms+0.0%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed10.66 ± (10.65 - 10.66) MB10.73 ± (10.73 - 10.74) MB+0.7%✅⬆️
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms250.11 ± (248.99 - 251.23) ms244.59 ± (242.22 - 246.96) ms-2.2%
process.time_to_main_ms452.99 ± (452.48 - 453.49) ms451.34 ± (450.95 - 451.73) ms-0.4%
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed49.39 ± (49.36 - 49.42) MB49.35 ± (49.33 - 49.38) MB-0.1%
runtime.dotnet.threads.count28 ± (28 - 28)28 ± (28 - 28)-0.1%
.NET 8 - Baseline
process.internal_duration_ms18.72 ± (18.70 - 18.75) ms18.83 ± (18.80 - 18.86) ms+0.6%✅⬆️
process.time_to_main_ms67.35 ± (67.25 - 67.44) ms67.45 ± (67.35 - 67.55) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed7.67 ± (7.67 - 7.68) MB7.68 ± (7.68 - 7.69) MB+0.2%✅⬆️
runtime.dotnet.threads.count10 ± (10 - 10)10 ± (10 - 10)+0.0%
.NET 8 - Bailout
process.internal_duration_ms18.79 ± (18.76 - 18.81) ms18.78 ± (18.75 - 18.80) ms-0.0%
process.time_to_main_ms68.42 ± (68.36 - 68.48) ms68.52 ± (68.47 - 68.57) ms+0.1%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed7.76 ± (7.75 - 7.77) MB7.74 ± (7.73 - 7.75) MB-0.2%
runtime.dotnet.threads.count11 ± (11 - 11)11 ± (11 - 11)+0.0%
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms177.17 ± (176.30 - 178.04) ms177.77 ± (176.87 - 178.66) ms+0.3%✅⬆️
process.time_to_main_ms434.46 ± (433.89 - 435.02) ms435.34 ± (434.65 - 436.03) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count0 ± (0 - 0)0 ± (0 - 0)+0.0%
runtime.dotnet.mem.committed36.65 ± (36.62 - 36.67) MB36.71 ± (36.68 - 36.74) MB+0.2%✅⬆️
runtime.dotnet.threads.count26 ± (26 - 26)27 ± (27 - 27)+0.5%✅⬆️

HttpMessageHandler

Metric Master (Mean ± 95% CI) Current (Mean ± 95% CI) Change Status
.NET Framework 4.8 - Baseline
duration195.34 ± (195.95 - 196.95) ms198.18 ± (197.90 - 198.83) ms+1.5%✅⬆️
.NET Framework 4.8 - Bailout
duration203.12 ± (202.51 - 203.97) ms199.69 ± (199.41 - 200.33) ms-1.7%
.NET Framework 4.8 - CallTarget+Inlining+NGEN
duration1153.57 ± (1155.32 - 1165.18) ms1151.55 ± (1156.51 - 1166.29) ms-0.2%
.NET Core 3.1 - Baseline
process.internal_duration_ms188.70 ± (188.37 - 189.02) ms188.73 ± (188.44 - 189.02) ms+0.0%✅⬆️
process.time_to_main_ms81.80 ± (81.55 - 82.05) ms81.63 ± (81.43 - 81.83) ms-0.2%
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%
runtime.dotnet.mem.committed16.11 ± (16.08 - 16.13) MB16.11 ± (16.08 - 16.13) MB-0.0%
runtime.dotnet.threads.count20 ± (19 - 20)20 ± (20 - 20)+0.2%✅⬆️
.NET Core 3.1 - Bailout
process.internal_duration_ms187.91 ± (187.59 - 188.22) ms188.55 ± (188.14 - 188.97) ms+0.3%✅⬆️
process.time_to_main_ms82.62 ± (82.44 - 82.79) ms83.17 ± (82.96 - 83.39) ms+0.7%✅⬆️
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%
runtime.dotnet.mem.committed16.17 ± (16.15 - 16.20) MB16.13 ± (16.10 - 16.16) MB-0.3%
runtime.dotnet.threads.count21 ± (20 - 21)21 ± (20 - 21)+0.2%✅⬆️
.NET Core 3.1 - CallTarget+Inlining+NGEN
process.internal_duration_ms422.67 ± (419.31 - 426.03) ms429.79 ± (426.96 - 432.63) ms+1.7%✅⬆️
process.time_to_main_ms480.43 ± (479.68 - 481.18) ms482.17 ± (481.48 - 482.86) ms+0.4%✅⬆️
runtime.dotnet.exceptions.count3 ± (3 - 3)3 ± (3 - 3)+0.0%
runtime.dotnet.mem.committed59.12 ± (59.00 - 59.24) MB58.98 ± (58.87 - 59.09) MB-0.2%
runtime.dotnet.threads.count29 ± (29 - 29)29 ± (29 - 29)-0.1%
.NET 6 - Baseline
process.internal_duration_ms192.98 ± (192.62 - 193.34) ms193.76 ± (193.36 - 194.17) ms+0.4%✅⬆️
process.time_to_main_ms70.83 ± (70.60 - 71.06) ms71.34 ± (71.16 - 71.53) ms+0.7%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed16.04 ± (15.90 - 16.19) MB16.37 ± (16.31 - 16.44) MB+2.1%✅⬆️
runtime.dotnet.threads.count19 ± (18 - 19)19 ± (19 - 19)+1.0%✅⬆️
.NET 6 - Bailout
process.internal_duration_ms193.02 ± (192.60 - 193.44) ms193.54 ± (193.13 - 193.95) ms+0.3%✅⬆️
process.time_to_main_ms71.91 ± (71.76 - 72.06) ms72.27 ± (72.10 - 72.44) ms+0.5%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed16.13 ± (16.00 - 16.27) MB16.38 ± (16.31 - 16.44) MB+1.5%✅⬆️
runtime.dotnet.threads.count20 ± (19 - 20)20 ± (20 - 20)+0.9%✅⬆️
.NET 6 - CallTarget+Inlining+NGEN
process.internal_duration_ms443.41 ± (439.98 - 446.83) ms443.47 ± (439.66 - 447.27) ms+0.0%✅⬆️
process.time_to_main_ms460.59 ± (459.90 - 461.28) ms460.41 ± (459.81 - 461.00) ms-0.0%
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed59.13 ± (59.00 - 59.26) MB58.81 ± (58.67 - 58.94) MB-0.5%
runtime.dotnet.threads.count29 ± (29 - 29)29 ± (29 - 29)+0.0%✅⬆️
.NET 8 - Baseline
process.internal_duration_ms192.52 ± (192.11 - 192.93) ms192.20 ± (191.90 - 192.51) ms-0.2%
process.time_to_main_ms70.57 ± (70.37 - 70.77) ms71.07 ± (70.83 - 71.30) ms+0.7%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed11.77 ± (11.74 - 11.79) MB11.73 ± (11.70 - 11.75) MB-0.3%
runtime.dotnet.threads.count18 ± (18 - 18)18 ± (18 - 18)-0.4%
.NET 8 - Bailout
process.internal_duration_ms192.55 ± (192.07 - 193.02) ms191.47 ± (191.06 - 191.88) ms-0.6%
process.time_to_main_ms71.90 ± (71.72 - 72.08) ms72.07 ± (71.93 - 72.21) ms+0.2%✅⬆️
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed11.76 ± (11.74 - 11.78) MB11.79 ± (11.76 - 11.83) MB+0.3%✅⬆️
runtime.dotnet.threads.count19 ± (19 - 19)19 ± (19 - 19)-0.0%
.NET 8 - CallTarget+Inlining+NGEN
process.internal_duration_ms371.25 ± (369.87 - 372.62) ms366.93 ± (365.30 - 368.55) ms-1.2%
process.time_to_main_ms446.99 ± (446.05 - 447.93) ms445.21 ± (444.32 - 446.10) ms-0.4%
runtime.dotnet.exceptions.count4 ± (4 - 4)4 ± (4 - 4)+0.0%
runtime.dotnet.mem.committed48.45 ± (48.42 - 48.48) MB48.34 ± (48.30 - 48.37) MB-0.2%
runtime.dotnet.threads.count29 ± (29 - 29)29 ± (28 - 29)-0.6%
Comparison explanation

Execution-time benchmarks measure the whole time it takes to execute a program, and are intended to measure the one-off costs. Cases where the execution time results for the PR are worse than latest master results are highlighted in **red**. The following thresholds were used for comparing the execution times:

  • Welch test with statistical test for significance of 5%
  • Only results indicating a difference greater than 5% and 5 ms are considered.

Note that these results are based on a single point-in-time result for each branch. For full results, see the dashboard.

Graphs show the p99 interval based on the mean and StdDev of the test run, as well as the mean value of the run (shown as a diamond below the graph).

Duration charts
FakeDbCommand (.NET Framework 4.8)
gantt
    title Execution time (ms) FakeDbCommand (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (69ms)  : 67, 70
    master - mean (68ms)  : 66, 70

    section Bailout
    This PR (8108) - mean (72ms)  : 71, 74
    master - mean (72ms)  : 71, 73

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (1,034ms)  : 954, 1114
    master - mean (1,020ms)  : 975, 1066

Loading
FakeDbCommand (.NET Core 3.1)
gantt
    title Execution time (ms) FakeDbCommand (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (106ms)  : 103, 109
    master - mean (106ms)  : 104, 108

    section Bailout
    This PR (8108) - mean (107ms)  : 106, 108
    master - mean (107ms)  : 106, 108

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (754ms)  : 710, 799
    master - mean (741ms)  : 684, 797

Loading
FakeDbCommand (.NET 6)
gantt
    title Execution time (ms) FakeDbCommand (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (94ms)  : 92, 96
    master - mean (94ms)  : 92, 96

    section Bailout
    This PR (8108) - mean (94ms)  : 93, 96
    master - mean (94ms)  : 94, 95

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (721ms)  : 683, 758
    master - mean (728ms)  : 700, 756

Loading
FakeDbCommand (.NET 8)
gantt
    title Execution time (ms) FakeDbCommand (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (92ms)  : 90, 94
    master - mean (92ms)  : 90, 94

    section Bailout
    This PR (8108) - mean (93ms)  : 92, 94
    master - mean (93ms)  : 92, 94

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (641ms)  : 629, 652
    master - mean (640ms)  : 625, 655

Loading
HttpMessageHandler (.NET Framework 4.8)
gantt
    title Execution time (ms) HttpMessageHandler (.NET Framework 4.8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (198ms)  : 192, 205
    master - mean (196ms)  : 189, 204

    section Bailout
    This PR (8108) - mean (200ms)  : 195, 204
    master - mean (203ms)  : 193, 213

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (1,161ms)  : 1089, 1234
    master - mean (1,160ms)  : 1086, 1234

Loading
HttpMessageHandler (.NET Core 3.1)
gantt
    title Execution time (ms) HttpMessageHandler (.NET Core 3.1)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (279ms)  : 273, 285
    master - mean (279ms)  : 273, 284

    section Bailout
    This PR (8108) - mean (280ms)  : 275, 286
    master - mean (279ms)  : 275, 283

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (941ms)  : 892, 991
    master - mean (933ms)  : 878, 989

Loading
HttpMessageHandler (.NET 6)
gantt
    title Execution time (ms) HttpMessageHandler (.NET 6)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (273ms)  : 268, 279
    master - mean (272ms)  : 266, 279

    section Bailout
    This PR (8108) - mean (274ms)  : 269, 279
    master - mean (273ms)  : 268, 279

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (934ms)  : 884, 985
    master - mean (936ms)  : 888, 984

Loading
HttpMessageHandler (.NET 8)
gantt
    title Execution time (ms) HttpMessageHandler (.NET 8)
    dateFormat  x
    axisFormat %Q
    todayMarker off
    section Baseline
    This PR (8108) - mean (273ms)  : 269, 278
    master - mean (273ms)  : 264, 283

    section Bailout
    This PR (8108) - mean (273ms)  : 268, 279
    master - mean (275ms)  : 260, 290

    section CallTarget+Inlining+NGEN
    This PR (8108) - mean (845ms)  : 828, 862
    master - mean (849ms)  : 823, 874

Loading

@pr-commenter
Copy link

pr-commenter bot commented Jan 27, 2026

Benchmarks

Benchmark execution time: 2026-01-27 15:54:59

Comparing candidate commit 4adff5d in PR branch duncan-harvey/datadog-serverless-compat-v1.2.0 with baseline commit f721ece in branch master.

Found 9 performance improvements and 14 performance regressions! Performance is the same for 155 metrics, 14 unstable metrics.

scenario:Benchmarks.Trace.ActivityBenchmark.StartStopWithChild net6.0

  • 🟥 execution_time [+117.169ms; +121.334ms] or [+145.798%; +150.981%]

scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces net472

  • 🟥 execution_time [+17.450ms; +18.738ms] or [+8.716%; +9.359%]

scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces net6.0

  • 🟥 execution_time [+101.638ms; +101.771ms] or [+101.769%; +101.903%]

scenario:Benchmarks.Trace.AgentWriterBenchmark.WriteAndFlushEnrichedTraces netcoreapp3.1

  • 🟥 execution_time [+88.722ms; +88.973ms] or [+81.104%; +81.335%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.AllCycleMoreComplexBody net6.0

  • 🟥 execution_time [+10.740ms; +16.195ms] or [+5.398%; +8.140%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.ObjectExtractorMoreComplexBody net6.0

  • 🟥 execution_time [+13.843ms; +19.186ms] or [+7.034%; +9.749%]

scenario:Benchmarks.Trace.Asm.AppSecBodyBenchmark.ObjectExtractorSimpleBody netcoreapp3.1

  • 🟥 execution_time [+12.649ms; +18.485ms] or [+6.365%; +9.301%]

scenario:Benchmarks.Trace.Asm.AppSecEncoderBenchmark.EncodeLegacyArgs netcoreapp3.1

  • 🟥 execution_time [+18.236ms; +18.852ms] or [+9.953%; +10.289%]

scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces net472

  • 🟥 execution_time [+36.924ms; +42.410ms] or [+21.027%; +24.151%]
  • 🟥 throughput [-161.561op/s; -127.953op/s] or [-13.629%; -10.794%]

scenario:Benchmarks.Trace.CIVisibilityProtocolWriterBenchmark.WriteAndFlushEnrichedTraces netcoreapp3.1

  • 🟩 execution_time [-57.304ms; -51.017ms] or [-22.991%; -20.469%]
  • 🟩 throughput [+107.865op/s; +144.111op/s] or [+7.986%; +10.670%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OptimizedCharSlice net6.0

  • 🟩 execution_time [-83.380µs; -79.046µs] or [-5.747%; -5.448%]
  • 🟩 throughput [+39.729op/s; +42.008op/s] or [+5.765%; +6.095%]

scenario:Benchmarks.Trace.CharSliceBenchmark.OptimizedCharSliceWithPool net6.0

  • 🟥 execution_time [+97.779µs; +104.608µs] or [+9.652%; +10.326%]
  • 🟥 throughput [-92.510op/s; -86.755op/s] or [-9.372%; -8.789%]

scenario:Benchmarks.Trace.Iast.StringAspectsBenchmark.StringConcatBenchmark net6.0

  • 🟥 throughput [-4030.819op/s; -2050.926op/s] or [-17.814%; -9.064%]

scenario:Benchmarks.Trace.Log4netBenchmark.EnrichedLog netcoreapp3.1

  • 🟥 execution_time [+24.631ms; +28.452ms] or [+14.073%; +16.256%]

scenario:Benchmarks.Trace.RedisBenchmark.SendReceive net6.0

  • 🟩 execution_time [-18.661ms; -14.188ms] or [-8.559%; -6.507%]
  • 🟩 throughput [+27765.890op/s; +39530.324op/s] or [+5.535%; +7.880%]

scenario:Benchmarks.Trace.RedisBenchmark.SendReceive netcoreapp3.1

  • 🟩 throughput [+19448.691op/s; +30200.015op/s] or [+5.198%; +8.072%]

scenario:Benchmarks.Trace.SingleSpanAspNetCoreBenchmark.SingleSpanAspNetCore netcoreapp3.1

  • 🟩 throughput [+14293457.512op/s; +15785222.519op/s] or [+6.334%; +6.995%]

scenario:Benchmarks.Trace.SpanBenchmark.StartFinishTwoScopes net6.0

  • 🟩 execution_time [-17.125ms; -13.088ms] or [-7.918%; -6.051%]

@duncanpharvey duncanpharvey merged commit bab1c28 into master Jan 27, 2026
146 checks passed
@duncanpharvey duncanpharvey deleted the duncan-harvey/datadog-serverless-compat-v1.2.0 branch January 27, 2026 18:29
@github-actions github-actions bot added this to the vNext-v3 milestone Jan 27, 2026
igoragoli pushed a commit that referenced this pull request Jan 29, 2026
…8108)

## Summary of changes

Update `Datadog.Serverless.Compat` to
[v1.2.0](https://github.com/DataDog/datadog-serverless-compat-dotnet/releases/tag/v1.2.0)
for `Datadog.AzureFunctions`.

## Reason for change

New Serverless Compatibility Layer package available.

## Implementation details

## Test coverage

## Other details
<!-- Fixes #{issue} -->


<!--  ⚠️ Note:

Where possible, please obtain 2 approvals prior to merging. Unless
CODEOWNERS specifies otherwise, for external teams it is typically best
to have one review from a team member, and one review from apm-dotnet.
Trivial changes do not require 2 reviews.

MergeQueue is NOT enabled in this repository. If you have write access
to the repo, the PR has 1-2 approvals (see above), and all of the
required checks have passed, you can use the Squash and Merge button to
merge the PR. If you don't have write access, or you need help, reach
out in the #apm-dotnet channel in Slack.
-->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants