[ https://issues.apache.org/jira/browse/GEODE-1712?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15408122#comment-15408122 ]
ASF GitHub Bot commented on GEODE-1712: --------------------------------------- Github user asfgit closed the pull request at: https://github.com/apache/incubator-geode/pull/226 > Need unit tests for classes using Integrated Security > ----------------------------------------------------- > > Key: GEODE-1712 > URL: https://issues.apache.org/jira/browse/GEODE-1712 > Project: Geode > Issue Type: Test > Components: tests > Affects Versions: 1.0.0-incubating.M1 > Reporter: Grace Meilen > Assignee: Grace Meilen > Labels: security > > Most of the classes using Integrated Security (GEODE-17) do not have Unit > Tests (some have Integrated Test coverage). > We need to wrap GeodeSecurityUtil in an interface (SecurityService) that can > be mocked so it's possible to write Unit Tests for all of the classes using > Integrated Security. > Client command classes that need Unit Tests: > * CommitCommand > * ContainsKey > * ContainsKey66 > * CreateRegion > * Destroy65 > * DestroyRegion > * Destroy > * ExecuteFunction65 > * ExecuteFunction66 > * ExecuteFunction > * Get70 > * GetAll70 > * GetAll651 > * GetAll > * GetAllWithCallback > * GetClientPartitionAttributesCommand66 > * GetClientPartitionAttributesCommand > * Invalidate > * KeySet > * Put61 > * Put65 > * Put > * RegisterInterest61 > * RegisterInterestList61 > * RegisterInterestList66 > * RegisterInterest > * RemoveAll > * Request > * UnregisterInterest > Non-core client command classes needing Unit Tests: > * CloseCQ > * ExecuteCQ > * ExecuteCQ61 > * GetCQStats > * GetDurableCQs > * MonitorCQ > * StopCQ > More classes needing Unit Tests: > * AcceptorImpl > * CacheClientProxy > * HandShake > * ServerConnection > * CreateAllterDestroyRegionCommands > * DataCommands > * IndexCommands > * DataCommandFunction > * CommandProcessor > * AccessControlMBean > * MBeanServerWrapper > * LoginHandlerInterceptor > * LuceneIndexCommands > * SystemManagementService -- This message was sent by Atlassian JIRA (v6.3.4#6332)