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

feat(firefox): support Response.text()/Response.json()/Response.buffer() #4063

Merged
merged 1 commit into from Feb 25, 2019

Conversation

aslushnikov
Copy link
Contributor

This patch:

  • implements Response.buffer() and other methods
  • splits out relevant tests into a separate test suites
  • implements testServer.enableGzip() method to optionally gzip
    certain routes in tests
  • adds tests to make sure Response.text() returns expected results
    for binary and compressed responses.

This patch:
- implements Response.buffer() and other methods
- splits out relevant tests into a separate test suites
- implements `testServer.enableGzip()` method to optionally gzip
  certain routes in tests
- adds tests to make sure `Response.text()` returns expected results
  for binary and compressed responses.
@aslushnikov aslushnikov changed the title feat(firefox): support Response.buffer(), Response.json() and Response.text() feat(firefox): support Response.text()/Response.json()/Response.buffer() Feb 25, 2019
@aslushnikov aslushnikov merged commit 56dafd7 into puppeteer:master Feb 25, 2019
@aslushnikov aslushnikov deleted the responses branch February 25, 2019 03:31
kiku-jw pushed a commit to kiku-jw/puppeteer that referenced this pull request Apr 6, 2019
…uppeteer#4063)

This patch:
- implements Response.buffer() and other methods
- splits out relevant tests into a separate test suites
- implements `testServer.enableGzip()` method to optionally gzip
  certain routes in tests
- adds tests to make sure `Response.text()` returns expected results
  for binary and compressed responses.
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

1 participant