Rename MeasuredOperationLatency to MeasuredOperation

README.md

@@ -195,7 +195,7 @@ Example.
 
 async Task MeasureCodeBlock(ServiceLevelIndicator serviceLevelIndicator)
 {
-    using var measuredOperation = serviceLevelIndicator.StartLatencyMeasureOperation("OperationName");
+    using var measuredOperation = serviceLevelIndicator.StartMeasuring("OperationName");
     // Do Work.
     measuredOperation.SetActivityStatusCode(System.Diagnostics.ActivityStatusCode.Ok);
 }

ServiceLevelIndicators.Asp/src/HttpContextExtensions.cs

@@ -7,38 +7,38 @@
 public static class HttpContextExtensions
 {
     /// <summary>
-    /// Gets the MeasuredOperationLatency from the IServiceLevelIndicatorFeature.
+    /// Gets the MeasuredOperation from the IServiceLevelIndicatorFeature.
     /// The method will throw an exception if the route is not configured to emit SLI metrics.
     /// </summary>
     /// <param name="context"></param>
-    /// <returns>MeasuredOperationLatency for the current API method.</returns>
+    /// <returns>MeasuredOperation for the current API method.</returns>
     /// <exception cref="ArgumentNullException"></exception>
-    /// <exception cref="InvalidOperationException">If the route does not emit SLI information and therefore MeasuredOperationLatency does not exist.</exception>
-    public static MeasuredOperationLatency GetMeasuredOperationLatency(this HttpContext context)
+    /// <exception cref="InvalidOperationException">If the route does not emit SLI information and therefore MeasuredOperation does not exist.</exception>
+    public static MeasuredOperation GetMeasuredOperation(this HttpContext context)
     {
         ArgumentNullException.ThrowIfNull(context);
 
-        return context.Features.GetRequiredFeature<IServiceLevelIndicatorFeature>().MeasuredOperationLatency;
+        return context.Features.GetRequiredFeature<IServiceLevelIndicatorFeature>().MeasuredOperation;
     }
 
     /// <summary>
-    /// Gets the MeasuredOperationLatency from the IServiceLevelIndicatorFeature.
+    /// Gets the MeasuredOperation from the IServiceLevelIndicatorFeature.
     /// </summary>
     /// <param name="context"></param>
-    /// <param name="measuredOperationLatency"></param>
-    /// <returns>true if MeasuredOperationLatency exists.</returns>
+    /// <param name="measuredOperation"></param>
+    /// <returns>true if MeasuredOperation exists.</returns>
     /// <exception cref="ArgumentNullException"></exception>
-    public static bool TryGetMeasuredOperationLatency(this HttpContext context, [MaybeNullWhen(false)] out MeasuredOperationLatency measuredOperationLatency)
+    public static bool TryGetMeasuredOperation(this HttpContext context, [MaybeNullWhen(false)] out MeasuredOperation measuredOperation)
     {
         ArgumentNullException.ThrowIfNull(context);
 
         if (context.Features.Get<IServiceLevelIndicatorFeature>() is IServiceLevelIndicatorFeature feature)
         {
-            measuredOperationLatency = feature.MeasuredOperationLatency;
+            measuredOperation = feature.MeasuredOperation;
             return true;
         }
 
-        measuredOperationLatency = null;
+        measuredOperation = null;
         return false;
     }
 }

ServiceLevelIndicators.Asp/src/IServiceLevelIndicatorFeature.cs

@@ -4,5 +4,5 @@
 /// </summary>
 public interface IServiceLevelIndicatorFeature
 {
-    MeasuredOperationLatency MeasuredOperationLatency { get; }
+    MeasuredOperation MeasuredOperation { get; }
 }

ServiceLevelIndicators.Asp/src/ServiceLevelIndicatorFeature.cs

@@ -2,7 +2,7 @@
 
 internal sealed class ServiceLevelIndicatorFeature : IServiceLevelIndicatorFeature
 {
-    public ServiceLevelIndicatorFeature(MeasuredOperationLatency measureOperationLatency) => MeasuredOperationLatency = measureOperationLatency;
+    public ServiceLevelIndicatorFeature(MeasuredOperation measureOperation) => MeasuredOperation = measureOperation;
 
-    public MeasuredOperationLatency MeasuredOperationLatency { get; }
+    public MeasuredOperation MeasuredOperation { get; }
 }

ServiceLevelIndicators.Asp/src/ServiceLevelIndicatorMiddleware.cs

@@ -32,7 +32,7 @@ public async Task InvokeAsync(HttpContext context)
         var operation = GetOperation(context, metadata);
         var attributes = GetMeasuredAttributes(context, metadata);
 
-        using var measuredOperation = _serviceLevelIndicator.StartLatencyMeasureOperation(operation, attributes);
+        using var measuredOperation = _serviceLevelIndicator.StartMeasuring(operation, attributes);
         SetCustomerResourceIdFromAttribute(context, metadata, measuredOperation);
         AddSliFeatureToHttpContext(context, measuredOperation);
         await _next(context);
@@ -47,14 +47,14 @@ public async Task InvokeAsync(HttpContext context)
     }
 
     [MethodImpl(MethodImplOptions.AggressiveInlining)]
-    private static void SetCustomerResourceIdFromAttribute(HttpContext context, EndpointMetadataCollection metadata, MeasuredOperationLatency measuredOperation)
+    private static void SetCustomerResourceIdFromAttribute(HttpContext context, EndpointMetadataCollection metadata, MeasuredOperation measuredOperation)
     {
         var customerResourceId = GetCustomerResourceIdAttributes(context, metadata);
         if (customerResourceId is not null)
             measuredOperation.CustomerResourceId = customerResourceId;
     }
 
-    private static void UpdateOperationWithResponseStatus(HttpContext context, MeasuredOperationLatency measuredOperation)
+    private static void UpdateOperationWithResponseStatus(HttpContext context, MeasuredOperation measuredOperation)
     {
         var statusCode = context.Response.StatusCode;
         measuredOperation.AddAttribute("http.response.status.code", statusCode);
@@ -129,7 +129,7 @@ private static string GetOperation(HttpContext context, EndpointMetadataCollecti
         return attributes;
     }
 
-    private void AddSliFeatureToHttpContext(HttpContext context, MeasuredOperationLatency measuredOperation)
+    private void AddSliFeatureToHttpContext(HttpContext context, MeasuredOperation measuredOperation)
     {
         if (context.Features.Get<IServiceLevelIndicatorFeature>() != null)
             throw new InvalidOperationException($"Another instance of {nameof(ServiceLevelIndicatorFeature)} already exists. Only one instance of {nameof(ServiceLevelIndicatorMiddleware)} can be configured for an application.");

ServiceLevelIndicators.Asp/src/WebEnrichmentContext.cs

@@ -3,10 +3,10 @@
 
 public class WebEnrichmentContext : IEnrichmentContext
 {
-    private readonly MeasuredOperationLatency _operation;
+    private readonly MeasuredOperation _operation;
     public HttpContext HttpContext { get; }
 
-    public WebEnrichmentContext(MeasuredOperationLatency operation, HttpContext httpContext)
+    public WebEnrichmentContext(MeasuredOperation operation, HttpContext httpContext)
     {
         _operation = operation;
         HttpContext = httpContext;

ServiceLevelIndicators.Asp/tests/ServiceLevelIndicatorAspTests.cs

@@ -285,7 +285,7 @@ void OnMeasurementRecorded(Instrument instrument, long measurement, ReadOnlySpan
     }
 
     [Fact]
-    public async Task GetMeasuredOperationLatency_will_throw_if_route_does_not_emit_SLI()
+    public async Task GetMeasuredOperation_will_throw_if_route_does_not_emit_SLI()
     {
         using var host = await TestHostBuilder.CreateHostWithoutSli();
 
@@ -298,7 +298,7 @@ public async Task GetMeasuredOperationLatency_will_throw_if_route_does_not_emit_
     }
 
     [Fact]
-    public async Task TryGetMeasuredOperationLatency_will_return_false_if_route_does_not_emit_SLI()
+    public async Task TryGetMeasuredOperation_will_return_false_if_route_does_not_emit_SLI()
     {
         using var host = await TestHostBuilder.CreateHostWithoutSli();
 
@@ -310,7 +310,7 @@ public async Task TryGetMeasuredOperationLatency_will_return_false_if_route_does
     }
 
     [Fact]
-    public async Task TryGetMeasuredOperationLatency_will_return_true_if_route_emits_SLI()
+    public async Task TryGetMeasuredOperation_will_return_true_if_route_emits_SLI()
     {
         _meterListener.SetMeasurementEventCallback<long>(OnMeasurementRecorded);
         _meterListener.Start();

ServiceLevelIndicators.Asp/tests/TestController.cs

@@ -24,16 +24,16 @@ public class TestController : ControllerBase
     [HttpGet("custom_attribute/{value}")]
     public IActionResult AddCustomAttribute(string value)
     {
-        HttpContext.GetMeasuredOperationLatency().AddAttribute("CustomAttribute", value);
+        HttpContext.GetMeasuredOperation().AddAttribute("CustomAttribute", value);
         return Ok(value);
     }
 
     [HttpGet("try_get_measured_operation_latency/{value}")]
     public IActionResult TryGetMeasuredOperationLatency(string value)
     {
-        if (HttpContext.TryGetMeasuredOperationLatency(out var measuredOperationLatency))
+        if (HttpContext.TryGetMeasuredOperation(out var measuredOperation))
         {
-            measuredOperationLatency.AddAttribute("CustomAttribute", value);
+            measuredOperation.AddAttribute("CustomAttribute", value);
             return Ok(true);
         }
         return Ok(false);

ServiceLevelIndicators/src/MeasuredOperationLatency.cs → ServiceLevelIndicators/src/MeasuredOperation.cs

@@ -4,19 +4,19 @@
 using System.Collections.Generic;
 using System.Diagnostics;
 
-public class MeasuredOperationLatency : IDisposable
+public class MeasuredOperation : IDisposable
 {
     private bool _disposed;
     private readonly ServiceLevelIndicator _serviceLevelIndicator;
     private readonly Stopwatch _stopWatch;
     private ActivityStatusCode _activityStatusCode = ActivityStatusCode.Unset;
     private readonly object _disposeLock = new();
 
-    public MeasuredOperationLatency(ServiceLevelIndicator serviceLevelIndicator, string operation, params KeyValuePair<string, object?>[] attributes) :
+    public MeasuredOperation(ServiceLevelIndicator serviceLevelIndicator, string operation, params KeyValuePair<string, object?>[] attributes) :
         this(serviceLevelIndicator, operation, serviceLevelIndicator.ServiceLevelIndicatorOptions.CustomerResourceId, attributes)
     { }
 
-    public MeasuredOperationLatency(ServiceLevelIndicator serviceLevelIndicator, string operation, string customerResourceId, params KeyValuePair<string, object?>[] attributes)
+    public MeasuredOperation(ServiceLevelIndicator serviceLevelIndicator, string operation, string customerResourceId, params KeyValuePair<string, object?>[] attributes)
     {
         _serviceLevelIndicator = serviceLevelIndicator;
         Operation = operation;
@@ -46,7 +46,7 @@ protected virtual void Dispose(bool disposing)
                     _stopWatch.Stop();
                     var elapsedTime = _stopWatch.ElapsedMilliseconds;
                     Attributes.Add(new KeyValuePair<string, object?>(_serviceLevelIndicator.ServiceLevelIndicatorOptions.ActivityStatusCodeAttributeName, _activityStatusCode.ToString()));
-                    _serviceLevelIndicator.RecordLatency(Operation, CustomerResourceId, elapsedTime, Attributes.ToArray());
+                    _serviceLevelIndicator.Record(Operation, CustomerResourceId, elapsedTime, Attributes.ToArray());
                 }
 
                 _disposed = true;

ServiceLevelIndicators/src/ServiceLevelIndicator.cs

@@ -20,10 +20,10 @@ public ServiceLevelIndicator(IOptions<ServiceLevelIndicatorOptions> options)
         _responseLatencyHistogram = ServiceLevelIndicatorOptions.Meter.CreateHistogram<long>(ServiceLevelIndicatorOptions.InstrumentName, "ms");
     }
 
-    public void RecordLatency(string operation, long elapsedTime, params KeyValuePair<string, object?>[] attributes) =>
-        RecordLatency(operation, ServiceLevelIndicatorOptions.CustomerResourceId, elapsedTime, attributes);
+    public void Record(string operation, long elapsedTime, params KeyValuePair<string, object?>[] attributes) =>
+        Record(operation, ServiceLevelIndicatorOptions.CustomerResourceId, elapsedTime, attributes);
 
-    public void RecordLatency(string operation, string customerResourseId, long elapsedTime, params KeyValuePair<string, object?>[] attributes)
+    public void Record(string operation, string customerResourseId, long elapsedTime, params KeyValuePair<string, object?>[] attributes)
     {
         var tagList = new TagList
         {
@@ -38,7 +38,7 @@ public void RecordLatency(string operation, string customerResourseId, long elap
         _responseLatencyHistogram.Record(elapsedTime, tagList);
     }
 
-    public MeasuredOperationLatency StartLatencyMeasureOperation(string operation, params KeyValuePair<string, object?>[] attributes) => new(this, operation, attributes);
+    public MeasuredOperation StartMeasuring(string operation, params KeyValuePair<string, object?>[] attributes) => new(this, operation, attributes);
 
     public static string CreateCustomerResourceId(Guid serviceId)
     {

ServiceLevelIndicators/tests/ServiceLevelIndicatorTests.cs

@@ -39,7 +39,7 @@ public ServiceLevelIndicatorTests(ITestOutputHelper output)
 
 
     [Fact]
-    public void Record_latency()
+    public void Record()
     {
         // Arrange
         var customerResourceId = "TestResourceId";
@@ -62,7 +62,7 @@ public void Record_latency()
         };
 
         // Act
-        serviceLevelIndicator.RecordLatency(operation, elapsedTime, attributes);
+        serviceLevelIndicator.Record(operation, elapsedTime, attributes);
 
         // Assert
         _expectedTags =
@@ -111,7 +111,7 @@ public async Task Will_measure_code_block()
 
         async Task MeasureCodeBlock(ServiceLevelIndicator serviceLevelIndicator)
         {
-            using var measuredOperation = serviceLevelIndicator.StartLatencyMeasureOperation("SleepWorker");
+            using var measuredOperation = serviceLevelIndicator.StartMeasuring("SleepWorker");
             await Task.Delay(sleepTime);
             measuredOperation.SetActivityStatusCode(System.Diagnostics.ActivityStatusCode.Ok);
         }
@@ -139,7 +139,7 @@ public void Customize_instrument_name()
         var elapsedTime = 30;
 
         // Act
-        serviceLevelIndicator.RecordLatency(operation, elapsedTime);
+        serviceLevelIndicator.Record(operation, elapsedTime);
 
         // Assert
         _expectedTags =