From b25debb0a6142e633edf195d44d474c1cb3ae5ab Mon Sep 17 00:00:00 2001 From: dev-lead Date: Sun, 10 May 2026 01:16:36 -0700 Subject: [PATCH] fix(delegation_test): remove stray '}' breaking go vet (closes #257) MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Commit 97768272 ("test(delegation): add isDeliveryConfirmedSuccess helper") left a stray closing brace on line 1265, immediately after the proper function-closing brace on 1264. Result: delegation_test.go:1265:1: expected declaration, found '}' `go vet ./internal/handlers/...` halts at this syntax error before any type-checking begins, masking other latent issues in the package. This is the THIRD silent main-breaker found in this debugging session, all from missing build/vet gates on PRs (see internal#219 — CI hardening RFC). --- workspace-server/internal/handlers/delegation_test.go | 1 - 1 file changed, 1 deletion(-) diff --git a/workspace-server/internal/handlers/delegation_test.go b/workspace-server/internal/handlers/delegation_test.go index 427e71b2..38c63206 100644 --- a/workspace-server/internal/handlers/delegation_test.go +++ b/workspace-server/internal/handlers/delegation_test.go @@ -1262,4 +1262,3 @@ func TestExecuteDelegation_CleanProxyResponse_Unchanged(t *testing.T) { t.Errorf("unmet sqlmock expectations: %v", err) } } -} -- 2.45.2