This is an automated email from the ASF dual-hosted git repository. slachiewicz pushed a commit to branch wagon-easymock in repository https://gitbox.apache.org/repos/asf/maven-wagon.git
commit 216d98b0795928a1d443b245249726bbef6069b6 Author: Sylwester Lachiewicz <[email protected]> AuthorDate: Wed Dec 31 17:35:19 2025 +0100 Refactor tests to use Mockito for mocking and assertions --- .../apache/maven/wagon/StreamingWagonTestCase.java | 4 +- .../java/org/apache/maven/wagon/WagonTestCase.java | 92 ++++++++-------------- 2 files changed, 34 insertions(+), 62 deletions(-) diff --git a/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java b/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java index 30246302..87cd7243 100644 --- a/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java +++ b/wagon-provider-test/src/main/java/org/apache/maven/wagon/StreamingWagonTestCase.java @@ -230,7 +230,7 @@ private int putStream() throws Exception { disconnectWagon(wagon); - verifyMock(progressAnswer, content.length()); + assertEquals(content.length(), progressAnswer.getSize()); return content.length(); } @@ -261,6 +261,6 @@ private void getStream(int expectedSize) throws Exception { disconnectWagon(wagon); - verifyMock(progressAnswer, expectedSize); + assertEquals(expectedSize, progressAnswer.getSize()); } } diff --git a/wagon-provider-test/src/main/java/org/apache/maven/wagon/WagonTestCase.java b/wagon-provider-test/src/main/java/org/apache/maven/wagon/WagonTestCase.java index 15914220..9b3e513e 100644 --- a/wagon-provider-test/src/main/java/org/apache/maven/wagon/WagonTestCase.java +++ b/wagon-provider-test/src/main/java/org/apache/maven/wagon/WagonTestCase.java @@ -39,21 +39,17 @@ import org.apache.maven.wagon.resource.Resource; import org.codehaus.plexus.PlexusTestCase; import org.codehaus.plexus.util.FileUtils; -import org.easymock.IAnswer; import org.junit.Assume; +import org.mockito.invocation.InvocationOnMock; +import org.mockito.stubbing.Answer; import org.slf4j.Logger; import org.slf4j.LoggerFactory; -import static org.easymock.EasyMock.anyInt; -import static org.easymock.EasyMock.anyObject; -import static org.easymock.EasyMock.anyString; -import static org.easymock.EasyMock.createMock; -import static org.easymock.EasyMock.eq; -import static org.easymock.EasyMock.expectLastCall; -import static org.easymock.EasyMock.getCurrentArguments; -import static org.easymock.EasyMock.replay; -import static org.easymock.EasyMock.reset; -import static org.easymock.EasyMock.verify; +import static org.mockito.ArgumentMatchers.any; +import static org.mockito.ArgumentMatchers.anyInt; +import static org.mockito.ArgumentMatchers.eq; +import static org.mockito.Mockito.doAnswer; +import static org.mockito.Mockito.mock; /** * @author <a href="mailto:[email protected]";>Jason van Zyl</a> @@ -61,11 +57,11 @@ public abstract class WagonTestCase extends PlexusTestCase { protected static Logger logger = LoggerFactory.getLogger(WagonTestCase.class); - static final class ProgressAnswer implements IAnswer { + static final class ProgressAnswer implements Answer<Object> { private int size; - public Object answer() throws Throwable { - int length = (Integer) getCurrentArguments()[2]; + public Object answer(InvocationOnMock invocation) { + int length = invocation.getArgument(2); size += length; return null; } @@ -110,7 +106,7 @@ public int getSize() { protected void setUp() throws Exception { checksumObserver = new ChecksumObserver(); - mockTransferListener = createMock(TransferListener.class); + mockTransferListener = mock(TransferListener.class); super.setUp(); } @@ -315,7 +311,7 @@ protected ProgressAnswer setupGetIfNewerTest(Wagon wagon, boolean expectedResult protected void assertGetIfNewerTest(ProgressAnswer progressAnswer, boolean expectedResult, int expectedSize) throws IOException { if (expectedResult) { - verifyMock(progressAnswer, expectedSize); + assertEquals(expectedSize, progressAnswer.getSize()); assertNotNull("check checksum is not null", checksumObserver.getActualChecksum()); @@ -329,10 +325,6 @@ protected void assertGetIfNewerTest(ProgressAnswer progressAnswer, boolean expec String destContent = FileUtils.fileRead(destFile); assertEquals(sourceContent, destContent); } else { - verify(mockTransferListener); - - reset(mockTransferListener); - assertNull("check checksum is null", checksumObserver.getActualChecksum()); assertFalse(destFile.exists()); @@ -349,11 +341,6 @@ private void replaceMockForSkippedGetIfNewer(Wagon wagon, int expectedSize) { // TODO: transfer skipped event? // mockTransferListener.transferSkipped( createTransferEvent( wagon, resource, TransferEvent.TRANSFER_STARTED, // TransferEvent.REQUEST_GET, destFile ) ); - - mockTransferListener.debug(anyString()); - expectLastCall().anyTimes(); - - replay(mockTransferListener); } public void testWagonPutDirectory() throws Exception { @@ -802,7 +789,7 @@ protected void putFile(String resourceName, String testFileName, String content) disconnectWagon(wagon); - verifyMock(progressAnswer, content.length()); + assertEquals(content.length(), progressAnswer.getSize()); } protected ProgressAnswer replayMockForPut(String resourceName, String content, Wagon wagon) { @@ -814,21 +801,22 @@ protected ProgressAnswer replayMockForPut(String resourceName, String content, W resource.setLastModified(sourceFile.lastModified()); mockTransferListener.transferStarted(createTransferEvent( wagon, resource, TransferEvent.TRANSFER_STARTED, TransferEvent.REQUEST_PUT, sourceFile)); - mockTransferListener.transferProgress( - eq(createTransferEvent( - wagon, resource, TransferEvent.TRANSFER_PROGRESS, TransferEvent.REQUEST_PUT, sourceFile)), - anyObject(byte[].class), - anyInt()); ProgressAnswer progressAnswer = new ProgressAnswer(); - expectLastCall().andStubAnswer(progressAnswer); - - mockTransferListener.debug(anyString()); - expectLastCall().anyTimes(); + doAnswer(progressAnswer) + .when(mockTransferListener) + .transferProgress( + eq(createTransferEvent( + wagon, + resource, + TransferEvent.TRANSFER_PROGRESS, + TransferEvent.REQUEST_PUT, + sourceFile)), + any(byte[].class), + anyInt()); mockTransferListener.transferCompleted(createTransferEvent( wagon, resource, TransferEvent.TRANSFER_COMPLETED, TransferEvent.REQUEST_PUT, sourceFile)); - replay(mockTransferListener); return progressAnswer; } @@ -861,15 +849,7 @@ protected void getFile(int expectedSize) throws Exception { disconnectWagon(wagon); - verifyMock(progressAnswer, expectedSize); - } - - protected void verifyMock(ProgressAnswer progressAnswer, int length) { - verify(mockTransferListener); - - assertEquals(length, progressAnswer.getSize()); - - reset(mockTransferListener); + assertEquals(expectedSize, progressAnswer.getSize()); } protected void disconnectWagon(Wagon wagon) throws ConnectionException { @@ -907,26 +887,18 @@ protected ProgressAnswer replaceMockForGet(Wagon wagon, int expectedSize) { TransferEvent te = createTransferEvent(wagon, resource, TransferEvent.TRANSFER_STARTED, TransferEvent.REQUEST_GET, null); mockTransferListener.transferStarted(te); - mockTransferListener.transferProgress( - eq(new TransferEvent(wagon, resource, TransferEvent.TRANSFER_PROGRESS, TransferEvent.REQUEST_GET)), - anyObject(byte[].class), - anyInt()); - ProgressAnswer progressAnswer = new ProgressAnswer(); - - if (assertOnTransferProgress()) { - expectLastCall().andAnswer(progressAnswer); - } else { - expectLastCall().andAnswer(progressAnswer); - expectLastCall().anyTimes(); - } - mockTransferListener.debug(anyString()); - expectLastCall().anyTimes(); + doAnswer(progressAnswer) + .when(mockTransferListener) + .transferProgress( + eq(new TransferEvent( + wagon, resource, TransferEvent.TRANSFER_PROGRESS, TransferEvent.REQUEST_GET)), + any(byte[].class), + anyInt()); mockTransferListener.transferCompleted(createTransferEvent( wagon, resource, TransferEvent.TRANSFER_COMPLETED, TransferEvent.REQUEST_GET, destFile)); - replay(mockTransferListener); return progressAnswer; }
