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 spring-projectsgh-70.
  • Loading branch information
jxblum committed Feb 22, 2020
1 parent 8e88abf commit 51e7b0e
Showing 1 changed file with 5 additions and 5 deletions.
Original file line number Diff line number Diff line change
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.