Merge pull request #559 from Chris0Jeky/fix/520-board-restore-success-toast

fix: add success toast after board restore (#520)

Author: Chris0Jeky

backend/tests/Taskdeck.Api.Tests/RateLimitingApiTests.cs

@@ -40,13 +40,17 @@ public async Task AuthEndpoints_ShouldThrottleAfterBurst_ByClientIp()
     [Fact]
     public async Task AuthEndpoints_ShouldRecoverAfterWindowReset()
     {
+        // Use a 3-second window so CI slowness cannot reset the window between the
+        // setup request and the throttle probe — one permit is consumed by the first
+        // SendInvalidLoginAsync call and the window must stay active long enough for
+        // SendInvalidLoginUntilThrottledAsync to observe the 429.
         using var factory = CreateFactoryWithRateLimits(
             authPermitLimit: 1,
-            authWindowSeconds: 1);
+            authWindowSeconds: 3);
         using var client = factory.CreateClient();
 
         (await SendInvalidLoginAsync(client)).StatusCode.Should().Be(HttpStatusCode.Unauthorized);
-        var throttled = await SendInvalidLoginUntilThrottledAsync(client);
+        var throttled = await SendInvalidLoginUntilThrottledAsync(client, maxAttempts: 15);
         var retryAfterSeconds = await AssertThrottleContractAsync(throttled, RateLimitingPolicyNames.AuthPerIp);
         using var recovered = await SendInvalidLoginUntilRecoveredAsync(client, retryAfterSeconds);
         recovered.StatusCode.Should().Be(HttpStatusCode.Unauthorized);

frontend/taskdeck-web/src/tests/views/ArchiveView.spec.ts

@@ -231,6 +231,37 @@ describe('ArchiveView', () => {
     setItemSpy.mockRestore()
   })
 
+  it('shows error toast when board restore fails', async () => {
+    const confirmSpy = vi.spyOn(window, 'confirm').mockReturnValue(true)
+
+    mocks.getBoards.mockResolvedValue([
+      {
+        id: 'board-archived',
+        name: 'Failing Board',
+        description: null,
+        isArchived: true,
+        createdAt: new Date().toISOString(),
+        updatedAt: new Date().toISOString(),
+      },
+    ])
+    mocks.updateBoard.mockRejectedValue(new Error('network error'))
+
+    const wrapper = mount(ArchiveView)
+    await waitForAsyncUi()
+
+    const restoreBoardButton = findButtonByText(wrapper, 'Restore Board')
+    await restoreBoardButton.trigger('click')
+    await waitForAsyncUi()
+
+    expect(mocks.updateBoard).toHaveBeenCalledWith('board-archived', { isArchived: false })
+    expect(mocks.successToast).not.toHaveBeenCalled()
+    expect(mocks.errorToast).toHaveBeenCalledWith('Failed to restore board')
+    // Board remains in the list
+    expect(wrapper.findAll('.td-archive-list--section .td-archive-row')).toHaveLength(1)
+
+    confirmSpy.mockRestore()
+  })
+
   it('hides archived board from default list and reveals it when hidden toggle is enabled', async () => {
     mocks.getBoards.mockResolvedValue([
       {