From 328a42144fd54ac99d1bf231e6114147e48b820f Mon Sep 17 00:00:00 2001 From: James O'Doherty Date: Fri, 29 May 2026 23:27:50 -0400 Subject: refactor: tidy project and stabilize E2E lifecycle tests - Improve routing logic in CLI for better readability. - Resolve flakiness in `TestNamespaceLifecycleAutomation` by increasing polling timeouts and adjusting tick intervals. - Verify project state with `go fmt`, `go vet`, and `golangci-lint`. - Ensure all unit and E2E tests pass deterministically. --- internal/cli/cli.go | 6 ++---- tests/e2e/lifecycle_test.go | 8 ++++---- 2 files changed, 6 insertions(+), 8 deletions(-) diff --git a/internal/cli/cli.go b/internal/cli/cli.go index 0e3b8ad..0a2ec96 100644 --- a/internal/cli/cli.go +++ b/internal/cli/cli.go @@ -42,13 +42,11 @@ func (a *App) Route() error { switch a.Args[1] { case "show-config": return a.showConfig() + case "profile": + return a.handleProfileCmd() } } - if len(a.Args) > 1 && a.Args[1] == "profile" { - return a.handleProfileCmd() - } - return a.Run() } diff --git a/tests/e2e/lifecycle_test.go b/tests/e2e/lifecycle_test.go index 67b1370..b6c001b 100644 --- a/tests/e2e/lifecycle_test.go +++ b/tests/e2e/lifecycle_test.go @@ -10,8 +10,8 @@ import ( ) func waitForPids(t *testing.T, pidsDir string, expectedCount int) { - timeout := time.After(2 * time.Second) - tick := time.NewTicker(50 * time.Millisecond) + timeout := time.After(5 * time.Second) + tick := time.NewTicker(100 * time.Millisecond) defer tick.Stop() for { @@ -38,8 +38,8 @@ func waitForPids(t *testing.T, pidsDir string, expectedCount int) { } func waitForLifecycle(t *testing.T, binaryPath, runtimeDir, profile string, expectedActive bool) { - timeout := time.After(2 * time.Second) - tick := time.NewTicker(50 * time.Millisecond) + timeout := time.After(5 * time.Second) + tick := time.NewTicker(100 * time.Millisecond) defer tick.Stop() for { -- cgit v1.2.3