From 5032d0430440def4df75beeab56979bbe3ee484f Mon Sep 17 00:00:00 2001 From: Sebastian Sdorra Date: Tue, 19 Aug 2014 22:00:04 +0200 Subject: [PATCH] remove powermock dependency, to fix build on java 8 --- pom.xml | 15 ------------- .../DefaultRepositoryManagerTest.java | 21 +++++++++---------- 2 files changed, 10 insertions(+), 26 deletions(-) diff --git a/pom.xml b/pom.xml index 23ea8daf86..e822496ea0 100644 --- a/pom.xml +++ b/pom.xml @@ -138,20 +138,6 @@ test - - org.powermock - powermock-module-junit4 - ${powermock.version} - test - - - - org.powermock - powermock-api-mockito - ${powermock.version} - test - - @@ -439,7 +425,6 @@ 1.9.5 1.3 4.11 - 1.5.3 1.7.7 diff --git a/scm-webapp/src/test/java/sonia/scm/repository/DefaultRepositoryManagerTest.java b/scm-webapp/src/test/java/sonia/scm/repository/DefaultRepositoryManagerTest.java index e0c67da45a..94c9c2b4df 100644 --- a/scm-webapp/src/test/java/sonia/scm/repository/DefaultRepositoryManagerTest.java +++ b/scm-webapp/src/test/java/sonia/scm/repository/DefaultRepositoryManagerTest.java @@ -35,16 +35,16 @@ package sonia.scm.repository; //~--- non-JDK imports -------------------------------------------------------- -import org.junit.Test; -import org.junit.runner.RunWith; +import com.google.common.collect.ImmutableSet; -import org.powermock.core.classloader.annotations.PrepareForTest; -import org.powermock.modules.junit4.PowerMockRunner; +import org.junit.Test; import sonia.scm.Type; import sonia.scm.config.ScmConfiguration; import sonia.scm.repository.api.HookContext; +import sonia.scm.repository.api.HookContextFactory; import sonia.scm.repository.api.HookFeature; +import sonia.scm.repository.spi.HookContextProvider; import sonia.scm.repository.xml.XmlRepositoryDAO; import sonia.scm.security.DefaultKeyGenerator; import sonia.scm.store.JAXBStoreFactory; @@ -65,8 +65,6 @@ import java.util.Set; * * @author Sebastian Sdorra */ -@RunWith(PowerMockRunner.class) -@PrepareForTest(HookContext.class) public class DefaultRepositoryManagerTest extends RepositoryManagerTestBase { @@ -81,12 +79,13 @@ public class DefaultRepositoryManagerTest extends RepositoryManagerTestBase @Override public HookContext createHookContext(Repository repository) { - HookContext ctx = mock(HookContext.class); + PreProcessorUtil ppu = mock(PreProcessorUtil.class); + HookContextProvider provider = mock(HookContextProvider.class); + Set features = ImmutableSet.of(); - when(ctx.isFeatureSupported(any(HookFeature.class))).thenReturn( - Boolean.FALSE); + when(provider.getSupportedFeatures()).thenReturn(features); - return ctx; + return new HookContextFactory(ppu).createContext(provider, repository); } //~--- get methods ---------------------------------------------------------- @@ -145,7 +144,7 @@ public class DefaultRepositoryManagerTest extends RepositoryManagerTestBase protected DefaultRepositoryManager createRepositoryManager( boolean archiveEnabled) { - Set handlerSet = new HashSet(); + Set handlerSet = new HashSet<>(); StoreFactory factory = new JAXBStoreFactory(); factory.init(contextProvider);