This PR is addressing the [recent Concourse 
failure](https://concourse.apachegeode-ci.info/teams/main/pipelines/apache-develop-main/jobs/UnitTest/builds/27).
  

It appears that:

* `powermock` was leaking its dependencies
* `Mockito` as provided by this version of `powermock` is not JDK11 compatabile
* `powermock` is not actually necessary for unit-level testing and may be 
removed here.
* Consequently, the actual dependencies should be specified instead.

All the more reason for us to be diligent in keeping our footprint small not 
leaking dependencies we do not intend to be shared downstream.

Noting that `precheckin` does not yet test JDK11: locally tested with command
```
JAVA_HOME="" ./gradlew geode-rebalancer:test --tests AutoBalancerJUnitTest 
-PtestJVM=/path/to/.jenv/versions/11.0 -PforceTest=true
```



[ Full content available at: https://github.com/apache/geode/pull/2797 ]
This message was relayed via gitbox.apache.org for 
[email protected]

Reply via email to