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

EXPERIMENTAL: Update to use alpha version of mockwebserver3-junit5 #500

Closed
wants to merge 1 commit into from

Conversation

sleberknight
Copy link
Member

@sleberknight sleberknight commented Jun 20, 2024

This is an experimental branch, not intended to commit to main since it is using an alpha version of OkHttp's MockWebServer which is using experimental APIs, subject to change. This is mainly to show how we can change when OkHttp 5.0.0 is released. It is also to ensure we keep up with changes to the alpha APIs.

Details:

  • Change POM to use mockwebserver3-junit5 5.0.0-alpha.14 instead of mockwebserver 4.12.0
  • Fix imports in RecordedRequests/Test (they're now just "import mockwebserver3.*"
  • Update RecordedRequestsTest to use MockWebServerExtension and inject it into the setup method. No longer need to explicitly close it, since the extension handles that. Change instantiations of MockResponse since they've changed them such that the Kotlin code uses JvmOverloads to generate a bunch of constructors for use by Java code.
  • Revert KiwiXmlAssertTest to use isExactlyInstanceOf instead of isInstanceOf and add a TODO whether we should actually do that or leave as isInstanceOf.

…nit5

This is an experimental branch, not intended to commit to main since
it is using an alpha version of OkHttp's MockWebServer which is using
experimental APIs, subject to change. This is mainly to show how we
can change when OkHttp 5.0.0 is released. It is also to ensure we keep up
with changes to the alpha APIs.

Details:

* Change POM to use mockwebserver3-junit5 5.0.0-alpha.14
  instead of mockwebserver 4.12.0
* Fix imports in RecordedRequests/Test (they're now just
  "import mockwebserver3.*"
* Update RecordedRequestsTest to use MockWebServerExtension
  and inject it into the setup method. No longer need to explicitly
  close it, since the extension handles that. Change instantiations
  of MockResponse since they've changed them such that the Kotlin
  code uses JvmOverloads to generate a bunch of constructors for
  use by Java code.
* Revert KiwiXmlAssertTest to use isExactlyInstanceOf instead of
  isInstanceOf and add a TODO whether we should actually do that
  or leave as isInstanceOf.
@sleberknight sleberknight added the investigation Something that needs to be investigated before implementation can proceed label Jun 20, 2024
@sleberknight sleberknight self-assigned this Jun 20, 2024
@sleberknight sleberknight changed the title EXPERIMENTAL branch: Update to use alpha version of mockwebserver3-ju… EXPERIMENTAL branch: Update to use alpha version of mockwebserver3-junit5 Jun 20, 2024
@sleberknight sleberknight changed the title EXPERIMENTAL branch: Update to use alpha version of mockwebserver3-junit5 EXPERIMENTAL: Update to use alpha version of mockwebserver3-junit5 Jun 20, 2024
Copy link

@sleberknight
Copy link
Member Author

Keeping the branch (for now) but closing the PR, since it is not intended to be merged.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigation Something that needs to be investigated before implementation can proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant