Surround manual code with Eclipse formatter off in partial update (#2685)

Surround manual code with Eclipse formatter off in partial update

Author: alzimmermsft

customization-base/pom.xml

@@ -53,7 +53,7 @@
     <dependency>
       <groupId>com.github.javaparser</groupId>
       <artifactId>javaparser-core</artifactId>
-      <version>3.25.6</version>
+      <version>3.25.10</version>
     </dependency>
     <dependency>
       <groupId>org.apache.ant</groupId>

javagen/src/main/java/com/azure/autorest/template/PackageInfoTemplate.java

@@ -7,6 +7,7 @@
 import com.azure.autorest.extension.base.plugin.JavaSettings;
 import com.azure.autorest.model.clientmodel.PackageInfo;
 import com.azure.autorest.model.javamodel.JavaFile;
+import com.azure.autorest.partialupdate.util.PartialUpdateHandler;
 
 import java.util.regex.Pattern;
 
@@ -35,15 +36,15 @@ public final void write(PackageInfo packageInfo, JavaFile javaFile) {
 
         javaFile.javadocComment((comment) -> {
             if (settings.isHandlePartialUpdate()) {
-                comment.line("<!-- start generated doc -->");
+                comment.line(PartialUpdateHandler.START_GENERATED_JAVA_DOC);
             }
 
             for (String desc : NEW_LINE.split(packageInfo.getDescription(), -1)) {
                 comment.description(desc);
             }
 
             if (settings.isHandlePartialUpdate()) {
-                comment.line("<!-- end generated doc -->");
+                comment.line(PartialUpdateHandler.END_GENERATED_JAVA_DOC);
             }
         });
 

partial-update-tests/existing/src/main/java/fixtures/bodystring/implementation/package-info.java

@@ -3,7 +3,7 @@
 // Code generated by Microsoft (R) AutoRest Code Generator.
 
 /**
- * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block.
+ * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block. And it shouldn't be formatted.
  * <p>
  * <!-- start generated doc -->
  * Package containing the implementations for AutoRestSwaggerBatService. Test Infrastructure for AutoRest Swagger BAT.

partial-update-tests/existing/src/main/java/fixtures/bodystring/package-info.java

@@ -7,6 +7,6 @@
  * Package containing the classes for AutoRestSwaggerBatService. Test Infrastructure for AutoRest Swagger BAT.
  * <!-- end generated doc -->
  * <p>
- * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block.
+ * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block. And it shouldn't be formatted.
  * */
 package fixtures.bodystring;

partial-update-tests/generated/src/main/java/fixtures/bodystring/StringOperationClient.java

@@ -23,9 +23,11 @@
 @ServiceClient(builder = AutoRestSwaggerBatServiceClientBuilder.class)
 public final class StringOperationClient {
 
+    // @formatter:off
     // 1. manually add one field
-    private EnumsImpl enumClient;
+    private EnumsImpl enumClient;// @formatter:on
 
+    // @formatter:off
     /**
      * 1. manually add one constructor
      *
@@ -34,7 +36,7 @@ public final class StringOperationClient {
      */
     StringOperationClient(StringOperationsImpl serviceClient, String test) {
         this.serviceClient = serviceClient;
-    }
+    }// @formatter:on
 
     @Generated
     private final StringOperationsImpl serviceClient;
@@ -70,16 +72,16 @@ public Response<BinaryData> getNullWithResponse(RequestOptions requestOptions) {
         return this.serviceClient.getNullWithResponse(requestOptions);
     }
 
+    // @formatter:off
     /**
      * 2. manually update method signature
      *
      * <p><strong>Header Parameters</strong>
      *
      * <table border="1">
-     * <caption>Header Parameters</caption>
-     * <tr><th>Name</th><th>Type</th><th>Required</th><th>Description</th></tr>
-     * <tr><td>Content-Type</td><td>String</td><td>No</td><td>The content type. Allowed values:
-     * "application/json".</td></tr>
+     *     <caption>Header Parameters</caption>
+     *     <tr><th>Name</th><th>Type</th><th>Required</th><th>Description</th></tr>
+     *     <tr><td>Content-Type</td><td>String</td><td>No</td><td>The content type. Allowed values: "application/json".</td></tr>
      * </table>
      *
      * You can add these to a request with {@link RequestOptions#addHeader}
@@ -101,8 +103,9 @@ public Response<BinaryData> getNullWithResponse(RequestOptions requestOptions) {
     @ServiceMethod(returns = ReturnType.SINGLE)
     public Response<Void> putNullWithResponse(RequestOptions requestOptions, String test) {
         return this.serviceClient.putNullWithResponse(requestOptions);
-    }
+    }// @formatter:on
 
+    // @formatter:off
     /**
      * 3. manually update method access from public to private
      *
@@ -122,7 +125,7 @@ public Response<Void> putNullWithResponse(RequestOptions requestOptions, String
     @ServiceMethod(returns = ReturnType.SINGLE)
     Response<BinaryData> getEmptyWithResponse(RequestOptions requestOptions) {
         return this.serviceClient.getEmptyWithResponse(requestOptions);
-    }
+    }// @formatter:on
 
     /**
      * Set string value empty ''.
@@ -340,14 +343,15 @@ public Response<BinaryData> getNullBase64UrlEncodedWithResponse(RequestOptions r
         return this.serviceClient.getNullBase64UrlEncodedWithResponse(requestOptions);
     }
 
+    // @formatter:off
     /**
      * 4. manually added method to test partial update
      *
      * @return a string
      */
     public String manuallyAddedMethod() {
         return "manuallyAddedMethod";
-    }
+    }// @formatter:on
 
     /**
      * This is an added method.

partial-update-tests/generated/src/main/java/fixtures/bodystring/implementation/package-info.java

@@ -2,8 +2,10 @@
 // Licensed under the MIT License.
 // Code generated by Microsoft (R) AutoRest Code Generator.
 /**
- * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block.
+ * <!-- @formatter:off -->
+ * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block. And it shouldn't be formatted.
  * <p>
+ * <!-- @formatter:on -->
  * <!-- start generated doc -->
  * Package containing the implementations for AutoRestSwaggerBatService.
  * Test Infrastructure for AutoRest Swagger BAT.

partial-update-tests/generated/src/main/java/fixtures/bodystring/package-info.java

@@ -6,7 +6,9 @@
  * Package containing the classes for AutoRestSwaggerBatService.
  * Test Infrastructure for AutoRest Swagger BAT.
  * <!-- end generated doc -->
+ * <!-- @formatter:off -->
  * <p>
- * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block.
+ * Some other custom Javadocs which shouldn't be overwritten as they aren't contained in the generated Javadoc block. And it shouldn't be formatted.
+ * <!-- @formatter:on -->
  */
 package fixtures.bodystring;

partial-update/pom.xml

@@ -21,7 +21,7 @@
         <dependency>
             <groupId>com.github.javaparser</groupId>
             <artifactId>javaparser-core</artifactId>
-            <version>3.25.6</version>
+            <version>3.25.10</version>
         </dependency>
         <dependency>
             <groupId>org.slf4j</groupId>