Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test client init to support headers argument #1966

Merged
merged 8 commits into from Dec 3, 2022

Conversation

roy-pstr
Copy link
Contributor

@roy-pstr roy-pstr commented Nov 28, 2022

The starting point for contributions should usually be a discussion

Simple documentation typos may be raised as stand-alone pull requests, but otherwise please ensure you've discussed your proposal prior to issuing a pull request.

This will help us direct work appropriately, and ensure that any suggested changes have been okayed by the maintainers.

starlette/testclient.py Outdated Show resolved Hide resolved
tests/test_testclient.py Outdated Show resolved Hide resolved
tests/test_testclient.py Outdated Show resolved Hide resolved
@Kludex Kludex added this to the Version 0.23.0 milestone Dec 2, 2022
roy-pstr and others added 4 commits December 2, 2022 21:59
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
@Kludex
Copy link
Sponsor Member

Kludex commented Dec 3, 2022

Thanks @roy-pstr ! :)

@Kludex
Copy link
Sponsor Member

Kludex commented Dec 3, 2022

Just for record keeping...

Even though it's possible to add the headers to the TestClient after the client is instantiated, it's more comfortable to add the headers' parameter. It's not only about practicality, but I believe people will make a good use of this feature.

Also, do not take this PR as an argument to add the other parameters supported by httpx client.

@Kludex Kludex merged commit b77a41d into encode:master Dec 3, 2022
aminalaee pushed a commit that referenced this pull request Feb 13, 2023
Co-authored-by: Marcelo Trylesinski <marcelotryle@gmail.com>
Co-authored-by: Roy Pasternak <roy@larium.co>
Co-authored-by: Roy Pasternak <roy-pstr>
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.

None yet

4 participants