LoggerProvider tests.

src/OpenTelemetry.Api/Logs/LoggerProvider.cs

@@ -29,13 +29,6 @@ internal class LoggerProvider : BaseProvider
 {
     private static readonly NoopLogger NoopLogger = new();
 
-    /// <summary>
-    /// Initializes a new instance of the <see cref="LoggerProvider"/> class.
-    /// </summary>
-    protected LoggerProvider()
-    {
-    }
-
     /// <summary>
     /// Gets a logger.
     /// </summary>

test/OpenTelemetry.Api.Tests/Logs/LogRecordDataTests.cs

@@ -14,6 +14,8 @@
 // limitations under the License.
 // </copyright>
 
+#nullable enable
+
 using System.Diagnostics;
 using Xunit;
 

test/OpenTelemetry.Api.Tests/Logs/LoggerProviderTests.cs

@@ -0,0 +1,112 @@
+// <copyright file="LoggerProviderTests.cs" company="OpenTelemetry Authors">
+// Copyright The OpenTelemetry Authors
+//
+// Licensed under the Apache License, Version 2.0 (the "License");
+// you may not use this file except in compliance with the License.
+// You may obtain a copy of the License at
+//
+//     http://www.apache.org/licenses/LICENSE-2.0
+//
+// Unless required by applicable law or agreed to in writing, software
+// distributed under the License is distributed on an "AS IS" BASIS,
+// WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+// See the License for the specific language governing permissions and
+// limitations under the License.
+// </copyright>
+
+#nullable enable
+
+using Xunit;
+
+namespace OpenTelemetry.Logs.Tests;
+
+public sealed class LoggerProviderTests
+{
+    [Fact]
+    public void NoopLoggerReturnedTest()
+    {
+        using var provider = new LoggerProvider();
+
+        var attributes = new Dictionary<string, object>()
+        {
+            ["key1"] = "value1",
+        };
+
+        var logger = provider.GetLogger(name: "TestLogger", version: "Version", attributes);
+
+        Assert.NotNull(logger);
+        Assert.Equal(typeof(NoopLogger), logger.GetType());
+
+        Assert.Equal(string.Empty, logger.Name);
+        Assert.Null(logger.Version);
+        Assert.Null(logger.Attributes);
+    }
+
+    [Fact]
+    public void LoggerReturnedWithInstrumentationScopeTest()
+    {
+        using var provider = new TestLoggerProvider();
+
+        var attributes = new Dictionary<string, object>()
+        {
+            ["key1"] = "value1",
+        };
+
+        var logger = provider.GetLogger(name: "TestLogger", version: "Version", attributes);
+
+        Assert.NotNull(logger);
+        Assert.Equal(typeof(TestLogger), logger.GetType());
+
+        Assert.Equal("TestLogger", logger.Name);
+        Assert.Equal("Version", logger.Version);
+        Assert.Equal(attributes, logger.Attributes);
+
+        logger = provider.GetLogger(name: "TestLogger", version: "Version");
+
+        Assert.NotNull(logger);
+        Assert.Equal(typeof(TestLogger), logger.GetType());
+
+        Assert.Equal("TestLogger", logger.Name);
+        Assert.Equal("Version", logger.Version);
+        Assert.Null(logger.Attributes);
+
+        logger = provider.GetLogger(name: "TestLogger");
+
+        Assert.NotNull(logger);
+        Assert.Equal(typeof(TestLogger), logger.GetType());
+
+        Assert.Equal("TestLogger", logger.Name);
+        Assert.Null(logger.Version);
+        Assert.Null(logger.Attributes);
+
+        logger = provider.GetLogger();
+
+        Assert.NotNull(logger);
+        Assert.Equal(typeof(TestLogger), logger.GetType());
+
+        Assert.Equal(string.Empty, logger.Name);
+        Assert.Null(logger.Version);
+        Assert.Null(logger.Attributes);
+    }
+
+    private sealed class TestLoggerProvider : LoggerProvider
+    {
+        protected override bool TryCreateLogger(string? name, out Logger? logger)
+        {
+            logger = new TestLogger(name);
+            return true;
+        }
+    }
+
+    private sealed class TestLogger : Logger
+    {
+        public TestLogger(string? name)
+            : base(name)
+        {
+        }
+
+        public override void EmitLog(in LogRecordData data, in LogRecordAttributeList attributes = default)
+        {
+        }
+    }
+}