MinimalViableSpacing recipe does not handle extends correctly

[JohannisK]

What's changed?

• Added test for implements and extends
• Fixed a bug when dealing with classes extending

[timtebeek]

Thanks for the quick fixes here!

I wonder if we can now also remove this oddly added whitespace:

rewrite/rewrite-java/src/main/java/org/openrewrite/java/internal/template/JavaTemplateJavaExtension.java

Line 168 in 6e66e18

c = c.getPadding().withExtends(c.getPadding().getExtends().withBefore(Space.format(" ")));

[JohannisK]

Thanks for the quick fixes here!

I wonder if we can now also remove this oddly added whitespace:

rewrite/rewrite-java/src/main/java/org/openrewrite/java/internal/template/JavaTemplateJavaExtension.java

Line 168 in 6e66e18

c = c.getPadding().withExtends(c.getPadding().getExtends().withBefore(Space.format(" ")));

It seems this takes care of the space before the extends keyword itself.

[timtebeek]

Thanks for the quick fixes here!
I wonder if we can now also remove this oddly added whitespace:

rewrite/rewrite-java/src/main/java/org/openrewrite/java/internal/template/JavaTemplateJavaExtension.java

Line 168 in 6e66e18

c = c.getPadding().withExtends(c.getPadding().getExtends().withBefore(Space.format(" ")));

It seems this takes care of the space before the extends keyword itself.

Hmm; that's what line 77 does as well; guess that's a separate code path then. No need to change that here.