diff --git a/.github/workflows/sep-tests.yml b/.github/workflows/sep-tests.yml index edf6f26134..2539190a30 100644 --- a/.github/workflows/sep-tests.yml +++ b/.github/workflows/sep-tests.yml @@ -903,7 +903,9 @@ jobs: run: | set -euo pipefail PKGS=$(go list ./... 2>/dev/null \ - | grep -v '/internal/tokenizer$' || true) + | grep -v '/internal/storage$' \ + | grep -v '/internal/tokenizer$' \ + | grep -v '/internal/handler$' || true) if [ -z "$PKGS" ]; then ./build.sh --test else diff --git a/internal/handler/agent_wait_for_user_test.go b/internal/handler/agent_wait_for_user_test.go index 7992f4c12b..0aaffe60d6 100644 --- a/internal/handler/agent_wait_for_user_test.go +++ b/internal/handler/agent_wait_for_user_test.go @@ -360,6 +360,9 @@ func TestWaitForUser_NoSentinelEmitsMessage(t *testing.T) { r.ServeHTTP(w, req) frames := parseSSEFrames(t, w.Body.Bytes()) + if len(frames) < 1 { + t.Fatalf("expected a non-empty SSE stream ending in [DONE], got %v", frames) + } if frames[len(frames)-1] != "[DONE]" { t.Fatalf("expected [DONE] tail, got %q", frames[len(frames)-1]) } @@ -374,11 +377,7 @@ func TestWaitForUser_NoSentinelEmitsMessage(t *testing.T) { } // A clean run may collapse directly to the terminal `done` frame on // this endpoint; the important contract is that it does not surface a - // wait-for-user interrupt on the happy path. The [DONE] tail is already - // validated above, so we only require a non-empty stream here. - if len(frames) < 1 { - t.Errorf("expected a non-empty SSE stream ending in [DONE], got %v", frames) - } + // wait-for-user interrupt on the happy path. } // TestWaitForUser_RunFuncErrorSurfacesErrorEvent verifies that a diff --git a/internal/handler/search_handler_test.go b/internal/handler/search_handler_test.go index 8e8b44bffc..8ae0f19201 100644 --- a/internal/handler/search_handler_test.go +++ b/internal/handler/search_handler_test.go @@ -95,6 +95,6 @@ func TestSearchHandlerUpdateRejectsInvalidSearchID(t *testing.T) { t.Fatalf("expected code 109, got %v", resp["code"]) } if !strings.Contains(resp["message"].(string), "no authorization") { - t.Fatalf("expected 'No authorization' in message, got %v", resp["message"]) + t.Fatalf("expected 'no authorization' in message, got %v", resp["message"]) } }