test: remove duplicate heredoc test in security.test.ts (#2583)

"should reject heredoc syntax in operator combinations" tested a single
case ("Input << EOF") that is fully covered by the broader "should reject
heredoc syntax" test (3 cases: << EOF, <<- HEREDOC, <<MARKER).

1 test removed, 0 expect() calls lost (the exact input pattern is covered
by the remaining test).

-- qa/dedup-scanner

Co-authored-by: spawn-qa-bot <qa@openrouter.ai>
Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-authored-by: L <6723574+louisgv@users.noreply.github.com>
This commit is contained in:
A 2026-03-13 09:50:48 -07:00 committed by GitHub
parent 2dead43404
commit afcc1665b2
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -483,10 +483,6 @@ describe("validatePrompt", () => {
expect(() => validatePrompt("Start server &")).toThrow("shell syntax");
});
it("should reject heredoc syntax in operator combinations", () => {
expect(() => validatePrompt("Input << EOF")).toThrow("shell syntax");
});
it("should accept legitimate uses of ampersand and pipes in text", () => {
expect(() => validatePrompt("Smith & Jones corporation")).not.toThrow();
expect(() => validatePrompt("Rock & roll music")).not.toThrow();