[onecollector] Prevent event full name mappings with "namespace.*" form (open-telemetry#2043)

Co-authored-by: Piotr Kiełkowicz <[email protected]>

Author: CodeBlanch

src/OpenTelemetry.Exporter.OneCollector/Internal/EventFullName.cs

@@ -86,5 +86,9 @@ internal void Validate(string key)
                 throw new OneCollectorExporterValidationException($"The event full name mapping value provided for key '{key}' is shorter or longer than what is allowed.");
             }
         }
+        else if (this.EventNamespace.Length != 0)
+        {
+            throw new OneCollectorExporterValidationException($"The event full name mapping value provided for key '{key}' has an invalid wild card pattern.");
+        }
     }
 }

test/OpenTelemetry.Exporter.OneCollector.Tests/OneCollectorLogExporterOptionsTests.cs

@@ -109,9 +109,14 @@ public void ValidDefaultEventFullNameTests(string defaultEventNamespace, string
     [InlineData("Prefix", "ABC")]
     [InlineData("Prefix", "_.EventName")]
     [InlineData("Prefix", "Namespace.Event_Name")]
+    [InlineData("Prefix", "Event_Name")]
     [InlineData("Prefix", "Namespace.")]
+    [InlineData("Prefix", ".")]
     [InlineData("*", "*.EventName")]
     [InlineData("*", "*.*")]
+    [InlineData("*", "Namespace.*")]
+    [InlineData("Valid", "Invalid*")]
+    [InlineData("Valid", "In*valid")]
     public void InvalidEventFullNameMappingTests(string key, string? value)
     {
         var options = new OneCollectorLogExporterOptions()
@@ -129,9 +134,14 @@ public void InvalidEventFullNameMappingTests(string key, string? value)
 
     [Theory]
     [InlineData("*", "ABCD")]
+    [InlineData("Some.Prefix", "ABCD")]
     [InlineData("*", "A.BC")]
+    [InlineData("Some.Prefix", "A.BC")]
     [InlineData("*", "A.B.C")]
+    [InlineData("Some.Prefix", "A.B.C")]
     [InlineData("*", "*")]
+    [InlineData("Some.Prefix", "*")]
+    [InlineData("Some_Name", "New.Name")]
     public void ValidEventFullNameMappingTests(string key, string? value)
     {
         var options = new OneCollectorLogExporterOptions()