Add missing method impl interface

openrewrite / rewrite-migrate-java

OpenRewrite recipes for migrating to newer versions of Java.

Apache License 2.0

92 stars 65 forks source link

Add missing method impl interface #460

Closed darylrobbins closed 2 months ago

darylrobbins commented 2 months ago

What's changed?

AddMissingMethodImplementation will no longer add methods to interfaces
Added tests for AddMissingMethodImplementation

What's your motivation?

Fixes #459

Checklist

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

darylrobbins commented 2 months ago

@timtebeek UpdateBeanManagerMethodsTest > fireEvent() is failing for both of the PR's so this doesn't look like anything to do with my changes. FWIW, the tests all pass for me locally.