Nemcio/SCM-Manager
1
0
Fork 0
mirror of https://github.com/scm-manager/scm-manager.git synced 2025-11-12 16:35:45 +01:00
Code Issues Packages Projects Releases Activity

fixes usage of named cache configurations, see issue #943

Browse Source
This commit is contained in:
Sebastian Sdorra
2017-08-14 16:04:30 +02:00
parent 785e1b12a9
commit f72648f646
2 changed files with 50 additions and 7 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
scm-webapp/src/main/java/sonia/scm/cache/GuavaCacheManager.java vendored
View File

@@ -78,15 +78,22 @@ public class GuavaCacheManager implements CacheManager, org.apache.shiro.cache.C
protected GuavaCacheManager(GuavaCacheManagerConfiguration config) {
defaultConfiguration = config.getDefaultCache();
for (GuavaNamedCacheConfiguration ncc : config.getCaches()) {
logger.debug("create cache {} from configured configuration {}", ncc.getName(), ncc);
cacheMap.put(ncc.getName(), new CacheWithConfiguration(
GuavaCaches.create(defaultConfiguration, ncc.getName()),
defaultConfiguration)
for (GuavaNamedCacheConfiguration namedCacheConfiguration : config.getCaches()) {
logger.debug("create cache {} from configured configuration {}",
namedCacheConfiguration.getName(), namedCacheConfiguration
);
cacheMap.put(namedCacheConfiguration.getName(), createCacheWithConfiguration(namedCacheConfiguration));
}
}
private CacheWithConfiguration createCacheWithConfiguration(GuavaNamedCacheConfiguration namedCacheConfiguration) {
return createCacheWithConfiguration(namedCacheConfiguration, namedCacheConfiguration.getName());
}
private CacheWithConfiguration createCacheWithConfiguration(GuavaCacheConfiguration configuration, String name) {
return new CacheWithConfiguration(GuavaCaches.create(configuration, name), configuration);
}
@Override
public void close() throws IOException {
logger.info("close guava cache manager");
@@ -110,7 +117,7 @@ public class GuavaCacheManager implements CacheManager, org.apache.shiro.cache.C
"cache {} does not exists, creating a new instance from default configuration: {}",
name, defaultConfiguration
);
cache = new CacheWithConfiguration(GuavaCaches.create(defaultConfiguration, name), defaultConfiguration);
cache = createCacheWithConfiguration(defaultConfiguration, name);
cacheMap.put(name, cache);
}

38
scm-webapp/src/test/java/sonia/scm/cache/GuavaCacheManagerTest.java vendored
View File

@@ -32,15 +32,33 @@
package sonia.scm.cache;
import com.google.common.collect.Lists;
import org.junit.Assert;
import org.junit.Test;
import org.junit.runner.RunWith;
import org.mockito.Answers;
import org.mockito.Mock;
import org.mockito.runners.MockitoJUnitRunner;
import java.util.List;
import static org.junit.Assert.assertEquals;
import static org.mockito.Mockito.when;
/**
*
* @author Sebastian Sdorra
*/
@RunWith(MockitoJUnitRunner.class)
public class GuavaCacheManagerTest extends CacheManagerTestBase
{
@Mock(answer = Answers.CALLS_REAL_METHODS)
private GuavaCacheConfiguration defaultConfiguration;
@Mock(answer = Answers.CALLS_REAL_METHODS)
private GuavaNamedCacheConfiguration configuration;
/**
* Method description
*
@@ -60,6 +78,24 @@ public class GuavaCacheManagerTest extends CacheManagerTestBase
((GuavaCache)c2).getWrappedCache()
);
}
@Test
public void configuration() {
when(configuration.getName()).thenReturn("my-crazy-cache");
when(configuration.getCopyStrategy()).thenReturn(CopyStrategy.READWRITE);
when(defaultConfiguration.getCopyStrategy()).thenReturn(CopyStrategy.READ);
List<GuavaNamedCacheConfiguration> configurations = Lists.newArrayList(configuration);
GuavaCacheManagerConfiguration managerConfiguration = new GuavaCacheManagerConfiguration(defaultConfiguration, configurations);
GuavaCacheManager guavaCacheManager = new GuavaCacheManager(managerConfiguration);
// default cache
GuavaSecurityCache<?,?> sorbotCache = guavaCacheManager.getCache("sorbot-cache");
assertEquals(CopyStrategy.READ, sorbotCache.copyStrategy);
// configured cache
GuavaSecurityCache<?,?> crazyCache = guavaCacheManager.getCache("my-crazy-cache");
assertEquals(CopyStrategy.READWRITE, crazyCache.copyStrategy);
}
}