Merge pull request #1540 from innoprej:build/quote-mockito-javaagent

PiperOrigin-RevId: 1000002188

Author: copybara-github

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>