[Improvement-12945][Tools] improvement: fix some sonarlint warning and file name spell error

[ZhongJinHacker]

Purpose of the pull request

fix issue #12945

improvement : fix some sonarlint warning and file name spell error

Brief change log

please check the code diff

Verify this pull request

Manually verified the change by testing locally.

[davidzollo]

thanks for your first contribution to Apache DolphinScheduler, It makes a lot of sense,Looking forward to your deep participation

this PR looks good to me. and please wait the CI check result

[EricGao888]

@ZhongJinHacker Could u plz run 'mvn spotless:apply' to fix formatting errors so that CI could pass? Thanks.

[ZhongJinHacker]

@EricGao888 ok, I had run 'mvn spotless:apply' in my local PC today, evertything is ok， and then I had pushed the fixed code. please review again. thank you for your suggestion

[codecov-commenter]

Codecov Report

Merging #12946 (f7d9585) into dev (27c37b8) will decrease coverage by 0.54%.
The diff coverage is 66.66%.

@@             Coverage Diff              @@
##                dev   #12946      +/-   ##
============================================
- Coverage     39.31%   38.77%   -0.55%     
- Complexity     4247     4291      +44     
============================================
  Files          1061     1080      +19     
  Lines         40050    41384    +1334     
  Branches       4604     4706     +102     
============================================
+ Hits          15746    16046     +300     
- Misses        22524    23558    +1034     
  Partials       1780     1780              

Impacted Files	Coverage Δ
...uler/tools/datasource/DolphinSchedulerManager.java	0.00% <0.00%> (ø)
...pache/dolphinscheduler/tools/demo/ProxyResult.java	50.00% <0.00%> (ø)
...hinscheduler/tools/demo/ProcessDefinitionDemo.java	41.16% <75.00%> (ø)
...che/dolphinscheduler/tools/demo/DemoConstants.java	100.00% <100.00%> (ø)
...r/tools/demo/ProxyProcessDefinitionController.java	88.88% <100.00%> (ø)
...erver/master/processor/queue/TaskEventService.java	75.00% <0.00%> (-5.36%)	⬇️
...dolphinscheduler/remote/future/ResponseFuture.java	81.96% <0.00%> (-1.64%)	⬇️
...olphinscheduler/plugin/task/python/PythonTask.java	9.72% <0.00%> (-0.14%)	⬇️
...erver/master/runner/WorkflowExecuteThreadPool.java	1.20% <0.00%> (-0.02%)	⬇️
.../org/apache/dolphinscheduler/spi/enums/DbType.java	0.00% <0.00%> (ø)
... and 27 more

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

[ZhongJinHacker]

SonarCloud Quality Gate failed.   

0 Bugs 0 Vulnerabilities 0 Security Hotspots 1 Code Smell

0.0% Coverage 0.0% Duplication

fix

[Radeity]

@ZhongJinHacker Please add license header for two UT files.

[ZhongJinHacker]

@Radeity fix, thank you

[ZhongJinHacker]

I have add unit tests to fix unittest coverage problem.
tips: some source code is hard to write unit test like DolphinSchedulerManager.java which need a real database datasource access in its constuctor method. It needs overrload a new constuctor method for DolphinSchedulerManager.
@EricGao888

[sonarqubecloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

61.5% Coverage
0.0% Duplication

[ZhongJinHacker]

@EricGao888 @davidzollo could you merge this PR ? all checks have passed. I can't wait to do something for the next issue

[github-actions]

This pull request has been automatically marked as stale because it has not had recent activity for 120 days. It will be closed in 7 days if no further activity occurs.

[github-actions]

This pull request has been closed because it has not had recent activity. You could reopen it if you try to continue your work, and anyone who are interested in it are encouraged to continue work on this pull request.