Simplify benchmarks

- Remove need for custom configuration now that `UseArtifactsOutput` is supported. - Use collection expression. - Simplify class declaration.
justeattakeaway · Aug 7, 2024 · 6c59e4f · 6c59e4f
1 parent bb85ee9
commit 6c59e4f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 26 deletions.
diff --git a/tests/HttpClientInterception.Benchmarks/CustomBenchmarkConfig.cs b/tests/HttpClientInterception.Benchmarks/CustomBenchmarkConfig.cs
diff --git a/tests/HttpClientInterception.Benchmarks/GitHubJsonSerializerContext.cs b/tests/HttpClientInterception.Benchmarks/GitHubJsonSerializerContext.cs
@@ -7,7 +7,5 @@
 namespace JustEat.HttpClientInterception;
 
 [JsonSerializable(typeof(GitHubOrganization))]
-internal sealed partial class GitHubJsonSerializerContext : JsonSerializerContext
-{
-}
+internal sealed partial class GitHubJsonSerializerContext : JsonSerializerContext;
 #endif
diff --git a/tests/HttpClientInterception.Benchmarks/InterceptionBenchmarks.cs b/tests/HttpClientInterception.Benchmarks/InterceptionBenchmarks.cs
@@ -4,11 +4,13 @@
 using System.Net.Http.Json;
 using System.Text.Json;
 using BenchmarkDotNet.Attributes;
+using BenchmarkDotNet.Diagnosers;
 using Refit;
 
 namespace JustEat.HttpClientInterception;
 
-[Config(typeof(CustomBenchmarkConfig))]
+[EventPipeProfiler(EventPipeProfile.CpuSampling)]
+[MemoryDiagnoser]
 public class InterceptionBenchmarks
 {
     private readonly HttpClientInterceptorOptions _options;
@@ -40,7 +42,7 @@ public InterceptionBenchmarks()
             .ForQuery(string.Empty)
             .Responds()
             .WithMediaType("application/octet-stream")
-            .WithContent(() => new byte[] { 0, 1, 2, 3, 4 })
+            .WithContent(() => [0, 1, 2, 3, 4])
             .RegisterWith(_options);
 
         builder