fixed RequestsPerSecond property in BDN (#6867)

had to add a null check to make sure we weren't missing reports
akkadotnet · Aug 1, 2023 · 33dda22 · 33dda22
1 parent 29cc42f
commit 33dda22
Showing 1 changed file with 9 additions and 5 deletions.
diff --git a/src/benchmark/Akka.Benchmarks/Configurations/Configs.cs b/src/benchmark/Akka.Benchmarks/Configurations/Configs.cs
@@ -36,13 +36,17 @@ public string GetValue(Summary summary, BenchmarkCase benchmarkCase, SummaryStyl
         {
             var benchmarkAttribute = benchmarkCase.Descriptor.WorkloadMethod.GetCustomAttribute<BenchmarkAttribute>();
             var totalOperations = benchmarkAttribute?.OperationsPerInvoke ?? 1;
-
-            var report = summary[benchmarkCase];
-            var nsPerOperation = report.ResultStatistics.Mean;
-            var operationsPerSecond = 1 / (nsPerOperation / 1e9);
 
+            if (summary.HasReport(benchmarkCase))
+            {
+                var report = summary[benchmarkCase];
+                var nsPerOperation = report.ResultStatistics.Mean;
+                var operationsPerSecond = 1 / (nsPerOperation / 1e9);
 
-            return operationsPerSecond.ToString("N2");  // or format as you like
+                return operationsPerSecond.ToString("N2");  // or format as you like
+            }
+
+            return "<not found>";
         }
     }