refactor: Clarify tracking context handling and rename test for inval…

…id tracking event name

Signed-off-by: André Silva <[email protected]>

src/OpenFeature.Providers.MultiProvider/MultiProvider.cs

@@ -133,7 +133,7 @@ public override void Track(string trackingEventName, EvaluationContext? evaluati
                 registeredProvider.Provider,
                 registeredProvider.Name,
                 registeredProvider.Status,
-                string.Empty); // Empty flag key for tracking context
+                string.Empty); // Tracking operations are not flag-specific, so the flag key is intentionally set to an empty string
 
             if (this._evaluationStrategy.ShouldTrackWithThisProvider(providerContext, evaluationContext, trackingEventName, trackingEventDetails))
             {

test/OpenFeature.Providers.MultiProvider.Tests/MultiProviderTrackingTests.cs

@@ -275,7 +275,7 @@ public async Task Track_WithComplexTrackingDetails_PropagatesAllDetails()
     [InlineData(null)]
     [InlineData("")]
     [InlineData(" ")]
-    public async Task Track_WhenEmptyTrackingDetails_DoesNotThrow(string? trackingEventName)
+    public async Task Track_WhenInvalidTrackingEventName_DoesNotCallProviders(string? trackingEventName)
     {
         // Arrange
         var providerEntries = new List<ProviderEntry>