Fix benchmarks

Fix benchmarks no longer running due to use of Artifacts Output and an incorrectly defined POCO property.

Author: martincostello

tests/HttpClientInterception.Benchmarks/CustomBenchmarkConfig.cs

@@ -0,0 +1,21 @@
+// Copyright (c) Just Eat, 2017. All rights reserved.
+// Licensed under the Apache 2.0 license. See the LICENSE file in the project root for full license information.
+
+using BenchmarkDotNet.Configs;
+using BenchmarkDotNet.Diagnosers;
+using BenchmarkDotNet.Jobs;
+
+namespace JustEat.HttpClientInterception;
+
+public class CustomBenchmarkConfig : ManualConfig
+{
+    public CustomBenchmarkConfig()
+        : base()
+    {
+        var job = Job.Default.WithArguments([new MsBuildArgument("/p:UseArtifactsOutput=false")]);
+
+        AddJob(job);
+        AddDiagnoser(MemoryDiagnoser.Default);
+        AddDiagnoser(new EventPipeProfiler(EventPipeProfile.CpuSampling));
+    }
+}

tests/HttpClientInterception.Benchmarks/InterceptionBenchmarks.cs

@@ -3,13 +3,11 @@
 
 using System.Text.Json;
 using BenchmarkDotNet.Attributes;
-using BenchmarkDotNet.Diagnosers;
 using Refit;
 
 namespace JustEat.HttpClientInterception;
 
-[EventPipeProfiler(EventPipeProfile.CpuSampling)]
-[MemoryDiagnoser]
+[Config(typeof(CustomBenchmarkConfig))]
 public class InterceptionBenchmarks
 {
     private readonly HttpClientInterceptorOptions _options;

tests/HttpClientInterception.Benchmarks/Organization.cs

@@ -7,7 +7,7 @@ public sealed class Organization
 {
     public string Login { get; set; }
 
-    public string Id { get; set; }
+    public long Id { get; set; }
 
     public string Url { get; set; }
 }