DELETE params does not behave like GET
Nobody has claimed this yet.
Assessment
- Difficulty
- 5/5
- Estimated time
- Over a week
- Newbie friendliness
- 30/100
- Issue type
- Feature
- Clarity
- Mostly clear
- Activity status
- Stale
- Tech stack
- python
- Domain
- api, testing-qa
Research direction
Start with the TestApp.delete entry in the linked API documentation and compare its params behavior with get() and post(). The issue names no source files or tests, so locate the request-handling and documentation paths before deciding the compatibility impact. Done means the intended DELETE parameter semantics are agreed, implemented, and documented.
Written by the indexing model from the issue text.
Description
Documentation Suggestion
https://docs.pylonsproject.org/projects/webtest/en/latest/api.html#webtest.app.TestApp.delete
The doc says that Do a DELETE request. Similar to get().
But actually, the parameter params of get() create a querystring.
And params in DELETE create an url encode version of the parameter in a body.
so it behave more like post() than get()
Describe the improvement
As an improvement, actually, a breaking change should be made, where params should act like get()
- Dominant language
- Python
- Stars
- 346
- Forks
- 113
- PR merge metrics
- No merged PRs in 30d
Getting set up
- No Dockerfile or Docker Compose file
- No pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from Pylons/webtest
-
Attempt at double decoding of PATH_INFO when using Django and CookieJarPossibly taken @gyanu2507 claimed this 45 days ago. Open
Difficulty 4/5 3-5 days Newbie friendliness 45/100
-
Difficulty 2/5 1-3 hours Newbie friendliness 35/100
-
Inline type hintsOpen
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
Difficulty 5/5 Over a week Newbie friendliness 25/100
-
Difficulty 3/5 1-2 days Newbie friendliness 42/100
Similar issues
-
Difficulty 1/5 Under an hour Newbie friendliness 72/100
-
[Bug]: Importing weave changes the process-wide localePossibly taken @himanshu748 claimed this today. Openbug
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
Maintainers usually reply within 1 day
-
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 74/100
-
UX
Difficulty 1/5 Under an hour Newbie friendliness 72/100
mediajunkie/piper-morgan-product#2008 · 2 comments ·