fixup! Support tracking

Signed-off-by: christian.lutnik <[email protected]>
open-feature · Dec 10, 2024 · c8dc923 · c8dc923
1 parent f3c1997
commit c8dc923
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 1 deletion.
diff --git a/src/OpenFeature/OpenFeatureClient.cs b/src/OpenFeature/OpenFeatureClient.cs
@@ -376,7 +376,7 @@ private async Task TriggerFinallyHooksAsync<T>(IReadOnlyList<Hook> hooks, HookCo
         /// <exception cref="ArgumentException">When trackingEventName is null or empty</exception>
         public void Track(string trackingEventName, EvaluationContext? evaluationContext = default, TrackingEventDetails? trackingEventDetails = default)
         {
-            if (string.IsNullOrEmpty(trackingEventName))
+            if (string.IsNullOrWhiteSpace(trackingEventName))
             {
                 throw new ArgumentException("Tracking event cannot be null or empty.", nameof(trackingEventName));
             }

diff --git a/test/OpenFeature.Tests/OpenFeatureClientTests.cs b/test/OpenFeature.Tests/OpenFeatureClientTests.cs
@@ -595,6 +595,16 @@ public async Task PassingAnEmptyStringAsTrackingEventName_ThrowsArgumentExceptio
             Assert.Throws<ArgumentException>(() => client.Track(""));
         }
 
+        [Fact]
+        public async Task PassingABlankStringAsTrackingEventName_ThrowsArgumentException()
+        {
+            var provider = new TestProvider();
+            await Api.Instance.SetProviderAsync(provider);
+            var client = Api.Instance.GetClient();
+
+            Assert.Throws<ArgumentException>(() => client.Track(" \n "));
+        }
+
         public static TheoryData<string, EvaluationContext?, EvaluationContext?, EvaluationContext?, string> GenerateMergeEvaluationContextTestData()
         {
             const string key = "key";