Skip to content

Commit b877e0c

Browse files
MackinnonBuckCopilot
authored andcommitted
Revert multi-client broadcast test skips for CLI 1.0.7
Remove the t.Skip/pytest.skip/Fact(Skip=...) additions that were disabling the multi-client broadcast tests across Go, Python, and C#. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent 978c63d commit b877e0c

File tree

3 files changed

+2
-3
lines changed

3 files changed

+2
-3
lines changed

dotnet/test/MultiClientTests.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -92,7 +92,7 @@ public async Task DisposeAsync()
9292

9393
private CopilotClient Client2 => _client2 ?? throw new InvalidOperationException("Client2 not initialized");
9494

95-
[Fact(Skip = "CLI 1.0.7 no longer broadcasts external_tool events to secondary clients")]
95+
[Fact]
9696
public async Task Both_Clients_See_Tool_Request_And_Completion_Events()
9797
{
9898
var tool = AIFunctionFactory.Create(MagicNumber, "magic_number");

go/internal/e2e/multi_client_test.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ func TestMultiClient(t *testing.T) {
4444
t.Cleanup(func() { client2.ForceStop() })
4545

4646
t.Run("both clients see tool request and completion events", func(t *testing.T) {
47-
t.Skip("Skipped: CLI 1.0.7 no longer broadcasts external_tool events to secondary clients")
4847
ctx.ConfigureForTest(t)
4948

5049
type SeedParams struct {

python/e2e/test_multi_client.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -189,7 +189,7 @@ async def test_both_clients_see_tool_request_and_completion_events(
189189
self, mctx: MultiClientContext
190190
):
191191
"""Both clients see tool request and completion events."""
192-
pytest.skip("CLI 1.0.7 no longer broadcasts external_tool events to secondary clients")
192+
193193

194194
class SeedParams(BaseModel):
195195
seed: str = Field(description="A seed value")

0 commit comments

Comments
 (0)