Add unit tests for MockDataBackend and QueryProcessor services

Author: jongalloway

tests/NLWebNet.Tests.MSTest/NLWebNet.Tests.MSTest.csproj

@@ -0,0 +1,24 @@
+<Project Sdk="Microsoft.NET.Sdk">
+
+  <PropertyGroup>
+    <TargetFramework>net10.0</TargetFramework>
+    <LangVersion>latest</LangVersion>
+    <ImplicitUsings>enable</ImplicitUsings>
+    <Nullable>enable</Nullable>
+    <IsPackable>false</IsPackable>
+  </PropertyGroup>
+
+  <ItemGroup>
+    <PackageReference Include="MSTest" Version="3.2.0" />
+    <PackageReference Include="Microsoft.Testing.Extensions.CodeCoverage" Version="17.10.1" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <ProjectReference Include="..\..\src\NLWebNet\NLWebNet.csproj" />
+  </ItemGroup>
+
+  <ItemGroup>
+    <Using Include="Microsoft.VisualStudio.TestTools.UnitTesting" />
+  </ItemGroup>
+
+</Project>

tests/NLWebNet.Tests.MSTest/Services/MockDataBackendTests.cs

@@ -0,0 +1,87 @@
+using NLWebNet.Models;
+using NLWebNet.Services;
+
+namespace NLWebNet.Tests.Services;
+
+[TestClass]
+public class MockDataBackendTests
+{
+    private MockDataBackend _mockDataBackend = null!;
+
+    [TestInitialize]
+    public void Initialize()
+    {
+        var logger = new TestLogger<MockDataBackend>();
+        _mockDataBackend = new MockDataBackend(logger);
+    }
+
+    [TestMethod]
+    public async Task SearchAsync_ReturnsResults()
+    {
+        // Arrange
+        var query = "NET Core"; // Use a term that appears in the sample data
+
+        // Act
+        var results = await _mockDataBackend.SearchAsync(query, null, 10, CancellationToken.None);
+        var resultsList = results.ToList();
+
+        // Assert
+        Assert.IsTrue(resultsList.Count > 0);
+        foreach (var result in resultsList)
+        {
+            Assert.IsNotNull(result.Name);
+            Assert.IsTrue(result.Name.Length > 0);
+            Assert.IsNotNull(result.Url);
+            Assert.IsTrue(result.Url.Length > 0);
+            Assert.IsNotNull(result.Description);
+            Assert.IsTrue(result.Description.Length > 0);
+        }
+    }
+
+    [TestMethod]
+    public async Task SearchAsync_WithSiteFilter_ReturnsFilteredResults()
+    {
+        // Arrange
+        var query = "documentation"; // Use a term that appears in the sample data
+        var site = "docs.microsoft.com"; // Use a site that exists in sample data
+
+        // Act
+        var results = await _mockDataBackend.SearchAsync(query, site, 10, CancellationToken.None);
+        var resultsList = results.ToList();
+
+        // Assert
+        Assert.IsTrue(resultsList.Count > 0);
+        foreach (var result in resultsList)
+        {
+            Assert.AreEqual(site, result.Site);
+        }
+    }
+
+    [TestMethod]
+    public async Task SearchAsync_RespectsMaxResults()
+    {
+        // Arrange
+        var query = "microsoft"; // Use a term that will match multiple results
+        var maxResults = 3;
+
+        // Act
+        var results = await _mockDataBackend.SearchAsync(query, null, maxResults, CancellationToken.None);
+        var resultsList = results.ToList();
+
+        // Assert
+        Assert.IsTrue(resultsList.Count <= maxResults);
+    }
+
+    [TestMethod]
+    [DataRow("")]
+    [DataRow(null)]
+    public async Task SearchAsync_WithEmptyQuery_ReturnsEmptyResults(string? query)
+    {
+        // Act
+        var results = await _mockDataBackend.SearchAsync(query!, null, 10, CancellationToken.None);
+        var resultsList = results.ToList();
+
+        // Assert
+        Assert.AreEqual(0, resultsList.Count);
+    }
+}

tests/NLWebNet.Tests.MSTest/Services/QueryProcessorTests.cs

@@ -0,0 +1,124 @@
+using Microsoft.Extensions.Logging;
+using Microsoft.Extensions.Options;
+using NLWebNet.Models;
+using NLWebNet.Services;
+
+namespace NLWebNet.Tests.Services;
+
+[TestClass]
+public class QueryProcessorTests
+{
+    private QueryProcessor _queryProcessor = null!;
+    private ILogger<QueryProcessor> _logger = null!;
+
+    [TestInitialize]
+    public void Initialize()
+    {
+        _logger = new TestLogger<QueryProcessor>();
+        _queryProcessor = new QueryProcessor(_logger);
+    }
+
+    [TestMethod]
+    public void ValidateRequest_WithValidRequest_ReturnsTrue()
+    {
+        // Arrange
+        var request = new NLWebRequest
+        {
+            Query = "test query",
+            Mode = QueryMode.List
+        };
+
+        // Act
+        var result = _queryProcessor.ValidateRequest(request);
+
+        // Assert
+        Assert.IsTrue(result);
+    }
+
+    [TestMethod]
+    public void ValidateRequest_WithEmptyQuery_ReturnsFalse()
+    {
+        // Arrange
+        var request = new NLWebRequest
+        {
+            Query = "",
+            Mode = QueryMode.List
+        };
+
+        // Act
+        var result = _queryProcessor.ValidateRequest(request);
+
+        // Assert
+        Assert.IsFalse(result);
+    }
+
+    [TestMethod]
+    public void ValidateRequest_WithNullQuery_ReturnsFalse()
+    {
+        // Arrange
+        var request = new NLWebRequest
+        {
+            Query = null!,
+            Mode = QueryMode.List
+        };
+
+        // Act
+        var result = _queryProcessor.ValidateRequest(request);
+
+        // Assert
+        Assert.IsFalse(result);
+    }
+
+    [TestMethod]
+    public void GenerateQueryId_WithNullRequestQueryId_GeneratesNewId()
+    {
+        // Arrange
+        var request = new NLWebRequest
+        {
+            Query = "test query",
+            QueryId = null
+        };
+
+        // Act
+        var result = _queryProcessor.GenerateQueryId(request);
+
+        // Assert
+        Assert.IsNotNull(result);
+        Assert.IsTrue(result.Length > 0);
+    }
+
+    [TestMethod]
+    public void GenerateQueryId_WithExistingQueryId_ReturnsExistingId()
+    {
+        // Arrange
+        var existingId = "existing-id";
+        var request = new NLWebRequest
+        {
+            Query = "test query",
+            QueryId = existingId
+        };
+
+        // Act
+        var result = _queryProcessor.GenerateQueryId(request);
+
+        // Assert
+        Assert.AreEqual(existingId, result);
+    }
+
+    [TestMethod]
+    public async Task ProcessQueryAsync_ReturnsQuery()
+    {
+        // Arrange
+        var request = new NLWebRequest
+        {
+            Query = "test query",
+            Mode = QueryMode.List
+        };
+
+        // Act
+        var result = await _queryProcessor.ProcessQueryAsync(request, CancellationToken.None);
+
+        // Assert
+        Assert.AreEqual("test query", result);
+    }
+}

tests/NLWebNet.Tests.MSTest/TestLogger.cs

@@ -0,0 +1,20 @@
+using Microsoft.Extensions.Logging;
+
+namespace NLWebNet.Tests;
+
+/// <summary>
+/// Simple test logger implementation for unit tests.
+/// </summary>
+/// <typeparam name="T">The type being logged.</typeparam>
+public class TestLogger<T> : ILogger<T>
+{
+    public IDisposable? BeginScope<TState>(TState state) where TState : notnull => null;
+
+    public bool IsEnabled(LogLevel logLevel) => true;
+
+    public void Log<TState>(LogLevel logLevel, EventId eventId, TState state, Exception? exception, Func<TState, Exception?, string> formatter)
+    {
+        // In a real test, you might want to capture log messages for assertions
+        Console.WriteLine($"[{logLevel}] {formatter(state, exception)}");
+    }
+}