mhansonp commented on a change in pull request #5645:
URL: https://github.com/apache/geode/pull/5645#discussion_r509431374
##########
File path:
geode-core/src/upgradeTest/java/org/apache/geode/internal/cache/rollingupgrade/RollingUpgradePartitionRegionClearServerVersionMismatch.java
##########
@@ -109,20 +111,28 @@ public void before() {
region.put("A", "ValueA");
region.put("B", "ValueB");
});
+ }
+
+ @After
+ public void after() {
+ locator.stop();
+ serverNew.stop();
+ serverOld.stop();
}
/**
- * testClient_ServerVersionMismatchException - validates that when a client
invokes a partitioned
- * region clear on a cluster where one server is running an unsupported
version for this feature
- * we return a ServerVersionMismatchException
+ * testClient_ServerOperationException - validates that when a client
invokes a partitioned region
+ * clear on a cluster where one server is running an unsupported version for
this feature we
+ * return a ServerOperationException
*/
@Test
- public void testClient_ServerVersionMismatchException() throws Exception {
+ public void testClient_ServerOperationExceptionCurrentServerVersion() throws
Exception {
IgnoredException.addIgnoredException(ServerOperationException.class);
- final int locatorPort = locator.getPort();
+
// Get a client VM
- ClientVM clientVM = cluster.startClientVM(3, c ->
c.withLocatorConnection(locatorPort));
+ int serverPort = serverNew.getPort();
+ clientVM = cluster.startClientVM(3, oldVersion, c ->
c.withServerConnection(serverPort));
Review comment:
Not sure I follow here, I have it broken out into two different client
side tests. client -> old server, client -> new server
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
