[fix-#12875]command line that -Dmaven.test.skip=true have no effect

[fuchanghai]

command line that -Dmaven.test.skip=true have no effect

Purpose of the pull request

Brief change log

command line that -Dmaven.test.skip=true have no effect

Verify this pull request

This pull request is code cleanup without any test coverage.

[codecov-commenter]

Codecov Report

Merging #12876 (f433cdc) into dev (87a88e3) will decrease coverage by 0.00%.
The diff coverage is n/a.

❗ Current head f433cdc differs from pull request most recent head 1debacc. Consider uploading reports for the commit 1debacc to get more accurate results

@@             Coverage Diff              @@
##                dev   #12876      +/-   ##
============================================
- Coverage     39.27%   39.27%   -0.01%     
  Complexity     4231     4231              
============================================
  Files          1051     1051              
  Lines         39849    39817      -32     
  Branches       4585     4567      -18     
============================================
- Hits          15652    15637      -15     
+ Misses        22422    22410      -12     
+ Partials       1775     1770       -5     

Impacted Files	Coverage Δ
...r/plugin/registry/zookeeper/ZookeeperRegistry.java	43.54% <0.00%> (-6.46%)	⬇️
...hinscheduler/plugin/alert/script/ScriptSender.java	70.58% <0.00%> (-1.64%)	⬇️
...r/plugin/task/sqoop/parameter/SqoopParameters.java	51.19% <0.00%> (-1.20%)	⬇️
...cheduler/plugin/alert/dingtalk/DingTalkSender.java	34.13% <0.00%> (-0.78%)	⬇️
...rver/master/runner/task/BlockingTaskProcessor.java	75.86% <0.00%> (-0.55%)	⬇️
...inscheduler/plugin/registry/etcd/EtcdRegistry.java	50.69% <0.00%> (-0.35%)	⬇️
.../dolphinscheduler/plugin/task/datax/DataxTask.java	36.62% <0.00%> (-0.26%)	⬇️
.../dolphinscheduler/plugin/task/spark/SparkTask.java	72.56% <0.00%> (-0.25%)	⬇️
...dolphinscheduler/service/storage/impl/S3Utils.java	0.00% <0.00%> (ø)
...eduler/api/dto/workflow/WorkflowCreateRequest.java	100.00% <0.00%> (ø)
... and 17 more

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

[sonarcloud]

Kudos, SonarCloud Quality Gate passed!   

0 Bugs
0 Vulnerabilities
0 Security Hotspots
0 Code Smells

No Coverage information
No Duplication information

[kezhenxu94]

To future reviewers and maybe the original author who adds the skip, you can use -DskipTests to skip running tests but still compile them, and -Dmaven.test.skip to entirely skip compiling the tests, there is no need to hardcode the skip=false in the pom.xml