refactor: extract tests to library test project

src/src/StevesBot.Library/Discord/ServicesExtensions.cs

@@ -17,7 +17,9 @@ public static class ServicesExtensions
         c.BaseAddress = new Uri(discordOptions.ApiUrl);
         c.DefaultRequestHeaders.Authorization = new("Bot", discordOptions.AppToken);
 
-        var userAgentString = $"DiscordBot (https://github.com/StevanFreeborn/steves-bot, {StevesBotInstrumentation.SourceVersion})";
+        var instrumentation = sp.GetRequiredService<IInstrumentation>();
+
+        var userAgentString = $"DiscordBot (https://github.com/StevanFreeborn/steves-bot, {instrumentation.SourceVersion})";
         c.DefaultRequestHeaders.Add("User-Agent", userAgentString);
       })
       .AddStandardResilienceHandler();

src/src/StevesBot.Library/Telemetry/HostExtensions.cs

@@ -11,11 +11,15 @@ namespace StevesBot.Library.Telemetry;
 
 public static class HostExtensions
 {
-  public static HostApplicationBuilder AddTelemetry(this HostApplicationBuilder builder)
+  public static IHostApplicationBuilder AddTelemetry(this IHostApplicationBuilder builder, Func<IInstrumentation> instrumentationFunc)
   {
     ArgumentNullException.ThrowIfNull(builder);
 
-    builder.Services.AddSingleton<StevesBotInstrumentation>();
+    builder.Services.AddSingleton(instrumentationFunc);
+
+    var instrumentation = builder.Services
+      .BuildServiceProvider()
+      .GetRequiredService<IInstrumentation>();
 
     var seq = new SeqOptions();
     builder.Configuration.GetSection(nameof(SeqOptions)).Bind(seq);
@@ -28,11 +32,11 @@ public static class HostExtensions
     builder.Services.AddOpenTelemetry()
       .ConfigureResource(resource =>
       {
-        resource.AddService(StevesBotInstrumentation.SourceName, StevesBotInstrumentation.SourceVersion);
+        resource.AddService(instrumentation.SourceName, instrumentation.SourceVersion);
         resource.AddAttributes(new Dictionary<string, object>
         {
-          ["service.name"] = StevesBotInstrumentation.SourceName,
+          ["service.name"] = instrumentation.SourceName,
-          ["service.version"] = StevesBotInstrumentation.SourceVersion,
+          ["service.version"] = instrumentation.SourceVersion,
           ["service.instance.id"] = Environment.MachineName,
           ["service.namespace"] = "stevesbot",
           ["service.environment"] = Environment.GetEnvironmentVariable("DOTNET_ENVIRONMENT") ?? "production",
@@ -49,7 +53,7 @@ public static class HostExtensions
       })
       .WithTracing(tb =>
       {
-        tb.AddSource(StevesBotInstrumentation.SourceName);
+        tb.AddSource(instrumentation.SourceName);
         tb.AddAspNetCoreInstrumentation();
         tb.AddHttpClientInstrumentation();
         tb.AddOtlpExporter(o =>

src/src/StevesBot.Library/Telemetry/IInstrumentation.cs

@@ -0,0 +1,10 @@
+using System.Diagnostics;
+
+namespace StevesBot.Library.Telemetry;
+
+public interface IInstrumentation : IDisposable
+{
+  string SourceName { get; }
+  string SourceVersion { get; }
+  ActivitySource Source { get; }
+}

src/src/StevesBot.Library/Telemetry/StevesBotInstrumentation.cs

@@ -1,15 +0,0 @@
-using System.Diagnostics;
-
-namespace StevesBot.Library.Telemetry;
-
-public sealed class StevesBotInstrumentation : IDisposable
-{
-  public const string SourceName = "StevesBot.Worker";
-  public const string SourceVersion = "0.0.0";
-  public ActivitySource Source { get; } = new(SourceName, SourceVersion);
-
-  public void Dispose()
-  {
-    Source.Dispose();
-  }
-}

src/src/StevesBot.Library/Telemetry/StevesBotWebhookInstrumentation.cs

@@ -0,0 +1,18 @@
+using System.Diagnostics;
+
+namespace StevesBot.Library.Telemetry;
+
+public sealed class StevesBotWebhookInstrumentation : IInstrumentation
+{
+  private const string SourceNameValue = "StevesBot.Webhook";
+  private const string SourceVersionValue = "0.0.0";
+
+  public string SourceName { get; } = SourceNameValue;
+  public string SourceVersion { get; } = SourceVersionValue;
+  public ActivitySource Source { get; } = new ActivitySource(SourceNameValue, SourceVersionValue);
+
+  public void Dispose()
+  {
+    Source.Dispose();
+  }
+}

src/src/StevesBot.Library/Telemetry/StevesBotWorkerInstrumentation.cs

@@ -0,0 +1,18 @@
+using System.Diagnostics;
+
+namespace StevesBot.Library.Telemetry;
+
+public sealed class StevesBotWorkerInstrumentation : IInstrumentation
+{
+  private const string SourceNameValue = "StevesBot.Worker";
+  private const string SourceVersionValue = "0.0.0";
+
+  public string SourceName { get; } = SourceNameValue;
+  public string SourceVersion { get; } = SourceVersionValue;
+  public ActivitySource Source { get; } = new ActivitySource(SourceNameValue, SourceVersionValue);
+
+  public void Dispose()
+  {
+    Source.Dispose();
+  }
+}

src/src/StevesBot.Webhook/Program.cs

@@ -1,5 +1,7 @@
 var builder = WebApplication.CreateBuilder(args);
 
+builder.AddTelemetry(static () => new StevesBotWebhookInstrumentation());
+
 builder.Services
   .AddOptionsWithValidateOnStart<SubscriptionOptions>()
   .BindConfiguration(nameof(SubscriptionOptions))

src/src/StevesBot.Webhook/Usings.cs

@@ -12,6 +12,7 @@ global using StevesBot.Library.Discord;
 global using StevesBot.Library.Discord.Common;
 global using StevesBot.Library.Discord.Rest;
 global using StevesBot.Library.Discord.Rest.Requests;
+global using StevesBot.Library.Telemetry;
 global using StevesBot.Webhook.YouTube;
 global using StevesBot.Webhook.YouTube.Data;
 global using StevesBot.Webhook.YouTube.Handlers;

src/src/StevesBot.Webhook/YouTube/Handlers/NotificationHandler.cs

@@ -40,7 +40,7 @@ internal static class NotificationHandler
       return Results.Ok();
     }
 
-    if (lastPostedStream.Value == videoId)
+    if (lastPostedStream.HasValue(videoId))
     {
       logger.LogInformation("Video ID {VideoId} has already been posted. Skipping notification.", videoId);
       return Results.Ok();

src/src/StevesBot.Webhook/appsettings.Example.json

@@ -25,5 +25,10 @@
   "DiscordNotificationOptions": {
     "ChannelId": "ChannelId",
     "MessageFormat": "MessageFormat"
+  },
+  "SeqOptions": {
+    "ServerUrl": "ServerUrl",
+    "ApiKeyHeader": "ApiKeyHeader",
+    "ApiKey": "ApiKey"
   }
 }

src/src/StevesBot.Worker/Program.cs

@@ -11,7 +11,7 @@ builder.Services.AddSingleton(static sp =>
   return discordOptions;
 });
 
-builder.AddTelemetry();
+builder.AddTelemetry(static () => new StevesBotWorkerInstrumentation());
 
 builder.Services.AddSingleton<IWebSocketFactory, WebSocketFactory>();
 builder.Services.AddSingleton(TimeProvider.System);

src/tests/StevesBot.Library.Tests/Unit/CreateMessageRequestTests.cs

@@ -1,4 +1,4 @@
-namespace StevesBot.Worker.Tests.Unit;
+namespace StevesBot.Library.Tests.Unit;
 
 public class CreateMessageRequestTests
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordClientOptionsTests.cs

@@ -1,4 +1,4 @@
-namespace StevesBot.Worker.Tests.Unit;
+namespace StevesBot.Library.Tests.Unit;
 
 public class DiscordClientOptionsTests
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordMessageReferenceTests.cs

@@ -1,4 +1,4 @@
-namespace StevesBot.Worker.Tests.Unit;
+namespace StevesBot.Library.Tests.Unit;
 
 public class DiscordMessageReferenceTests
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordMessageReferenceTypesTests.cs

@@ -1,4 +1,4 @@
-namespace StevesBot.Worker.Tests.Unit;
+namespace StevesBot.Library.Tests.Unit;
 
 public class DiscordMessageReferenceTypesTests
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordMessageTests.cs

@@ -0,0 +1 @@
+namespace StevesBot.Library.Tests.Unit;

src/tests/StevesBot.Library.Tests/Unit/DiscordRestClientExceptionTests.cs

@@ -1,4 +1,4 @@
-namespace StevesBot.Worker.Tests.Unit;
+namespace StevesBot.Library.Tests.Unit;
 
 public class DiscordRestClientExceptionTests
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordRestClientTests.cs

@@ -1,6 +1,4 @@
-using Microsoft.AspNetCore.Http.HttpResults;
+namespace StevesBot.Library.Tests.Unit;
-
-namespace StevesBot.Worker.Tests.Unit;
 
 public sealed class DiscordRestClientTests : IDisposable
 {

src/tests/StevesBot.Library.Tests/Unit/DiscordUserTests.cs

@@ -0,0 +1 @@
+namespace StevesBot.Library.Tests.Unit;

src/tests/StevesBot.Library.Tests/Unit/HostExtensionsTests.cs

@@ -0,0 +1,6 @@
+namespace StevesBot.Library.Tests.Unit;
+
+public class HostExtensionsTests
+{
+
+}

src/tests/StevesBot.Library.Tests/Unit/SeqOptionsTests.cs

@@ -0,0 +1,66 @@
+namespace StevesBot.Library.Tests.Unit;
+
+public class SeqOptionsTests
+{
+  [Fact]
+  public void Constructor_WhenCalled_ItShouldReturnInstance()
+  {
+    var result = new SeqOptions();
+
+    result.ServerUrl.Should().Be(string.Empty);
+    result.ApiKeyHeader.Should().Be(string.Empty);
+    result.ApiKey.Should().Be(string.Empty);
+    result.IsEnabled.Should().BeFalse();
+    result.LogEndpoint.Should().Be($"{string.Empty}/ingest/otlp/v1/logs");
+    result.TraceEndpoint.Should().Be($"{string.Empty}/ingest/otlp/v1/traces");
+    result.AuthHeader.Should().Be($"{string.Empty}={string.Empty}");
+  }
+
+  [Fact]
+  public void Constructor_WhenPropertiesSet_ItShouldReturnCorrectValues()
+  {
+    var serverUrl = "http://example.com";
+    var apiKeyHeader = "ApiKeyHeader";
+    var apiKey = "ApiKeyValue";
+
+    var result = new SeqOptions
+    {
+      ServerUrl = serverUrl,
+      ApiKeyHeader = apiKeyHeader,
+      ApiKey = apiKey
+    };
+
+    result.ServerUrl.Should().Be(serverUrl);
+    result.ApiKeyHeader.Should().Be(apiKeyHeader);
+    result.ApiKey.Should().Be(apiKey);
+    result.IsEnabled.Should().BeTrue();
+    result.LogEndpoint.Should().Be($"{serverUrl}/ingest/otlp/v1/logs");
+    result.TraceEndpoint.Should().Be($"{serverUrl}/ingest/otlp/v1/traces");
+    result.AuthHeader.Should().Be($"{apiKeyHeader}={apiKey}");
+  }
+
+  [Theory]
+  [InlineData(null, null, null, false)]
+  [InlineData("", "", "", false)]
+  [InlineData("http://example.com", null, null, false)]
+  [InlineData("http://example.com", "", "", false)]
+  [InlineData(null, "ApiKeyHeader", null, false)]
+  [InlineData(null, null, "ApiKeyValue", false)]
+  [InlineData("http://example.com", "ApiKeyHeader", "ApiKeyValue", true)]
+  [InlineData("http://example.com", "", "ApiKeyValue", false)]
+  [InlineData("", "ApiKeyHeader", "ApiKeyValue", false)]
+  [InlineData("http://example.com", "ApiKeyHeader", null, false)]
+  [InlineData("", "ApiKeyHeader", null, false)]
+  [InlineData(null, "ApiKeyHeader", "ApiKeyValue", false)]
+  public void IsEnabled_WhenPropertiesAreEmpty_ItShouldReturnFalse(string? serverUrl, string? apiKeyHeader, string? apiKey, bool expected)
+  {
+    var result = new SeqOptions
+    {
+      ServerUrl = serverUrl!,
+      ApiKeyHeader = apiKeyHeader!,
+      ApiKey = apiKey!
+    };
+
+    result.IsEnabled.Should().Be(expected);
+  }
+}

src/tests/StevesBot.Library.Tests/Unit/StevesBotWebhookInstrumentationTests.cs

@@ -0,0 +1,32 @@
+namespace StevesBot.Library.Tests.Unit;
+
+public sealed class StevesBotWebhookInstrumentationTests : IDisposable
+{
+  private readonly StevesBotWebhookInstrumentation _instrumentation = new();
+
+  [Fact]
+  public void SourceName_WhenCalled_ItShouldReturnCorrectName()
+  {
+    _instrumentation.SourceName.Should().Be("StevesBot.Webhook");
+  }
+
+  [Fact]
+  public void SourceVersion_WhenCalled_ItShouldReturnCorrectVersion()
+  {
+    _instrumentation.SourceVersion.Should().Be("0.0.0");
+  }
+
+  [Fact]
+  public void Constructor_WhenCalled_ItShouldReturnInstance()
+  {
+    using var result = new StevesBotWorkerInstrumentation();
+    using var expectedSource = new ActivitySource(_instrumentation.SourceName, _instrumentation.SourceVersion);
+
+    result.Source.Should().BeEquivalentTo(expectedSource);
+  }
+
+  public void Dispose()
+  {
+    _instrumentation.Dispose();
+  }
+}

src/tests/StevesBot.Library.Tests/Unit/StevesBotWorkerInstrumentationTests.cs

@@ -0,0 +1,32 @@
+namespace StevesBot.Library.Tests.Unit;
+
+public sealed class StevesBotWorkerInstrumentationTests : IDisposable
+{
+  private readonly StevesBotWorkerInstrumentation _instrumentation = new();
+
+  [Fact]
+  public void SourceName_WhenCalled_ItShouldReturnCorrectName()
+  {
+    _instrumentation.SourceName.Should().Be("StevesBot.Worker");
+  }
+
+  [Fact]
+  public void SourceVersion_WhenCalled_ItShouldReturnCorrectVersion()
+  {
+    _instrumentation.SourceVersion.Should().Be("0.0.0");
+  }
+
+  [Fact]
+  public void Constructor_WhenCalled_ItShouldReturnInstance()
+  {
+    using var result = new StevesBotWorkerInstrumentation();
+    using var expectedSource = new ActivitySource(_instrumentation.SourceName, _instrumentation.SourceVersion);
+
+    result.Source.Should().BeEquivalentTo(expectedSource);
+  }
+
+  public void Dispose()
+  {
+    _instrumentation.Dispose();
+  }
+}

src/tests/StevesBot.Library.Tests/Usings.cs

@@ -1,5 +1,16 @@
+global using System.Diagnostics;
+global using System.Net;
+global using System.Text.Json;
+
 global using Microsoft.Extensions.DependencyInjection;
+global using Microsoft.Extensions.Logging;
+
+global using Moq;
+
+global using RichardSzalay.MockHttp;
 
 global using StevesBot.Library.Discord;
 global using StevesBot.Library.Discord.Common;
 global using StevesBot.Library.Discord.Rest;
+global using StevesBot.Library.Discord.Rest.Requests;
+global using StevesBot.Library.Telemetry;