Skip to content

Conversation

@adambro
Copy link
Contributor

@adambro adambro commented May 27, 2016

This is a typical scenario on tests, where REQUEST_URI contains path and
all params to perform a request. No need to set QUERY_STRING in
environment separately.

This is a typical scenario on tests, where REQUEST_URI contains path and
all params to perform a request. No need to set QUERY_STRING in
environment separately.
@coveralls
Copy link

Coverage Status

Coverage increased (+0.006%) to 96.609% when pulling ab5fa78 on adambro:request_uri_with_params into 30cfe3c on slimphp:3.x.

@akrabat
Copy link
Member

akrabat commented Jul 26, 2016

Which tests don't set up the Request object correctly?

@adambro
Copy link
Contributor Author

adambro commented Jul 26, 2016

I've meant my own test cases where I use the mock factory to create
request. It is not about tests from framework itself.

Anyway REQUEST_URI can contain query string parameters so would be nice to
account for it.

On 26 Jul 2016 11:16, "Rob Allen" [email protected] wrote:

Which tests don't set up the Request object correctly?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1896 (comment), or mute
the thread
https://github.com/notifications/unsubscribe-auth/AAnoES8eVt-N5mndIYiehqQXy1fhq0Rjks5qZdBZgaJpZM4IobbU
.

@akrabat akrabat added this to the 3.5.0 milestone Jul 26, 2016
@akrabat akrabat merged commit ab5fa78 into slimphp:3.x Jul 26, 2016
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.

3 participants