Improve httpclient 4->5 AuthScheme-related migration

[pdelagrave]

What's changed?

The main UpgradeApacheHttpClient_5 declarative recipe now:

• Refactors the BasicSchemeFactory() and DigestSchemeFactory() constructors usage to remove the deprecated argument use.
• Rename classes AuthSchemeProvider and AuthSchemes with their new names.
• Rename method AuthScheme.getSchemeName() to AuthScheme.getName()

What's your motivation?

Fix a reported issue with missing auth migrations in http client.

Any additional context

The Apache HttpClient release notes and migration guide are missing many renamed classes, methods, etc. Hence adding more support as required.

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

[pdelagrave]

I've converted it all to declarative as we can simply use org.openrewrite.java.DeleteMethodArgument.

[pdelagrave]

Because the first draft of that change was in its own recipe we had to include a description.

description: RFC 7616 mandates the use of UTF-8. This constructor is now deprecated and httpclient v5 ignores any provided charset, ensuring that only UTF-8 is used.

The recipeList: items doesn't support being documented with a description. Has it ever been considered?

It could be useful for example in this case where the recipe called is very generic, and the recipe name + arguments don't provide enough context to explain why we're doing that transformation.

[timtebeek]

Great to see how simple this has become with the changes upstream; thanks for the (re)work!