diff options
| author | James O'Doherty <james@theodohertyfamily.com> | 2026-05-29 21:16:13 -0400 |
|---|---|---|
| committer | James O'Doherty <james@theodohertyfamily.com> | 2026-05-29 21:16:13 -0400 |
| commit | 0f3806f77164af99466bfc8c0d7d5f85f9ec078f (patch) | |
| tree | 034b3c7d2688345a01b2d7fa26575944242e6b8c /internal/cli/cli_test.go | |
| parent | d2173cdbc03884ecd9534e9369f8ebe1634f7e9c (diff) | |
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.
Diffstat (limited to 'internal/cli/cli_test.go')
| -rw-r--r-- | internal/cli/cli_test.go | 2 |
1 files changed, 0 insertions, 2 deletions
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 } |