test(gom-43): prevent nil dereference in StreamOptions test

SantiagoDePolonia · claude · SantiagoDePolonia · commit 06b6da1a4bf7 · 2026-02-24T12:36:11.000+01:00
Use t.Fatalf for status assertion and add nil guard on
capturedResponsesReq before accessing StreamOptions.

Co-Authored-By: Claude Opus 4.6 &lt;noreply@anthropic.com&gt;
diff --git a/internal/server/handlers_test.go b/internal/server/handlers_test.go
@@ -609,9 +609,12 @@ func TestStreamingResponses_DoesNotInjectStreamOptions(t *testing.T) {
 	}
 
 	if rec.Code != http.StatusOK {
-		t.Errorf("expected status 200, got %d", rec.Code)
+		t.Fatalf("expected status 200, got %d", rec.Code)
 	}
 
+	if provider.capturedResponsesReq == nil {
+		t.Fatalf("expected capturedResponsesReq to be set, got nil")
+	}
 	if provider.capturedResponsesReq.StreamOptions != nil {
 		t.Errorf("Responses streaming should NOT have StreamOptions injected, got: %+v", provider.capturedResponsesReq.StreamOptions)
 	}

Original file line number	Diff line number	Diff line change
`@@ -609,9 +609,12 @@ func TestStreamingResponses_DoesNotInjectStreamOptions(t *testing.T) {`
`609`	`609`	`}`
`610`	`610`
`611`	`611`	`if rec.Code != http.StatusOK {`
`612`		`- t.Errorf("expected status 200, got %d", rec.Code)`
	`612`	`+ t.Fatalf("expected status 200, got %d", rec.Code)`
`613`	`613`	`}`
`614`	`614`
	`615`	`+ if provider.capturedResponsesReq == nil {`
	`616`	`+ t.Fatalf("expected capturedResponsesReq to be set, got nil")`
	`617`	`+ }`
`615`	`618`	`if provider.capturedResponsesReq.StreamOptions != nil {`
`616`	`619`	`t.Errorf("Responses streaming should NOT have StreamOptions injected, got: %+v", provider.capturedResponsesReq.StreamOptions)`
`617`	`620`	`}`