[branch-2.9] Rename test file name from *Test2.java to *Test.java to run all tests correctly

[equanz]

Fixes #16854

cherry-pick: #13644

Motivation

It seems some test cases like **/*Test2.java aren't run on the default test lifecycle.
This issue is caused by #10148. I'd like to fix it.

Modifications

Originally:

• Rename test file name to **/*Test.java
• Fix some codes to follow changes
  • [Broker] Apply PolicyHierarchyValue to inactiveTopicPolicies #12687
  • Refactored PulsarResources to avoid building metadata paths everywhere #11693
  • Optimize isValidPath check in MetadataStore #12663
  • [Transaction] Fix transaction system topic create in loop. #12749
  • [Enhancement] avoid duplicate deletion of schema #11640

Additionally:

• Fix conflict and keep existing changes
  • https://github.com/apache/pulsar/pull/14657/files#diff-8696b1e2ba4ee355256e31b33ccc4f1cab77f1c6c79f7b4ab508b73d43d8ad55L477-R471
• Revert some codes because the change is not cherry-picked to branch-2.9 yet
  • [Broker] Apply PolicyHierarchyValue to inactiveTopicPolicies #12687
• Cherry-pick additional changes about test code because existing cherry-pick commit is imcomplete
  • existing cherry-pick commit: 4f65f65
  • original commit: ecd275d

Verifying this change

• Make sure that the change passes the CI checks.

This change is a trivial rework / code cleanup without any test coverage.

Does this pull request potentially affect one of the following parts:

• Dependencies (does it add or upgrade a dependency): (no)
• The public API: (no)
• The schema: (no)
• The default values of configurations: (no)
• The wire protocol: (no)
• The rest endpoints: (no)
• The admin cli options: (no)
• Anything that affects deployment: (no)

Documentation

• no-need-doc

[equanz]

/pulsarbot run-failure-checks