Skip to content

test(method-override): preserve query parameters during query override#4995

Open
xu91102 wants to merge 1 commit into
honojs:mainfrom
xu91102:codex/middleware-test-scout
Open

test(method-override): preserve query parameters during query override#4995
xu91102 wants to merge 1 commit into
honojs:mainfrom
xu91102:codex/middleware-test-scout

Conversation

@xu91102

@xu91102 xu91102 commented Jun 2, 2026

Copy link
Copy Markdown

What this adds

Adds one test for query-based methodOverride().

The test verifies that the _method query parameter is removed before the request is re-dispatched, while unrelated query parameters are preserved for the overridden route.

This is a test-only change.

The author should do the following, if applicable

  • Add tests
  • Run tests
  • bun run format:fix && bun run lint:fix to format the code
  • Add TSDoc/JSDoc to document the code

Verified locally with:

  • npx vitest --run src/middleware/method-override/index.test.ts
  • npx eslint src/middleware/method-override/index.test.ts
  • npx prettier --check src/middleware/method-override/index.test.ts
  • npx tsc --noEmit --pretty false
  • git diff --check

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant