forked from square/okhttp
-
Notifications
You must be signed in to change notification settings - Fork 0
[pull] master from square:master #111
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
Open
pull
wants to merge
36
commits into
AKJUS:master
Choose a base branch
from
square:master
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
If Robolectric not active, then unit tests for Android will fail annoyingly.
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…ugin to v2.1.21-2.0.2 (#8847) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
…8848) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….13.0.0 (#8851) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Change MockWebServer to use Okio's new Socket interface This replaces the Stream interface introduced in a recent alpha release. * apiDump * Only buffer once --------- Co-authored-by: Jesse Wilson <[email protected]>
Also remove @ExperimentalOkHttpApi in a few places. Also rename 'shutdown' to 'close' in MockWebServer and Dispatcher. Also rename QueueDispatcher.enqueueResponse() to QueueDispatcher.enqueue. Co-authored-by: Jesse Wilson <[email protected]>
* Change RecordedRequest.body to a ByteString Immutable is good. * apiDump --------- Co-authored-by: Jesse Wilson <[email protected]>
* Fix a liveness bug in MockWebServer It was possible to configure delays that would be honored after MockWebServer was shut down. With this fix any delay will be abandoned once the server is closed. * Update mockwebserver/src/main/kotlin/mockwebserver3/internal/SleepNanos.kt Co-authored-by: Jake Wharton <[email protected]> --------- Co-authored-by: Jesse Wilson <[email protected]> Co-authored-by: Jake Wharton <[email protected]>
Co-authored-by: Jesse Wilson <[email protected]>
* Decompose the request line into individual properties Replace the 'path' property with the URL's encodedPath property. Rename the 'requestUrl' property to 'url'. * Spotless * Fix up DnsOverHttpsTest * Fix some test failures
* Change some MockWebServer ergonomics Now calling MockWebServer.port and similar functions will not implicity start the server. It is instead necessary to explicitly call the start() function. The problem with the old behavior was that reading the port field could fail with an IOException, which is weird behavior for reading a property. * Fixup test * Fix another test --------- Co-authored-by: Jesse Wilson <[email protected]>
* New @startstop extension for JUnit 5 This is a new take on our JUnit 5 extension. Rather than creating and managing the MockWebServer instance in the extension, we let the user create it and all the extension does is manage the lifecycle. Note that this annotation doesn't require any external configuration - it doesn't need a system-property to opt-into 'automatic' extensions, and it doesn't require a class-level extension either. * apiDump --------- Co-authored-by: Jesse Wilson <[email protected]>
* New @startstop extension for JUnit 5 This is a new take on our JUnit 5 extension. Rather than creating and managing the MockWebServer instance in the extension, we let the user create it and all the extension does is manage the lifecycle. Note that this annotation doesn't require any external configuration - it doesn't need a system-property to opt-into 'automatic' extensions, and it doesn't require a class-level extension either. * apiDump * Switch to @startstop for JUnit 5 tests * Fixup visibility --------- Co-authored-by: Jesse Wilson <[email protected]>
* New @startstop extension for JUnit 5 This is a new take on our JUnit 5 extension. Rather than creating and managing the MockWebServer instance in the extension, we let the user create it and all the extension does is manage the lifecycle. Note that this annotation doesn't require any external configuration - it doesn't need a system-property to opt-into 'automatic' extensions, and it doesn't require a class-level extension either. * apiDump * Switch to @startstop for JUnit 5 tests * Fixup visibility * Drop the parameters-based JUnit 5 extension I think the @startstop one is sufficient. --------- Co-authored-by: Jesse Wilson <[email protected]>
* Permit RecordedRequest.body to be null or empty Null if we didn't capture a body, and empty if we captured an empty body. * Track API change * Fixup more tests * spotless --------- Co-authored-by: Jesse Wilson <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
They're stable now. Co-authored-by: Jesse Wilson <[email protected]>
Also rename 'sequenceNumber' to 'exchangeIndex'. This may be useful to test features like connection reuse. Co-authored-by: Jesse Wilson <[email protected]>
* Replace SocketPolicy with a new type, SocketEffect SocketPolicy encapsulated two things: - a trigger (request start, response body, etc) - an effect (close the socket, close the stream, stall, etc.) It also had some special cases for effects like failing the TLS handshake. With this PR there's a new class, SocketEffect, that determines what to do. It's assigned to one of 5 fields for different triggers. # Conflicts: # mockwebserver/src/main/kotlin/mockwebserver3/MockWebServer.kt * apiDump * Track API change * Track more API changes * Spotless --------- Co-authored-by: Jesse Wilson <[email protected]>
* Update dependency org.robolectric:robolectric to v4.15 * Skip test on early robolectric --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Yuri Schimke <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Change RecordedRequest.chunkSizes to be nullable Now that it's all Kotlin we can usefully distinguish between null (not chunked) and empty list (chunked with zero bytes). Also remove the experimental annotation from RecordedRequest and delete the experimental annotation. * apiDump --------- Co-authored-by: Jesse Wilson <[email protected]>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
….0 (#8875) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
as old link points to 404 now.
…8879) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
* Update dependency org.junit.platform:junit-platform-console to v1.13.2 * Update dependency org.junit.vintage:junit-vintage-engine to v5.13.2 (#8882) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.1)
Can you help keep this open source service alive? 💖 Please sponsor : )