Jmockit to Mockito problem

[Gwynner-MJ]

Caused by: java.lang.RuntimeException: Error while visiting xxxxx\xxxxTest.java: java.lang.IndexOutOfBoundsException: Index 0 out of bounds for length 0 java.base/jdk.internal.util.Preconditions.outOfBounds(Preconditions.java:100) java.base/jdk.internal.util.Preconditions.outOfBoundsCheckIndex(Preconditions.java:106) java.base/jdk.internal.util.Preconditions.checkIndex(Preconditions.java:302) java.base/java.util.Objects.checkIndex(Objects.java:385) java.base/java.util.ArrayList.get(ArrayList.java:427) org.openrewrite.java.testing.jmockit.SetupStatementsRewriter.rewriteMethodBody(SetupStatementsRewriter.java:53) org.openrewrite.java.testing.jmockit.JMockitBlockToMockito$RewriteJMockitBlockVisitor.visitMethodDeclaration(JMockitBlockToMockito.java:69) org.openrewrite.java.testing.jmockit.JMockitBlockToMockito$RewriteJMockitBlockVisitor.visitMethodDeclaration(JMockitBlockToMockito.java:59) org.openrewrite.java.tree.J$MethodDeclaration.acceptJava(J.java:3808) org.openrewrite.java.tree.J.accept(J.java:58) org.openrewrite.TreeVisitor.visit(TreeVisitor.java:250) org.openrewrite.TreeVisitor.visitAndCast(TreeVisitor.java:320) org.openrewrite.java.JavaVisitor.visitRightPadded(JavaVisitor.java:1379) org.openrewrite.java.JavaVisitor.lambda$visitBlock$4(JavaVisitor.java:397) org.openrewrite.internal.ListUtils.map(ListUtils.java:244) org.openrewrite.internal.ListUtils.map(ListUtils.java:267)
the test code

the new Expectations mock a static method

i don't know how to resolve it

[timtebeek]

Thanks for the report! That seems to be an issue with the recipe, on this line

rewrite-testing-frameworks/src/main/java/org/openrewrite/java/testing/jmockit/SetupStatementsRewriter.java

Line 53 in 9089e95

J.Block expectationsBlock = (J.Block) nc.getBody().getStatements().get(0);

It looks like there might be an unexpected empty block there; can you correlate that with the test code you have there?

[Gwynner-MJ]

If this is the case, the mock pile will report an out-of-bounds error.

[timtebeek]

Thanks for the example! Could you explain "If this is the case, the mock pile will report an out-of-bounds error." a bit more? I haven't used JMockit myself, so it's not immediately clear if this usage is valid and should be support, and what the replacement should then be.