Remove RemoveMethodCallVisitor

[jevanlingen]

What's changed?

Remove RemoveMethodInvocationsVisitor and update the RemoveUnneededAssertion recipe as it did use that vistor.

What's your motivation?

As the core's RemoveMethodInvocationsVisitor recipe can now also remove static methods (openrewrite/rewrite#4754), there is not need for the RemoveMethodCallVisitor anymore.

Any additional context

I went a little overboard with improving the RemoveUnneededAssertion recipe. I did not only replace the remove-method-invocation visitors, but also rewrote most of the other stuff. It used it's own form of TriFunction (called a InvokeRemoveMethodCallVisitor), which made the code a little hard to get. I reverted it to a native method, which is more in line with the rest of the code we tend to write.

Checklist

• I've added unit tests to cover both positive and negative cases
• I've read and applied the recipe conventions and best practices
• I've used the IntelliJ IDEA auto-formatter on affected files

[timtebeek]

Great to see the reduction here; thanks for this and the change upstream!