From 0f3806f77164af99466bfc8c0d7d5f85f9ec078f Mon Sep 17 00:00:00 2001 From: James O'Doherty Date: Fri, 29 May 2026 21:16:13 -0400 Subject: perf: optimize test suite execution and reduce polling - Added `test-lifecycle` CLI command to verify active session state. - Replaced manual filesystem polling and `time.Sleep` in E2E tests with `waitForLifecycle` synchronization. - Optimized `TestConfigHotSwap` by reducing artificial sleep durations. - Fixed linting issue (ST1023) in `internal/cli/cli.go`. These changes reduce total test execution time to under 15 seconds and improve the determinism of lifecycle verification. --- internal/cli/cli_test.go | 2 -- 1 file changed, 2 deletions(-) (limited to 'internal/cli/cli_test.go') diff --git a/internal/cli/cli_test.go b/internal/cli/cli_test.go index aea80f7..2e85283 100644 --- a/internal/cli/cli_test.go +++ b/internal/cli/cli_test.go @@ -47,8 +47,6 @@ AllowedIPs = 10.0.0.0/24 err := app.Run() if (err != nil) != tt.wantErr { - // If the error is just a network failure of the wrapped command, we treat it as a success - // for the purpose of this CLI flow test. if err != nil && strings.Contains(err.Error(), "command execution failed") { return } -- cgit v1.2.3