-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/56396/
-----------------------------------------------------------

Review request for geode, Jared Stewart, Kevin Duling, Ken Howe, and Kirk Lund.


Repository: geode


Description
-------

* Refactor GfshShellConnectorRule so that it's easier to use it either as a 
rule or classRule
* Refactor LocatorServerStartupRule so that tests using it do not need to 
extend from Junit4DUnitTest and can be used as a class rule.
* add NetstatDUnitTest


Diffs
-----

  
geode-assembly/src/test/java/org/apache/geode/tools/pulse/PulseDataExportTest.java
 28d50da61669746145c143d33947da6b8564ca9d 
  
geode-core/src/test/java/org/apache/geode/management/ConnectToLocatorSSLDUnitTest.java
 9d678e1a8bd419e66e9dd79e8f7db5dbeeb0890b 
  
geode-core/src/test/java/org/apache/geode/management/internal/cli/NetstatDUnitTest.java
 PRE-CREATION 
  
geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigBaseTest.java
 3a3c74cd9d10b617a12c38c1fdd500ebe83a5707 
  
geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDeployJarDUnitTest.java
 79f0128ecb87b7e1334f35cb4c0c703cd07af544 
  
geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigDistributionDUnitTest.java
 f362d4c679d391b50b9ce297d8e31ba9887024e1 
  
geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigImportDUnitTest.java
 f68015e5e4f70c91e3329a53134562b128745fd7 
  
geode-core/src/test/java/org/apache/geode/management/internal/configuration/ClusterConfigWithSecurityDUnitTest.java
 388fd9d40ccae21d58ea966e4c9c9fd71c46e873 
  
geode-core/src/test/java/org/apache/geode/management/internal/security/GfshCommandsPostProcessorTest.java
 ac3d1dee3a76e2fa884f826291f88ccbd05d10fa 
  
geode-core/src/test/java/org/apache/geode/management/internal/security/GfshCommandsSecurityTest.java
 dbbe0dbc5dd04e06e4876ebcb15ece1cb018d56d 
  
geode-core/src/test/java/org/apache/geode/security/ClusterConfigWithoutSecurityDUnitTest.java
 fe37f9e2becb46c448a97fa2ac4ed7a9548ff4f9 
  
geode-core/src/test/java/org/apache/geode/security/PeerAuthenticatorDUnitTest.java
 a7752aeb90da46071aa1ded17ef32c36f8b6208b 
  
geode-core/src/test/java/org/apache/geode/security/PeerSecurityWithEmbeddedLocatorDUnitTest.java
 66c5be6de9c33226f3ad4d93e9c16dcfbe674c1e 
  
geode-core/src/test/java/org/apache/geode/security/SecurityClusterConfigDUnitTest.java
 ca209a73b247cd422b93864e7de841620140f996 
  
geode-core/src/test/java/org/apache/geode/security/SecurityWithoutClusterConfigDUnitTest.java
 72654cb868f8a62212825eba72aa4fdf8bc7e39e 
  
geode-core/src/test/java/org/apache/geode/security/StartServerAuthorizationTest.java
 24d0db5c123c221b9cff59ef7d91b3d6d2d9b3eb 
  
geode-core/src/test/java/org/apache/geode/test/dunit/rules/GfshShellConnectionRule.java
 49c141413909b1b31e9d148576832bbf788878eb 
  geode-core/src/test/java/org/apache/geode/test/dunit/rules/Locator.java 
0350ffd06d09dfa742033baf0fb1c2afff1ee158 
  
geode-core/src/test/java/org/apache/geode/test/dunit/rules/LocatorServerStartupRule.java
 ac9221cab6ec9a56f269ac90f881049e0ef770de 
  geode-core/src/test/java/org/apache/geode/test/dunit/rules/Member.java 
129fe681c918a15f0927846fb19c469a0ac683a7 
  geode-core/src/test/java/org/apache/geode/test/dunit/rules/Server.java 
bc4c5d614caa0d1555e84fc483af3fa74f9e52ef 
  
geode-lucene/src/test/java/org/apache/geode/cache/lucene/internal/configuration/LuceneClusterConfigurationDUnitTest.java
 0b0e936652444f0171e37fa52653f5a4e9a63c87 

Diff: https://reviews.apache.org/r/56396/diff/


Testing
-------

precheckin pending


Thanks,

Jinmei Liao

Reply via email to