Issue checkstyle#85: resolved position helper bug

Author: Anmol202005

src/main/java/org/checkstyle/autofix/PositionHelper.java

@@ -17,13 +17,15 @@
 
 package org.checkstyle.autofix;
 
+import java.util.List;
 import java.util.concurrent.CancellationException;
 import java.util.function.Function;
 
 import org.openrewrite.Cursor;
 import org.openrewrite.PrintOutputCapture;
 import org.openrewrite.TreeVisitor;
 import org.openrewrite.internal.RecipeRunException;
+import org.openrewrite.java.tree.Comment;
 import org.openrewrite.java.tree.J;
 
 public final class PositionHelper {
@@ -65,6 +67,12 @@ private static int computePosition(
                     public PrintOutputCapture<TreeVisitor<?, ?>> append(String text) {
                         if (targetElement.isScope(getContext().getCursor().getValue())) {
                             super.append(targetElement.getPrefix().getWhitespace());
+
+                            final List<Comment> comments = targetElement.getPrefix().getComments();
+                            for (Comment comment : comments) {
+                                super.append(comment.printComment(cursor) + comment.getSuffix());
+                            }
+
                             throw new CancellationException();
                         }
                         return super.append(text);

src/test/resources/org/checkstyle/autofix/recipe/finallocalvariable/singlelocaltest/InputSingleLocalTest.java

@@ -26,7 +26,8 @@ public void basicLocalVariables() {
         List<String> items = new ArrayList<>();            // violation, "should be declared final"
         StringBuilder builder = new StringBuilder();       // violation, "should be declared final"
         Date currentDate = new Date();                     // violation, "should be declared final"
-        File tempFile = new File("temp.txt");     // violation, "should be declared final"
+        File // prefix comment preceding target element
+        tempFile = new File("temp.txt");     // violation, "should be declared final"
     }
 
     public void methodWithCalculations() {

src/test/resources/org/checkstyle/autofix/recipe/finallocalvariable/singlelocaltest/OutputSingleLocalTest.java

@@ -26,7 +26,8 @@ public void basicLocalVariables() {
         final List<String> items = new ArrayList<>();
         final StringBuilder builder = new StringBuilder();
         final Date currentDate = new Date();
-        final File tempFile = new File("temp.txt");
+        final File // prefix comment preceding target element
+        tempFile = new File("temp.txt");
     }
 
     public void methodWithCalculations() {

src/test/resources/org/checkstyle/autofix/recipe/upperell/hexoctalliteral/InputHexOctalLiteral.java

@@ -18,7 +18,8 @@ public class InputHexOctalLiteral {
     private long hexUpper = 0X2DEFl;    // violation 'Should use uppercase 'L'.'
     private long octal = 0777l;         // violation 'Should use uppercase 'L'.'
     private long binary = 0b1010l;      // violation 'Should use uppercase 'L'.'
-    private long decimal = 12345l;      // violation 'Should use uppercase 'L'.'
+    private long decimal = // comment preceding a target element
+                           12345l;      // violation 'Should use uppercase 'L'.'
 
     public void calculateValues() {
         // violation below, 'Should use uppercase 'L'.'

src/test/resources/org/checkstyle/autofix/recipe/upperell/hexoctalliteral/OutputHexOctalLiteral.java

@@ -18,7 +18,8 @@ public class OutputHexOctalLiteral {
     private long hexUpper = 0X2DEFL;
     private long octal = 0777L;
     private long binary = 0b1010L;
-    private long decimal = 12345L;
+    private long decimal = // comment preceding a target element
+                           12345L;
 
     public void calculateValues() {
         long hexResult = 0xDEADBEEFL + 0xDEADBEFl; //suppressed violation for 0xDEADBEFl