Add tests for NoOpTelemetryService and conditional registration

Co-authored-by: JFenderson <24466206+JFenderson@users.noreply.github.com>

Author: Copilot

Backend/Podium/Podium.Tests/Unit/Extensions/MonitoringExtensionsTests.cs

@@ -0,0 +1,105 @@
+using Xunit;
+using FluentAssertions;
+using Microsoft.Extensions.DependencyInjection;
+using Microsoft.Extensions.Configuration;
+using Podium.API.Extensions;
+using Podium.Core.Interfaces;
+using Podium.Infrastructure.Telemetry;
+
+namespace Podium.Tests.Unit.Extensions
+{
+    /// <summary>
+    /// Unit tests for MonitoringExtensions to verify conditional telemetry service registration.
+    /// </summary>
+    public class MonitoringExtensionsTests
+    {
+        [Fact]
+        public void AddPodiumTelemetryServices_WithApplicationInsightsDisabled_RegistersNoOpService()
+        {
+            // Arrange
+            var services = new ServiceCollection();
+            services.AddLogging(); // Add logging infrastructure
+            var configuration = new ConfigurationBuilder()
+                .AddInMemoryCollection(new Dictionary<string, string?>
+                {
+                    { "ENABLE_APPLICATION_INSIGHTS", "false" }
+                })
+                .Build();
+
+            // Act
+            services.AddPodiumTelemetryServices(configuration);
+            var serviceProvider = services.BuildServiceProvider();
+
+            // Assert
+            var telemetryService = serviceProvider.GetService<ITelemetryService>();
+            telemetryService.Should().NotBeNull();
+            telemetryService.Should().BeOfType<NoOpTelemetryService>();
+        }
+
+        [Fact]
+        public void AddPodiumTelemetryServices_WithNoConnectionString_RegistersNoOpService()
+        {
+            // Arrange
+            var services = new ServiceCollection();
+            services.AddLogging(); // Add logging infrastructure
+            var configuration = new ConfigurationBuilder()
+                .AddInMemoryCollection(new Dictionary<string, string?>())
+                .Build();
+
+            // Act
+            services.AddPodiumTelemetryServices(configuration);
+            var serviceProvider = services.BuildServiceProvider();
+
+            // Assert
+            var telemetryService = serviceProvider.GetService<ITelemetryService>();
+            telemetryService.Should().NotBeNull();
+            telemetryService.Should().BeOfType<NoOpTelemetryService>();
+        }
+
+        [Fact]
+        public void AddPodiumTelemetryServices_WithEmptyConnectionString_RegistersNoOpService()
+        {
+            // Arrange
+            var services = new ServiceCollection();
+            services.AddLogging(); // Add logging infrastructure
+            var configuration = new ConfigurationBuilder()
+                .AddInMemoryCollection(new Dictionary<string, string?>
+                {
+                    { "APPLICATIONINSIGHTS_CONNECTION_STRING", "" },
+                    { "ENABLE_APPLICATION_INSIGHTS", "true" }
+                })
+                .Build();
+
+            // Act
+            services.AddPodiumTelemetryServices(configuration);
+            var serviceProvider = services.BuildServiceProvider();
+
+            // Assert
+            var telemetryService = serviceProvider.GetService<ITelemetryService>();
+            telemetryService.Should().NotBeNull();
+            telemetryService.Should().BeOfType<NoOpTelemetryService>();
+        }
+
+        [Fact]
+        public void AddPodiumTelemetryServices_WithApplicationInsightsDisabled_DoesNotRegisterMetricsService()
+        {
+            // Arrange
+            var services = new ServiceCollection();
+            services.AddLogging(); // Add logging infrastructure
+            var configuration = new ConfigurationBuilder()
+                .AddInMemoryCollection(new Dictionary<string, string?>
+                {
+                    { "ENABLE_APPLICATION_INSIGHTS", "false" }
+                })
+                .Build();
+
+            // Act
+            services.AddPodiumTelemetryServices(configuration);
+            var serviceProvider = services.BuildServiceProvider();
+
+            // Assert
+            var metricsService = serviceProvider.GetService<MetricsService>();
+            metricsService.Should().BeNull();
+        }
+    }
+}

Backend/Podium/Podium.Tests/Unit/Telemetry/NoOpTelemetryServiceTests.cs

@@ -0,0 +1,120 @@
+using Xunit;
+using Moq;
+using FluentAssertions;
+using Microsoft.Extensions.Logging;
+using Podium.Infrastructure.Telemetry;
+
+namespace Podium.Tests.Unit.Telemetry
+{
+    /// <summary>
+    /// Unit tests for NoOpTelemetryService to ensure it doesn't throw exceptions
+    /// and properly logs debug messages when Application Insights is disabled.
+    /// </summary>
+    public class NoOpTelemetryServiceTests
+    {
+        private readonly Mock<ILogger<NoOpTelemetryService>> _mockLogger;
+        private readonly NoOpTelemetryService _service;
+
+        public NoOpTelemetryServiceTests()
+        {
+            _mockLogger = new Mock<ILogger<NoOpTelemetryService>>();
+            _service = new NoOpTelemetryService(_mockLogger.Object);
+        }
+
+        [Fact]
+        public void TrackAuthenticationAttempt_DoesNotThrowException()
+        {
+            // Arrange
+            var userId = "test-user";
+            var success = true;
+            var reason = "Valid credentials";
+
+            // Act
+            var action = () => _service.TrackAuthenticationAttempt(userId, success, reason);
+
+            // Assert
+            action.Should().NotThrow();
+        }
+
+        [Fact]
+        public void TrackVideoUpload_DoesNotThrowException()
+        {
+            // Arrange
+            var userId = "test-user";
+            var fileSizeBytes = 1024L;
+            var duration = TimeSpan.FromSeconds(30);
+            var success = true;
+
+            // Act
+            var action = () => _service.TrackVideoUpload(userId, fileSizeBytes, duration, success);
+
+            // Assert
+            action.Should().NotThrow();
+        }
+
+        [Fact]
+        public void TrackDatabaseQuery_DoesNotThrowException()
+        {
+            // Arrange
+            var queryName = "GetStudentById";
+            var duration = TimeSpan.FromMilliseconds(50);
+
+            // Act
+            var action = () => _service.TrackDatabaseQuery(queryName, duration);
+
+            // Assert
+            action.Should().NotThrow();
+        }
+
+        [Fact]
+        public void TrackApiEndpoint_DoesNotThrowException()
+        {
+            // Arrange
+            var endpoint = "/api/students";
+            var responseTime = TimeSpan.FromMilliseconds(100);
+            var statusCode = 200;
+
+            // Act
+            var action = () => _service.TrackApiEndpoint(endpoint, responseTime, statusCode);
+
+            // Assert
+            action.Should().NotThrow();
+        }
+
+        [Fact]
+        public void TrackException_DoesNotThrowException()
+        {
+            // Arrange
+            var exception = new Exception("Test exception");
+            var properties = new Dictionary<string, string> { { "Key", "Value" } };
+
+            // Act
+            var action = () => _service.TrackException(exception, properties);
+
+            // Assert
+            action.Should().NotThrow();
+        }
+
+        [Fact]
+        public void TrackAuthenticationAttempt_LogsDebugMessage()
+        {
+            // Arrange
+            var userId = "test-user";
+            var success = true;
+            var reason = "Valid credentials";
+
+            // Act
+            _service.TrackAuthenticationAttempt(userId, success, reason);
+
+            // Assert
+            _mockLogger.Verify(
+                x => x.Log(
+                    LogLevel.Debug,
+                    It.IsAny<EventId>(),
+                    It.Is<It.IsAnyType>((v, t) => v.ToString()!.Contains("Application Insights disabled")),
+                    It.IsAny<Exception>(),
+                    It.IsAny<Func<It.IsAnyType, Exception?, string>>()),
+                Times.Once);
+        }
+    }
+}