[
https://issues.apache.org/jira/browse/GEODE-6052?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16699743#comment-16699743
]
ASF subversion and git services commented on GEODE-6052:
--------------------------------------------------------
Commit 8aba8b9f8f0301370cf55fcd8824b2a80e21f3d1 in geode's branch
refs/heads/develop from [~mcmellawatt]
[ https://gitbox.apache.org/repos/asf?p=geode.git;h=8aba8b9 ]
GEODE-6052: Use constructor DI to avoid PowerMock in JMX tests (#2891)
> Refactor JMX tests to avoid use of PowerMock
> --------------------------------------------
>
> Key: GEODE-6052
> URL: https://issues.apache.org/jira/browse/GEODE-6052
> Project: Geode
> Issue Type: Task
> Components: tests
> Reporter: Ryan McMahon
> Assignee: Ryan McMahon
> Priority: Major
> Labels: pull-request-available
> Time Spent: 2h 10m
> Remaining Estimate: 0h
>
> PowerMock is troublesome because it introduces global state which pollutes
> other tests, but it has been used to mock static methods in the
> MBeanJMXAdapterTest and MBeanProxyFactoryTest. We should determine what test
> and API changes would be required to eliminate PowerMock from these tests,
> which is currently used to mock these static methods:
> 1. InternalDistributedSystem.getConnectedInstance()
> 2. LogService.getLogger()
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
