[Improvement-12296][Test] Close static mock

[rickchengx]

Purpose of the pull request

• close: [Improvement][Test] Close static mock #12296
• related: [Improvement][Test] Remove dependency of powermock #11405

Brief change log

In some UT cases, static mocks are not closed after used. This may cause unexpected UT failure. The best practice is to handle it with try-with-resource, see the code below for an example:

dolphinscheduler/dolphinscheduler-common/src/test/java/org/apache/dolphinscheduler/common/utils/FileUtilsTest.java

Lines 39 to 43 in a973054

	try (MockedStatic<DateUtils> mockedDateUtils = Mockito.mockStatic(DateUtils.class)) {
	mockedDateUtils.when(() -> DateUtils.getCurrentTime(YYYYMMDDHHMMSS)).thenReturn("20190101101059");
	Assert.assertEquals("/tmp/dolphinscheduler/download/20190101101059/test",
	FileUtils.getDownloadFilename("test"));
	}

Verify this pull request

manually tested

[codecov-commenter]

Codecov Report

Merging #12300 (e032794) into dev (a973054) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##                dev   #12300   +/-   ##
=========================================
  Coverage     38.53%   38.53%           
  Complexity     4075     4075           
=========================================
  Files          1023     1023           
  Lines         38267    38267           
  Branches       4393     4393           
=========================================
  Hits          14748    14748           
  Misses        21796    21796           
  Partials       1723     1723           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

[rickchengx]

rebased dev to remove conflicts

[EricGao888]

LGTM, waiting for CI to pass

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
0.0% Duplication

[EricGao888]

If no more comments, I will proceed to merge this PR later this afternoon.