align iteration mode

dotnet · May 23, 2018 · 8ae23e8 · 8ae23e8
1 parent e34fc7a
commit 8ae23e8
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 2 deletions.
diff --git a/src/BenchmarkDotNet/Reports/Measurement.cs b/src/BenchmarkDotNet/Reports/Measurement.cs
@@ -1,4 +1,5 @@
 using System;
+using System.Linq;
 using BenchmarkDotNet.Engines;
 using BenchmarkDotNet.Environments;
 using BenchmarkDotNet.Extensions;
@@ -13,6 +14,8 @@ public struct Measurement : IComparable<Measurement>
     {
         private static readonly Measurement Error = new Measurement(-1, IterationMode.Unknown, 0, 0, 0);
 
+        private static readonly int IterationModeNameMaxWidth = Enum.GetNames(typeof(IterationMode)).Max(text => text.Length);
+
         public IterationMode IterationMode { get; }
 
         public int LaunchIndex { get; }
@@ -48,10 +51,13 @@ public Measurement(int launchIndex, IterationMode iterationMode, int iterationIn
 
         public string ToOutputLine()
         {
+            string alignedIterationMode = IterationMode.ToString().PadRight(IterationModeNameMaxWidth, ' ');
+
             // Usually, a benchmarks takes more than 10 iterations (rarely more than 99)
             // PadLeft(2, ' ') looks like a good trade-off between alignment and amount of characters
             string alignedIterationIndex = IterationIndex.ToString().PadLeft(2, ' ');
-            return $"{IterationMode} {alignedIterationIndex}: {GetDisplayValue()}";
+
+            return $"{alignedIterationMode} {alignedIterationIndex}: {GetDisplayValue()}";
         }
 
         private string GetDisplayValue() => $"{Operations} op, {Nanoseconds.ToStr("0.00")} ns, {GetAverageTime()}";

diff --git a/tests/BenchmarkDotNet.Tests/Engine/EngineFactoryTests.cs b/tests/BenchmarkDotNet.Tests/Engine/EngineFactoryTests.cs
@@ -80,7 +80,7 @@ public void ForJobsWithExplicitUnrollFactorTheGlobalSetupIsCalledAndMultiActionC
         public void ForJobsThatDontRequirePilotTheGlobalSetupIsCalledAndMultiActionCodeGetsJitted() 
             => AssertGlobalSetupWasCalledAndMultiActionGotJitted(Job.Default.WithInvocationCount(100));
 
-        public void AssertGlobalSetupWasCalledAndMultiActionGotJitted(Job job)
+        private void AssertGlobalSetupWasCalledAndMultiActionGotJitted(Job job)
         {
             var engineParameters = CreateEngineParameters(mainSingleAction: Throwing, mainMultiAction: Instant16, job: job);