[Test] bUnit: IssuesPage filter and search wiring (#125)

tests/Web.Tests.Bunit/Components/Features/Issues/IssuesPageFilterTests.cs

@@ -0,0 +1,260 @@
+// ============================================
+// Copyright (c) 2026. All rights reserved.
+// File Name :     IssuesPageFilterTests.cs
+// Company :       mpaulosky
+// Author :        Matthew Paulosky
+// Solution Name : IssueManager
+// Project Name :  Web.Tests.Bunit
+// =============================================
+
+namespace Web.Components.Features.Issues;
+
+/// <summary>
+/// bUnit tests for IssuesPage filter and search wiring.
+/// Verifies that filter values are correctly forwarded to <see cref="IIssueApiClient.GetAllAsync"/>.
+///
+/// Tests marked <c>[Fact(Skip = "Pending #116")]</c> depend on the filter-bug fix and/or
+/// the updated <see cref="IIssueApiClient.GetAllAsync"/> signature from issue #116:
+/// <c>GetAllAsync(page, pageSize, searchTerm, authorName, statusName, categoryName, cancellationToken)</c>.
+///
+/// Closes #125. Depends on #116.
+/// </summary>
+[ExcludeFromCodeCoverage]
+public class IssuesPageFilterTests : ComponentTestBase
+{
+	private readonly IIssueApiClient _mockIssueClient;
+
+	/// <summary>
+	/// Pre-seeded category for dropdown tests.
+	/// </summary>
+	private static readonly CategoryDto BugCategory = new(
+		ObjectId.GenerateNewId(),
+		"Bug",
+		"Bug category",
+		DateTime.UtcNow,
+		null,
+		false,
+		UserDto.Empty);
+
+	/// <summary>
+	/// Pre-seeded status for dropdown tests.
+	/// </summary>
+	private static readonly StatusDto OpenStatus = new(
+		ObjectId.GenerateNewId(),
+		"Open",
+		"Open status",
+		DateTime.UtcNow,
+		null,
+		false,
+		UserDto.Empty);
+
+	/// <summary>
+	/// Initializes a new instance of the <see cref="IssuesPageFilterTests"/> class,
+	/// registering a mock <see cref="IIssueApiClient"/> and pre-seeding dropdown data.
+	/// </summary>
+	public IssuesPageFilterTests()
+	{
+		_mockIssueClient = Substitute.For<IIssueApiClient>();
+		_mockIssueClient
+			.GetAllAsync(Arg.Any<int>(), Arg.Any<int>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>())
+			.Returns(Task.FromResult(PaginatedResponse<IssueDto>.Empty));
+		TestContext.Services.AddSingleton(_mockIssueClient);
+
+		// Seed category and status dropdown data so filters have selectable options.
+		// (ComponentTestBase registers the mocks; we reconfigure them here with real data.)
+		var categoryClient = TestContext.Services.GetRequiredService<ICategoryApiClient>();
+		categoryClient.GetAllAsync(Arg.Any<CancellationToken>())
+			.Returns(Task.FromResult<IEnumerable<CategoryDto>>([BugCategory]));
+
+		var statusClient = TestContext.Services.GetRequiredService<IStatusApiClient>();
+		statusClient.GetAllAsync(Arg.Any<CancellationToken>())
+			.Returns(Task.FromResult<IEnumerable<StatusDto>>([OpenStatus]));
+	}
+
+	/// <summary>
+	/// Verifies that on component mount GetAllAsync is called once with the default
+	/// parameters: page=1, pageSize=20, searchTerm=null, authorName=null.
+	/// </summary>
+	[Fact]
+	public void LoadIssues_OnInit_CallsApiWithDefaultParams()
+	{
+		// Act
+		TestContext.Render<IssuesPage>();
+
+		// Assert — exactly one call with the default/null filter values
+		_ = _mockIssueClient.Received(1)
+			.GetAllAsync(1, 20, null, null, null, null, Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that typing a search term and clicking Search passes <c>searchTerm</c>
+	/// to <see cref="IIssueApiClient.GetAllAsync"/>.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: <c>LoadIssues</c> currently calls
+	/// <c>GetAllAsync(page, 20)</c> without forwarding <c>_searchTerm</c>.
+	/// Remove the Skip attribute once the component bug is fixed.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: LoadIssues does not yet forward _searchTerm to GetAllAsync")]
+	public async Task SearchBox_WhenFilled_PassesSearchTermToApi()
+	{
+		// Arrange
+		var cut = TestContext.Render<IssuesPage>();
+
+		// Act — type a search term then click Search
+		await cut.Find("#search").InputAsync(new ChangeEventArgs { Value = "my-bug" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert — the second call (after init) should carry the search term
+		_ = _mockIssueClient.Received()
+			.GetAllAsync(1, 20, "my-bug", null, null, null, Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that selecting a status from the dropdown and clicking Search
+	/// passes <c>statusName</c> to <see cref="IIssueApiClient.GetAllAsync"/>.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: <c>IIssueApiClient.GetAllAsync</c> does not yet
+	/// include a <c>statusName</c> parameter.
+	/// When #116 adds the parameter, update the assertion to:
+	/// <code>GetAllAsync(1, 20, null, null, "Open", null, Arg.Any&lt;CancellationToken&gt;())</code>
+	/// and remove the Skip attribute.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: IIssueApiClient.GetAllAsync does not yet include statusName parameter")]
+	public async Task StatusFilter_WhenSelected_PassesStatusNameToApi()
+	{
+		// Arrange
+		var cut = TestContext.Render<IssuesPage>();
+
+		// Act — select "Open" from the status filter, then click Search
+		await cut.Find("#status-filter").ChangeAsync(new ChangeEventArgs { Value = "Open" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert
+		// TODO (#116): Replace with the specific statusName assertion once the parameter is added:
+		// _mockIssueClient.Received().GetAllAsync(1, 20, null, null, "Open", null, Arg.Any<CancellationToken>());
+		_ = _mockIssueClient.Received()
+			.GetAllAsync(1, 20, Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that selecting a category from the dropdown and clicking Search
+	/// passes <c>categoryName</c> to <see cref="IIssueApiClient.GetAllAsync"/>.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: <c>IIssueApiClient.GetAllAsync</c> does not yet
+	/// include a <c>categoryName</c> parameter.
+	/// When #116 adds the parameter, update the assertion to:
+	/// <code>GetAllAsync(1, 20, null, null, null, "Bug", Arg.Any&lt;CancellationToken&gt;())</code>
+	/// and remove the Skip attribute.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: IIssueApiClient.GetAllAsync does not yet include categoryName parameter")]
+	public async Task CategoryFilter_WhenSelected_PassesCategoryNameToApi()
+	{
+		// Arrange
+		var cut = TestContext.Render<IssuesPage>();
+
+		// Act — select "Bug" from the category filter, then click Search
+		await cut.Find("#category-filter").ChangeAsync(new ChangeEventArgs { Value = "Bug" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert
+		// TODO (#116): Replace with the specific categoryName assertion once the parameter is added:
+		// _mockIssueClient.Received().GetAllAsync(1, 20, null, null, null, "Bug", Arg.Any<CancellationToken>());
+		_ = _mockIssueClient.Received()
+			.GetAllAsync(1, 20, Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that combining search term, status, and category all flow through
+	/// together to a single <see cref="IIssueApiClient.GetAllAsync"/> call.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: requires both the interface update (statusName/categoryName)
+	/// and the component bug fix (forwarding all filters in LoadIssues).
+	/// When #116 lands, update the assertion to:
+	/// <code>GetAllAsync(1, 20, "crash", null, "Open", "Bug", Arg.Any&lt;CancellationToken&gt;())</code>
+	/// and remove the Skip attribute.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: GetAllAsync missing statusName/categoryName params; LoadIssues does not forward filters")]
+	public async Task MultipleFilters_AllPassedToApiCombined()
+	{
+		// Arrange
+		var cut = TestContext.Render<IssuesPage>();
+
+		// Act — set all three filters then click Search
+		await cut.Find("#search").InputAsync(new ChangeEventArgs { Value = "crash" });
+		await cut.Find("#status-filter").ChangeAsync(new ChangeEventArgs { Value = "Open" });
+		await cut.Find("#category-filter").ChangeAsync(new ChangeEventArgs { Value = "Bug" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert
+		// TODO (#116): Replace with the full assertion once the interface and component are fixed:
+		// _mockIssueClient.Received().GetAllAsync(1, 20, "crash", null, "Open", "Bug", Arg.Any<CancellationToken>());
+		_ = _mockIssueClient.Received()
+			.GetAllAsync(1, 20, Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that clicking Clear resets all filter fields to <c>null</c> and
+	/// calls <see cref="IIssueApiClient.GetAllAsync"/> with the default parameters.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: once the component forwards filters to LoadIssues,
+	/// we can assert that the post-clear call uses all-null filter args.
+	/// When #116 lands, update the assertion to verify a third call with all nulls:
+	/// <code>GetAllAsync(1, 20, null, null, null, null, Arg.Any&lt;CancellationToken&gt;())</code>
+	/// and remove the Skip attribute.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: LoadIssues does not yet forward filter values; full clear verification needs the fix")]
+	public async Task ClearButton_ResetsAllFiltersAndCallsApiWithDefaults()
+	{
+		// Arrange — render and set a search term
+		var cut = TestContext.Render<IssuesPage>();
+		await cut.Find("#search").InputAsync(new ChangeEventArgs { Value = "something" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Act — click Clear
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Clear")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert — three total calls (init, search, clear); the clear call must use page 1 and null filters
+		// TODO (#116): Once fixed, assert the third call uses all null filter params:
+		// _mockIssueClient.Received(3).GetAllAsync(Arg.Any<int>(), ...);
+		// And use ReceivedCalls() to inspect the last call specifically.
+		_ = _mockIssueClient.Received(3)
+			.GetAllAsync(Arg.Any<int>(), Arg.Any<int>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+	}
+
+	/// <summary>
+	/// Verifies that changing a filter resets the current page back to 1 before
+	/// calling <see cref="IIssueApiClient.GetAllAsync"/>.
+	/// </summary>
+	/// <remarks>
+	/// Skipped pending issue #116: the Search button triggers <c>LoadIssues(1)</c>
+	/// directly, so pagination reset already happens structurally — but it cannot
+	/// be meaningfully verified until filter params are wired through LoadIssues.
+	/// Remove the Skip attribute and verify the page argument is 1 once #116 lands.
+	/// </remarks>
+	[Fact(Skip = "Pending #116: pagination reset verification requires filter-wiring fix from #116")]
+	public async Task FilterChange_ResetsPageToOne()
+	{
+		// Arrange — render (triggers init call at page 1)
+		var cut = TestContext.Render<IssuesPage>();
+
+		// Act — change the search term and click Search (should always use page 1)
+		await cut.Find("#search").InputAsync(new ChangeEventArgs { Value = "reset-test" });
+		await cut.FindAll("button").First(b => b.TextContent.Trim() == "Search")
+			.ClickAsync(new MouseEventArgs());
+
+		// Assert — the Search call must target page 1 regardless of prior navigation
+		_ = _mockIssueClient.Received()
+			.GetAllAsync(1, 20, Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<string?>(), Arg.Any<CancellationToken>());
+	}
+}