test(naming): regression for get_current_location title-on-claim fix

Two xUnit cases pinned to PowerShellTools.GetCurrentLocation:

- GetCurrentLocation_ConsoleSwitched_SetsWindowTitle verifies that
  SetWindowTitleAsync is called when FindReadyPipeAsync reports a
  newly-claimed (unowned) pipe. Without 5b3ae35's fix this test
  fails with Moq.MockException ("Expected invocation on the mock at
  least once, but was never performed").

- GetCurrentLocation_NotSwitched_DoesNotSetWindowTitle is the
  companion guard: established consoles must not re-trigger
  TryAssignNameToPid + SetWindowTitleAsync on every call, otherwise
  the round-trip is wasted (TryAssignNameToPid is idempotent and
  returns null on the second call) and the established title would
  be at risk of accidental rename if the queue logic ever changes.

Verified locally by toggling the if (consoleSwitched) block in
PowerShellTools.cs:109 — the first test fails as designed when the
guard is removed and passes when restored.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

Author: yotsuda

Tests/Unit/Proxy/PowerShellToolsTests.cs

@@ -135,6 +135,77 @@ public async Task GetCurrentLocation_PipeError_ReturnsErrorMessage()
         Assert.Contains("Pipe communication failed", result);
     }
 
+    [Fact]
+    public async Task GetCurrentLocation_ConsoleSwitched_SetsWindowTitle()
+    {
+        // Regression: GetCurrentLocation previously discarded FindReadyPipeAsync's
+        // consoleSwitched flag with `_`, so a get_current_location call that
+        // claimed an unowned (Import-Module-only) console left the window title
+        // at the OnImport placeholder "#PID ____" instead of assigning a name.
+        // Arrange: discovery reports a switched (newly-claimed) pipe.
+        _mockPipeDiscoveryService
+            .Setup(s => s.FindReadyPipeAsync(It.IsAny<string>(), It.IsAny<CancellationToken>()))
+            .ReturnsAsync(new PipeDiscoveryResult(TestPipeName, true, new List<string>(), null));
+
+        _mockPowerShellService
+            .Setup(s => s.GetCurrentLocationFromPipeAsync(TestPipeName, It.IsAny<CancellationToken>()))
+            .ReturnsAsync("Location [FileSystem]: C:\\test");
+
+        _mockPowerShellService
+            .Setup(s => s.SetWindowTitleAsync(TestPipeName, It.IsAny<string>(), It.IsAny<CancellationToken>()))
+            .Returns(Task.CompletedTask);
+
+        _mockPipeDiscoveryService
+            .Setup(s => s.CollectAllCachedOutputsAsync(It.IsAny<string>(), TestPipeName, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(new CachedOutputResult("", ""));
+
+        // Act
+        var result = await PowerShellTools.GetCurrentLocation(
+            _mockPowerShellService.Object,
+            _mockPipeDiscoveryService.Object,
+            agent_id: TestAgentId);
+
+        // Assert: SetWindowTitleAsync must be invoked at least once for the
+        // switched pipe. Title text itself is generated by ConsoleSessionManager
+        // and varies per run, so accept any non-empty string.
+        _mockPowerShellService.Verify(
+            s => s.SetWindowTitleAsync(TestPipeName, It.IsAny<string>(), It.IsAny<CancellationToken>()),
+            Times.AtLeastOnce);
+        Assert.Contains("Location [FileSystem]", result);
+    }
+
+    [Fact]
+    public async Task GetCurrentLocation_NotSwitched_DoesNotSetWindowTitle()
+    {
+        // Companion guard: an established pipe (consoleSwitched=false) must not
+        // re-trigger the title-assignment path. Setting the title every call
+        // would either redo the round-trip wastefully (TryAssignNameToPid is
+        // idempotent so it returns null) or — if the name dictionary ever
+        // changes — risk renaming a console mid-session.
+        _mockPipeDiscoveryService
+            .Setup(s => s.FindReadyPipeAsync(It.IsAny<string>(), It.IsAny<CancellationToken>()))
+            .ReturnsAsync(new PipeDiscoveryResult(TestPipeName, false, new List<string>(), null));
+
+        _mockPowerShellService
+            .Setup(s => s.GetCurrentLocationFromPipeAsync(TestPipeName, It.IsAny<CancellationToken>()))
+            .ReturnsAsync("Location [FileSystem]: C:\\test");
+
+        _mockPipeDiscoveryService
+            .Setup(s => s.CollectAllCachedOutputsAsync(It.IsAny<string>(), TestPipeName, It.IsAny<CancellationToken>()))
+            .ReturnsAsync(new CachedOutputResult("", ""));
+
+        // Act
+        await PowerShellTools.GetCurrentLocation(
+            _mockPowerShellService.Object,
+            _mockPipeDiscoveryService.Object,
+            agent_id: TestAgentId);
+
+        // Assert
+        _mockPowerShellService.Verify(
+            s => s.SetWindowTitleAsync(It.IsAny<string>(), It.IsAny<string>(), It.IsAny<CancellationToken>()),
+            Times.Never);
+    }
+
     #endregion
 
     #region InvokeExpression Tests