…possible for future error occurrences. (#853)

…possible for future error occurrences. (#853)

…possible for future error occurrences. (#853)

…possible for future error occurrences. (#853)

* WiP

* WiP 2 - am able to get `.build()` added, but seems to drop the after padding (including the newline), leading to it changing wrapping.

* Expanding tests and conversions to include headers methods

* Adapted the failing recipe to add as much debug/retry information as possible for future error occurrences. (#853)

* Reapplying paddings from arguments as they were prior to chaining `.build()`, and backpatching the parameter types onto the various method invocations. Feels like there could be a lot of improvement still with how this done, as it feels pretty hacky right now.

* wip

* WiP - Ripped out a lot

* wip classpath jars

* iterate over a list of old to new replacement methods

* use template for adding .build() on MockResponseBuilder

* try to handle chained call

* inline static patchReturnTypeAndName method call

* Apply suggestions from code review

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update src/test/java/org/openrewrite/java/testing/junit5/UpgradeOkHttpMockWebServerTest.java

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* Update src/main/java/org/openrewrite/java/testing/junit5/UpdateMockWebServerMockResponse.java

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>

* patch .build() return type and name

* replace edits lost in merge

* update recipes.csv

* add test for method name changes and test with MockResponse return type

* fix unit tests

* clean up

* convert MockResponse "with" methods

* fix line wrap

* Use separate test classes for the two cases

* Remove unused field

* remove duplicate code in patchReturnTypeAndName

* Inline the delegating `patchReturnTypeAndName`

* Replace the ChangePackage with a call to UpdateMockWebServerMockResponse

* Use `ChangeMethodName` instead of `MethodInvocationReplacement`

* Inline the `patchReturnTypeAndName` used once

* Move existing recipes into `getRecipeList()`

* Move replacements down to only usage

* Change `methodInvocationsToAdjust` to local field

* Drop unnecessary list of indices

* Always carry over prefix from arg

* Simplify `patchBuilderBuildReturnTypeAndName`

* Use inlined `JavaType.ShallowClass.build`

* Inline `patchBuilderBuildReturnTypeAndName`

* Final touches

* Change return type before renaming method; add missing imports

---------

Co-authored-by: Jente Sondervorst <jentesondervorst@gmail.com>
Co-authored-by: David Grieve <david@moderne.io>
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Tim te Beek <tim@moderne.io>