Remove 'org.apache.geode:geode-membership' and 'org.apache.geode:geode-tcp-server' provided dependency declarations.

Add Javadoc to the GemFireUtilsTests to remove use of internal Apache Geode APIs (AbstractRegion).

Author: jxblum

spring-session-data-geode/spring-session-data-geode.gradle

@@ -22,11 +22,10 @@ dependencies {
 
 	provided "javax.servlet:javax.servlet-api"
 
-	// TODO: Remove dependency declarations when all internal Apache Geode API usage is removed; Only required by tests!
+	// TODO: Remove dependency declarations when all internal Apache Geode API usage is removed; Only required by test,
+	//  and specifically, the GemFireUtilsTests! :P
 	provided "org.apache.geode:geode-logging:$apacheGeodeVersion"
-	provided "org.apache.geode:geode-membership:$apacheGeodeVersion" // Need for Geode Serialization
 	provided "org.apache.geode:geode-serialization:$apacheGeodeVersion" // Need for Geode Serialization
-	provided "org.apache.geode:geode-tcp-server:$apacheGeodeVersion" // Need for Geode Serialization
 
 	testCompile "org.assertj:assertj-core"
 	testCompile "junit:junit"

spring-session-data-geode/src/test/java/org/springframework/session/data/gemfire/support/GemFireUtilsTests.java

@@ -175,6 +175,7 @@ public void clientRegionWithPoolIsNonLocalClientRegion() {
 
 	@Test
 	@SuppressWarnings("rawtypes")
+	// TODO: Remove use of Apache Geode internal API (AbstractRegion).
 	public void clientRegionWithServerProxyIsNonLocalClientRegion() {
 
 		ClientCache mockClientCache = mock(ClientCache.class);
@@ -198,6 +199,7 @@ public void clientRegionWithServerProxyIsNonLocalClientRegion() {
 
 	@Test
 	@SuppressWarnings("rawtypes")
+	// TODO: Remove use of Apache Geode internal API (AbstractRegion).
 	public void clientRegionWithNoPoolAndNoServerProxyIsNotNonLocalClientRegion() {
 
 		ClientCache mockClientCache = mock(ClientCache.class);
@@ -242,6 +244,7 @@ public void nonClientRegionWithPoolIsNotNonLocalClientRegion() {
 
 	@Test
 	@SuppressWarnings("rawtypes")
+	// TODO: Remove use of Apache Geode internal API (AbstractRegion).
 	public void peerRegionWithServerProxyIsNotNonLocalClientRegion() {
 
 		Cache mockPeerCache = mock(Cache.class);