Fix ConfigurePDXCommandIntegrationTest: Quote parameter values containing '='

Spring Shell 3.x splits parameter values on '=' signs unless they are quoted.
Added comprehensive class-level Javadoc explaining why quotes are required
and the impact of the GfshParser.splitUserInput() behavior.

Changes:
- Added 30+ line class-level documentation explaining Spring Shell 3.x parsing
- Quoted all --auto-serializable-classes and --portable-auto-serializable-classes
  parameter values containing '=' (e.g., "com.company.DomainObject.*#identity=id")
- Without quotes: parser splits into ["...#identity", "id"] (2 args)
- With quotes: parser preserves ["...#identity=id"] (1 arg)

This prevents AutoSerializableManager from failing with 'Unable to correctly
process auto serialization init value' when it expects 'param=value' format
but receives only 'param' due to the split.

Tests fixed (4):
- commandShouldSucceedWhenConfiguringAutoSerializableClassesWithPersistence
- commandShouldSucceedWhenConfiguringAutoSerializableClassesWithoutPersistence
- commandShouldSucceedWhenConfiguringPortableAutoSerializableClassesWithPersistence
- commandShouldSucceedWhenConfiguringPortableAutoSerializableClassesWithoutPersistence

All 6 ConfigurePDXCommandIntegrationTest tests now pass.

Author: JinwooHwang

geode-gfsh/src/integrationTest/java/org/apache/geode/management/internal/cli/commands/ConfigurePDXCommandIntegrationTest.java

@@ -25,6 +25,36 @@
 import org.apache.geode.test.junit.rules.GfshCommandRule;
 import org.apache.geode.test.junit.rules.LocatorStarterRule;
 
+/**
+ * Integration tests for the 'configure pdx' gfsh command.
+ *
+ * <p><b>IMPORTANT - Spring Shell 3.x Parameter Quoting:</b></p>
+ * <p>Parameter values containing '=' signs MUST be quoted to prevent incorrect parsing.
+ * Spring Shell 3.x's {@link org.apache.geode.management.internal.cli.GfshParser#splitUserInput}
+ * splits unquoted tokens on '=' by default, treating them as separate arguments.</p>
+ *
+ * <p><b>Why Quotes Are Required:</b></p>
+ * <ul>
+ *   <li><b>Without quotes:</b> {@code --auto-serializable-classes=com.company.DomainObject.*#identity=id}
+ *       <br>Parser sees: {@code ["com.company.DomainObject.*#identity", "id"]} (2 separate values)</li>
+ *   <li><b>With quotes:</b> {@code --auto-serializable-classes="com.company.DomainObject.*#identity=id"}
+ *       <br>Parser sees: {@code ["com.company.DomainObject.*#identity=id"]} (single value)</li>
+ * </ul>
+ *
+ * <p><b>Impact:</b> PDX auto-serialization patterns use the format {@code pattern#param=value}.
+ * Without quotes, the {@code =value} portion is lost, causing
+ * {@link org.apache.geode.pdx.internal.AutoSerializableManager} to fail with:
+ * <pre>"Unable to correctly process auto serialization init value: pattern#param"</pre>
+ * because it expects {@code param=value} but receives only {@code param}.</p>
+ *
+ * <p><b>GfshParser Behavior:</b> The parser explicitly checks for quoted strings and bypasses
+ * the '=' splitting logic when quotes are detected (see {@code GfshParser.splitUserInput()}).
+ * This is the intended mechanism for passing parameter values containing delimiter characters.</p>
+ *
+ * @see org.apache.geode.management.internal.cli.GfshParser#splitUserInput
+ * @see org.apache.geode.pdx.ReflectionBasedAutoSerializer
+ * @see org.apache.geode.pdx.internal.AutoSerializableManager
+ */
 @Category({ClientServerTest.class})
 public class ConfigurePDXCommandIntegrationTest {
   private static final String BASE_COMMAND_STRING = "configure pdx ";
@@ -61,7 +91,7 @@ public void commandShouldSucceedWhenUsingDefaults() {
   @Test
   public void commandShouldSucceedWhenConfiguringAutoSerializableClassesWithPersistence() {
     gfsh.executeAndAssertThat(BASE_COMMAND_STRING
-        + "--read-serialized=true --disk-store=myDiskStore --ignore-unread-fields=true --auto-serializable-classes=com.company.DomainObject.*#identity=id")
+        + "--read-serialized=true --disk-store=myDiskStore --ignore-unread-fields=true --auto-serializable-classes=\"com.company.DomainObject.*#identity=id\"")
         .statusIsSuccess();
 
     String sharedConfigXml = locator.getLocator().getConfigurationPersistenceService()
@@ -79,7 +109,7 @@ public void commandShouldSucceedWhenConfiguringAutoSerializableClassesWithPersis
   @Test
   public void commandShouldSucceedWhenConfiguringAutoSerializableClassesWithoutPersistence() {
     gfsh.executeAndAssertThat(BASE_COMMAND_STRING
-        + "--read-serialized=false --ignore-unread-fields=false --auto-serializable-classes=com.company.DomainObject.*#identity=id")
+        + "--read-serialized=false --ignore-unread-fields=false --auto-serializable-classes=\"com.company.DomainObject.*#identity=id\"")
         .statusIsSuccess();
 
     String sharedConfigXml = locator.getLocator().getConfigurationPersistenceService()
@@ -97,7 +127,7 @@ public void commandShouldSucceedWhenConfiguringAutoSerializableClassesWithoutPer
   @Test
   public void commandShouldSucceedWhenConfiguringPortableAutoSerializableClassesWithPersistence() {
     gfsh.executeAndAssertThat(BASE_COMMAND_STRING
-        + "--read-serialized=true --disk-store=myDiskStore --ignore-unread-fields=true --portable-auto-serializable-classes=com.company.DomainObject.*#identity=id")
+        + "--read-serialized=true --disk-store=myDiskStore --ignore-unread-fields=true --portable-auto-serializable-classes=\"com.company.DomainObject.*#identity=id\"")
         .statusIsSuccess();
 
     String sharedConfigXml = locator.getLocator().getConfigurationPersistenceService()
@@ -117,7 +147,7 @@ public void commandShouldSucceedWhenConfiguringPortableAutoSerializableClassesWi
   @Test
   public void commandShouldSucceedWhenConfiguringPortableAutoSerializableClassesWithoutPersistence() {
     gfsh.executeAndAssertThat(BASE_COMMAND_STRING
-        + "--read-serialized=false --ignore-unread-fields=false --portable-auto-serializable-classes=com.company.DomainObject.*#identity=id")
+        + "--read-serialized=false --ignore-unread-fields=false --portable-auto-serializable-classes=\"com.company.DomainObject.*#identity=id\"")
         .statusIsSuccess();
 
     String sharedConfigXml = locator.getLocator().getConfigurationPersistenceService()