Avoid duplicate vararg handling with Mockito (#2201)

As @meestchoo correctly suspected in #2161, vararg expansion in
conjunction with the Mockito MockMaker is handled by both Mockito and
Spock.
The easy fix is to just not use Mockito var expansion.

Fixes #2161

---------

Co-authored-by: AndreasTu <[email protected]>

Author: beatbrot

docs/release_notes.adoc

@@ -55,6 +55,10 @@ spockPull:1910[]
 will now require that the spec is annotated with <<parallel_execution.adoc#isolated-execution, @Isolated>> or `@ResourceLock(org.spockframework.runtime.model.parallel.Resources.META_CLASS_REGISTRY)`. See <<interaction_based_testing.adoc#global-mocks-parallel-execution, Global mocks and parallel execution>> spockPull:1848[]
 * `@TempDir` `spock.tempDir.keep` has been replaced by `spock.tempdir.cleanup`. See <<extensions.adoc#temp-dir-cleanup,TempDir Cleanup>> spockPull:1525[]
 
+=== Misc
+
+* Fix vararg handling in SpyStatic spockIssue:2161[]
+
 == 2.4-M6 (2025-04-15)
 
 === Highlights

spock-core/src/main/java/org/spockframework/mock/runtime/mockito/MockitoMockMakerImpl.java

@@ -262,7 +262,7 @@ public SpockMockHandler(IProxyBasedMockInterceptor mockInterceptor) {
     @Override
     public Object handle(Invocation invocation) {
       Object mock = invocation.getMock();
-      return mockInterceptor.intercept(mock, invocation.getMethod(), invocation.getArguments(), new IResponseGenerator() {
+      return mockInterceptor.intercept(mock, invocation.getMethod(), invocation.getRawArguments(), new IResponseGenerator() {
         @Override
         public Object respond(IMockInvocation __) {
           try {

spock-specs/src/test/groovy/org/spockframework/mock/runtime/mockito/MockitoStaticMocksSpec.groovy

@@ -22,6 +22,7 @@ import org.mockito.exceptions.base.MockitoException
 import org.spockframework.mock.CannotCreateMockException
 import org.spockframework.mock.MockUtil
 import org.spockframework.runtime.InvalidSpecException
+import spock.lang.Issue
 import spock.lang.Shared
 import spock.lang.Specification
 import spock.mock.MockMakers
@@ -478,6 +479,18 @@ class MockitoStaticMocksSpec extends Specification {
     ex.message.contains("It is not possible to mock static methods of java.lang.Thread")
   }
 
+  @Issue("https://github.com/spockframework/spock/issues/2161")
+  def "SpyStatic with varargs in method"() {
+    given:
+    SpyStatic(StaticClass)
+    StaticClass.staticVarargsMethod("test") >> true
+    StaticClass.staticVarargsMethod("test2") >> false
+
+    expect:
+    StaticClass.staticVarargsMethod("test")
+    !StaticClass.staticVarargsMethod("test2")
+  }
+
   static class StaticClass {
 
     String instanceMethod() {
@@ -491,6 +504,11 @@ class MockitoStaticMocksSpec extends Specification {
     static String staticMethod2() {
       return REAL_VALUE
     }
+
+    @SuppressWarnings('unused')
+    static boolean staticVarargsMethod(String str, String... varargs) {
+      return true
+    }
   }
 
   static class StaticClass2 {