Some compilation issues are occuring after running switch pattern matching recipe

[Jenson3210]

What's changed?

If any of the branches returns a value, we do (for now) not convert it to a switch as it is invalid.
Besides this change: If any of the values throws, we do allow this.
We used to unwrap if-statements if they were the only one -> no longer happening as only expressions and Throws are allowed to be written in that syntax.

I know we can enhance to do return from switch but I first want to stabilize the existing recipe before working on:

• Enhance IfElseIfConstructToSwitch to support "return" statements #776
• Convert assigning Switch statements to Switch expressions #718

What's your motivation?

Clean up compilation issues occuring during run of the recipe.

Anyone you would like to review specifically?

@timtebeek

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

[github-actions]

Some suggestions could not be made:

• src/test/java/org/openrewrite/java/migrate/util/MigrateCollectionsSingletonMapTest.java
  • lines 108-108

[github-actions]

Some suggestions could not be made:

• src/test/java/org/openrewrite/java/migrate/util/MigrateCollectionsSingletonMapTest.java
  • lines 108-108

[github-actions]

Some suggestions could not be made:

• src/test/java/org/openrewrite/java/migrate/util/MigrateCollectionsSingletonMapTest.java
  • lines 108-108

[timtebeek]

Thanks for the isolated fixes here! Makes reviewing a lot easier.