-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/42729/
-----------------------------------------------------------
(Updated Jan. 25, 2016, 10:51 p.m.)
Review request for geode, Jens Deppe and Jinmei Liao.
Changes
-------
Removed custom TestSuite class.
Bugs: GEODE-851
https://issues.apache.org/jira/browse/GEODE-851
Repository: geode
Description (updated)
-------
GEODE-851: Cleanup tests in management.internal.security package
* Rename tests so that gradle runs them all
* Flag unimplemented tests with @Ignore
* Rename tests to represent what they are actually testing
* Correct the @Category on each test
There are three TODOs in the test code. 2 of these look like potential bugs
that I will file after committing the rename of these tests.
Diffs (updated)
-----
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/AuthorizeOperationForDistributedSystemMXBeanIntegrationTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/AuthorizeOperationForMBeansIntegrationTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/AuthorizeOperationForRegionCommandsIntegrationTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/JSONAuthCodeTest.java
c408116
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/JSONAuthorizationDetailsIntegrationTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/JSONAuthorizationTest.java
bfd8152
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/OperationCodesForDataCommandsIntegrationTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/OperationCodesForDistributedSystemMXBeanTest.java
PRE-CREATION
gemfire-core/src/test/java/com/gemstone/gemfire/management/internal/security/ResourceOperationJUnit.java
0d680e6
Diff: https://reviews.apache.org/r/42729/diff/
Testing
-------
com.gemstone.gemfire.management.internal.security tests
Thanks,
Kirk Lund
