build: quote the Mockito -javaagent path in surefire argLine

The -javaagent argument added in #1483 is set per module as
-javaagent:${org.mockito:mockito-core:jar} without quotes. Surefire
splits argLine into JVM arguments at unquoted whitespace, so when the
local Maven repository path contains a space (for example a Windows
user profile such as C:\Users\First Last) the forked JVM receives a
truncated agent path and exits before running any test:

  Error opening zip file or JAR manifest missing : C:\Users\First

Quote the value in every module that sets it, which is how
jacoco-maven-plugin already passes its own agent argument when the
path contains a space, and note in the parent pom that the quotes must
stay. CI runs on Ubuntu with a space-free repository path, so it could
not catch this.

Author: innoprej

a2a/pom.xml

@@ -26,7 +26,7 @@
     <junit4.version>4.13.2</junit4.version>
     <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely on
          runtime self-attachment, which newer JDKs block. -->
-    <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+    <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
   </properties>
 
   <dependencies>

contrib/firestore-session-service/pom.xml

@@ -31,7 +31,7 @@
     <properties>
         <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely
              on runtime self-attachment, which newer JDKs block. -->
-        <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+        <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
     </properties>
 
     <dependencies>

contrib/langchain4j/pom.xml

@@ -31,7 +31,7 @@
     <properties>
         <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely
              on runtime self-attachment, which newer JDKs block. -->
-        <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+        <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
     </properties>
 
     <dependencyManagement>

contrib/planners/pom.xml

@@ -31,7 +31,7 @@
     <properties>
         <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely
              on runtime self-attachment, which newer JDKs block. -->
-        <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+        <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
     </properties>
 
     <dependencies>

contrib/spring-ai/pom.xml

@@ -33,7 +33,7 @@
         <testcontainers.version>1.21.3</testcontainers.version>
         <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely
              on runtime self-attachment, which newer JDKs block. -->
-        <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+        <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
     </properties>
 
     <dependencyManagement>

core/pom.xml

@@ -30,7 +30,7 @@
   <properties>
     <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely on
          runtime self-attachment, which newer JDKs block. -->
-    <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+    <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
   </properties>
 
   <dependencies>

dev/pom.xml

@@ -28,7 +28,7 @@
   <properties>
     <!-- Load mockito-core as a startup -javaagent so the inline mock maker does not rely on
          runtime self-attachment, which newer JDKs block. -->
-    <mockito.javaagent.arg>-javaagent:${org.mockito:mockito-core:jar}</mockito.javaagent.arg>
+    <mockito.javaagent.arg>"-javaagent:${org.mockito:mockito-core:jar}"</mockito.javaagent.arg>
   </properties>
 
   <dependencyManagement>

pom.xml

@@ -86,7 +86,8 @@
     <!-- Prepended to the surefire JVM args. Empty by default; any module whose tests resolve
          mockito-core (directly or transitively) must set it to
          "-javaagent:${org.mockito:mockito-core:jar}" so the inline mock maker loads as a
-         startup agent instead of self-attaching at runtime, which newer JDKs block. -->
+         startup agent instead of self-attaching at runtime, which newer JDKs block. Keep the
+         quotes: they make a jar path with spaces a single JVM argument. -->
     <mockito.javaagent.arg/>
     <surefire.argLine>${mockito.javaagent.arg} @{jacoco.agent.argLine} --add-opens=java.base/java.nio=ALL-UNNAMED --add-opens=java.base/sun.nio.ch=ALL-UNNAMED --add-opens=java.base/java.util.concurrent=ALL-UNNAMED --add-opens=java.base/java.text=ALL-UNNAMED --add-opens=java.base/java.util=ALL-UNNAMED --add-opens=java.base/java.lang=ALL-UNNAMED --add-opens=java.base/java.lang.reflect=ALL-UNNAMED --add-opens=java.base/jdk.internal.misc=ALL-UNNAMED -Dio.netty.tryReflectionSetAccessible=true</surefire.argLine>
   </properties>