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

Align Fetch Request tests with WebKit ones #2549

Merged
merged 2 commits into from Feb 3, 2016

Conversation

youennf
Copy link
Contributor

@youennf youennf commented Feb 1, 2016

Adding new tests coming from WebKit:

  • Checking how headers are cloned when cloning a request
  • Various (good and failing) JSON parsing when calling Request.json()
  • Checking that a HEAD request cannot take a body
    Making some minor clean-up to some tests.

@hoppipolla-critic-bot
Copy link

Critic review: https://critic.hoppipolla.co.uk/r/6159

This is an external review system which you may optionally use for the code review of your pull request.

In order to help critic track your changes, please do not make in-place history rewrites (e.g. via git rebase -i or git commit --amend) when updating this pull request.

var badJSONValues = ["undefined", "{", "a", "["];
badJSONValues.forEach(function(value) {
promise_test(function(test) {
var request = new Request("", {"method": "POST", "body": "{"});
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This isn't using the test value as the body.

@jdm
Copy link
Contributor

jdm commented Feb 1, 2016

This looks fine with that one change made.

jdm added a commit that referenced this pull request Feb 3, 2016
Align Fetch Request tests with WebKit ones
@jdm jdm merged commit ecd723d into web-platform-tests:master Feb 3, 2016
@jdm
Copy link
Contributor

jdm commented Feb 3, 2016

Thanks!

@youennf youennf deleted the align-with-webkit-tests branch April 21, 2016 13:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants