Return-Path: X-Original-To: archive-asf-public-internal@cust-asf2.ponee.io Delivered-To: archive-asf-public-internal@cust-asf2.ponee.io Received: from cust-asf.ponee.io (cust-asf.ponee.io [163.172.22.183]) by cust-asf2.ponee.io (Postfix) with ESMTP id AEC8F200C77 for ; Mon, 1 May 2017 21:49:16 +0200 (CEST) Received: by cust-asf.ponee.io (Postfix) id AD46C160BB9; Mon, 1 May 2017 19:49:16 +0000 (UTC) Delivered-To: archive-asf-public@cust-asf.ponee.io Received: from mail.apache.org (hermes.apache.org [140.211.11.3]) by cust-asf.ponee.io (Postfix) with SMTP id 3027F160BA0 for ; Mon, 1 May 2017 21:49:15 +0200 (CEST) Received: (qmail 40425 invoked by uid 500); 1 May 2017 19:49:14 -0000 Mailing-List: contact commits-help@geode.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@geode.apache.org Delivered-To: mailing list commits@geode.apache.org Received: (qmail 40413 invoked by uid 99); 1 May 2017 19:49:14 -0000 Received: from git1-us-west.apache.org (HELO git1-us-west.apache.org) (140.211.11.23) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 01 May 2017 19:49:14 +0000 Received: by git1-us-west.apache.org (ASF Mail Server at git1-us-west.apache.org, from userid 33) id 441E1DF9A3; Mon, 1 May 2017 19:49:14 +0000 (UTC) Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: klund@apache.org To: commits@geode.apache.org Date: Mon, 01 May 2017 19:49:14 -0000 Message-Id: <10ddecb19a444624a550e1ed0de51c5e@git.apache.org> X-Mailer: ASF-Git Admin Mailer Subject: [01/36] geode git commit: GEODE-2632: change dependencies on GemFireCacheImpl to InternalCache archived-at: Mon, 01 May 2017 19:49:16 -0000 Repository: geode Updated Branches: refs/heads/feature/GEODE-2632-10 [created] dc820690c http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheCreationJUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheCreationJUnitTest.java b/geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheCreationJUnitTest.java index 6035c7a..c728cb0 100644 --- a/geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheCreationJUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/internal/cache/xmlcache/CacheCreationJUnitTest.java @@ -21,7 +21,7 @@ import java.util.HashMap; import java.util.List; import java.util.Map; -import org.apache.geode.cache.GemFireCache; +import org.apache.geode.cache.Region; import org.apache.geode.cache.wan.GatewayReceiver; import org.apache.geode.cache.wan.GatewayReceiverFactory; import org.junit.Before; @@ -34,14 +34,14 @@ import org.mockito.MockitoAnnotations; import org.apache.geode.cache.server.CacheServer; import org.apache.geode.internal.cache.CacheServerImpl; import org.apache.geode.internal.cache.CacheServerLauncher; -import org.apache.geode.internal.cache.GemFireCacheImpl; +import org.apache.geode.internal.cache.InternalCache; import org.apache.geode.test.junit.categories.UnitTest; @Category(UnitTest.class) public class CacheCreationJUnitTest { @Mock - private GemFireCacheImpl cache; + private InternalCache cache; @Before public void setUp() { @@ -56,130 +56,126 @@ public class CacheCreationJUnitTest { RegionCreation declarativeRegion = mock(RegionCreation.class); when(declarativeRegion.getName()).thenReturn("testRegion"); - Map declarativeRegions = new HashMap(); + Map> declarativeRegions = new HashMap<>(); declarativeRegions.put("testRegion", declarativeRegion); - when(cache.getRegion("testRegion")).thenReturn(null); + when(this.cache.getRegion("testRegion")).thenReturn(null); - cacheCreation.initializeRegions(declarativeRegions, cache); + cacheCreation.initializeRegions(declarativeRegions, this.cache); - verify(declarativeRegion, times(1)).createRoot(cache); + verify(declarativeRegion, times(1)).createRoot(this.cache); } @Test public void defaultCacheServerIsNotCreatedWithDefaultPortWhenNoDeclarativeServerIsConfigured() { - Boolean disableDefaultCacheServer = false; - Integer configuredServerPort = null; - String configuredServerBindAddress = null; - CacheCreation cacheCreation = new CacheCreation(); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); List cacheServers = new ArrayList<>(); - when(cache.getCacheServers()).thenReturn(cacheServers); + when(this.cache.getCacheServers()).thenReturn(cacheServers); + + Boolean disableDefaultCacheServer = false; + Integer configuredServerPort = null; + String configuredServerBindAddress = null; - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); - verify(cache, never()).addCacheServer(); + verify(this.cache, never()).addCacheServer(); } @Test public void defaultCacheServerIsNotCreatedWhenDisableDefaultCacheServerIsTrue() { - Boolean disableDefaultCacheServer = true; - Integer configuredServerPort = null; - String configuredServerBindAddress = null; - CacheCreation cacheCreation = new CacheCreation(); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); List cacheServers = new ArrayList<>(); - when(cache.getCacheServers()).thenReturn(cacheServers); + when(this.cache.getCacheServers()).thenReturn(cacheServers); + + Boolean disableDefaultCacheServer = true; + Integer configuredServerPort = null; + String configuredServerBindAddress = null; - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); - verify(cache, never()).addCacheServer(); + verify(this.cache, never()).addCacheServer(); } @Test public void defaultCacheServerIsCreatedWithConfiguredPortWhenNoDeclarativeServerIsConfigured() { - Boolean disableDefaultCacheServer = false; - Integer configuredServerPort = 9999; - String configuredServerBindAddress = null; - CacheCreation cacheCreation = new CacheCreation(); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); List cacheServers = new ArrayList<>(); - when(cache.getCacheServers()).thenReturn(cacheServers); + when(this.cache.getCacheServers()).thenReturn(cacheServers); + + Boolean disableDefaultCacheServer = false; + Integer configuredServerPort = 9999; + String configuredServerBindAddress = null; - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); - verify(cache, times(1)).addCacheServer(); + verify(this.cache, times(1)).addCacheServer(); verify(mockServer).setPort(9999); } @Test public void declarativeCacheServerIsCreatedWithConfiguredServerPort() { - Integer configuredServerPort = 9999; - String configuredServerBindAddress = null; - Boolean disableDefaultCacheServer = false; - CacheCreation cacheCreation = new CacheCreation(); CacheServerCreation br1 = new CacheServerCreation(cacheCreation, false); br1.setPort(8888); cacheCreation.getCacheServers().add(br1); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + Integer configuredServerPort = 9999; + String configuredServerBindAddress = null; + Boolean disableDefaultCacheServer = false; + + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); - verify(cache, times(1)).addCacheServer(); + verify(this.cache, times(1)).addCacheServer(); verify(mockServer).setPort(configuredServerPort); } @Test public void cacheServerCreationIsSkippedWhenAServerExistsForAGivenPort() { - Integer configuredServerPort = null; - String configuredServerBindAddress = null; - Boolean disableDefaultCacheServer = false; - CacheCreation cacheCreation = new CacheCreation(); CacheServerCreation br1 = new CacheServerCreation(cacheCreation, false); br1.setPort(40406); cacheCreation.getCacheServers().add(br1); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); when(mockServer.getPort()).thenReturn(40406); List cacheServers = new ArrayList<>(); cacheServers.add(mockServer); - when(cache.getCacheServers()).thenReturn(cacheServers); + when(this.cache.getCacheServers()).thenReturn(cacheServers); - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + Integer configuredServerPort = null; + String configuredServerBindAddress = null; + Boolean disableDefaultCacheServer = false; - verify(cache, never()).addCacheServer(); + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); + + verify(this.cache, never()).addCacheServer(); } @Test public void userCanCreateMultipleCacheServersDeclaratively() { - Integer configuredServerPort = null; - String configuredServerBindAddress = null; - Boolean disableDefaultCacheServer = false; - CacheCreation cacheCreation = new CacheCreation(); CacheServerCreation br1 = new CacheServerCreation(cacheCreation, false); br1.setPort(40406); @@ -189,28 +185,32 @@ public class CacheCreationJUnitTest { cacheCreation.getCacheServers().add(br2); CacheServerImpl mockServer = mock(CacheServerImpl.class); - when(cache.addCacheServer()).thenReturn(mockServer); + when(this.cache.addCacheServer()).thenReturn(mockServer); - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + Integer configuredServerPort = null; + String configuredServerBindAddress = null; + Boolean disableDefaultCacheServer = false; + + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); - verify(cache, times(2)).addCacheServer(); + verify(this.cache, times(2)).addCacheServer(); verify(mockServer).configureFrom(br1); verify(mockServer).configureFrom(br2); } @Test(expected = RuntimeException.class) public void shouldThrowExceptionWhenUserTriesToDeclareMultipleCacheServersWithPort() { - Integer configuredServerPort = 50505; - String configuredServerBindAddress = "localhost[50505]"; - Boolean disableDefaultCacheServer = false; - CacheCreation cacheCreation = new CacheCreation(); cacheCreation.getCacheServers().add(new CacheServerCreation(cacheCreation, false)); cacheCreation.getCacheServers().add(new CacheServerCreation(cacheCreation, false)); - cacheCreation.startCacheServers(cacheCreation.getCacheServers(), cache, configuredServerPort, - configuredServerBindAddress, disableDefaultCacheServer); + Integer configuredServerPort = 50505; + String configuredServerBindAddress = "localhost[50505]"; + Boolean disableDefaultCacheServer = false; + + cacheCreation.startCacheServers(cacheCreation.getCacheServers(), this.cache, + configuredServerPort, configuredServerBindAddress, disableDefaultCacheServer); } @Test @@ -219,16 +219,16 @@ public class CacheCreationJUnitTest { GatewayReceiver receiver = mock(GatewayReceiver.class); cacheCreation.addGatewayReceiver(receiver); cacheCreation.addRootRegion(new RegionCreation(cacheCreation, "region")); - GemFireCacheImpl cache = mock(GemFireCacheImpl.class); + InternalCache internalCache = mock(InternalCache.class); GatewayReceiverFactory receiverFactory = mock(GatewayReceiverFactory.class); - when(cache.createGatewayReceiverFactory()).thenReturn(receiverFactory); + when(internalCache.createGatewayReceiverFactory()).thenReturn(receiverFactory); when(receiverFactory.create()).thenReturn(receiver); - InOrder inOrder = inOrder(cache, receiverFactory); - cacheCreation.create(cache); + InOrder inOrder = inOrder(internalCache, receiverFactory); + cacheCreation.create(internalCache); - inOrder.verify(cache).basicCreateRegion(eq("region"), any()); - inOrder.verify(cache).createGatewayReceiverFactory(); + // inOrder.verify(cache).basicCreateRegion(eq("region"), any()); + inOrder.verify(internalCache).createGatewayReceiverFactory(); inOrder.verify(receiverFactory).create(); } } http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java b/geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java index 5b07f28..4b7ba9c 100644 --- a/geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java +++ b/geode-core/src/test/java/org/apache/geode/management/internal/cli/commands/DiskStoreCommandsDUnitTest.java @@ -60,6 +60,7 @@ import org.apache.geode.internal.AvailablePort; import org.apache.geode.internal.AvailablePortHelper; import org.apache.geode.internal.cache.DiskStoreImpl; import org.apache.geode.internal.cache.GemFireCacheImpl; +import org.apache.geode.internal.cache.InternalCache; import org.apache.geode.internal.cache.PartitionedRegion; import org.apache.geode.internal.cache.SnapshotTestUtil; import org.apache.geode.internal.cache.persistence.PersistentMemberManager; @@ -96,7 +97,7 @@ import java.util.concurrent.TimeUnit; * The DiskStoreCommandsDUnitTest class is a distributed test suite of test cases for testing the * disk store commands that are part of Gfsh. *

- * + * * @see org.apache.geode.management.internal.cli.commands.DiskStoreCommands * @see org.junit.Assert * @see org.junit.Test @@ -202,7 +203,7 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { regionFactory.setScope(Scope.DISTRIBUTED_ACK); try { regionFactory.create(regionName); - } catch (DistributedSystemDisconnectedException e) { + } catch (DistributedSystemDisconnectedException ignore) { // okay to ignore } } @@ -215,7 +216,7 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { public boolean done() { Cache cache = getCache(); PersistentMemberManager memberManager = - ((GemFireCacheImpl) cache).getPersistentMemberManager(); + ((InternalCache) cache).getPersistentMemberManager(); return !memberManager.getWaitingRegions().isEmpty(); } @@ -840,13 +841,13 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { Cache cache = getCache(); assertNotNull(cache); - GemFireCacheImpl gfc = (GemFireCacheImpl) cache; - Collection diskStoreList = gfc.listDiskStores(); + InternalCache internalCache = (InternalCache) cache; + Collection diskStoreList = internalCache.listDiskStores(); assertNotNull(diskStoreList); assertFalse(diskStoreList.isEmpty()); assertTrue(diskStoreList.size() == 1); - for (DiskStoreImpl diskStore : diskStoreList) { + for (DiskStore diskStore : diskStoreList) { assertTrue(diskStore.getName().equals(diskStoreName)); break; } @@ -892,8 +893,8 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { getSystem(localProps); Cache cache = getCache(); assertNotNull(cache); - GemFireCacheImpl gfc = (GemFireCacheImpl) cache; - Collection diskStores = gfc.listDiskStores(); + InternalCache internalCache = (InternalCache) cache; + Collection diskStores = internalCache.listDiskStores(); assertNotNull(diskStores); assertTrue(diskStores.isEmpty()); return null; @@ -905,7 +906,7 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { /** * 1) Create a disk-store in a member, get the disk-dirs. 2) Close the member. 3) Execute the * command. 4) Restart the member. 5) Check if the disk-store is altered. - * + * * @throws IOException * @throws ClassNotFoundException */ @@ -1190,9 +1191,7 @@ public class DiskStoreCommandsDUnitTest extends CliCommandTestBase { commandStringBuilder.addOption(CliStrings.CREATE_DISK_STORE__TIME_INTERVAL, "2023"); commandStringBuilder.addOption(CliStrings.CREATE_DISK_STORE__WRITE_BUFFER_SIZE, "3110"); commandStringBuilder.addOption(CliStrings.CREATE_DISK_STORE__DIRECTORY_AND_SIZE, - diskStore1Dir1.getAbsolutePath() + "#1452637463"); - commandStringBuilder.addOption(CliStrings.CREATE_DISK_STORE__DIRECTORY_AND_SIZE, - diskStore1Dir2.getAbsolutePath()); + diskStore1Dir1.getAbsolutePath() + "#1452637463," + diskStore1Dir2.getAbsolutePath()); cmdResult = executeCommand(commandStringBuilder.toString()); assertEquals(Result.Status.OK, cmdResult.getStatus()); String stringResult = commandResultToString(cmdResult); http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit3CacheTestCase.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit3CacheTestCase.java b/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit3CacheTestCase.java index 1966a08..5d74ca4 100644 --- a/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit3CacheTestCase.java +++ b/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit3CacheTestCase.java @@ -26,8 +26,10 @@ import org.apache.geode.cache.RegionExistsException; import org.apache.geode.cache.TimeoutException; import org.apache.geode.cache.client.ClientCache; import org.apache.geode.cache.client.ClientCacheFactory; +import org.apache.geode.cache.client.internal.InternalClientCache; import org.apache.geode.cache30.CacheSerializableRunnable; import org.apache.geode.internal.cache.GemFireCacheImpl; +import org.apache.geode.internal.cache.InternalCache; import org.apache.geode.test.dunit.IgnoredException; import org.apache.geode.test.dunit.internal.JUnit3DistributedTestCase; @@ -78,19 +80,19 @@ public abstract class JUnit3CacheTestCase extends JUnit3DistributedTestCase /** * Return a cache for obtaining regions, created lazily. */ - public final Cache getCache() { + public final InternalCache getCache() { return delegate.getCache(); } - public final Cache getCache(final CacheFactory factory) { + public final InternalCache getCache(final CacheFactory factory) { return delegate.getCache(factory); } - public final Cache getCache(final boolean client) { + public final InternalCache getCache(final boolean client) { return delegate.getCache(client); } - public final Cache getCache(final boolean client, final CacheFactory factory) { + public final InternalCache getCache(final boolean client, final CacheFactory factory) { return delegate.getCache(client, factory); } @@ -99,7 +101,7 @@ public abstract class JUnit3CacheTestCase extends JUnit3DistributedTestCase * * @since GemFire 6.5 */ - public final ClientCache getClientCache(final ClientCacheFactory factory) { + public final InternalClientCache getClientCache(final ClientCacheFactory factory) { return delegate.getClientCache(factory); } http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit4CacheTestCase.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit4CacheTestCase.java b/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit4CacheTestCase.java index 4ccf8c2..5b1629b 100644 --- a/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit4CacheTestCase.java +++ b/geode-core/src/test/java/org/apache/geode/test/dunit/cache/internal/JUnit4CacheTestCase.java @@ -39,10 +39,12 @@ import org.apache.geode.cache.TimeoutException; import org.apache.geode.cache.client.ClientCache; import org.apache.geode.cache.client.ClientCacheFactory; import org.apache.geode.cache.client.PoolManager; +import org.apache.geode.cache.client.internal.InternalClientCache; import org.apache.geode.cache30.CacheSerializableRunnable; import org.apache.geode.distributed.internal.DistributionConfig; import org.apache.geode.distributed.internal.DistributionMessageObserver; import org.apache.geode.internal.cache.GemFireCacheImpl; +import org.apache.geode.internal.cache.InternalCache; import org.apache.geode.internal.cache.InternalRegionArguments; import org.apache.geode.internal.cache.LocalRegion; import org.apache.geode.internal.cache.xmlcache.CacheCreation; @@ -79,7 +81,7 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase *

* Field is static so it doesn't get serialized with SerializableRunnable inner classes. */ - private static Cache cache; + private static InternalCache cache; private final CacheTestFixture cacheTestFixture; @@ -112,20 +114,20 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase try { System.setProperty( DistributionConfig.GEMFIRE_PREFIX + "DISABLE_DISCONNECT_DS_ON_CACHE_CLOSE", "true"); - Cache newCache; + InternalCache newCache; if (client) { System.setProperty(DistributionConfig.GEMFIRE_PREFIX + "locators", ""); System.setProperty(DistributionConfig.GEMFIRE_PREFIX + MCAST_PORT, "0"); - newCache = (Cache) new ClientCacheFactory(getSystem().getProperties()).create(); + newCache = (InternalCache) new ClientCacheFactory(getSystem().getProperties()).create(); } else { if (factory == null) { - newCache = CacheFactory.create(getSystem()); + newCache = (InternalCache) CacheFactory.create(getSystem()); } else { Properties props = getSystem().getProperties(); for (Map.Entry entry : props.entrySet()) { factory.set((String) entry.getKey(), (String) entry.getValue()); } - newCache = factory.create(); + newCache = (InternalCache) factory.create(); } } cache = newCache; @@ -154,7 +156,7 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase try { System.setProperty( DistributionConfig.GEMFIRE_PREFIX + "DISABLE_DISCONNECT_DS_ON_CACHE_CLOSE", "true"); - Cache newCache = CacheFactory.create(getLonerSystem()); + InternalCache newCache = (InternalCache) CacheFactory.create(getLonerSystem()); cache = newCache; } catch (CacheExistsException e) { Assert.fail("the cache already exists", e); // TODO: remove error handling @@ -238,24 +240,24 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase /** * Return a cache for obtaining regions, created lazily. */ - public final Cache getCache() { + public final InternalCache getCache() { return getCache(false); } - public final Cache getCache(final CacheFactory factory) { + public final InternalCache getCache(final CacheFactory factory) { return getCache(false, factory); } - public final Cache getCache(final Properties properties) { + public final InternalCache getCache(final Properties properties) { getSystem(properties); return getCache(); } - public final Cache getCache(final boolean client) { + public final InternalCache getCache(final boolean client) { return getCache(client, null); } - public final Cache getCache(final boolean client, final CacheFactory factory) { + public final InternalCache getCache(final boolean client, final CacheFactory factory) { synchronized (JUnit4CacheTestCase.class) { final GemFireCacheImpl gemFireCache = GemFireCacheImpl.getInstance(); if (gemFireCache != null && !gemFireCache.isClosed() @@ -288,7 +290,7 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase * * @since GemFire 6.5 */ - public final ClientCache getClientCache(final ClientCacheFactory factory) { + public final InternalClientCache getClientCache(final ClientCacheFactory factory) { synchronized (JUnit4CacheTestCase.class) { final GemFireCacheImpl gemFireCache = GemFireCacheImpl.getInstance(); if (gemFireCache != null && !gemFireCache.isClosed() @@ -308,12 +310,12 @@ public abstract class JUnit4CacheTestCase extends JUnit4DistributedTestCase if (cache == null || cache.isClosed()) { cache = null; disconnectFromDS(); - cache = (Cache) factory.create(); + cache = (InternalCache) factory.create(); } if (cache != null) { IgnoredException.addIgnoredException("java.net.ConnectException"); } - return (ClientCache) cache; + return (InternalClientCache) cache; } } http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVM.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVM.java b/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVM.java index 3e8630b..d02ce74 100644 --- a/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVM.java +++ b/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVM.java @@ -24,14 +24,10 @@ import org.apache.logging.log4j.Logger; import org.apache.geode.internal.logging.LogService; -/** - * - */ public class RemoteDUnitVM extends UnicastRemoteObject implements RemoteDUnitVMIF { - private static final Logger logger = LogService.getLogger(); - public RemoteDUnitVM() throws RemoteException { + RemoteDUnitVM() throws RemoteException { super(); } @@ -40,37 +36,38 @@ public class RemoteDUnitVM extends UnicastRemoteObject implements RemoteDUnitVMI * the object. Does this synchronously (does not spawn a thread). This method is used by the unit * test framework, dunit. * - * @param obj the object to execute the method on + * @param target the object to execute the method on * @param methodName the name of the method to execute * @return the result of method execution */ - public MethExecutorResult executeMethodOnObject(Object obj, String methodName) { - String name = obj.getClass().getName() + "." + methodName + " on object: " + obj; + @Override + public MethExecutorResult executeMethodOnObject(Object target, String methodName) { + String name = target.getClass().getName() + '.' + methodName + " on object: " + target; long start = start(name); - MethExecutorResult result = MethExecutor.executeObject(obj, methodName); + MethExecutorResult result = MethExecutor.executeObject(target, methodName); logDelta(name, start, result); return result; } protected long start(String name) { - logger.info("Received method: " + name); - long start = System.nanoTime(); - return start; + logger.info("Received method: {}", name); + return System.nanoTime(); } - protected void logDelta(String name, long start, MethExecutorResult result) { + private void logDelta(String name, long start, MethExecutorResult result) { long delta = TimeUnit.NANOSECONDS.toMillis(System.nanoTime() - start); - logger.info("Got result: " + result.toString() + " from " + name + " (took " + delta + " ms)"); + logger.info("Got result: {} from {} (took {} ms)", result, name, delta); } /** * Executes a given instance method on a given object with the given arguments. */ - public MethExecutorResult executeMethodOnObject(Object obj, String methodName, Object[] args) { - String name = obj.getClass().getName() + "." + methodName - + (args != null ? " with " + args.length + " args" : "") + " on object: " + obj; + @Override + public MethExecutorResult executeMethodOnObject(Object target, String methodName, Object[] args) { + String name = target.getClass().getName() + '.' + methodName + + (args != null ? " with " + args.length + " args" : "") + " on object: " + target; long start = start(name); - MethExecutorResult result = MethExecutor.executeObject(obj, methodName, args); + MethExecutorResult result = MethExecutor.executeObject(target, methodName, args); logDelta(name, start, result); return result; } @@ -85,7 +82,7 @@ public class RemoteDUnitVM extends UnicastRemoteObject implements RemoteDUnitVMI * @return the result of method execution */ public MethExecutorResult executeMethodOnClass(String className, String methodName) { - String name = className + "." + methodName; + String name = className + '.' + methodName; long start = start(name); MethExecutorResult result = MethExecutor.execute(className, methodName); logDelta(name, start, result); @@ -96,10 +93,11 @@ public class RemoteDUnitVM extends UnicastRemoteObject implements RemoteDUnitVMI /** * Executes a given static method in a given class with the given arguments. */ + @Override public MethExecutorResult executeMethodOnClass(String className, String methodName, Object[] args) { String name = - className + "." + methodName + (args != null ? " with " + args.length + " args" : ""); + className + '.' + methodName + (args != null ? " with " + args.length + " args" : ""); long start = start(name); MethExecutorResult result = MethExecutor.execute(className, methodName, args); logDelta(name, start, result); @@ -107,22 +105,25 @@ public class RemoteDUnitVM extends UnicastRemoteObject implements RemoteDUnitVMI } public void executeTask(int tsid, int type, int index) throws RemoteException { - throw new UnsupportedOperationException(); - + throw new UnsupportedOperationException("executeTask is not implemented"); } public void runShutdownHook() throws RemoteException { - + throw new UnsupportedOperationException("runShutdownHook is not implemented"); } public void notifyDynamicActionComplete(int actionId) throws RemoteException { - throw new UnsupportedOperationException(); - + throw new UnsupportedOperationException("notifyDynamicActionComplete is not implemented"); } + @Override public void shutDownVM() throws RemoteException { ChildVM.stopVM(); } - public void disconnectVM() throws RemoteException {} + public void disconnectVM() throws RemoteException { + throw new UnsupportedOperationException("disconnectVM is not implemented"); + } + + private static final long serialVersionUID = 251934856609958734L; } http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVMIF.java ---------------------------------------------------------------------- diff --git a/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVMIF.java b/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVMIF.java index d2cc533..9db6b06 100644 --- a/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVMIF.java +++ b/geode-core/src/test/java/org/apache/geode/test/dunit/standalone/RemoteDUnitVMIF.java @@ -21,12 +21,12 @@ import hydra.MethExecutorResult; public interface RemoteDUnitVMIF extends Remote { - MethExecutorResult executeMethodOnObject(Object o, String methodName) throws RemoteException; + MethExecutorResult executeMethodOnObject(Object target, String methodName) throws RemoteException; - MethExecutorResult executeMethodOnObject(Object o, String methodName, Object[] args) + MethExecutorResult executeMethodOnObject(Object target, String methodName, Object[] args) throws RemoteException; - MethExecutorResult executeMethodOnClass(String name, String methodName, Object[] args) + MethExecutorResult executeMethodOnClass(String className, String methodName, Object[] args) throws RemoteException; void shutDownVM() throws RemoteException; http://git-wip-us.apache.org/repos/asf/geode/blob/dc820690/geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java ---------------------------------------------------------------------- diff --git a/geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java b/geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java index 07e3171..f680b63 100644 --- a/geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java +++ b/geode-cq/src/main/java/org/apache/geode/cache/query/internal/cq/CqQueryImpl.java @@ -215,7 +215,8 @@ public abstract class CqQueryImpl implements InternalCqQuery { } // Set Query ExecutionContext, that will be used in later execution. - this.setQueryExecutionContext(new QueryExecutionContext(null, this.cqService.getCache())); + this.setQueryExecutionContext( + new QueryExecutionContext(null, (InternalCache) this.cqService.getCache())); } /**