Skip to content

Comments

[12.x] Add missing tests for Request::fullUrlWithoutQuery#58814

Merged
taylorotwell merged 1 commit intolaravel:12.xfrom
miladev95:test-full-url-without-query
Feb 14, 2026
Merged

[12.x] Add missing tests for Request::fullUrlWithoutQuery#58814
taylorotwell merged 1 commit intolaravel:12.xfrom
miladev95:test-full-url-without-query

Conversation

@miladev95
Copy link
Contributor

This PR adds test coverage for the Request::fullUrlWithoutQuery method which was previously untested.

The new tests cover:

  • Removing a single query parameter by string key
  • Removing a single query parameter by array
  • Removing multiple query parameters at once
  • Removing the only query parameter (resulting in no query string)
  • Removing a non-existent key from a URL without query string
  • Root URL with query string
  • URL with trailing slash

@taylorotwell taylorotwell merged commit a46e50c into laravel:12.x Feb 14, 2026
47 of 72 checks passed
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.

2 participants