Skip to content

Commit

Permalink
Remove assertion on the cache instance asserting whether the cache wa…
Browse files Browse the repository at this point in the history
…s an instance of o.a.g.internal.cache.GemFireCacheImpl.

Remove assertion on the Region instance asserting whether the Region was an instance of o.a.g.internal.cache.AbstractRegion.

Resolves gh-70.
  • Loading branch information
jxblum committed Feb 22, 2020
1 parent 8e88abf commit 51e7b0e
Showing 1 changed file with 5 additions and 5 deletions.
Expand Up @@ -25,8 +25,6 @@

import org.apache.geode.cache.Region;
import org.apache.geode.cache.client.ClientCache;
import org.apache.geode.internal.cache.AbstractRegion;
import org.apache.geode.internal.cache.GemFireCacheImpl;

import org.springframework.beans.factory.annotation.Autowired;
import org.springframework.boot.SpringBootConfiguration;
Expand All @@ -37,6 +35,7 @@
import org.springframework.data.gemfire.tests.integration.IntegrationTestsSupport;
import org.springframework.data.gemfire.tests.mock.annotation.EnableGemFireMockObjects;
import org.springframework.geode.boot.autoconfigure.CachingProviderAutoConfiguration;
import org.springframework.geode.util.GeodeAssertions;
import org.springframework.test.context.junit4.SpringRunner;

import example.test.service.TestCacheableService;
Expand Down Expand Up @@ -75,10 +74,11 @@ public class AutoConfiguredCachingUnitTests extends IntegrationTestsSupport {
public void setup() {

assertThat(this.clientCache).isNotNull();
assertThat(this.clientCache).isNotInstanceOf(GemFireCacheImpl.class);
assertThat(this.randomNumbers).isNotNull();
// TODO: Why is AbstractRegion internal!?!?! #argh
assertThat(this.randomNumbers).isNotInstanceOf(AbstractRegion.class);

GeodeAssertions.assertThat(this.clientCache).isNotInstanceOfGemFireCacheImpl();
GeodeAssertions.assertThat(this.randomNumbers).isNotInstanceOfAbstractRegion();

assertThat(this.cacheableService).isNotNull();
assertThat(this.cacheableService.isCacheMiss()).isFalse();
}
Expand Down

0 comments on commit 51e7b0e

Please sign in to comment.