This method doesn't add any real value.

…229)c

In SLF4J, the last argument of the logging methods will be treated specially and removed from the formatted string argument array if it's a Throwable.

When the last argument was a `.toString()` call we stripped it even if it was a `Throwable` which changed the behaviour.

https://www.slf4j.org/faq.html#paramException

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.text.FindAndReplace?organizationId=ODQ2MGExMTUtNDg0My00N2EwLTgzMGMtNGE1NGExMTBmZDkw#defaults%3DW3sidmFsdWUiOiI6anVuaXQtKC4qKTpsYXRlc3QucmVsZWFzZSIsIm5hbWUiOiJmaW5kIn0seyJ2YWx1ZSI6IjpqdW5pdC0kMTo1LisiLCJuYW1lIjoicmVwbGFjZSJ9LHsidmFsdWUiOiJUcnVlIiwibmFtZSI6InJlZ2V4In0seyJ2YWx1ZSI6ImJ1aWxkLmdyYWRsZS5rdHMiLCJuYW1lIjoiZmlsZVBhdHRlcm4ifV0%3D

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/builder/zYwEKw0Gm?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.xml.security.RemoveOwaspSuppressions?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl#defaults%3DW3sidmFsdWUiOiIyMDI1LTA2LTMwIiwibmFtZSI6ImN1dE9mZkRhdGUifV0%3D

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.gradle.UpdateGradleWrapper?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl#defaults%3DW3sidmFsdWUiOmZhbHNlLCJuYW1lIjoiYWRkSWZNaXNzaW5nIn1d

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.text.FindAndReplace?organizationId=ODQ2MGExMTUtNDg0My00N2EwLTgzMGMtNGE1NGExMTBmZDkw#defaults%3DW3sibmFtZSI6ImZpbmQiLCJ2YWx1ZSI6Imp1bml0LWp1cGl0ZXItZW5naW5lOjUuKyJ9LHsibmFtZSI6InJlcGxhY2UiLCJ2YWx1ZSI6Imp1bml0LWp1cGl0ZXItZW5naW5lOjUuMTMuMyJ9LHsibmFtZSI6ImZpbGVQYXR0ZXJuIiwidmFsdWUiOiIqKi9idWlsZC5ncmFkbGUua3RzIn1d

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.ReorderAnnotations?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl

Co-authored-by: Moderne <team@moderne.io>

* tests: Don't if-wrap logs when args are getters

* more and better tests
started to implement

* Update src/main/java/org/openrewrite/java/logging/slf4j/WrapExpensiveLogStatementsInConditionals.java

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

* Update src/main/java/org/openrewrite/java/logging/slf4j/WrapExpensiveLogStatementsInConditionals.java

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

* implemented more
added more tests

* clean up

* Adopt `@ValueSource(strings = {...})` for simplicity

* Illustrate a case missed so far

* Minor changes

* Fixed test

* Condense logic

---------

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Co-authored-by: Tim te Beek <tim@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.staticanalysis.RemoveRedundantNullCheckBeforeInstanceof?organizationId=ODQ2MGExMTUtNDg0My00N2EwLTgzMGMtNGE1NGExMTBmZDkw

Co-authored-by: Moderne <team@moderne.io>

Use this link to re-run the recipe: https://app.moderne.io/recipes/org.openrewrite.recipes.rewrite.OpenRewriteRecipeBestPractices?organizationId=QUxML09wZW4gU291cmNlL09wZW5SZXdyaXRl

Co-authored-by: Moderne <team@moderne.io>