jinmeiliao commented on pull request #6789: URL: https://github.com/apache/geode/pull/6789#issuecomment-904810506
Instead of using different vm for different version client, a parameterized test would be more appropriate since we should keep adding newer versions to the test. Try add this test to `AuthExpirationDUnitTest` as that is already a parameterized test. This could be just a another test method added there. This tests a single server with different version of client. I would suggest also adding a test to the `AuthExpirationMultiServerDUnitTest`, and test those `onRegion`, `onServer` and `onServers` calls and verify behavior is as expected. -- 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. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
