Replace Powermock with Mockito

Alluxio / alluxio

Alluxio, data orchestration for analytics and machine learning in the cloud

https://www.alluxio.io

Apache License 2.0

6.8k stars 2.94k forks source link

Replace Powermock with Mockito #15003

Open apc999 opened 2 years ago

apc999 commented 2 years ago

Summary

Powermock seems no longer maintained https://github.com/powermock/powermock/issues/1117 where the last commit on the master was in 2020
Using Powermock with Java 11 causing excessive amount of "An illegal reflective access operation has occurred" warnings https://github.com/powermock/powermock/issues/969

This issue is asking to replace Powermock (mostly used for mocking static method and whiteboxing to access private method) with Mockito equivalent.

Urgency

Low

xwzbupt commented 2 years ago

/assign @xwzbupt

dbw9580 commented 2 years ago

Tracking list of modules that are using PowerMock:

[ ] alluxio-core-client-fs
[ ] alluxio-core-client-hdfs
[ ] alluxio-core-common
[ ] alluxio-core-server-master
[ ] alluxio-integration-fuse
[ ] alluxio-integration-tools-validation
[x] ~alluxio-job-server~ #15178
[x] ~alluxio-shell~ #15091
[ ] alluxio-table-server-common
[ ] alluxio-table-server-master
[ ] alluxio-tests
[ ] alluxio-underfs-hdfs
[ ] alluxio-underfs-kodo
[ ] alluxio-underfs-obs
[ ] alluxio-underfs-oss
[ ] alluxio-underfs-s3a
[ ] alluxio-underfs-swift

yabola commented 2 years ago

I will finish the alluxio-job-server module @dbw9580

YangchenYe323 commented 2 years ago

I want to work on this for alluxio-core-client-fs

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale because it has not had recent activity. It will be closed in two weeks if no further activity occurs. Thank you for your contributions.