From 83ac5277b1b0eeedbe00be33b310ca9d448c64d5 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Leonard=20Br=C3=BCnings?= Date: Wed, 13 Dec 2023 17:56:34 +0100 Subject: [PATCH] Use local class for test to avoid cross contamination --- .../org/spockframework/smoke/mock/GroovyMocks.groovy | 10 ++++++---- 1 file changed, 6 insertions(+), 4 deletions(-) diff --git a/spock-specs/src/test/groovy/org/spockframework/smoke/mock/GroovyMocks.groovy b/spock-specs/src/test/groovy/org/spockframework/smoke/mock/GroovyMocks.groovy index 25f6a9136f..c71b824f81 100644 --- a/spock-specs/src/test/groovy/org/spockframework/smoke/mock/GroovyMocks.groovy +++ b/spock-specs/src/test/groovy/org/spockframework/smoke/mock/GroovyMocks.groovy @@ -24,7 +24,7 @@ class GroovyMocks extends Specification { then: CannotCreateMockException e = thrown() - e.message == 'Cannot create mock for class java.util.ArrayList. The given type is already mocked by Spock.' + e.message == 'Cannot create mock for class org.spockframework.smoke.mock.GroovyMocks$LocalClassForMocking. The given type is already mocked by Spock.' where: [typeA, typeB] << ([['Mock', 'Stub', 'Spy']] * 2).combinations() @@ -33,16 +33,18 @@ class GroovyMocks extends Specification { void createMock(String type) { switch (type) { case 'Mock': - GroovyMock(global: true, ArrayList) + GroovyMock(global: true, LocalClassForMocking) break case 'Stub': - GroovyStub(global: true, ArrayList) + GroovyStub(global: true, LocalClassForMocking) break case 'Spy': - GroovySpy(global: true, ArrayList) + GroovySpy(global: true, LocalClassForMocking) break default: throw new IllegalArgumentException(type) } } + + class LocalClassForMocking {} }